djsmithme / Cache

Src/Cache/Cache.php

Spacing +16 added lines, -16 removed lines

@@ -37,13 +37,13 @@ 
     public function set($key, $value, $ttl = null)
     {
         //convert to timestamp from now.
-        if ($ttl instanceof \DateInterval){
+        if ($ttl instanceof \DateInterval) {
             $dateTime = new \DateTime();
-            $dateTime->add( $ttl );
+            $dateTime->add($ttl);
             $ttl = $dateTime->getTimestamp() - time();
         }
 
-        if (!is_int($ttl) ||  $ttl === null){
+        if (!is_int($ttl) || $ttl === null) {
             throw new InvalidArgumentException('$ttl can only be an instance of \DateInterval, int or null');
         }
 
@@ -86,7 +86,7 @@ 
     {
         $this->_isValidKey($key);
 
-        if ($this->cache->has($key)){
+        if ($this->cache->has($key)) {
             return $this->cache->get($key);
         }
 
@@ -127,7 +127,7 @@ 
         $this->_isTraversable($keys);
         $result = [];
 
-        foreach ((array)$keys as $key){
+        foreach ((array)$keys as $key) {
             $cachedItem = $this->cache->get($key);
             $result[$key] = (null !== $cachedItem) ? $cachedItem : $default;
         }
@@ -155,11 +155,11 @@ 
         $this->_isTraversable($values);
         $results = [];
 
-        foreach ((array)$values as $key => $value){
+        foreach ((array)$values as $key => $value) {
             $results[] = $this->cache->set($key, $value, $ttl);
         }
 
-        if ($this->_hasFailure($results)){
+        if ($this->_hasFailure($results)) {
             return false;
         }
 
@@ -182,11 +182,11 @@ 
         $this->_isTraversable($keys);
         $results = [];
 
-        foreach ((array)$keys as $key){
+        foreach ((array)$keys as $key) {
             $results[] = $this->cache->delete($key);
         }
 
-        if ($this->_hasFailure($results)){
+        if ($this->_hasFailure($results)) {
             return false;
         }
 
@@ -209,8 +209,8 @@ 
      * @param $key
      * @throws InvalidArgumentException
      */
-    private function _isValidKey($key){
-        if (!is_string($key)){
+    private function _isValidKey($key) {
+        if (!is_string($key)) {
             throw new InvalidArgumentException('provided key must be a valid string');
         }
     }
@@ -223,9 +223,9 @@ 
      * @return bool
      * @internal
      */
-    private function _isTraversable($data){
+    private function _isTraversable($data) {
 
-        if (is_array($data) || $data instanceof \Traversable){
+        if (is_array($data) || $data instanceof \Traversable) {
             return true;
         }
 
@@ -239,7 +239,7 @@ 
      */
     private function _hasKeys(array $arr)
     {
-        if (array() === $arr || array_keys($arr) === range(0, count($arr) - 1)){
+        if (array() === $arr || array_keys($arr) === range(0, count($arr) - 1)) {
             throw new InvalidArgumentException('Keys missing');
         }
 
@@ -253,8 +253,8 @@ 
      * @return bool
      * @internal
      */
-    private function _hasFailure(array $results){
-        if (in_array(false, $results)){
+    private function _hasFailure(array $results) {
+        if (in_array(false, $results)) {
             return true;
         }
         return false;

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

Tests/Cache/CachePrivateTest.php

Spacing +12 added lines, -12 removed lines

@@ -36,7 +36,7 @@ 
     /**
      * Test that array has keys.
      */
-    public function testArrayHasKeys(){
+    public function testArrayHasKeys() {
         $data = ['a' => 'foo', 'b' => 'bar'];
         $method = $this->getCacheMethod('_hasKeys');
         $actual = $method->invokeArgs($this->cache, [$data]);
@@ -46,9 +46,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'];
         $method = $this->getCacheMethod('_hasKeys');
         $method->invokeArgs($this->cache, [$data]);
     }
@@ -56,8 +56,8 @@ 
     /**
      * Test array contains a failure.
      */
-    public function testHasFailure(){
-        $results = [true,true,true,false,true];
+    public function testHasFailure() {
+        $results = [true, true, true, false, true];
         $method = $this->getCacheMethod('_hasFailure');
         $actual = $method->invokeArgs($this->cache, [$results]);
         $expected = true;
@@ -67,8 +67,8 @@ 
     /**
      *
      */
-    public function testHasNoFailure(){
-        $results = [true,true,true,true,true];
+    public function testHasNoFailure() {
+        $results = [true, true, true, true, true];
         $method = $this->getCacheMethod('_hasFailure');
         $actual = $method->invokeArgs($this->cache, [$results]);
         $expected = false;
@@ -78,8 +78,8 @@ 
     /**
      * Test that array is Traversable
      */
-    public function testIsTraversable(){
-        $data = ['a','b','c','d'];
+    public function testIsTraversable() {
+        $data = ['a', 'b', 'c', 'd'];
         $method = $this->getCacheMethod('_isTraversable');
         $actual = $method->invokeArgs($this->cache, [$data]);
         $expected = true;
@@ -89,7 +89,7 @@ 
     /**
      * Test that string is not Traversable
      */
-    public function testIsNotTraversable(){
+    public function testIsNotTraversable() {
         $this->expectException(InvalidArgumentException::class);
         $data = 'some-random-string';
         $method = $this->getCacheMethod('_isTraversable');
@@ -99,7 +99,7 @@ 
     /**
      * Test that instance of Iterator is traversable
      */
-    public function testIsTraversableIterator(){
+    public function testIsTraversableIterator() {
         $iterator = new IteratorMock();
         $method = $this->getCacheMethod('_isTraversable');
         $actual = $method->invokeArgs($this->cache, [$iterator]);
@@ -112,7 +112,7 @@ 
      * @param $method
      * @return \ReflectionMethod
      */
-    private function getCacheMethod($method){
+    private function getCacheMethod($method) {
         $reflector = new \ReflectionClass('\Ds\Cache\Cache');
         $method = $reflector->getMethod($method);
         $method->setAccessible(true);

Tests/Cache/CacheTest.php

Spacing +18 added lines, -18 removed lines

@@ -39,7 +39,7 @@ 
     {
         $key = 21021000;
         $this->expectException(InvalidArgumentException::class);
-        $this->cache->set($key,'value');
+        $this->cache->set($key, 'value');
     }
 
     /**
@@ -205,7 +205,7 @@ 
     /**
      *
      */
-    public function testGetMultiple(){
+    public function testGetMultiple() {
 
         $expected = [
             'foo' => 'fooValue',
@@ -215,7 +215,7 @@ 
 
         $i = 0;
 
-        foreach ($expected as $key => $value){
+        foreach ($expected as $key => $value) {
 
             $this->storageMock->expects($this->at($i))
                 ->method('get')
@@ -235,7 +235,7 @@ 
     /**
      *
      */
-    public function testSetMultiple(){
+    public function testSetMultiple() {
 
         $keys = [
             'foo' => 'fooValue',
@@ -249,7 +249,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(
@@ -261,14 +261,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',
@@ -282,11 +282,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;
             }
 
@@ -301,20 +301,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(
@@ -330,12 +330,12 @@ 
     /**
      *
      */
-    public function testDeleteMultipleFailure(){
-        $keys = ['foo','bar','baz'];
+    public function testDeleteMultipleFailure() {
+        $keys = ['foo', 'bar', 'baz'];
         $expected = false;
         $deleteStatus = true;
-        foreach ($keys as $i => $key){
-            if ($i === 1){
+        foreach ($keys as $i => $key) {
+            if ($i === 1) {
                 $deleteStatus = false;
             }
             $this->storageMock->expects($this->at($i))
@@ -352,7 +352,7 @@ 
     /**
      * Test that storage clear is called.
      */
-    public function testclear(){
+    public function testclear() {
         $this->storageMock
             ->expects($this->once())
             ->method('clear');

Tests/Cache/Storage/FileStorageTest.php

Spacing +25 added lines, -25 removed lines

@@ -29,8 +29,8 @@ 
      */
     public function setUp()
     {
-        $this->testDir = __DIR__ . '/cacheTest';
-        $this->cacheStorage = new FileStorage( $this->testDir, new \DateInterval('P1M'));
+        $this->testDir = __DIR__.'/cacheTest';
+        $this->cacheStorage = new FileStorage($this->testDir, new \DateInterval('P1M'));
     }
 
     /**
@@ -45,7 +45,7 @@ 
     /**
      * Test that CacheException is thrown if directory is not valid/writable.
      */
-    public function testUnwritableDirectory(){
+    public function testUnwritableDirectory() {
 
         $this->expectException(CacheException::class);
         $this->cacheStorage = new FileStorage('php://memory', new \DateInterval('P1M'));
@@ -54,68 +54,68 @@ 
     /**
      * Test that set is called.
      */
-    public function testSet(){
-        $actual = $this->cacheStorage->set('key','value',60*60);
+    public function testSet() {
+        $actual = $this->cacheStorage->set('key', 'value', 60 * 60);
         $this->assertEquals(true, $actual);
     }
 
     /**
      * Test has when not value is found.
      */
-    public function testHasNoValue(){
+    public function testHasNoValue() {
         $this->assertEquals($this->cacheStorage->has('someRandomKey'), false);
     }
 
     /**
      * Test has when value is found.
      */
-    public function testHas(){
-        $this->cacheStorage->set('foo','bar');
+    public function testHas() {
+        $this->cacheStorage->set('foo', 'bar');
         $this->assertEquals($this->cacheStorage->has('foo'), true);
     }
 
     /**
      * Test has when value is found but has expired.
      */
-    public function testHasExpired(){
-        $this->cacheStorage->set('expired','bar', -1200);
+    public function testHasExpired() {
+        $this->cacheStorage->set('expired', 'bar', -1200);
         $this->assertEquals($this->cacheStorage->has('expired'), false);
     }
 
     /**
      * Test that clear is called.
      */
-    public function testClear(){
+    public function testClear() {
        $this->assertEquals($this->cacheStorage->clear(), true);
     }
 
     /**
      * Test that get returns a found key.
      */
-    public function testGet(){
+    public function testGet() {
         $expected = 'bat';
-        $this->cacheStorage->set('baz',$expected);
+        $this->cacheStorage->set('baz', $expected);
         $actual = $this->cacheStorage->get('baz');
         $this->assertEquals($expected, $actual);
     }
 
-    public function testDelete(){
+    public function testDelete() {
         $this->cacheStorage->delete('baz');
         $this->assertEquals($this->cacheStorage->has('baz'), false);
     }
 
-    public function testCreateCacheItem(){
+    public function testCreateCacheItem() {
 
         $reflector = new \ReflectionClass(FileStorage::class);
         $method = $reflector->getMethod('_createCacheItem');
         $method->setAccessible(true);
 
-        $key= 'my-key_+';
+        $key = 'my-key_+';
         $value = 'my-value';
         $ttl = 60 * 60 * 7;
 
         $expected = [
-            'file' => $this->testDir . DIRECTORY_SEPARATOR . implode('.',[$key, FileStorage::EXTENSION]),
+            'file' => $this->testDir.DIRECTORY_SEPARATOR.implode('.', [$key, FileStorage::EXTENSION]),
             'data' => json_encode([$ttl, $value])
         ];
 
@@ -123,7 +123,7 @@ 
         $this->assertEquals($expected, $actual);
     }
 
-    public function testCreateCacheItemInvalidKey(){
+    public function testCreateCacheItemInvalidKey() {
 
         $this->expectException(InvalidArgumentException::class);
         $reflector = new \ReflectionClass(FileStorage::class);
@@ -135,18 +135,18 @@ 
         $method->invokeArgs($this->cacheStorage, [$key, $value, $ttl]);
     }
 
-    public function testFetchCacheNoItem(){
+    public function testFetchCacheNoItem() {
         $reflector = new \ReflectionClass(FileStorage::class);
         $method = $reflector->getMethod('_fetchCacheFile');
         $method->setAccessible(true);
-        $key= 'unknown';
+        $key = 'unknown';
         $actual = $method->invokeArgs($this->cacheStorage, [$key]);
         $this->assertEquals(false, $actual);
     }
 
-    public function testFetchCacheValid(){
+    public function testFetchCacheValid() {
 
-        $key= 'my-item';
+        $key = 'my-item';
         $ttl = 60 * 60;
         $value = 'some-value';
 
@@ -164,8 +164,8 @@ 
         $this->assertEquals($expected, $actual);
     }
 
-    public function testFetchCacheExpired(){
-        $key= 'my-item';
+    public function testFetchCacheExpired() {
+        $key = 'my-item';
         $ttl = -1500;
         $value = 'some-value';
         $this->cacheStorage->set($key, $value, $ttl);
@@ -183,7 +183,7 @@ 
     {
         $dir = $this->testDir;
 
-        if (is_dir( $dir)){
+        if (is_dir($dir)) {
             array_map('unlink', glob("$dir/*.*"));
         }
         \rmdir($dir);