refactor: add centralized reset in getMockAppConfig for test isolation

- Add reset() method to AppConfigOverwrite that clears overWrite and deleted arrays and returns self
- Integrate reset() directly into getMockAppConfig() to ensure clean state on every call
- All tests now automatically get clean AppConfig state without explicit reset calls
- Prevents state pollution across test suites by resetting at the source
- Simplifies test code by removing need for separate reset wrapper method

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>

Author: vitormattos

tests/php/Unit/Handler/CertificateEngine/AEngineHandlerTest.php

@@ -40,10 +40,6 @@ public function setUp(): void {
 		$this->urlGenerator = \OCP\Server::get(IURLGenerator::class);
 		$this->caIdentifierService = \OCP\Server::get(CaIdentifierService::class);
 		$this->logger = \OCP\Server::get(LoggerInterface::class);
-
-		$this->appConfig->deleteKey(Application::APP_ID, 'certificate_engine');
-		$this->appConfig->deleteKey(Application::APP_ID, 'identify_methods');
-		$this->appConfig->deleteKey(Application::APP_ID, 'config_path');
 	}
 
 	private function getInstance(): OpenSslHandler {

tests/php/Unit/TestCase.php

@@ -79,6 +79,9 @@ public static function getMockAppConfig(): IAppConfig {
 			));
 			$service = \OCP\Server::get(\OCP\IAppConfig::class);
 		}
+		if ($service instanceof AppConfigOverwrite) {
+			$service->reset();
+		}
 		return $service;
 	}
 

tests/php/lib/AppConfigOverwrite.php

@@ -138,6 +138,12 @@ public function deleteKey(string $app, string $key): void {
 		$this->markDeleted($app, $key);
 	}
 
+	public function reset(): self {
+		$this->overWrite = [];
+		$this->deleted = [];
+		return $this;
+	}
+
 	private function isDeleted(string $app, string $key): bool {
 		return isset($this->deleted[$app][$key]);
 	}