djsmithme / Cache

Tests/Cache/CacheTest.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Ds\Cache\Cache;
 use Ds\Cache\CacheStorageInterface;
-use Ds\Cache\NullStorage;
 use Psr\SimpleCache\InvalidArgumentException;
 use Tests\Cache\Mock\IteratorMock;
 

Spacing +29 added lines, -29 removed lines

@@ -40,7 +40,7 @@ 
     {
         $key = 21021000;
         $this->expectException(InvalidArgumentException::class);
-        $this->cache->set($key,'value');
+        $this->cache->set($key, 'value');
     }
 
     /**
@@ -206,7 +206,7 @@ 
     /**
      *
      */
-    public function testGetMultiple(){
+    public function testGetMultiple() {
 
         $expected = [
             'foo' => 'fooValue',
@@ -216,7 +216,7 @@ 
 
         $i = 0;
 
-        foreach ($expected as $key => $value){
+        foreach ($expected as $key => $value) {
 
             $this->storageMock->expects($this->at($i))
                 ->method('get')
@@ -236,7 +236,7 @@ 
     /**
      *
      */
-    public function testSetMultiple(){
+    public function testSetMultiple() {
 
         $keys = [
             'foo' => 'fooValue',
@@ -250,7 +250,7 @@ 
         $i = 0;
         $expires = 60 * 60;
 
-        foreach ($keys as $key => $value){
+        foreach ($keys as $key => $value) {
             $this->storageMock->expects($this->at($i))
                 ->method('set')
                 ->with(
@@ -262,14 +262,14 @@ 
             $i++;
         }
 
-        $actual = $this->cache->setMultiple($keys,$expires);
+        $actual = $this->cache->setMultiple($keys, $expires);
         $this->assertEquals($expected, $actual);
     }
 
     /**
      *
      */
-    public function testSetMultipleWithFailure(){
+    public function testSetMultipleWithFailure() {
 
         $keys = [
             'foo' => 'fooValue',
@@ -283,11 +283,11 @@ 
         $i = 0;
         $expires = 60 * 60;
 
-        foreach ($keys as $key => $value){
+        foreach ($keys as $key => $value) {
 
             $status = $addStatus;
 
-            if ($i === 1){
+            if ($i === 1) {
                 $status = false;
             }
 
@@ -302,20 +302,20 @@ 
             $i++;
         }
 
-        $actual = $this->cache->setMultiple($keys,$expires);
+        $actual = $this->cache->setMultiple($keys, $expires);
         $this->assertEquals($expected, $actual);
     }
 
     /**
      *
      */
-    public function testDeleteMultiple(){
+    public function testDeleteMultiple() {
 
-        $keys = ['foo','bar','baz'];
+        $keys = ['foo', 'bar', 'baz'];
         $deleteStatus = true;
         $expected = true;
 
-        foreach ($keys as $i => $key){
+        foreach ($keys as $i => $key) {
             $this->storageMock->expects($this->at($i))
                 ->method('delete')
                 ->with(
@@ -331,15 +331,15 @@ 
     /**
      *
      */
-    public function testDeleteMultipleFailure(){
+    public function testDeleteMultipleFailure() {
 
-        $keys = ['foo','bar','baz'];
+        $keys = ['foo', 'bar', 'baz'];
         $expected = false;
         $deleteStatus = true;
 
-        foreach ($keys as $i => $key){
+        foreach ($keys as $i => $key) {
 
-            if ($i === 1){
+            if ($i === 1) {
                 $deleteStatus = false;
             }
 
@@ -358,7 +358,7 @@ 
     /**
      * Test that storage clear is called.
      */
-    public function testclear(){
+    public function testclear() {
         $this->storageMock->expects($this->once())
             ->method('clear');
         $this->cache->clear();
@@ -367,7 +367,7 @@ 
     /**
      * Test that array has keys.
      */
-    public function testArrayHasKeys(){
+    public function testArrayHasKeys() {
         $data = ['a' => 'foo', 'b' => 'bar'];
 
         $reflector = new \ReflectionClass('\Ds\Cache\Cache');
@@ -381,9 +381,9 @@ 
     /**
      * Test exception thrown when array is missing keys.
      */
-    public function testArrayHasNoKeys(){
+    public function testArrayHasNoKeys() {
         $this->expectException(InvalidArgumentException::class);
-        $data = ['foo','bar'];
+        $data = ['foo', 'bar'];
         $reflector = new \ReflectionClass(Cache::class);
         $method = $reflector->getMethod('_hasKeys');
         $method->setAccessible(true);
@@ -393,8 +393,8 @@ 
     /**
      * Test array contains a failure.
      */
-    public function testHasFailure(){
-        $results = [true,true,true,false,true];
+    public function testHasFailure() {
+        $results = [true, true, true, false, true];
         $reflector = new \ReflectionClass('\Ds\Cache\Cache');
         $method = $reflector->getMethod('_hasFailure');
         $method->setAccessible(true);
@@ -406,8 +406,8 @@ 
     /**
      *
      */
-    public function testHasNoFailure(){
-        $results = [true,true,true,true,true];
+    public function testHasNoFailure() {
+        $results = [true, true, true, true, true];
         $reflector = new \ReflectionClass('\Ds\Cache\Cache');
         $method = $reflector->getMethod('_hasFailure');
         $method->setAccessible(true);
@@ -419,8 +419,8 @@ 
     /**
      * Test that array is Traversable
      */
-    public function testIsTraversable(){
-        $data = ['a','b','c','d'];
+    public function testIsTraversable() {
+        $data = ['a', 'b', 'c', 'd'];
         $reflector = new \ReflectionClass('\Ds\Cache\Cache');
         $method = $reflector->getMethod('_isTraversable');
         $method->setAccessible(true);
@@ -432,7 +432,7 @@ 
     /**
      * Test that string is not Traversable
      */
-    public function testIsNotTraversable(){
+    public function testIsNotTraversable() {
         $this->expectException(InvalidArgumentException::class);
         $data = 'some-random-string';
         $reflector = new \ReflectionClass('\Ds\Cache\Cache');
@@ -444,7 +444,7 @@ 
     /**
      * Test that instance of Iterator is traversable
      */
-    public function testIsTraversableIterator(){
+    public function testIsTraversableIterator() {
         $iterator = new IteratorMock();
         $reflector = new \ReflectionClass('\Ds\Cache\Cache');
         $method = $reflector->getMethod('_isTraversable');

Tests/Cache/Mock/IteratorMock.php

Spacing +6 added lines, -6 removed lines

@@ -9,10 +9,10 @@
  *
  * @package Tests\Cache\Mock
  */
-class IteratorMock implements \Iterator{
-    public function current(){}
-    public function next(){}
-    public function key(){}
-    public function valid(){}
-    public function rewind(){}
+class IteratorMock implements \Iterator {
+    public function current() {}
+    public function next() {}
+    public function key() {}
+    public function valid() {}
+    public function rewind() {}
 }

Tests/Cache/Mock/AbstractStorageMock.php

Spacing +6 added lines, -6 removed lines

@@ -11,10 +11,10 @@
  *
  * @package Tests\Cache\Mock
  */
-class AbstractStorageMock extends AbstractStorage{
-    public function set($key, $value, $ttl = null){}
-    public function has($key){}
-    public function get($key){}
-    public function delete($key){}
-    public function clear(){}
+class AbstractStorageMock extends AbstractStorage {
+    public function set($key, $value, $ttl = null) {}
+    public function has($key) {}
+    public function get($key) {}
+    public function delete($key) {}
+    public function clear() {}
 }
\ No newline at end of file