Automatically add phpunit group name to a plugin php test (#16615)

* Automatically add plugin group to every test

* prevent possible error

* add test

2 files changed, 27 insertions, 1 deletions

diff --git a/tests/PHPUnit/Framework/TestCase/SystemTestCase.php b/tests/PHPUnit/Framework/TestCase/SystemTestCase.php
index 20ebb007bb..da1e1dfb23 100644
--- a/tests/PHPUnit/Framework/TestCase/SystemTestCase.php
+++ b/tests/PHPUnit/Framework/TestCase/SystemTestCase.php
@@ -58,6 +58,19 @@ abstract class SystemTestCase extends TestCase
      */
     public static $fixture;
 
+    public function setGroups(array $groups): void
+    {
+        $pluginName = explode('\\', get_class($this));
+        if (!empty($pluginName[2]) && !empty($pluginName[1]) && $pluginName[1] === 'Plugins') {
+            // we assume \Piwik\Plugins\PluginName nanmespace...
+            if (!in_array($pluginName[2], $groups, true)) {
+                $groups[] = $pluginName[2];
+            }
+        }
+
+        parent::setGroups($groups);
+    }
+
     public static function setUpBeforeClass(): void
     {
         Log::debug("Setting up " . get_called_class());
diff --git a/tests/PHPUnit/Framework/TestCase/UnitTestCase.php b/tests/PHPUnit/Framework/TestCase/UnitTestCase.php
index d81f000a3d..56428a92de 100644
--- a/tests/PHPUnit/Framework/TestCase/UnitTestCase.php
+++ b/tests/PHPUnit/Framework/TestCase/UnitTestCase.php
@@ -26,6 +26,19 @@ abstract class UnitTestCase extends \PHPUnit\Framework\TestCase
      */
     protected $environment;
 
+    public function setGroups(array $groups): void
+    {
+        $pluginName = explode('\\', get_class($this));
+        if (!empty($pluginName[2]) && !empty($pluginName[1]) && $pluginName[1] === 'Plugins') {
+            // we assume \Piwik\Plugins\PluginName nanmespace...
+            if (!in_array($pluginName[2], $groups, true)) {
+                $groups[] = $pluginName[2];
+            }
+        }
+
+        parent::setGroups($groups);
+    }
+
     public function setUp(): void
     {
         parent::setUp();
@@ -41,7 +54,7 @@ abstract class UnitTestCase extends \PHPUnit\Framework\TestCase
 
         // make sure the global container exists for the next test case that is executed (since logging can be done
         // before a test sets up an environment)
-        $nextTestEnviornment = new Environment($environment = null, array(), $postBootstrappedEvent = false);
+        $nextTestEnviornment = new Environment($environment = null, array());
         $nextTestEnviornment->init();
 
         parent::tearDown();