2.0 cakeplugin tests #87

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

sitedyno commented May 19, 2011

Keep application plugins out of core tests.

sitedyno added some commits May 19, 2011

@sitedyno sitedyno Move CakePlugin::unload() to setUp.
If the tests are run in an app with plugins the first test would fail
due to the application's plugins showing in CakePlugin::loaded().
c987bbc
@sitedyno sitedyno Update the object cache.
Keep application plugins out of core tests, otherwise tests will fail.
be1e379
Owner

lorenzo commented May 19, 2011

Can you submit again the patch but also leaving the unload() call in tearDown() ? that way you won't contaminate the other tests when running the complete testsuite

Contributor

sitedyno commented May 19, 2011

Ok, but I will add CakePlugin::unload() to CakePluginTest::testLoadSingle(), otherwise it will fail.

sitedyno closed this May 19, 2011

Owner

lorenzo commented May 19, 2011

Ok, good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment