wpstudio/majestic
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Correction TODO comments and remove them

Browse Source
This commit is contained in:
Vyacheslav Agafonov
2011-12-01 18:58:15 +04:00
parent bafe38ccf0
commit d7bae53911
2 changed files with 0 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tests/cache/MemcacheCacheTest.php vendored
View File

@ -36,14 +36,10 @@ class MemcacheCacheTest extends PHPUnit_Framework_TestCase
$this->assertAttributeInstanceOf('TestCache', 'connection', $memcache); $this->assertAttributeInstanceOf('TestCache', 'connection', $memcache);
} }
/**
* @TODO: MemcacheCache::__construct - empty config array passes with no host params
*/
public function testWrongConfig() public function testWrongConfig()
{ {
$config = array('some' => array()); $config = array('some' => array());
$this->setExpectedException('InitializationException', 'Configuration must have a'); $this->setExpectedException('InitializationException', 'Configuration must have a');
$memcache = new MemcacheCache($config); $memcache = new MemcacheCache($config);
} }

3
tests/redis/RedisManagerTest.php
View File

@ -42,9 +42,6 @@ class RedisManagerTest extends PHPUnit_Framework_TestCase
set_new_overload(array($this, 'newCallback')); set_new_overload(array($this, 'newCallback'));
} }
/**
* @TODO: line 34: $config = Config::get('Redis')->$name; - check for Redis config existence. Agafonov: 'check added'
*/
public function testConnectNoAnyConfig() public function testConnectNoAnyConfig()
{ {
$this->setExpectedException('GeneralException', 'Redis config no existence'); $this->setExpectedException('GeneralException', 'Redis config no existence');