Skip to content

Commit

Permalink
Merge pull request #119 from Jakub-Fajkus/fix-configuration-problems
Browse files Browse the repository at this point in the history
Fix configuration errors
  • Loading branch information
Spamercz authored Apr 4, 2022
2 parents 5a28197 + 9880b1a commit b67ac20
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
2 changes: 0 additions & 2 deletions src/Kdyby/Redis/DI/Config/SessionClientSchema.php
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,6 @@ public function complete($value, \Nette\Schema\Context $context)
$value = $this->expandParameters($value);
}

$value = $this->getSchema()->complete($value, $context);

return $value;
}

Expand Down
5 changes: 5 additions & 0 deletions tests/KdybyTests/Redis/ExtensionTest.phpt
Original file line number Diff line number Diff line change
Expand Up @@ -139,6 +139,11 @@ class ExtensionTest extends \Tester\TestCase
$config->addConfig(__DIR__ . '/files/session.neon');
$dic = $config->createContainer();
Assert::true($dic->getService('redis.client') instanceof Kdyby\Redis\RedisClient);

/** @var \Kdyby\Redis\RedisClient $client */
$client = $dic->getService('redis.sessionHandler_client');
Assert::same(1, $client->getDatabase());

Assert::true($dic->hasService('redis.sessionHandler'));
Assert::type(\Kdyby\Redis\RedisSessionHandler::class, $dic->getService('redis.sessionHandler'));
Assert::false($dic->hasService('redis.cacheJournal'));
Expand Down

0 comments on commit b67ac20

Please sign in to comment.