bug fix for issue #232

Author: nyamsprod

src/Cache.php

@@ -18,6 +18,7 @@
 use DateInterval;
 use FilesystemIterator;
 use Generator;
+use InvalidArgumentException;
 use Psr\SimpleCache\CacheInterface;
 use Traversable;
 use function chmod;
@@ -88,6 +89,10 @@ public function __construct(string $cache_path = '')
             $this->mkdir($cache_path); // ensure that the parent path exists
         }
 
+        if (! is_writable($cache_path.DIRECTORY_SEPARATOR)) {
+            throw new InvalidArgumentException(sprintf('cache path does not exist or is not writable: %s', $cache_path));
+        }
+
         $this->cache_path = $cache_path;
     }
 

tests/CacheTest.php

@@ -69,8 +69,14 @@ public function testConstructorOnParentCachePathIsNotExisted()
 
     public function testSetOnNotWritableCachePath()
     {
+        self::expectException(\InvalidArgumentException::class);
         $cache = new Cache('/bin');
-        self::assertFalse($cache->set('key', 'value'));
+    }
+
+    public function testSetOnNotExistingCachePath()
+    {
+        self::expectException(\InvalidArgumentException::class);
+        $cache = new Cache('/foo/bar');
     }
 
     /**