From 1e0e870e18e8be6b35fd1ce374fc04c5f024bed5 Mon Sep 17 00:00:00 2001 From: Bjoern Schiessle Date: Wed, 13 Aug 2014 11:41:11 +0200 Subject: [PATCH] don't clean up the test environment if a data provider was finished --- tests/testcleanuplistener.php | 31 +++++++++++++++++-------------- 1 file changed, 17 insertions(+), 14 deletions(-) diff --git a/tests/testcleanuplistener.php b/tests/testcleanuplistener.php index 8a9e9e6efb..7b442bbd4f 100644 --- a/tests/testcleanuplistener.php +++ b/tests/testcleanuplistener.php @@ -41,20 +41,23 @@ class TestCleanupListener implements PHPUnit_Framework_TestListener { } public function endTestSuite(PHPUnit_Framework_TestSuite $suite) { - if ($this->cleanStorages() && $this->isShowSuiteWarning()) { - printf("TestSuite '%s': Did not clean up storages\n", $suite->getName()); - } - if ($this->cleanFileCache() && $this->isShowSuiteWarning()) { - printf("TestSuite '%s': Did not clean up file cache\n", $suite->getName()); - } - if ($this->cleanStrayDataFiles() && $this->isShowSuiteWarning()) { - printf("TestSuite '%s': Did not clean up data dir\n", $suite->getName()); - } - if ($this->cleanStrayHooks() && $this->isShowSuiteWarning()) { - printf("TestSuite '%s': Did not clean up hooks\n", $suite->getName()); - } - if ($this->cleanProxies() && $this->isShowSuiteWarning()) { - printf("TestSuite '%s': Did not clean up proxies\n", $suite->getName()); + // don't clean up the test environment if a data provider finished + if (!($suite instanceof PHPUnit_Framework_TestSuite_DataProvider)) { + if ($this->cleanStorages() && $this->isShowSuiteWarning()) { + printf("TestSuite '%s': Did not clean up storages\n", $suite->getName()); + } + if ($this->cleanFileCache() && $this->isShowSuiteWarning()) { + printf("TestSuite '%s': Did not clean up file cache\n", $suite->getName()); + } + if ($this->cleanStrayDataFiles() && $this->isShowSuiteWarning()) { + printf("TestSuite '%s': Did not clean up data dir\n", $suite->getName()); + } + if ($this->cleanStrayHooks() && $this->isShowSuiteWarning()) { + printf("TestSuite '%s': Did not clean up hooks\n", $suite->getName()); + } + if ($this->cleanProxies() && $this->isShowSuiteWarning()) { + printf("TestSuite '%s': Did not clean up proxies\n", $suite->getName()); + } } }