diff options
author | diosmosis <benaka@piwik.pro> | 2015-11-23 16:53:36 +0300 |
---|---|---|
committer | diosmosis <benaka@piwik.pro> | 2015-11-23 16:53:36 +0300 |
commit | 4c722433777bd84bd603419ed04667fe82c22f62 (patch) | |
tree | a17502cbe524ae3f1dcaae24b5259008a10c903f /tests/PHPUnit/Fixtures | |
parent | bf5bd787aad9f0a3a1f7cacead62590fe599fa90 (diff) |
Fixing issues causing omnifixture setup to fail.
Diffstat (limited to 'tests/PHPUnit/Fixtures')
4 files changed, 29 insertions, 7 deletions
diff --git a/tests/PHPUnit/Fixtures/ManySitesImportedLogsWithXssAttempts.php b/tests/PHPUnit/Fixtures/ManySitesImportedLogsWithXssAttempts.php index b516c85d65..227e45bd4a 100644 --- a/tests/PHPUnit/Fixtures/ManySitesImportedLogsWithXssAttempts.php +++ b/tests/PHPUnit/Fixtures/ManySitesImportedLogsWithXssAttempts.php @@ -57,6 +57,11 @@ class ManySitesImportedLogsWithXssAttempts extends ManySitesImportedLogs self::createWebsite($this->dateTime, $ecommerce = 0, $siteName = 'Piwik test two', $siteUrl = 'http://example-site-two.com'); } + + if (!self::siteCreated($idSite = 3)) { + self::createWebsite($this->dateTime, $ecommerce = 0, $siteName = 'Piwik test three', + $siteUrl = 'http://example-site-three.com'); + } } public function addAnnotations() diff --git a/tests/PHPUnit/Fixtures/ManyVisitsWithGeoIP.php b/tests/PHPUnit/Fixtures/ManyVisitsWithGeoIP.php index c413635c8e..ce454b38d2 100644 --- a/tests/PHPUnit/Fixtures/ManyVisitsWithGeoIP.php +++ b/tests/PHPUnit/Fixtures/ManyVisitsWithGeoIP.php @@ -265,13 +265,6 @@ class ManyVisitsWithGeoIP extends Fixture public static function unsetLocationProvider() { - // also fails on other PHP, is it really needed? - return; - - // this randomly fails on PHP 5.3 - if(strpos(PHP_VERSION, '5.3') === 0) { - return; - } try { LocationProvider::setCurrentProvider('default'); } catch(Exception $e) { diff --git a/tests/PHPUnit/Fixtures/ManyVisitsWithMockLocationProvider.php b/tests/PHPUnit/Fixtures/ManyVisitsWithMockLocationProvider.php index dd2a2c773c..9fbd6f986d 100644 --- a/tests/PHPUnit/Fixtures/ManyVisitsWithMockLocationProvider.php +++ b/tests/PHPUnit/Fixtures/ManyVisitsWithMockLocationProvider.php @@ -33,6 +33,8 @@ class ManyVisitsWithMockLocationProvider extends Fixture $this->setUpWebsitesAndGoals(); $this->setMockLocationProvider(); $this->trackVisits(); + + ManyVisitsWithGeoIP::unsetLocationProvider(); } public function tearDown() diff --git a/tests/PHPUnit/Fixtures/OmniFixture.php b/tests/PHPUnit/Fixtures/OmniFixture.php index 123e214bd3..3c08313018 100644 --- a/tests/PHPUnit/Fixtures/OmniFixture.php +++ b/tests/PHPUnit/Fixtures/OmniFixture.php @@ -32,11 +32,29 @@ class OmniFixture extends Fixture public $fixtures = array(); + private function requireAllFixtures() + { + $fixturesToLoad = array( + '/tests/PHPUnit/Fixtures/*.php', + '/tests/UI/Fixtures/*.php', + '/plugins/*/tests/Fixtures/*.php', + '/plugins/*/Test/Fixtures/*.php', + ); + + foreach($fixturesToLoad as $fixturePath) { + foreach (glob(PIWIK_INCLUDE_PATH . $fixturePath) as $file) { + require_once $file; + } + } + } + /** * Constructor. */ public function __construct() { + $this->requireAllFixtures(); + $date = $this->month . '-01'; $classes = get_declared_classes(); @@ -94,6 +112,8 @@ class OmniFixture extends Fixture public function setUp() { foreach ($this->fixtures as $fixture) { + echo "Setting up " . get_class($fixture) . "...\n"; + $fixture->setUp(); } @@ -108,6 +128,8 @@ class OmniFixture extends Fixture public function tearDown() { foreach ($this->fixtures as $fixture) { + echo "Tearing down " . get_class($fixture) . "...\n"; + $fixture->tearDown(); } } |