refactor: cover phpunit postgresql bridge with mago

Author: norberttech

Justfile

@@ -79,7 +79,8 @@ analyze-mago *args:
         src/bridge/filesystem/async-aws \
         src/bridge/filesystem/azure \
         src/bridge/monolog/http \
-        src/bridge/monolog/telemetry
+        src/bridge/monolog/telemetry \
+        src/bridge/phpunit/postgresql
 
 # Auto-fix code style (Mago format + lint --fix) and GitHub Actions findings (zizmor --fix).
 fix:

src/bridge/phpunit/postgresql/src/Flow/Bridge/PHPUnit/PostgreSQL/Subscriber/TestPreparationStartedSubscriber.php

@@ -37,21 +37,24 @@ private static function hasSkipAttribute(Test $test): bool
 
         $reflectionClass = new ReflectionClass($test->className());
 
-        if ($reflectionClass->getAttributes(SkipTransactionRollback::class)) {
+        if ($reflectionClass->getAttributes(SkipTransactionRollback::class) !== []) {
             return true;
         }
 
         if (
             $reflectionClass->hasMethod($test->methodName())
-            && $reflectionClass->getMethod($test->methodName())->getAttributes(SkipTransactionRollback::class)
+            && $reflectionClass->getMethod($test->methodName())->getAttributes(SkipTransactionRollback::class) !== []
         ) {
             return true;
         }
 
-        while (($reflectionClass = $reflectionClass->getParentClass()) && $reflectionClass->name !== TestCase::class) {
-            if ($reflectionClass->getAttributes(SkipTransactionRollback::class)) {
+        $parent = $reflectionClass->getParentClass();
+
+        while ($parent !== false && $parent->name !== TestCase::class) {
+            if ($parent->getAttributes(SkipTransactionRollback::class) !== []) {
                 return true;
             }
+            $parent = $parent->getParentClass();
         }
 
         return false;