Browse Source

Make sure id() also returns a non empty value for CLI faking.

mscherer 9 years ago
parent
commit
70bd81ae8e
2 changed files with 6 additions and 4 deletions
  1. 1 0
      src/Network/Session.php
  2. 5 4
      tests/TestCase/Network/SessionTest.php

+ 1 - 0
src/Network/Session.php

@@ -310,6 +310,7 @@ class Session
 
         if ($this->_isCLI) {
             $_SESSION = [];
+            $this->id('cli');
 
             return $this->_started = true;
         }

+ 5 - 4
tests/TestCase/Network/SessionTest.php

@@ -287,14 +287,15 @@ class SessionTest extends TestCase
         $session = new Session();
         $result = $session->id();
         $expected = session_id();
-        $this->assertEquals($expected, $result);
+        $this->assertNotEmpty($result);
+        $this->assertSame($expected, $result);
 
         $session->id('MySessionId');
-        $this->assertEquals('MySessionId', $session->id());
-        $this->assertEquals('MySessionId', session_id());
+        $this->assertSame('MySessionId', $session->id());
+        $this->assertSame('MySessionId', session_id());
 
         $session->id('');
-        $this->assertEquals('', session_id());
+        $this->assertSame('', session_id());
     }
 
     /**