From a9033a0dbfaf11e2e4f79a05e9a34e3e2af6679a Mon Sep 17 00:00:00 2001 From: diosmosis <benaka@piwik.pro> Date: Fri, 25 Dec 2015 05:50:58 -0800 Subject: [PATCH] Remove change to site settings in InvalidVisits.php after it is no longer needed so when fixtures are setup together, the change won't carry over into other fixtures. --- tests/PHPUnit/Fixtures/InvalidVisits.php | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/tests/PHPUnit/Fixtures/InvalidVisits.php b/tests/PHPUnit/Fixtures/InvalidVisits.php index 5518a7a163..da1038eba2 100644 --- a/tests/PHPUnit/Fixtures/InvalidVisits.php +++ b/tests/PHPUnit/Fixtures/InvalidVisits.php @@ -112,6 +112,12 @@ class InvalidVisits extends Fixture $t->setUrl("http://their.stuff.com/back/to/the/future"); $t->doTrackPageView("ignored, not from my.stuff.com"); + // undo exclude unknown urls change (important when multiple fixtures are setup together, as is done in OmniFixture) + API::getInstance()->updateSite($idSite, $siteName = null, $urls, $ecommerce = null, $siteSearch = null, + $searchKeywordParameters = null, $searchCategoryParameters = null, $excludedIps = null, $excludedQueryParams = null, + $timezone = null, $currency = null, $group = null, $startDate = null, $excludedUserAgents = null, + $keepUrlFragments = null, $type = null, $settings = null, $excludeUnknownUrls = 0); + try { @$t->setAttributionInfo(array()); self::fail(); -- GitLab