test: Suppress PHPUnit error handler warnings

Author: distantnative

tests/Cms/App/AppTest.php

@@ -11,6 +11,7 @@
 use Kirby\Session\Session;
 use Kirby\Toolkit\Str;
 use ReflectionMethod;
+use Whoops\Handler\PrettyPageHandler;
 
 /**
  * @coversDefaultClass \Kirby\Cms\App
@@ -29,6 +30,7 @@ public function setUp(): void
 
 	public function tearDown(): void
 	{
+		App::destroy();
 		Dir::remove(static::TMP);
 		$_SERVER = $this->_SERVER;
 	}
@@ -456,7 +458,6 @@ public function testOptionWithDotNotation()
 
 	public function testOptionFromPlugin()
 	{
-		App::destroy();
 		App::plugin('namespace/plugin', [
 			'options' => [
 				'key' => 'A',
@@ -500,8 +501,6 @@ public function testOptionFromPlugin()
 
 	public function testOptions()
 	{
-		App::destroy();
-
 		$app = new App([
 			'roots' => [
 				'index' => '/dev/null'
@@ -520,8 +519,6 @@ public function testOptions()
 
 	public function testOptionsFromFile()
 	{
-		App::destroy();
-
 		$app = new App([
 			'roots' => [
 				'index'  => '/dev/null',
@@ -545,8 +542,6 @@ public function testOptionsFromFile()
 
 	public function testOptionsFromFileWithEnv1()
 	{
-		App::destroy();
-
 		$app = new App([
 			'roots' => [
 				'index'  => '/dev/null',
@@ -571,8 +566,6 @@ public function testOptionsFromFileWithEnv1()
 
 	public function testOptionsFromFileWithEnv2()
 	{
-		App::destroy();
-
 		$app = new App([
 			'roots' => [
 				'index'  => '/dev/null',
@@ -597,10 +590,8 @@ public function testOptionsFromFileWithEnv2()
 		], $app->options());
 	}
 
-	public function testOptionsOnReady()
+	public function testOptionsOnReady(): void
 	{
-		App::destroy();
-
 		$app = new App([
 			'cli' => false,
 			'roots' => [
@@ -626,25 +617,31 @@ public function testOptionsOnReady()
 		]);
 
 		$this->assertSame([
-			'ready' => $ready,
-			'whoops' => true,
-			'test' => '/dev/null',
+			'ready'        => $ready,
+			'whoops'       => true,
+			'test'         => '/dev/null',
 			'another.test' => 'foo',
-			'debug' => true,
-			'home' => 'test',
-			'error' => 'another-test',
-			'slugs' => 'de'
+			'debug'        => true,
+			'home'         => 'test',
+			'error'        => 'another-test',
+			'slugs'        => 'de'
 		], $app->options());
 
 		$whoopsMethod = new ReflectionMethod(App::class, 'whoops');
 		$whoopsMethod->setAccessible(true);
 		$whoopsHandler = $whoopsMethod->invoke($app)->getHandlers()[0];
-		$this->assertInstanceOf('Whoops\Handler\PrettyPageHandler', $whoopsHandler);
+		$this->assertInstanceOf(PrettyPageHandler::class, $whoopsHandler);
 
 		$this->assertSame('test', $app->site()->homePageId());
 		$this->assertSame('another-test', $app->site()->errorPageId());
 
 		$this->assertSame('ss', Str::$language['ß']);
+
+		// Suppresses PHPUnit warnings:
+		// Test code or tested code did not remove its own error handlers
+		// Test code or tested code did not remove its own exception handlers
+		restore_error_handler();
+		restore_exception_handler();
 	}
 
 	public function testRolesFromFixtures()
@@ -730,7 +727,6 @@ public function testSession()
 
 	public function testInstance()
 	{
-		App::destroy();
 		$this->assertNull(App::instance(null, true));
 
 		$instance1 = new App();
@@ -951,8 +947,6 @@ public function testUrl($url, $expected)
 
 	public function testUrlFromEnvWithDetection()
 	{
-		App::destroy();
-
 		$app = new App([
 			'roots' => [
 				'index'  => '/dev/null',
@@ -971,8 +965,6 @@ public function testUrlFromEnvWithDetection()
 
 	public function testUrlFromEnvWithOverride()
 	{
-		App::destroy();
-
 		$app = new App([
 			'roots' => [
 				'index'  => '/dev/null',