support SkipDatabaseRollback also on parent classes

README.md

@@ -68,6 +68,9 @@ With PHPUnit you can skip this bundle's database rollback handling for specific
 #[SkipDatabaseRollback] // this will skip it for all tests in a class
 public class MyTest extends \PHPUnit\Framework\TestCase {}
 
+#[SkipDatabaseRollback] // also supported on (abstract) parent classes. So all tests in child classes will skip the rollback logic.
+abstract class MyAbstractTest extends \PHPUnit\Framework\TestCase {}
+
 #[SkipDatabaseRollback] // this will skip it for only one test method
 public function MyTest() {}
 ```

src/PHPUnit/PHPUnitExtension.php

@@ -13,6 +13,7 @@
 use PHPUnit\Event\TestRunner\FinishedSubscriber as TestRunnerFinishedSubscriber;
 use PHPUnit\Event\TestRunner\Started as TestRunnerStartedEvent;
 use PHPUnit\Event\TestRunner\StartedSubscriber as TestRunnerStartedSubscriber;
+use PHPUnit\Framework\TestCase;
 use PHPUnit\Runner\Extension\Extension;
 use PHPUnit\Runner\Extension\Facade;
 use PHPUnit\Runner\Extension\ParameterCollection;
@@ -93,8 +94,21 @@ public static function hasSkipAttribute(Test $test): bool
             return true;
         }
 
-        return $reflectionClass->hasMethod($methodName = $test->methodName())
-            && $reflectionClass->getMethod($methodName)->getAttributes(SkipDatabaseRollback::class);
+        if ($reflectionClass->hasMethod($methodName = $test->methodName())
+            && $reflectionClass->getMethod($methodName)->getAttributes(SkipDatabaseRollback::class)
+        ) {
+            return true;
+        }
+
+        while (($reflectionClass = $reflectionClass->getParentClass())
+            && $reflectionClass->name !== TestCase::class
+        ) {
+            if ($reflectionClass->getAttributes(SkipDatabaseRollback::class)) {
+                return true;
+            }
+        }
+
+        return false;
     }
 
     public function bootstrap(Configuration $configuration, Facade $facade, ParameterCollection $parameters): void

tests/Functional/AbstractTestClassWithSkipAttribute.php

@@ -0,0 +1,13 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Functional;
+
+use DAMA\DoctrineTestBundle\PHPUnit\SkipDatabaseRollback;
+use PHPUnit\Framework\TestCase;
+
+#[SkipDatabaseRollback]
+abstract class AbstractTestClassWithSkipAttribute extends TestCase
+{
+}

tests/Functional/PhpunitWithClassAttributeTest.php

@@ -2,12 +2,11 @@
 
 declare(strict_types=1);
 
-namespace Functional;
+namespace Tests\Functional;
 
 use DAMA\DoctrineTestBundle\PHPUnit\SkipDatabaseRollback;
 use PHPUnit\Framework\Attributes\Depends;
 use PHPUnit\Framework\TestCase;
-use Tests\Functional\FunctionalTestTrait;
 
 #[SkipDatabaseRollback]
 class PhpunitWithClassAttributeTest extends TestCase

tests/Functional/PhpunitWithExtendingClassAttributeTest.php

@@ -0,0 +1,27 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Functional;
+
+use PHPUnit\Framework\Attributes\Depends;
+
+class PhpunitWithExtendingClassAttributeTest extends AbstractTestClassWithSkipAttribute
+{
+    use FunctionalTestTrait;
+
+    public function testTransactionalBehaviorDisabledWithAttributeOnClassLevel(): void
+    {
+        $this->insertRow();
+        $this->assertRowCount(1);
+    }
+
+    #[Depends('testTransactionalBehaviorDisabledWithAttributeOnClassLevel')]
+    public function testChangesFromPreviousTestAreVisibleWhenDisabledWithAttributeOnClassLevel(): void
+    {
+        $this->assertRowCount(1);
+
+        // cleanup persisted row to not affect any other tests afterwards
+        $this->connection->executeQuery('DELETE FROM test');
+    }
+}