[#ZF-9188] Failures in Zend_Cache_Manager - Zend Framework Issue Tracker

Details

Type: Unit Tests: Problem
Status: Resolved
Priority: Major
Resolution: Fixed
Affects Version/s: 1.10.1, 1.10.2, 1.10.3
Fix Version/s: 1.10.5
Component/s: Zend_Cache
Labels:
None

Description

Failures

PHPUnit 3.4.5 by Sebastian Bergmann.

.EEEE......E

Time: 16 seconds, Memory: 6.50Mb

There were 5 errors:

1) Zend_Cache_ManagerTest::testLazyLoadsDefaultPageCache
Zend_Cache_Exception: A cache configuration templatedoes not exist with the name "tagCache"

/var/www/ZendFramework-trunk/library/Zend/Cache/Manager.php:324

2) Zend_Cache_ManagerTest::testCanOverrideCacheFrontendNameConfiguration
Zend_Cache_Exception: A cache configuration templatedoes not exist with the name "tagCache"

/var/www/ZendFramework-trunk/library/Zend/Cache/Manager.php:324

3) Zend_Cache_ManagerTest::testCanMergeTemplateCacheOptionsFromZendConfig
Zend_Cache_Exception: A cache configuration templatedoes not exist with the name "tagCache"

/var/www/ZendFramework-trunk/library/Zend/Cache/Manager.php:324

4) Zend_Cache_ManagerTest::testCanOverrideCacheBackendendNameConfiguration
Zend_Cache_Exception: A cache configuration templatedoes not exist with the name "tagCache"

/var/www/ZendFramework-trunk/library/Zend/Cache/Manager.php:324

5) Zend_Cache_ManagerTest::testGettingPageCacheAlsoCreatesTagCache
Zend_Cache_Exception: cache_dir must be a directory

/var/www/ZendFramework-trunk/library/Zend/Cache.php:208
/var/www/ZendFramework-trunk/library/Zend/Cache/Backend/File.php:154
/var/www/ZendFramework-trunk/library/Zend/Cache/Backend/File.php:121
/var/www/ZendFramework-trunk/library/Zend/Cache.php:152
/var/www/ZendFramework-trunk/library/Zend/Cache.php:93
/var/www/ZendFramework-trunk/library/Zend/Cache/Manager.php:230
/var/www/ZendFramework-trunk/library/Zend/Cache/Manager.php:223

FAILURES!
Tests: 12, Assertions: 7, Errors: 5.

Changes made in Zend_Cache_Manager should be reflected in tests units.

More Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Download All

Attachments

Zend_Cache_ManagerTest.patch

16/Feb/10 4:12 AM

3 kB

Ramon Henrique Ornelas

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Ramon Henrique Ornelas added a comment - 16/Feb/10 4:13 AM

Attached the patch of the correction.

Show

Ramon Henrique Ornelas added a comment - 16/Feb/10 4:13 AM Attached the patch of the correction.

Hide

Permalink

Pádraic Brady added a comment - 28/Apr/10 9:12 AM

Fixed previously? Just checked and cannot reproduce any of the detailed errors.

Show

Pádraic Brady added a comment - 28/Apr/10 9:12 AM Fixed previously? Just checked and cannot reproduce any of the detailed errors.

Hide

Permalink

Ramon Henrique Ornelas added a comment - 28/Apr/10 7:52 PM

@Padraic

Changes made in revision r20602 in class Zend_Cache_Manager::$_optionTemplates option tagcache renamed to pagetag, in test units Zend_Cache_ManagerTest makes use tagcache.

Other problem in revision r20582 in Zend_Cache_Manager::$_optionTemplates['page']['name'] changed to Capture in test units Zend_Cache_ManagerTest in line 73 current code

$this->assertTrue($manager->getCache('page') instanceof Zend_Cache_Frontend_Output);

Should be

$this->assertTrue($manager->getCache('page') instanceof Zend_Cache_Frontend_Capture);

Thanks

Show

Ramon Henrique Ornelas added a comment - 28/Apr/10 7:52 PM @Padraic Changes made in revision r20602 in class Zend_Cache_Manager::$_optionTemplates option tagcache renamed to pagetag, in test units Zend_Cache_ManagerTest makes use tagcache. Other problem in revision r20582 in Zend_Cache_Manager::$_optionTemplates['page']['name'] changed to Capture in test units Zend_Cache_ManagerTest in line 73 current code

$this->assertTrue($manager->getCache('page') instanceof Zend_Cache_Frontend_Output);

Should be

$this->assertTrue($manager->getCache('page') instanceof Zend_Cache_Frontend_Capture);

Thanks

Hide

Permalink

Pádraic Brady added a comment - 29/Apr/10 8:05 AM

ManagerTest has been omitted from AllTests - reopening for resolution shortly.

Show

Pádraic Brady added a comment - 29/Apr/10 8:05 AM ManagerTest has been omitted from AllTests - reopening for resolution shortly.

Hide

Permalink

Pádraic Brady added a comment - 29/Apr/10 8:14 AM

Committed patch as r22056. 1 remaining error to resolve concerning a non-existing "frontend" index.

Show

Pádraic Brady added a comment - 29/Apr/10 8:14 AM Committed patch as r22056. 1 remaining error to resolve concerning a non-existing "frontend" index.

Hide

Permalink

Pádraic Brady added a comment - 29/Apr/10 8:20 AM

If it's not resolved now, I'll eat my hat. Not that I have a hat .

Show

Pádraic Brady added a comment - 29/Apr/10 8:20 AM If it's not resolved now, I'll eat my hat. Not that I have a hat

Hide

Permalink

Ramon Henrique Ornelas added a comment - 29/Apr/10 10:15 AM

Thanks @PadraicB .

Show

Ramon Henrique Ornelas added a comment - 29/Apr/10 10:15 AM Thanks @PadraicB

People

Assignee:

Pádraic Brady

Reporter:

Ramon Henrique Ornelas

Vote (0)

Watch (1)

Dates

Created:

16/Feb/10 4:03 AM

Updated:

29/Apr/10 10:15 AM

Resolved:

29/Apr/10 8:20 AM