diff --git a/tests/PHPUnit/Integration/Menu/MenuReportingTest.php b/tests/PHPUnit/Integration/Menu/MenuReportingTest.php index 44001094856ed5b29de50c0572c0b0a4c55a7709..a9d06afa13dcb8dda47bbd985dc9454516160daa 100644 --- a/tests/PHPUnit/Integration/Menu/MenuReportingTest.php +++ b/tests/PHPUnit/Integration/Menu/MenuReportingTest.php @@ -27,6 +27,8 @@ class ReportingTest extends IntegrationTestCase public function setUp() { + parent::setUp(); + PluginManager::getInstance()->unloadPlugins(); $this->menu = MenuReporting::getInstance(); } diff --git a/tests/PHPUnit/Integration/Tracker/RequestTest.php b/tests/PHPUnit/Integration/Tracker/RequestTest.php index 32a00aedd50893f8ee5b295aa9ad81a7e838cb1f..f6fd1f48cd17ca6964818dc0ddad5e6484d5d421 100644 --- a/tests/PHPUnit/Integration/Tracker/RequestTest.php +++ b/tests/PHPUnit/Integration/Tracker/RequestTest.php @@ -19,15 +19,6 @@ use Piwik\Tracker\Request; use Piwik\Tests\Framework\TestCase\IntegrationTestCase; use Piwik\Tracker\TrackerConfig; -class TestRequest extends Request { - - public function setIsAuthenticated() - { - $this->isAuthenticated = true; - } - -} - /** * @group RequestTest * @group Request @@ -335,6 +326,19 @@ class RequestTest extends IntegrationTestCase $this->buildCustomVars(array('key' => 'val', 'key2' => 'val2'))); } + public function test_getIdSite_shouldTriggerEventAndReturnThatIdSite() + { + $self = $this; + Piwik::addAction('Tracker.Request.getIdSite', function (&$idSite, $params) use ($self) { + $self->assertSame(14, $idSite); + $self->assertEquals(array('idsite' => '14'), $params); + $idSite = 12; + }); + + $request = $this->buildRequest(array('idsite' => '14')); + $this->assertSame(12, $request->getIdSite()); + } + private function assertCustomVariablesInVisitScope($expectedCvars, $cvarsJsonEncoded) { $request = $this->buildRequest(array('_cvar' => $cvarsJsonEncoded)); @@ -384,3 +388,11 @@ class RequestTest extends IntegrationTestCase return new TestRequest($params, $token); } } + +class TestRequest extends Request +{ + public function setIsAuthenticated() + { + $this->isAuthenticated = true; + } +} diff --git a/tests/PHPUnit/Integration/CronArchive/FixedSiteIdsTest.php b/tests/PHPUnit/Unit/CronArchive/FixedSiteIdsTest.php similarity index 97% rename from tests/PHPUnit/Integration/CronArchive/FixedSiteIdsTest.php rename to tests/PHPUnit/Unit/CronArchive/FixedSiteIdsTest.php index 3f74af2bec54d33bbeab18b5d195c25eb7c57bd0..cb34283064bf8eade8057b7a661d20da95805bbc 100644 --- a/tests/PHPUnit/Integration/CronArchive/FixedSiteIdsTest.php +++ b/tests/PHPUnit/Unit/CronArchive/FixedSiteIdsTest.php @@ -6,7 +6,7 @@ * @license http://www.gnu.org/licenses/gpl-3.0.html GPL v3 or later */ -namespace Piwik\Tests\Integration\CronArchive; +namespace Piwik\Tests\Unit\CronArchive; use Piwik\CronArchive\FixedSiteIds; diff --git a/tests/PHPUnit/Unit/Tracker/RequestTest.php b/tests/PHPUnit/Unit/Tracker/RequestTest.php index 17dce6b3c10951eadf40c6be493bfa1f846b102b..3db5c22a4ab9afd049970f8b074c14672647fbee 100644 --- a/tests/PHPUnit/Unit/Tracker/RequestTest.php +++ b/tests/PHPUnit/Unit/Tracker/RequestTest.php @@ -476,19 +476,6 @@ class RequestTest extends UnitTestCase $this->assertSame(14, $request->getIdSite()); } - public function test_getIdSite_shouldTriggerEventAndReturnThatIdSite() - { - $self = $this; - Piwik::addAction('Tracker.Request.getIdSite', function (&$idSite, $params) use ($self) { - $self->assertSame(14, $idSite); - $self->assertEquals(array('idsite' => '14'), $params); - $idSite = 12; - }); - - $request = $this->buildRequest(array('idsite' => '14')); - $this->assertSame(12, $request->getIdSite()); - } - /** * @expectedException \Piwik\Exception\UnexpectedWebsiteFoundException * @expectedExceptionMessage Invalid idSite: '0'