@@ -4,7 +4,6 @@ |
||
4 | 4 | |
5 | 5 | use Ds\Cache\Cache; |
6 | 6 | use Ds\Cache\CacheStorageInterface; |
7 | -use Ds\Cache\NullStorage; |
|
8 | 7 | use Psr\SimpleCache\InvalidArgumentException; |
9 | 8 | use Tests\Cache\Mock\IteratorMock; |
10 | 9 |
@@ -40,7 +40,7 @@ discard block |
||
40 | 40 | { |
41 | 41 | $key = 21021000; |
42 | 42 | $this->expectException(InvalidArgumentException::class); |
43 | - $this->cache->set($key,'value'); |
|
43 | + $this->cache->set($key, 'value'); |
|
44 | 44 | } |
45 | 45 | |
46 | 46 | /** |
@@ -206,7 +206,7 @@ discard block |
||
206 | 206 | /** |
207 | 207 | * |
208 | 208 | */ |
209 | - public function testGetMultiple(){ |
|
209 | + public function testGetMultiple() { |
|
210 | 210 | |
211 | 211 | $expected = [ |
212 | 212 | 'foo' => 'fooValue', |
@@ -216,7 +216,7 @@ discard block |
||
216 | 216 | |
217 | 217 | $i = 0; |
218 | 218 | |
219 | - foreach ($expected as $key => $value){ |
|
219 | + foreach ($expected as $key => $value) { |
|
220 | 220 | |
221 | 221 | $this->storageMock->expects($this->at($i)) |
222 | 222 | ->method('get') |
@@ -236,7 +236,7 @@ discard block |
||
236 | 236 | /** |
237 | 237 | * |
238 | 238 | */ |
239 | - public function testSetMultiple(){ |
|
239 | + public function testSetMultiple() { |
|
240 | 240 | |
241 | 241 | $keys = [ |
242 | 242 | 'foo' => 'fooValue', |
@@ -250,7 +250,7 @@ discard block |
||
250 | 250 | $i = 0; |
251 | 251 | $expires = 60 * 60; |
252 | 252 | |
253 | - foreach ($keys as $key => $value){ |
|
253 | + foreach ($keys as $key => $value) { |
|
254 | 254 | $this->storageMock->expects($this->at($i)) |
255 | 255 | ->method('set') |
256 | 256 | ->with( |
@@ -262,14 +262,14 @@ discard block |
||
262 | 262 | $i++; |
263 | 263 | } |
264 | 264 | |
265 | - $actual = $this->cache->setMultiple($keys,$expires); |
|
265 | + $actual = $this->cache->setMultiple($keys, $expires); |
|
266 | 266 | $this->assertEquals($expected, $actual); |
267 | 267 | } |
268 | 268 | |
269 | 269 | /** |
270 | 270 | * |
271 | 271 | */ |
272 | - public function testSetMultipleWithFailure(){ |
|
272 | + public function testSetMultipleWithFailure() { |
|
273 | 273 | |
274 | 274 | $keys = [ |
275 | 275 | 'foo' => 'fooValue', |
@@ -283,11 +283,11 @@ discard block |
||
283 | 283 | $i = 0; |
284 | 284 | $expires = 60 * 60; |
285 | 285 | |
286 | - foreach ($keys as $key => $value){ |
|
286 | + foreach ($keys as $key => $value) { |
|
287 | 287 | |
288 | 288 | $status = $addStatus; |
289 | 289 | |
290 | - if ($i === 1){ |
|
290 | + if ($i === 1) { |
|
291 | 291 | $status = false; |
292 | 292 | } |
293 | 293 | |
@@ -302,20 +302,20 @@ discard block |
||
302 | 302 | $i++; |
303 | 303 | } |
304 | 304 | |
305 | - $actual = $this->cache->setMultiple($keys,$expires); |
|
305 | + $actual = $this->cache->setMultiple($keys, $expires); |
|
306 | 306 | $this->assertEquals($expected, $actual); |
307 | 307 | } |
308 | 308 | |
309 | 309 | /** |
310 | 310 | * |
311 | 311 | */ |
312 | - public function testDeleteMultiple(){ |
|
312 | + public function testDeleteMultiple() { |
|
313 | 313 | |
314 | - $keys = ['foo','bar','baz']; |
|
314 | + $keys = ['foo', 'bar', 'baz']; |
|
315 | 315 | $deleteStatus = true; |
316 | 316 | $expected = true; |
317 | 317 | |
318 | - foreach ($keys as $i => $key){ |
|
318 | + foreach ($keys as $i => $key) { |
|
319 | 319 | $this->storageMock->expects($this->at($i)) |
320 | 320 | ->method('delete') |
321 | 321 | ->with( |
@@ -331,15 +331,15 @@ discard block |
||
331 | 331 | /** |
332 | 332 | * |
333 | 333 | */ |
334 | - public function testDeleteMultipleFailure(){ |
|
334 | + public function testDeleteMultipleFailure() { |
|
335 | 335 | |
336 | - $keys = ['foo','bar','baz']; |
|
336 | + $keys = ['foo', 'bar', 'baz']; |
|
337 | 337 | $expected = false; |
338 | 338 | $deleteStatus = true; |
339 | 339 | |
340 | - foreach ($keys as $i => $key){ |
|
340 | + foreach ($keys as $i => $key) { |
|
341 | 341 | |
342 | - if ($i === 1){ |
|
342 | + if ($i === 1) { |
|
343 | 343 | $deleteStatus = false; |
344 | 344 | } |
345 | 345 | |
@@ -358,7 +358,7 @@ discard block |
||
358 | 358 | /** |
359 | 359 | * Test that storage clear is called. |
360 | 360 | */ |
361 | - public function testclear(){ |
|
361 | + public function testclear() { |
|
362 | 362 | $this->storageMock->expects($this->once()) |
363 | 363 | ->method('clear'); |
364 | 364 | $this->cache->clear(); |
@@ -367,7 +367,7 @@ discard block |
||
367 | 367 | /** |
368 | 368 | * Test that array has keys. |
369 | 369 | */ |
370 | - public function testArrayHasKeys(){ |
|
370 | + public function testArrayHasKeys() { |
|
371 | 371 | $data = ['a' => 'foo', 'b' => 'bar']; |
372 | 372 | |
373 | 373 | $reflector = new \ReflectionClass('\Ds\Cache\Cache'); |
@@ -381,9 +381,9 @@ discard block |
||
381 | 381 | /** |
382 | 382 | * Test exception thrown when array is missing keys. |
383 | 383 | */ |
384 | - public function testArrayHasNoKeys(){ |
|
384 | + public function testArrayHasNoKeys() { |
|
385 | 385 | $this->expectException(InvalidArgumentException::class); |
386 | - $data = ['foo','bar']; |
|
386 | + $data = ['foo', 'bar']; |
|
387 | 387 | $reflector = new \ReflectionClass(Cache::class); |
388 | 388 | $method = $reflector->getMethod('_hasKeys'); |
389 | 389 | $method->setAccessible(true); |
@@ -393,8 +393,8 @@ discard block |
||
393 | 393 | /** |
394 | 394 | * Test array contains a failure. |
395 | 395 | */ |
396 | - public function testHasFailure(){ |
|
397 | - $results = [true,true,true,false,true]; |
|
396 | + public function testHasFailure() { |
|
397 | + $results = [true, true, true, false, true]; |
|
398 | 398 | $reflector = new \ReflectionClass('\Ds\Cache\Cache'); |
399 | 399 | $method = $reflector->getMethod('_hasFailure'); |
400 | 400 | $method->setAccessible(true); |
@@ -406,8 +406,8 @@ discard block |
||
406 | 406 | /** |
407 | 407 | * |
408 | 408 | */ |
409 | - public function testHasNoFailure(){ |
|
410 | - $results = [true,true,true,true,true]; |
|
409 | + public function testHasNoFailure() { |
|
410 | + $results = [true, true, true, true, true]; |
|
411 | 411 | $reflector = new \ReflectionClass('\Ds\Cache\Cache'); |
412 | 412 | $method = $reflector->getMethod('_hasFailure'); |
413 | 413 | $method->setAccessible(true); |
@@ -419,8 +419,8 @@ discard block |
||
419 | 419 | /** |
420 | 420 | * Test that array is Traversable |
421 | 421 | */ |
422 | - public function testIsTraversable(){ |
|
423 | - $data = ['a','b','c','d']; |
|
422 | + public function testIsTraversable() { |
|
423 | + $data = ['a', 'b', 'c', 'd']; |
|
424 | 424 | $reflector = new \ReflectionClass('\Ds\Cache\Cache'); |
425 | 425 | $method = $reflector->getMethod('_isTraversable'); |
426 | 426 | $method->setAccessible(true); |
@@ -432,7 +432,7 @@ discard block |
||
432 | 432 | /** |
433 | 433 | * Test that string is not Traversable |
434 | 434 | */ |
435 | - public function testIsNotTraversable(){ |
|
435 | + public function testIsNotTraversable() { |
|
436 | 436 | $this->expectException(InvalidArgumentException::class); |
437 | 437 | $data = 'some-random-string'; |
438 | 438 | $reflector = new \ReflectionClass('\Ds\Cache\Cache'); |
@@ -444,7 +444,7 @@ discard block |
||
444 | 444 | /** |
445 | 445 | * Test that instance of Iterator is traversable |
446 | 446 | */ |
447 | - public function testIsTraversableIterator(){ |
|
447 | + public function testIsTraversableIterator() { |
|
448 | 448 | $iterator = new IteratorMock(); |
449 | 449 | $reflector = new \ReflectionClass('\Ds\Cache\Cache'); |
450 | 450 | $method = $reflector->getMethod('_isTraversable'); |
@@ -9,10 +9,10 @@ |
||
9 | 9 | * |
10 | 10 | * @package Tests\Cache\Mock |
11 | 11 | */ |
12 | -class IteratorMock implements \Iterator{ |
|
13 | - public function current(){} |
|
14 | - public function next(){} |
|
15 | - public function key(){} |
|
16 | - public function valid(){} |
|
17 | - public function rewind(){} |
|
12 | +class IteratorMock implements \Iterator { |
|
13 | + public function current() {} |
|
14 | + public function next() {} |
|
15 | + public function key() {} |
|
16 | + public function valid() {} |
|
17 | + public function rewind() {} |
|
18 | 18 | } |
@@ -11,10 +11,10 @@ |
||
11 | 11 | * |
12 | 12 | * @package Tests\Cache\Mock |
13 | 13 | */ |
14 | -class AbstractStorageMock extends AbstractStorage{ |
|
15 | - public function set($key, $value, $ttl = null){} |
|
16 | - public function has($key){} |
|
17 | - public function get($key){} |
|
18 | - public function delete($key){} |
|
19 | - public function clear(){} |
|
14 | +class AbstractStorageMock extends AbstractStorage { |
|
15 | + public function set($key, $value, $ttl = null) {} |
|
16 | + public function has($key) {} |
|
17 | + public function get($key) {} |
|
18 | + public function delete($key) {} |
|
19 | + public function clear() {} |
|
20 | 20 | } |
21 | 21 | \ No newline at end of file |