refs #4233 setting group on class level to make sure all tests within…

… this class have this group
matomo-org · Nov 13, 2013 · b2f0936 · b2f0936
1 parent 3d02064
commit b2f0936
Show file tree

Hide file tree

Showing 17 changed files with 77 additions and 35 deletions.
diff --git a/tests/PHPUnit/Integration/Core/AccessTest.php b/tests/PHPUnit/Integration/Core/AccessTest.php
@@ -8,6 +8,11 @@
 use Piwik\Access;
 use Piwik\AuthResult;
 
+/**
+ * Class Core_AccessTest
+ *
+ * @group Core
+ */
 class Core_AccessTest extends DatabaseTestCase
 {
     public function setUp()

diff --git a/tests/PHPUnit/Integration/Core/ArchiveProcessingTest.php b/tests/PHPUnit/Integration/Core/ArchiveProcessingTest.php
@@ -38,7 +38,12 @@ public function public_isArchiveTemporary()
     }
 }
 
-class ArchiveProcessingTest extends DatabaseTestCase
+/**
+ * Class Core_ArchiveProcessorTest
+ *
+ * @group Core
+ */
+class Core_ArchiveProcessingTest extends DatabaseTestCase
 {
     public function setUp()
     {

diff --git a/tests/PHPUnit/Integration/Core/LogTest.php b/tests/PHPUnit/Integration/Core/LogTest.php
@@ -15,6 +15,11 @@
 use Piwik\Log;
 use Piwik\Plugins\TestPlugin\TestLoggingUtility;
 
+/**
+ * Class Core_LogTest
+ *
+ * @group Core
+ */
 class Core_LogTest extends DatabaseTestCase
 {
     const TESTMESSAGE = 'test%smessage';

diff --git a/tests/PHPUnit/Integration/Core/OptionTest.php b/tests/PHPUnit/Integration/Core/OptionTest.php
@@ -11,6 +11,11 @@
 
 require_once "Option.php";
 
+/**
+ * Class Core_OptionTest
+ *
+ * @group Core
+ */
 class Core_OptionTest extends DatabaseTestCase
 {
     /**

diff --git a/tests/PHPUnit/Integration/Core/PiwikTest.php b/tests/PHPUnit/Integration/Core/PiwikTest.php
@@ -12,6 +12,11 @@
 use Piwik\Plugins\SitesManager\API;
 use Piwik\Translate;
 
+/**
+ * Class Core_PiwikTest
+ *
+ * @group Core
+ */
 class Core_PiwikTest extends DatabaseTestCase
 {
     /**

diff --git a/tests/PHPUnit/Integration/Core/SqlTest.php b/tests/PHPUnit/Integration/Core/SqlTest.php
@@ -6,6 +6,8 @@
  *
  * @link http://piwik.org
  * @license http://www.gnu.org/licenses/gpl-3.0.html GPL v3 or later
+ *
+ * @group Core
  */
 class Core_SqlTest extends DatabaseTestCase
 {

diff --git a/tests/PHPUnit/Integration/Core/Tracker/ActionTest.php b/tests/PHPUnit/Integration/Core/Tracker/ActionTest.php
@@ -12,6 +12,8 @@
  *
  * @link http://piwik.org
  * @license http://www.gnu.org/licenses/gpl-3.0.html GPL v3 or later
+ *
+ * @group Core
  */
 class Core_Tracker_ActionTest extends DatabaseTestCase
 {

diff --git a/tests/PHPUnit/Integration/Core/Tracker/VisitTest.php b/tests/PHPUnit/Integration/Core/Tracker/VisitTest.php
@@ -11,6 +11,11 @@
 use Piwik\Tracker\Request;
 use Piwik\Tracker\VisitExcluded;
 
+/**
+ * Class Core_Tracker_VisitTest
+ *
+ * @group Core
+ */
 class Core_Tracker_VisitTest extends DatabaseTestCase
 {
     public function setUp()

diff --git a/tests/PHPUnit/Integration/Core/UpdaterTest.php b/tests/PHPUnit/Integration/Core/UpdaterTest.php
@@ -7,6 +7,11 @@
  */
 use Piwik\Updater;
 
+/**
+ * Class Core_UpdaterTest
+ *
+ * @group Core
+ */
 class Core_UpdaterTest extends DatabaseTestCase
 {
     /**

diff --git a/tests/PHPUnit/Integration/Core/WidgetsListTest.php b/tests/PHPUnit/Integration/Core/WidgetsListTest.php
@@ -10,6 +10,11 @@
 use Piwik\Plugins\Goals\API;
 use Piwik\WidgetsList;
 
+/**
+ * Class Core_WidgetsListTest
+ *
+ * @group Core
+ */
 class Core_WidgetsListTest extends DatabaseTestCase
 {
     /**

diff --git a/tests/PHPUnit/Integration/Plugins/LoginTest.php b/tests/PHPUnit/Integration/Plugins/LoginTest.php
@@ -14,6 +14,11 @@
 
 require_once 'Login/Auth.php';
 
+/**
+ * Class Plugins_LoginTest
+ *
+ * @group Plugins
+ */
 class Plugins_LoginTest extends DatabaseTestCase
 {
     public function setUp()

diff --git a/tests/PHPUnit/Integration/Plugins/MobileMessagingTest.php b/tests/PHPUnit/Integration/Plugins/MobileMessagingTest.php
@@ -13,6 +13,11 @@
 use Piwik\Plugins\ScheduledReports\API as APIScheduledReports;
 use Piwik\Plugins\SitesManager\API as APISitesManager;
 
+/**
+ * Class Plugins_MobileMessagingTest
+ *
+ * @group Plugins
+ */
 class Plugins_MobileMessagingTest extends DatabaseTestCase
 {
     protected $idSiteAccess;

diff --git a/tests/PHPUnit/Integration/Plugins/MultiSitesTest.php b/tests/PHPUnit/Integration/Plugins/MultiSitesTest.php
@@ -10,6 +10,11 @@
 use Piwik\Plugins\MultiSites\API as APIMultiSites;
 use Piwik\Plugins\SitesManager\API as APISitesManager;
 
+/**
+ * Class Plugins_MultiSitesTest
+ *
+ * @group Plugins
+ */
 class Plugins_MultiSitesTest extends DatabaseTestCase
 {
     protected $idSiteAccess;

diff --git a/tests/PHPUnit/Integration/Plugins/ScheduledReportsTest.php b/tests/PHPUnit/Integration/Plugins/ScheduledReportsTest.php
@@ -19,6 +19,11 @@
 
 require_once 'ScheduledReports/ScheduledReports.php';
 
+/**
+ * Class Plugins_ScheduledReportsTest
+ *
+ * @group Plugins
+ */
 class Plugins_ScheduledReportsTest extends DatabaseTestCase
 {
     private $idSite = 1;

diff --git a/tests/PHPUnit/Integration/Plugins/SegmentEditorTest.php b/tests/PHPUnit/Integration/Plugins/SegmentEditorTest.php
@@ -11,6 +11,11 @@
 use Piwik\Plugins\SegmentEditor\API;
 use Piwik\Plugins\SitesManager\API as APISitesManager;
 
+/**
+ * Class Plugins_SegmentEditorTest
+ *
+ * @group Plugins
+ */
 class Plugins_SegmentEditorTest extends DatabaseTestCase
 {
     public function setUp()

diff --git a/tests/PHPUnit/Integration/Plugins/SitesManagerTest.php b/tests/PHPUnit/Integration/Plugins/SitesManagerTest.php
@@ -10,6 +10,11 @@
 use Piwik\Plugins\UsersManager\API as APIUsersManager;
 use Piwik\Site;
 
+/**
+ * Class Plugins_SitesManagerTest
+ *
+ * @group Plugins
+ */
 class Plugins_SitesManagerTest extends DatabaseTestCase
 {
     public function setUp()

diff --git a/tests/PHPUnit/Integration/Plugins/UsersManagerTest.php b/tests/PHPUnit/Integration/Plugins/UsersManagerTest.php
@@ -9,6 +9,8 @@
  *
  * @link http://piwik.org
  * @license http://www.gnu.org/licenses/gpl-3.0.html GPL v3 or later
+ *
+ * @group Plugins
  */
 class Plugins_UsersManagerTest extends DatabaseTestCase
 {
@@ -69,9 +71,6 @@ private function _checkUserHasNotChanged($user, $newPassword, $newEmail = null,
         $this->assertEquals($user, $userAfter);
     }
 
-    /**
-     * @group Plugins
-     */
     public function testAllSuperUserIncluded()
     {
         Config::getInstance()->superuser = array(
@@ -118,8 +117,6 @@ public function testAllSuperUserIncluded()
 
     /**
      * bad password => exception
-     *
-     * @group Plugins
      */
     public function testUpdateUserBadpasswd()
     {
@@ -160,7 +157,6 @@ public function getAddUserInvalidLoginData()
     /**
      *
      * @dataProvider getAddUserInvalidLoginData
-     * @group Plugins
      */
     public function testAddUserWrongLogin($userLogin, $password, $email, $alias)
     {
@@ -175,8 +171,6 @@ public function testAddUserWrongLogin($userLogin, $password, $email, $alias)
 
     /**
      * existing login => exception
-     *
-     * @group Plugins
      */
     public function testAddUserExistingLogin()
     {
@@ -206,7 +200,6 @@ public function getWrongPasswordTestData()
     /**
      *
      * @dataProvider getWrongPasswordTestData
-     * @group Plugins
      */
     public function testAddUserWrongPassword($userLogin, $password, $email, $alias)
     {
@@ -235,7 +228,6 @@ public function getWrongEmailTestData()
     /**
      *
      * @dataProvider getWrongEmailTestData
-     * @group Plugins
      */
     public function testAddUserWrongEmail($userLogin, $password, $email, $alias)
     {
@@ -250,8 +242,6 @@ public function testAddUserWrongEmail($userLogin, $password, $email, $alias)
 
     /**
      * empty email => exception
-     *
-     * @group Plugins
      */
     public function testAddUserEmptyEmail()
     {
@@ -267,8 +257,6 @@ public function testAddUserEmptyEmail()
 
     /**
      * empty alias => use login
-     *
-     * @group Plugins
      */
     public function testAddUserEmptyAlias()
     {
@@ -281,8 +269,6 @@ public function testAddUserEmptyAlias()
 
     /**
      * no alias => use login
-     *
-     * @group Plugins
      */
     public function testAddUserNoAliasSpecified()
     {
@@ -295,8 +281,6 @@ public function testAddUserNoAliasSpecified()
 
     /**
      * normal test case
-     *
-     * @group Plugins
      */
     public function testAddUser()
     {
@@ -329,8 +313,6 @@ public function testAddUser()
 
     /**
      * user doesnt exist => exception
-     *
-     * @group Plugins
      */
     public function testSeleteUserDoesntExist()
     {
@@ -347,8 +329,6 @@ public function testSeleteUserDoesntExist()
 
     /**
      * empty name, doesnt exists =>exception
-     *
-     * @group Plugins
      */
     public function testDeleteUserEmptyUser()
     {
@@ -363,8 +343,6 @@ public function testDeleteUserEmptyUser()
 
     /**
      * null user,, doesnt exists => exception
-     *
-     * @group Plugins
      */
     public function testDeleteUserNullUser()
     {
@@ -379,8 +357,6 @@ public function testDeleteUserNullUser()
 
     /**
      * normal case, user deleted
-     *
-     * @group Plugins
      */
     public function testDeleteUser()
     {
@@ -419,8 +395,6 @@ public function testDeleteUser()
 
     /**
      * no user => exception
-     *
-     * @group Plugins
      */
     public function testGetUserNoUser()
     {
@@ -437,8 +411,6 @@ public function testGetUserNoUser()
 
     /**
      * normal case
-     *
-     * @group Plugins
      */
     public function test_GetUser()
     {
@@ -462,8 +434,6 @@ public function test_GetUser()
 
     /**
      * no user => empty array
-     *
-     * @group Plugins
      */
     public function testGetUsersNoUser()
     {
@@ -473,8 +443,6 @@ public function testGetUsersNoUser()
     /**
      * normal case
      * as well as selecting specific user names, comma separated
-     *
-     * @group Plugins
      */
     public function testGetUsers()
     {