PHPLIB-1659: Fix dropping collections after successful tests (#1653)

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

@@ -17,7 +17,6 @@
 use MongoDB\Operation\DatabaseCommand;
 use MongoDB\Operation\ListCollections;
 use stdClass;
-use Throwable;
 use UnexpectedValueException;
 
 use function array_intersect_key;
@@ -70,15 +69,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();