PHPLIB-1659: Fix dropping collections after successful tests

cb33c1c introduced onNotSuccessfulTest(), but that is the inverse logic. The internal TestCase::status() method seems to be the only replacement for TestCase::hasFailed(), which was removed in PHPUnit 10.

Author: jmikola

tests/FunctionalTestCase.php

@@ -70,15 +70,12 @@ public function setUp(): void
         $this->configuredFailPoints = [];
     }
 
-    protected function onNotSuccessfulTest(Throwable $t): never
-    {
-        $this->cleanupCollections();
-
-        throw $t;
-    }
-
     public function tearDown(): void
     {
+        if ($this->status()->isSuccess()) {
+            $this->cleanupCollections();
+        }
+
         $this->disableFailPoints();
 
         parent::tearDown();