serialize data before writing it to redis

Makefile

@@ -25,3 +25,6 @@ tests: vendor
 
 coverage: vendor
 	vendor/bin/tester -s -p phpdbg --colors 1 -C --coverage ./coverage.xml --coverage-src ./src tests/cases
+
+coverage-html:
+	vendor/bin/tester -s -p phpdbg --colors 1 -C --coverage ./coverage.html --coverage-src ./src tests/cases

src/Caching/RedisStorage.php

@@ -5,7 +5,6 @@
 use Nette\Caching\Cache;
 use Nette\Caching\IStorage;
 use Predis\Client;
-use Throwable;
 
 final class RedisStorage implements IStorage
 {
@@ -29,12 +28,12 @@ public function getClient(): Client
 	 */
 	public function write(string $key, $data, array $dependencies): void
 	{
-		$this->client->set($key, $data);
+		$this->client->set($key, serialize($data));
 
 		if (isset($dependencies[Cache::EXPIRATION])) {
 			$expiration = (int) $dependencies[Cache::EXPIRATION];
 
-			if ($dependencies[Cache::SLIDING] !== true) {
+			if (!isset($dependencies[Cache::SLIDING]) || $dependencies[Cache::SLIDING] !== true) {
 				$this->client->expireat($key, time() + $expiration);
 			} else {
 				$this->client->expire($key, $expiration);
@@ -44,17 +43,22 @@ public function write(string $key, $data, array $dependencies): void
 
 	/**
 	 * @return mixed
-	 * @phpcsSuppress SlevomatCodingStandard.TypeHints.TypeHintDeclaration.MissingParameterTypeHint
 	 */
 	public function read(string $key)
 	{
-		$val = $this->client->get($key);
+		$serializedValue = $this->client->get($key);
 
-		try {
-			return $val;
-		} catch (Throwable $e) {
+		if ($serializedValue === null) {
 			return null;
 		}
+
+		if ($serializedValue === 'b:0;') {
+			return false;
+		}
+
+		$value = @unserialize($serializedValue);
+
+		return $value === false ? null : $value;
 	}
 
 	public function lock(string $key): void

tests/cases/RedisStorage.phpt

@@ -12,7 +12,7 @@ use Tester\Assert;
 require_once __DIR__ . '/../bootstrap.php';
 
 test(function (): void {
-	$storage = (object) [];
+	$storage = (object) ['unserialized' => 'unserialized'];
 
 	$conn = Mockery::mock(ConnectionInterface::class)
 		->shouldReceive('executeCommand')
@@ -36,4 +36,8 @@ test(function (): void {
 	Assert::same('bat', $redis->read('foo'));
 	$redis->remove('foo');
 	Assert::null($redis->read('foo'));
+	Assert::null($redis->read('unserialized'));
+
+	$redis->write('false', false, []);
+	Assert::false($redis->read('false'));
 });