Unverified Commit 8146809d authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3172757 by Erik Frèrejean, raman.b: Session manager destroy misses isCli check

(cherry picked from commit 86ef18f1ceecc618bf44ea6b7cb0adca5ea974d2)
parent b794a339
Loading
Loading
Loading
Loading
+4 −0
Original line number Diff line number Diff line
@@ -257,6 +257,10 @@ public function delete($uid) {
   * {@inheritdoc}
   */
  public function destroy() {
    if ($this->isCli()) {
      return;
    }

    session_destroy();

    // Unset the session cookies.
+22 −0
Original line number Diff line number Diff line
<?php

namespace Drupal\KernelTests\Core\Session;

use Drupal\KernelTests\KernelTestBase;

/**
 * Tests starting and destroying a session from the CLI.
 *
 * @group Session
 */
class SessionManagerDestroyNoCliCheckTest extends KernelTestBase {

  /**
   * Tests starting and destroying a session from the CLI.
   */
  public function testCallSessionManagerStartAndDestroy() {
    $this->assertFalse(\Drupal::service('session_manager')->start());
    $this->assertNull(\Drupal::service('session_manager')->destroy());
  }

}