@@ -10,104 +10,104 @@ |
||
10 | 10 | |
11 | 11 | abstract class AbstractTest extends SapphireTest |
12 | 12 | { |
13 | - protected $usesDatabase = true; |
|
14 | - |
|
15 | - protected function setUp() |
|
16 | - { |
|
17 | - parent::setUp(); |
|
18 | - |
|
19 | - TestCookieStore::$override_headers_sent = false; |
|
20 | - |
|
21 | - Injector::inst()->registerService( |
|
22 | - new TestCookieStore(), |
|
23 | - CookieStore::class |
|
24 | - ); |
|
25 | - |
|
26 | - DBDatetime::set_mock_now('2010-03-15 12:00:00'); |
|
27 | - } |
|
28 | - |
|
29 | - protected function tearDown() |
|
30 | - { |
|
31 | - DBDatetime::clear_mock_now(); |
|
32 | - |
|
33 | - parent::tearDown(); |
|
34 | - } |
|
35 | - |
|
36 | - abstract protected function getStore(); |
|
37 | - |
|
38 | - /** |
|
39 | - * Test how this store handles large volumes of data (>1000 characters) |
|
40 | - */ |
|
41 | - public function testStoreLargeData() |
|
42 | - { |
|
43 | - $session = uniqid(); |
|
44 | - $store = $this->getStore(); |
|
45 | - |
|
46 | - // Test new session is blank |
|
47 | - $result = $store->read($session); |
|
48 | - $this->assertEmpty($result); |
|
49 | - |
|
50 | - // Save data against session |
|
51 | - $data1 = array( |
|
52 | - 'Large' => str_repeat('A', 600), |
|
53 | - 'Content' => str_repeat('B', 600) |
|
54 | - ); |
|
55 | - $store->write($session, serialize($data1)); |
|
56 | - $result = $store->read($session); |
|
57 | - $this->assertEquals($data1, unserialize($result)); |
|
58 | - } |
|
59 | - |
|
60 | - /** |
|
61 | - * Test storage of data |
|
62 | - */ |
|
63 | - public function testStoreData() |
|
64 | - { |
|
65 | - $session = uniqid(); |
|
66 | - $store = $this->getStore(); |
|
67 | - |
|
68 | - // Test new session is blank |
|
69 | - $result = $store->read($session); |
|
70 | - $this->assertEmpty($result); |
|
71 | - |
|
72 | - // Save data against session |
|
73 | - $data1 = array( |
|
74 | - 'Color' => 'red', |
|
75 | - 'Animal' => 'elephant' |
|
76 | - ); |
|
77 | - $store->write($session, serialize($data1)); |
|
78 | - $result = $store->read($session); |
|
79 | - $this->assertEquals($data1, unserialize($result)); |
|
80 | - |
|
81 | - // Save larger data |
|
82 | - $data2 = array( |
|
83 | - 'Color' => 'blue', |
|
84 | - 'Animal' => str_repeat('bat', 100) |
|
85 | - ); |
|
86 | - $store->write($session, serialize($data2)); |
|
87 | - $result = $store->read($session); |
|
88 | - $this->assertEquals($data2, unserialize($result)); |
|
89 | - } |
|
90 | - |
|
91 | - /** |
|
92 | - * Test expiry of data |
|
93 | - */ |
|
94 | - public function testExpiry() |
|
95 | - { |
|
96 | - $session1 = uniqid(); |
|
97 | - $store = $this->getStore(); |
|
98 | - |
|
99 | - // Store data now |
|
100 | - $data1 = array( |
|
101 | - 'Food' => 'Pizza' |
|
102 | - ); |
|
103 | - $store->write($session1, serialize($data1)); |
|
104 | - $result1 = $store->read($session1); |
|
105 | - $this->assertEquals($data1, unserialize($result1)); |
|
106 | - |
|
107 | - // Go to the future and test that the expiry is accurate |
|
108 | - DBDatetime::set_mock_now('2040-03-16 12:00:00'); |
|
109 | - $result2 = $store->read($session1); |
|
110 | - |
|
111 | - $this->assertEmpty($result2); |
|
112 | - } |
|
13 | + protected $usesDatabase = true; |
|
14 | + |
|
15 | + protected function setUp() |
|
16 | + { |
|
17 | + parent::setUp(); |
|
18 | + |
|
19 | + TestCookieStore::$override_headers_sent = false; |
|
20 | + |
|
21 | + Injector::inst()->registerService( |
|
22 | + new TestCookieStore(), |
|
23 | + CookieStore::class |
|
24 | + ); |
|
25 | + |
|
26 | + DBDatetime::set_mock_now('2010-03-15 12:00:00'); |
|
27 | + } |
|
28 | + |
|
29 | + protected function tearDown() |
|
30 | + { |
|
31 | + DBDatetime::clear_mock_now(); |
|
32 | + |
|
33 | + parent::tearDown(); |
|
34 | + } |
|
35 | + |
|
36 | + abstract protected function getStore(); |
|
37 | + |
|
38 | + /** |
|
39 | + * Test how this store handles large volumes of data (>1000 characters) |
|
40 | + */ |
|
41 | + public function testStoreLargeData() |
|
42 | + { |
|
43 | + $session = uniqid(); |
|
44 | + $store = $this->getStore(); |
|
45 | + |
|
46 | + // Test new session is blank |
|
47 | + $result = $store->read($session); |
|
48 | + $this->assertEmpty($result); |
|
49 | + |
|
50 | + // Save data against session |
|
51 | + $data1 = array( |
|
52 | + 'Large' => str_repeat('A', 600), |
|
53 | + 'Content' => str_repeat('B', 600) |
|
54 | + ); |
|
55 | + $store->write($session, serialize($data1)); |
|
56 | + $result = $store->read($session); |
|
57 | + $this->assertEquals($data1, unserialize($result)); |
|
58 | + } |
|
59 | + |
|
60 | + /** |
|
61 | + * Test storage of data |
|
62 | + */ |
|
63 | + public function testStoreData() |
|
64 | + { |
|
65 | + $session = uniqid(); |
|
66 | + $store = $this->getStore(); |
|
67 | + |
|
68 | + // Test new session is blank |
|
69 | + $result = $store->read($session); |
|
70 | + $this->assertEmpty($result); |
|
71 | + |
|
72 | + // Save data against session |
|
73 | + $data1 = array( |
|
74 | + 'Color' => 'red', |
|
75 | + 'Animal' => 'elephant' |
|
76 | + ); |
|
77 | + $store->write($session, serialize($data1)); |
|
78 | + $result = $store->read($session); |
|
79 | + $this->assertEquals($data1, unserialize($result)); |
|
80 | + |
|
81 | + // Save larger data |
|
82 | + $data2 = array( |
|
83 | + 'Color' => 'blue', |
|
84 | + 'Animal' => str_repeat('bat', 100) |
|
85 | + ); |
|
86 | + $store->write($session, serialize($data2)); |
|
87 | + $result = $store->read($session); |
|
88 | + $this->assertEquals($data2, unserialize($result)); |
|
89 | + } |
|
90 | + |
|
91 | + /** |
|
92 | + * Test expiry of data |
|
93 | + */ |
|
94 | + public function testExpiry() |
|
95 | + { |
|
96 | + $session1 = uniqid(); |
|
97 | + $store = $this->getStore(); |
|
98 | + |
|
99 | + // Store data now |
|
100 | + $data1 = array( |
|
101 | + 'Food' => 'Pizza' |
|
102 | + ); |
|
103 | + $store->write($session1, serialize($data1)); |
|
104 | + $result1 = $store->read($session1); |
|
105 | + $this->assertEquals($data1, unserialize($result1)); |
|
106 | + |
|
107 | + // Go to the future and test that the expiry is accurate |
|
108 | + DBDatetime::set_mock_now('2040-03-16 12:00:00'); |
|
109 | + $result2 = $store->read($session1); |
|
110 | + |
|
111 | + $this->assertEmpty($result2); |
|
112 | + } |
|
113 | 113 | } |
@@ -9,20 +9,20 @@ |
||
9 | 9 | |
10 | 10 | class DatabaseStoreTest extends AbstractTest |
11 | 11 | { |
12 | - protected function setUp() |
|
13 | - { |
|
14 | - parent::setUp(); |
|
12 | + protected function setUp() |
|
13 | + { |
|
14 | + parent::setUp(); |
|
15 | 15 | |
16 | - if (!DB::get_conn() instanceof MySQLDatabase) { |
|
17 | - $this->markTestSkipped('Only MySQL databases are supported'); |
|
18 | - } |
|
19 | - } |
|
16 | + if (!DB::get_conn() instanceof MySQLDatabase) { |
|
17 | + $this->markTestSkipped('Only MySQL databases are supported'); |
|
18 | + } |
|
19 | + } |
|
20 | 20 | |
21 | - protected function getStore() |
|
22 | - { |
|
23 | - $store = Injector::inst()->get(DatabaseStore::class); |
|
24 | - $store->setKey(uniqid()); |
|
21 | + protected function getStore() |
|
22 | + { |
|
23 | + $store = Injector::inst()->get(DatabaseStore::class); |
|
24 | + $store->setKey(uniqid()); |
|
25 | 25 | |
26 | - return $store; |
|
27 | - } |
|
26 | + return $store; |
|
27 | + } |
|
28 | 28 | } |
@@ -25,6 +25,7 @@ discard block |
||
25 | 25 | |
26 | 26 | /** |
27 | 27 | * @param SessionHandlerInterface[] |
28 | + * @param BaseStore[] $handlers |
|
28 | 29 | * |
29 | 30 | * @return $this |
30 | 31 | */ |
@@ -38,6 +39,7 @@ discard block |
||
38 | 39 | |
39 | 40 | /** |
40 | 41 | * @param string |
42 | + * @param string $key |
|
41 | 43 | * |
42 | 44 | * @return $this |
43 | 45 | */ |
@@ -9,152 +9,152 @@ |
||
9 | 9 | class HybridSession extends BaseStore |
10 | 10 | { |
11 | 11 | |
12 | - /** |
|
13 | - * List of session handlers |
|
14 | - * |
|
15 | - * @var array |
|
16 | - */ |
|
17 | - protected $handlers = []; |
|
18 | - |
|
19 | - /** |
|
20 | - * True if this session store has been initialised |
|
21 | - * |
|
22 | - * @var bool |
|
23 | - */ |
|
24 | - protected static $enabled = false; |
|
25 | - |
|
26 | - /** |
|
27 | - * @param SessionHandlerInterface[] |
|
28 | - * |
|
29 | - * @return $this |
|
30 | - */ |
|
31 | - public function setHandlers($handlers) |
|
32 | - { |
|
33 | - $this->handlers = $handlers; |
|
34 | - $this->setKey($this->getKey()); |
|
35 | - |
|
36 | - return $this; |
|
37 | - } |
|
38 | - |
|
39 | - /** |
|
40 | - * @param string |
|
41 | - * |
|
42 | - * @return $this |
|
43 | - */ |
|
44 | - public function setKey($key) |
|
45 | - { |
|
46 | - parent::setKey($key); |
|
47 | - |
|
48 | - foreach ($this->getHandlers() as $handler) { |
|
49 | - $handler->setKey($key); |
|
50 | - } |
|
51 | - |
|
52 | - return $this; |
|
53 | - } |
|
54 | - |
|
55 | - /** |
|
56 | - * @return SessionHandlerInterface[] |
|
57 | - */ |
|
58 | - public function getHandlers() |
|
59 | - { |
|
60 | - return $this->handlers ?: []; |
|
61 | - } |
|
62 | - |
|
63 | - /** |
|
64 | - * @param string $save_path |
|
65 | - * @param string $name |
|
66 | - * |
|
67 | - * @return bool |
|
68 | - */ |
|
69 | - public function open($save_path, $name) |
|
70 | - { |
|
71 | - foreach ($this->getHandlers() as $handler) { |
|
72 | - $handler->open($save_path, $name); |
|
73 | - } |
|
74 | - |
|
75 | - return true; |
|
76 | - } |
|
77 | - |
|
78 | - /** |
|
79 | - * @return bool |
|
80 | - */ |
|
81 | - public function close() |
|
82 | - { |
|
83 | - foreach ($this->getHandlers() as $handler) { |
|
84 | - $handler->close(); |
|
85 | - } |
|
86 | - |
|
87 | - return true; |
|
88 | - } |
|
89 | - |
|
90 | - /** |
|
91 | - * @param string $session_id |
|
92 | - * |
|
93 | - * @return string |
|
94 | - */ |
|
95 | - public function read($session_id) |
|
96 | - { |
|
97 | - foreach ($this->getHandlers() as $handler) { |
|
98 | - if ($data = $handler->read($session_id)) { |
|
99 | - return $data; |
|
100 | - } |
|
101 | - } |
|
102 | - |
|
103 | - return ''; |
|
104 | - } |
|
105 | - |
|
106 | - public function write($session_id, $session_data) |
|
107 | - { |
|
108 | - foreach ($this->getHandlers() as $handler) { |
|
109 | - if ($handler->write($session_id, $session_data)) { |
|
110 | - return true; |
|
111 | - } |
|
112 | - } |
|
113 | - |
|
114 | - return false; |
|
115 | - } |
|
116 | - |
|
117 | - public function destroy($session_id) |
|
118 | - { |
|
119 | - foreach ($this->getHandlers() as $handler) { |
|
120 | - $handler->destroy($session_id); |
|
121 | - } |
|
122 | - |
|
123 | - return true; |
|
124 | - } |
|
125 | - |
|
126 | - public function gc($maxlifetime) |
|
127 | - { |
|
128 | - foreach ($this->getHandlers() as $handler) { |
|
129 | - $handler->gc($maxlifetime); |
|
130 | - } |
|
131 | - } |
|
132 | - |
|
133 | - /** |
|
134 | - * Register the session handler as the default |
|
135 | - * |
|
136 | - * @param string $key Desired session key |
|
137 | - */ |
|
138 | - public static function init($key = null) |
|
139 | - { |
|
140 | - $instance = Injector::inst()->get(__CLASS__); |
|
141 | - |
|
142 | - if (empty($key)) { |
|
143 | - user_error( |
|
144 | - 'HybridSession::init() was not given a $key. Disabling cookie-based storage', |
|
145 | - E_USER_WARNING |
|
146 | - ); |
|
147 | - } else { |
|
148 | - $instance->setKey($key); |
|
149 | - } |
|
150 | - |
|
151 | - session_set_save_handler($instance, true); |
|
152 | - |
|
153 | - self::$enabled = true; |
|
154 | - } |
|
155 | - |
|
156 | - public static function is_enabled() |
|
157 | - { |
|
158 | - return self::$enabled; |
|
159 | - } |
|
12 | + /** |
|
13 | + * List of session handlers |
|
14 | + * |
|
15 | + * @var array |
|
16 | + */ |
|
17 | + protected $handlers = []; |
|
18 | + |
|
19 | + /** |
|
20 | + * True if this session store has been initialised |
|
21 | + * |
|
22 | + * @var bool |
|
23 | + */ |
|
24 | + protected static $enabled = false; |
|
25 | + |
|
26 | + /** |
|
27 | + * @param SessionHandlerInterface[] |
|
28 | + * |
|
29 | + * @return $this |
|
30 | + */ |
|
31 | + public function setHandlers($handlers) |
|
32 | + { |
|
33 | + $this->handlers = $handlers; |
|
34 | + $this->setKey($this->getKey()); |
|
35 | + |
|
36 | + return $this; |
|
37 | + } |
|
38 | + |
|
39 | + /** |
|
40 | + * @param string |
|
41 | + * |
|
42 | + * @return $this |
|
43 | + */ |
|
44 | + public function setKey($key) |
|
45 | + { |
|
46 | + parent::setKey($key); |
|
47 | + |
|
48 | + foreach ($this->getHandlers() as $handler) { |
|
49 | + $handler->setKey($key); |
|
50 | + } |
|
51 | + |
|
52 | + return $this; |
|
53 | + } |
|
54 | + |
|
55 | + /** |
|
56 | + * @return SessionHandlerInterface[] |
|
57 | + */ |
|
58 | + public function getHandlers() |
|
59 | + { |
|
60 | + return $this->handlers ?: []; |
|
61 | + } |
|
62 | + |
|
63 | + /** |
|
64 | + * @param string $save_path |
|
65 | + * @param string $name |
|
66 | + * |
|
67 | + * @return bool |
|
68 | + */ |
|
69 | + public function open($save_path, $name) |
|
70 | + { |
|
71 | + foreach ($this->getHandlers() as $handler) { |
|
72 | + $handler->open($save_path, $name); |
|
73 | + } |
|
74 | + |
|
75 | + return true; |
|
76 | + } |
|
77 | + |
|
78 | + /** |
|
79 | + * @return bool |
|
80 | + */ |
|
81 | + public function close() |
|
82 | + { |
|
83 | + foreach ($this->getHandlers() as $handler) { |
|
84 | + $handler->close(); |
|
85 | + } |
|
86 | + |
|
87 | + return true; |
|
88 | + } |
|
89 | + |
|
90 | + /** |
|
91 | + * @param string $session_id |
|
92 | + * |
|
93 | + * @return string |
|
94 | + */ |
|
95 | + public function read($session_id) |
|
96 | + { |
|
97 | + foreach ($this->getHandlers() as $handler) { |
|
98 | + if ($data = $handler->read($session_id)) { |
|
99 | + return $data; |
|
100 | + } |
|
101 | + } |
|
102 | + |
|
103 | + return ''; |
|
104 | + } |
|
105 | + |
|
106 | + public function write($session_id, $session_data) |
|
107 | + { |
|
108 | + foreach ($this->getHandlers() as $handler) { |
|
109 | + if ($handler->write($session_id, $session_data)) { |
|
110 | + return true; |
|
111 | + } |
|
112 | + } |
|
113 | + |
|
114 | + return false; |
|
115 | + } |
|
116 | + |
|
117 | + public function destroy($session_id) |
|
118 | + { |
|
119 | + foreach ($this->getHandlers() as $handler) { |
|
120 | + $handler->destroy($session_id); |
|
121 | + } |
|
122 | + |
|
123 | + return true; |
|
124 | + } |
|
125 | + |
|
126 | + public function gc($maxlifetime) |
|
127 | + { |
|
128 | + foreach ($this->getHandlers() as $handler) { |
|
129 | + $handler->gc($maxlifetime); |
|
130 | + } |
|
131 | + } |
|
132 | + |
|
133 | + /** |
|
134 | + * Register the session handler as the default |
|
135 | + * |
|
136 | + * @param string $key Desired session key |
|
137 | + */ |
|
138 | + public static function init($key = null) |
|
139 | + { |
|
140 | + $instance = Injector::inst()->get(__CLASS__); |
|
141 | + |
|
142 | + if (empty($key)) { |
|
143 | + user_error( |
|
144 | + 'HybridSession::init() was not given a $key. Disabling cookie-based storage', |
|
145 | + E_USER_WARNING |
|
146 | + ); |
|
147 | + } else { |
|
148 | + $instance->setKey($key); |
|
149 | + } |
|
150 | + |
|
151 | + session_set_save_handler($instance, true); |
|
152 | + |
|
153 | + self::$enabled = true; |
|
154 | + } |
|
155 | + |
|
156 | + public static function is_enabled() |
|
157 | + { |
|
158 | + return self::$enabled; |
|
159 | + } |
|
160 | 160 | } |
@@ -9,83 +9,83 @@ |
||
9 | 9 | |
10 | 10 | class HybridSessionTest extends SapphireTest |
11 | 11 | { |
12 | - /** |
|
13 | - * @var BaseStore |
|
14 | - */ |
|
15 | - protected $handler; |
|
16 | - |
|
17 | - /** |
|
18 | - * @var HybridSession |
|
19 | - */ |
|
20 | - protected $instance; |
|
21 | - |
|
22 | - protected function setUp() |
|
23 | - { |
|
24 | - parent::setUp(); |
|
25 | - |
|
26 | - $this->handler = $this->createMock(TestCookieStore::class); |
|
27 | - |
|
28 | - $this->instance = new HybridSession(); |
|
29 | - } |
|
30 | - |
|
31 | - public function testSetHandlersAlsoSetsKeyToEachHandler() |
|
32 | - { |
|
33 | - $this->instance->setKey('foobar'); |
|
34 | - $this->handler->expects($this->once())->method('setKey')->with('foobar'); |
|
35 | - $this->instance->setHandlers([$this->handler]); |
|
36 | - } |
|
37 | - |
|
38 | - public function testOpenDelegatesToAllHandlers() |
|
39 | - { |
|
40 | - $this->handler->expects($this->once())->method('open')->with('foo', 'bar'); |
|
41 | - $this->instance->setHandlers([$this->handler]); |
|
42 | - $this->assertTrue($this->instance->open('foo', 'bar'), 'Method returns true after delegation'); |
|
43 | - } |
|
44 | - |
|
45 | - public function testCloseDelegatesToAllHandlers() |
|
46 | - { |
|
47 | - $this->handler->expects($this->once())->method('close'); |
|
48 | - $this->instance->setHandlers([$this->handler]); |
|
49 | - $this->assertTrue($this->instance->close(), 'Method returns true after delegation'); |
|
50 | - } |
|
51 | - |
|
52 | - public function testReadReturnsEmptyStringWithNoHandlers() |
|
53 | - { |
|
54 | - $this->handler->expects($this->once())->method('read')->with('foosession')->willReturn(false); |
|
55 | - $this->instance->setHandlers([$this->handler]); |
|
56 | - $this->assertSame('', $this->instance->read('foosession')); |
|
57 | - } |
|
58 | - |
|
59 | - public function testReadReturnsHandlerDelegateResult() |
|
60 | - { |
|
61 | - $this->handler->expects($this->once())->method('read')->with('foo.session')->willReturn('success!'); |
|
62 | - $this->instance->setHandlers([$this->handler]); |
|
63 | - $this->assertSame('success!', $this->instance->read('foo.session')); |
|
64 | - } |
|
65 | - |
|
66 | - public function testWriteDelegatesToHandlerAndReturnsTrue() |
|
67 | - { |
|
68 | - $this->handler->expects($this->once())->method('write')->with('foo', 'bar')->willReturn(true); |
|
69 | - $this->instance->setHandlers([$this->handler]); |
|
70 | - $this->assertTrue($this->instance->write('foo', 'bar')); |
|
71 | - } |
|
72 | - |
|
73 | - public function testWriteReturnsFalseWithNoHandlers() |
|
74 | - { |
|
75 | - $this->assertFalse($this->instance->write('no', 'handlers')); |
|
76 | - } |
|
77 | - |
|
78 | - public function testDestroyDelegatesToHandler() |
|
79 | - { |
|
80 | - $this->handler->expects($this->once())->method('destroy')->with('sessid1234'); |
|
81 | - $this->instance->setHandlers([$this->handler]); |
|
82 | - $this->assertTrue($this->instance->destroy('sessid1234'), 'Method returns true after delegation'); |
|
83 | - } |
|
84 | - |
|
85 | - public function testGcDelegatesToHandlers() |
|
86 | - { |
|
87 | - $this->handler->expects($this->once())->method('gc')->with(12345); |
|
88 | - $this->instance->setHandlers([$this->handler]); |
|
89 | - $this->instance->gc(12345); |
|
90 | - } |
|
12 | + /** |
|
13 | + * @var BaseStore |
|
14 | + */ |
|
15 | + protected $handler; |
|
16 | + |
|
17 | + /** |
|
18 | + * @var HybridSession |
|
19 | + */ |
|
20 | + protected $instance; |
|
21 | + |
|
22 | + protected function setUp() |
|
23 | + { |
|
24 | + parent::setUp(); |
|
25 | + |
|
26 | + $this->handler = $this->createMock(TestCookieStore::class); |
|
27 | + |
|
28 | + $this->instance = new HybridSession(); |
|
29 | + } |
|
30 | + |
|
31 | + public function testSetHandlersAlsoSetsKeyToEachHandler() |
|
32 | + { |
|
33 | + $this->instance->setKey('foobar'); |
|
34 | + $this->handler->expects($this->once())->method('setKey')->with('foobar'); |
|
35 | + $this->instance->setHandlers([$this->handler]); |
|
36 | + } |
|
37 | + |
|
38 | + public function testOpenDelegatesToAllHandlers() |
|
39 | + { |
|
40 | + $this->handler->expects($this->once())->method('open')->with('foo', 'bar'); |
|
41 | + $this->instance->setHandlers([$this->handler]); |
|
42 | + $this->assertTrue($this->instance->open('foo', 'bar'), 'Method returns true after delegation'); |
|
43 | + } |
|
44 | + |
|
45 | + public function testCloseDelegatesToAllHandlers() |
|
46 | + { |
|
47 | + $this->handler->expects($this->once())->method('close'); |
|
48 | + $this->instance->setHandlers([$this->handler]); |
|
49 | + $this->assertTrue($this->instance->close(), 'Method returns true after delegation'); |
|
50 | + } |
|
51 | + |
|
52 | + public function testReadReturnsEmptyStringWithNoHandlers() |
|
53 | + { |
|
54 | + $this->handler->expects($this->once())->method('read')->with('foosession')->willReturn(false); |
|
55 | + $this->instance->setHandlers([$this->handler]); |
|
56 | + $this->assertSame('', $this->instance->read('foosession')); |
|
57 | + } |
|
58 | + |
|
59 | + public function testReadReturnsHandlerDelegateResult() |
|
60 | + { |
|
61 | + $this->handler->expects($this->once())->method('read')->with('foo.session')->willReturn('success!'); |
|
62 | + $this->instance->setHandlers([$this->handler]); |
|
63 | + $this->assertSame('success!', $this->instance->read('foo.session')); |
|
64 | + } |
|
65 | + |
|
66 | + public function testWriteDelegatesToHandlerAndReturnsTrue() |
|
67 | + { |
|
68 | + $this->handler->expects($this->once())->method('write')->with('foo', 'bar')->willReturn(true); |
|
69 | + $this->instance->setHandlers([$this->handler]); |
|
70 | + $this->assertTrue($this->instance->write('foo', 'bar')); |
|
71 | + } |
|
72 | + |
|
73 | + public function testWriteReturnsFalseWithNoHandlers() |
|
74 | + { |
|
75 | + $this->assertFalse($this->instance->write('no', 'handlers')); |
|
76 | + } |
|
77 | + |
|
78 | + public function testDestroyDelegatesToHandler() |
|
79 | + { |
|
80 | + $this->handler->expects($this->once())->method('destroy')->with('sessid1234'); |
|
81 | + $this->instance->setHandlers([$this->handler]); |
|
82 | + $this->assertTrue($this->instance->destroy('sessid1234'), 'Method returns true after delegation'); |
|
83 | + } |
|
84 | + |
|
85 | + public function testGcDelegatesToHandlers() |
|
86 | + { |
|
87 | + $this->handler->expects($this->once())->method('gc')->with(12345); |
|
88 | + $this->instance->setHandlers([$this->handler]); |
|
89 | + $this->instance->gc(12345); |
|
90 | + } |
|
91 | 91 | } |
@@ -7,21 +7,21 @@ |
||
7 | 7 | |
8 | 8 | class TestCookieStore extends CookieStore implements TestOnly |
9 | 9 | { |
10 | - /** |
|
11 | - * Override value of 'headers_sent' but only if running tests. |
|
12 | - * |
|
13 | - * Set to true or false, or null to not override |
|
14 | - * |
|
15 | - * @var string |
|
16 | - */ |
|
17 | - public static $override_headers_sent = null; |
|
10 | + /** |
|
11 | + * Override value of 'headers_sent' but only if running tests. |
|
12 | + * |
|
13 | + * Set to true or false, or null to not override |
|
14 | + * |
|
15 | + * @var string |
|
16 | + */ |
|
17 | + public static $override_headers_sent = null; |
|
18 | 18 | |
19 | - protected function canWrite() |
|
20 | - { |
|
21 | - if (self::$override_headers_sent !== null) { |
|
22 | - return !self::$override_headers_sent; |
|
23 | - } |
|
19 | + protected function canWrite() |
|
20 | + { |
|
21 | + if (self::$override_headers_sent !== null) { |
|
22 | + return !self::$override_headers_sent; |
|
23 | + } |
|
24 | 24 | |
25 | - parent::canWrite(); |
|
26 | - } |
|
25 | + parent::canWrite(); |
|
26 | + } |
|
27 | 27 | } |