Fix the global state unit tests

contao · May 6, 2022 · 32306fa · 32306fa
1 parent a5c8c84
commit 32306fa
Show file tree

Hide file tree

Showing 6 changed files with 37 additions and 17 deletions.
diff --git a/calendar-bundle/tests/EventListener/SitemapListenerTest.php b/calendar-bundle/tests/EventListener/SitemapListenerTest.php
@@ -23,6 +23,13 @@
 
 class SitemapListenerTest extends ContaoTestCase
 {
+    protected function tearDown(): void
+    {
+        unset($GLOBALS['TL_CONFIG']);
+
+        parent::tearDown();
+    }
+
     public function testNothingIsAddedIfNoPublishedCalendar(): void
     {
         $adapters = [

diff --git a/core-bundle/src/Resources/contao/library/Contao/Input.php b/core-bundle/src/Resources/contao/library/Contao/Input.php
@@ -39,23 +39,23 @@ class Input
 	/**
 	 * Parameters set via setGet() are stored by request
 	 *
-	 * @var \WeakMap<Request,array<string,array|string>>
+	 * @var \WeakMap<Request,array<string,array|string>>|null
 	 */
-	private static \WeakMap $setGet;
+	private static \WeakMap|null $setGet = null;
 
 	/**
 	 * Parameters set via setPost() are stored by request
 	 *
-	 * @var \WeakMap<Request,array<string,array|string>>
+	 * @var \WeakMap<Request,array<string,array|string>>|null
 	 */
-	private static \WeakMap $setPost;
+	private static \WeakMap|null $setPost = null;
 
 	/**
 	 * Parameters set via setCookie() are stored by request
 	 *
-	 * @var \WeakMap<Request,array<string,array|string>>
+	 * @var \WeakMap<Request,array<string,array|string>>|null
 	 */
-	private static \WeakMap $setCookie;
+	private static \WeakMap|null $setCookie = null;
 
 	/**
 	 * Clean the global GPC arrays

diff --git a/core-bundle/tests/Contao/DcaExtractorTest.php b/core-bundle/tests/Contao/DcaExtractorTest.php
@@ -12,9 +12,11 @@
 
 namespace Contao\CoreBundle\Tests\Contao;
 
+use Contao\Config;
 use Contao\CoreBundle\Config\ResourceFinder;
 use Contao\CoreBundle\Tests\TestCase;
 use Contao\DcaExtractor;
+use Contao\DcaLoader;
 use Contao\System;
 use Doctrine\DBAL\Connection;
 use Symfony\Component\Config\FileLocator;
@@ -47,6 +49,10 @@ protected function setUp(): void
 
     protected function tearDown(): void
     {
+        unset($GLOBALS['TL_MIME'], $GLOBALS['TL_TEST'], $GLOBALS['TL_LANG'], $GLOBALS['TL_DCA']);
+
+        $this->resetStaticProperties([System::class, DcaExtractor::class, Config::class, DcaLoader::class]);
+
         parent::tearDown();
 
         (new Filesystem())->remove($this->getTempDir());

diff --git a/core-bundle/tests/Contao/EnvironmentTest.php b/core-bundle/tests/Contao/EnvironmentTest.php
@@ -17,6 +17,7 @@
 use Contao\System;
 use Symfony\Bridge\PhpUnit\ExpectDeprecationTrait;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\HttpFoundation\IpUtils;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\RequestStack;
 
@@ -50,7 +51,14 @@ protected function setUp(): void
     protected function tearDown(): void
     {
         $this->restoreServerEnvGetPost();
-        $this->resetStaticProperties([Environment::class, [Environment::class, ['strSapi']], System::class]);
+
+        $this->resetStaticProperties([
+            Environment::class,
+            [Environment::class, ['strSapi']],
+            System::class,
+            Request::class,
+            IpUtils::class,
+        ]);
 
         parent::tearDown();
     }

diff --git a/core-bundle/tests/Twig/Loader/ContaoFilesystemLoaderTest.php b/core-bundle/tests/Twig/Loader/ContaoFilesystemLoaderTest.php
@@ -30,6 +30,13 @@
 
 class ContaoFilesystemLoaderTest extends TestCase
 {
+    protected function tearDown(): void
+    {
+        unset($GLOBALS['objPage']);
+
+        parent::tearDown();
+    }
+
     public function testAddsPath(): void
     {
         $loader = $this->getContaoFilesystemLoader();
@@ -201,8 +208,6 @@ public function testGetCacheKeyDelegatesToThemeTemplate(): void
             Path::join($basePath, 'templates/my/theme/text.html.twig'),
             Path::normalize($loader->getCacheKey('@Contao/text.html.twig'))
         );
-
-        unset($GLOBALS['objPage']);
     }
 
     public function testGetsSourceContext(): void
@@ -243,8 +248,6 @@ public function testGetSourceContextDelegatesToThemeTemplate(): void
 
         $this->assertSame('@Contao_Theme_my_theme/text.html.twig', $source->getName());
         $this->assertSame(Path::join($basePath, 'templates/my/theme/text.html.twig'), Path::normalize($source->getPath()));
-
-        unset($GLOBALS['objPage']);
     }
 
     public function testGetsSourceContextFromHtml5File(): void
@@ -303,8 +306,6 @@ public function testExistsDelegatesToThemeTemplate(): void
 
         $this->assertTrue($loader->exists('@Contao/text.html.twig'));
         $this->assertFalse($loader->exists('@Contao/foo.html.twig'));
-
-        unset($GLOBALS['objPage']);
     }
 
     /**
@@ -390,8 +391,6 @@ public function testIsFreshDelegatesToThemeTemplate(): void
         $GLOBALS['objPage'] = $page;
 
         $this->assertFalse($loader->isFresh('@Contao/text.html.twig', $cacheTime));
-
-        unset($GLOBALS['objPage']);
     }
 
     public function testGetsHierarchy(): void
@@ -622,8 +621,6 @@ public function testThrowsInvalidThemePathExceptionWhenGeneratingSlug(): void
         $GLOBALS['objPage'] = $page;
 
         $this->assertFalse($loader->exists('@Contao/foo.html.twig'));
-
-        unset($GLOBALS['objPage']);
     }
 
     private function getTemplateLocator(string $projectDir = '/', array $themePaths = [], array $bundles = [], array $bundlesMetadata = []): TemplateLocator

diff --git a/test-case/src/ContaoTestCase.php b/test-case/src/ContaoTestCase.php
@@ -275,6 +275,7 @@ protected function backupServerEnvGetPost(): void
             '_ENV' => $_ENV,
             '_GET' => $_GET,
             '_POST' => $_POST,
+            '_COOKIE' => $_COOKIE,
         ];
     }
 
@@ -284,6 +285,7 @@ protected function restoreServerEnvGetPost(): void
         $_ENV = $this->backupServerEnvGetPost['_ENV'] ?? $_ENV;
         $_GET = $this->backupServerEnvGetPost['_GET'] ?? [];
         $_POST = $this->backupServerEnvGetPost['_POST'] ?? [];
+        $_COOKIE = $this->backupServerEnvGetPost['_COOKIE'] ?? [];
     }
 
     /**