Passed
Push — master ( 2dbe85...59ef95 )
by Dan
07:33 queued 01:43
created
Tests/Cache/CacheTest.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,6 @@
 block discarded – undo
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
 
Please login to merge, or discard this patch.
Spacing   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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');
Please login to merge, or discard this patch.
Tests/Cache/Mock/IteratorMock.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -9,10 +9,10 @@
 block discarded – undo
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
 }
Please login to merge, or discard this patch.
Tests/Cache/Mock/AbstractStorageMock.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -11,10 +11,10 @@
 block discarded – undo
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
Please login to merge, or discard this patch.