refactor(tests): teardown is now always called - also when skipped

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Author: susnux

tests/lib/Repair/RepairCollationTest.php

@@ -31,10 +31,9 @@ public function getAllNonUTF8BinTables(IDBConnection $connection) {
 /**
  * Tests for the converting of MySQL tables to InnoDB engine
  *
- * @group DB
- *
  * @see \OC\Repair\RepairMimeTypes
  */
+#[\PHPUnit\Framework\Attributes\Group('DB')]
 class RepairCollationTest extends TestCase {
 
 	private TestCollationRepair $repair;
@@ -48,13 +47,12 @@ protected function setUp(): void {
 		parent::setUp();
 
 		$this->connection = Server::get(ConnectionAdapter::class);
-		$this->config = Server::get(IConfig::class);
 		if ($this->connection->getDatabaseProvider() !== IDBConnection::PLATFORM_MYSQL) {
 			$this->markTestSkipped('Test only relevant on MySql');
 		}
 
 		$this->logger = $this->createMock(LoggerInterface::class);
-
+		$this->config = Server::get(IConfig::class);
 		$dbPrefix = $this->config->getSystemValueString('dbtableprefix');
 		$this->tableName = $this->getUniqueID($dbPrefix . '_collation_test');
 		$this->connection->prepare("CREATE TABLE $this->tableName(text VARCHAR(16)) COLLATE utf8_unicode_ci")->execute();
@@ -63,7 +61,11 @@ protected function setUp(): void {
 	}
 
 	protected function tearDown(): void {
-		$this->connection->getInner()->createSchemaManager()->dropTable($this->tableName);
+		$this->connection = Server::get(ConnectionAdapter::class);
+		if ($this->connection->getDatabaseProvider() === IDBConnection::PLATFORM_MYSQL) {
+			// tear down only needed on MySQL
+			$this->connection->getInner()->createSchemaManager()->dropTable($this->tableName);
+		}
 		parent::tearDown();
 	}