Fix phpunit errors

Author: paulbalandan

tests/system/Autoloader/AutoloaderTest.php

@@ -43,6 +43,20 @@ final class AutoloaderTest extends CIUnitTestCase
      */
     private Closure $classLoader;
 
+    public static function setUpBeforeClass(): void
+    {
+        parent::setUpBeforeClass();
+
+        set_error_handler(null);
+    }
+
+    public static function tearDownAfterClass(): void
+    {
+        parent::tearDownAfterClass();
+
+        restore_error_handler();
+    }
+
     protected function setUp(): void
     {
         parent::setUp();
@@ -396,10 +410,6 @@ public function testAutoloaderLoadsNonClassFiles(): void
     #[RunInSeparateProcess]
     public function testLoadHelpers(): void
     {
-        // Workaround for errors on PHPUnit 10 and PHP 8.3.
-        // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
-        restore_error_handler();
-
         $config            = new Autoload();
         $config->helpers[] = 'form';
 

tests/system/CodeIgniterTest.php

@@ -48,15 +48,25 @@ final class CodeIgniterTest extends CIUnitTestCase
     private CodeIgniter $codeigniter;
     protected $routes;
 
+    public static function setUpBeforeClass(): void
+    {
+        parent::setUpBeforeClass();
+
+        set_error_handler(null);
+    }
+
+    public static function tearDownAfterClass(): void
+    {
+        parent::tearDownAfterClass();
+
+        restore_error_handler();
+    }
+
     protected function setUp(): void
     {
         parent::setUp();
         $this->resetServices();
 
-        // Workaround for errors on PHPUnit 10 and PHP 8.3.
-        // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
-        restore_error_handler();
-
         $_SERVER['SERVER_PROTOCOL'] = 'HTTP/1.1';
 
         $this->codeigniter = new MockCodeIgniter(new App());

tests/system/Helpers/URLHelper/MiscUrlTest.php

@@ -38,6 +38,20 @@ final class MiscUrlTest extends CIUnitTestCase
 {
     private App $config;
 
+    public static function setUpBeforeClass(): void
+    {
+        parent::setUpBeforeClass();
+
+        set_error_handler(null);
+    }
+
+    public static function tearDownAfterClass(): void
+    {
+        parent::tearDownAfterClass();
+
+        restore_error_handler();
+    }
+
     protected function setUp(): void
     {
         parent::setUp();
@@ -63,10 +77,6 @@ protected function tearDown(): void
     #[RunInSeparateProcess]
     public function testPreviousURLUsesSessionFirst(): void
     {
-        // Workaround for errors on PHPUnit 10 and PHP 8.3.
-        // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
-        restore_error_handler();
-
         $uri1 = 'http://example.com/one?two';
         $uri2 = 'http://example.com/two?foo';
 
@@ -98,10 +108,6 @@ private function createRequest(string $uri): void
     #[RunInSeparateProcess]
     public function testPreviousURLUsesRefererIfNeeded(): void
     {
-        // Workaround for errors on PHPUnit 10 and PHP 8.3.
-        // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
-        restore_error_handler();
-
         $uri1 = 'http://example.com/one?two';
 
         $_SERVER['HTTP_REFERER'] = $uri1;

tests/system/View/ParserPluginTest.php

@@ -29,6 +29,20 @@ final class ParserPluginTest extends CIUnitTestCase
     private Parser $parser;
     private Validation $validator;
 
+    public static function setUpBeforeClass(): void
+    {
+        parent::setUpBeforeClass();
+
+        set_error_handler(null);
+    }
+
+    public static function tearDownAfterClass(): void
+    {
+        parent::tearDownAfterClass();
+
+        restore_error_handler();
+    }
+
     protected function setUp(): void
     {
         parent::setUp();
@@ -51,10 +65,6 @@ public function testCurrentURL(): void
     #[RunInSeparateProcess]
     public function testPreviousURL(): void
     {
-        // Workaround for errors on PHPUnit 10 and PHP 8.3.
-        // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
-        restore_error_handler();
-
         $template = '{+ previous_url +}';
 
         // Ensure a previous URL exists to work with.