Merge pull request #9080 from paulbalandan/update-to-phpunit-11

chore: Update phpunit to `^10.5.16 || ^11.2`
codeigniter4 · Jul 28, 2024 · 9c99605 · 9c99605
2 parents 24d3c5f + e33e6f2
commit 9c99605
Show file tree

Hide file tree

Showing 27 changed files with 63 additions and 156 deletions.
diff --git a/admin/framework/composer.json b/admin/framework/composer.json
@@ -23,7 +23,7 @@
  "kint-php/kint": "^5.0.4",
  "mikey179/vfsstream": "^1.6",
  "nexusphp/cs-config": "^3.6",
- "phpunit/phpunit": "^10.5.16",
+ "phpunit/phpunit": "^10.5.16 || ^11.2",
  "predis/predis": "^1.1 || ^2.0"
  },
  "suggest": {

diff --git a/composer.json b/composer.json
@@ -25,8 +25,8 @@
  "phpstan/extension-installer": "^1.4",
  "phpstan/phpstan": "^1.11",
  "phpstan/phpstan-strict-rules": "^1.6",
- "phpunit/phpcov": "^9.0.2",
- "phpunit/phpunit": "^10.5.16",
+ "phpunit/phpcov": "^9.0.2 || ^10.0",
+ "phpunit/phpunit": "^10.5.16 || ^11.2",
  "predis/predis": "^1.1 || ^2.0",
  "rector/rector": "1.2.2"
  },

diff --git a/tests/system/Autoloader/AutoloaderTest.php b/tests/system/Autoloader/AutoloaderTest.php
@@ -25,6 +25,7 @@
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\PreserveGlobalState;
 use PHPUnit\Framework\Attributes\RunInSeparateProcess;
+use PHPUnit\Framework\Attributes\WithoutErrorHandler;
 use RuntimeException;
 use UnnamespacedClass;
 
@@ -394,12 +395,9 @@ public function testAutoloaderLoadsNonClassFiles(): void
 
  #[PreserveGlobalState(false)]
  #[RunInSeparateProcess]
+ #[WithoutErrorHandler]
  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';
 

diff --git a/tests/system/CodeIgniterTest.php b/tests/system/CodeIgniterTest.php
@@ -34,6 +34,7 @@
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\RunTestsInSeparateProcesses;
+use PHPUnit\Framework\Attributes\WithoutErrorHandler;
 use Tests\Support\Filters\Customfilter;
 use Tests\Support\Filters\RedirectFilter;
 
@@ -48,15 +49,12 @@ final class CodeIgniterTest extends CIUnitTestCase
  private CodeIgniter $codeigniter;
  protected $routes;
 
+ #[WithoutErrorHandler]
  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());

diff --git a/tests/system/Commands/GenerateKeyTest.php b/tests/system/Commands/GenerateKeyTest.php
@@ -19,6 +19,7 @@
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\PreserveGlobalState;
 use PHPUnit\Framework\Attributes\RunInSeparateProcess;
+use PHPUnit\Framework\Attributes\WithoutErrorHandler;
 
 /**
  * @internal
@@ -31,6 +32,7 @@ final class GenerateKeyTest extends CIUnitTestCase
  private string $envPath;
  private string $backupEnvPath;
 
+ #[WithoutErrorHandler]
  protected function setUp(): void
  {
  parent::setUp();
@@ -43,10 +45,6 @@ protected function setUp(): void
  }
 
  $this->resetEnvironment();
-
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
  }
 
  protected function tearDown(): void

diff --git a/tests/system/CommonFunctionsSendTest.php b/tests/system/CommonFunctionsSendTest.php
@@ -17,22 +17,20 @@
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\PreserveGlobalState;
 use PHPUnit\Framework\Attributes\RunInSeparateProcess;
+use PHPUnit\Framework\Attributes\WithoutErrorHandler;
 
 /**
  * @internal
  */
 #[Group('SeparateProcess')]
 final class CommonFunctionsSendTest extends CIUnitTestCase
 {
+ #[WithoutErrorHandler]
  protected function setUp(): void
  {
  parent::setUp();
 
  unset($_ENV['foo'], $_SERVER['foo']);
-
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
  }
 
  /**

diff --git a/tests/system/CommonFunctionsTest.php b/tests/system/CommonFunctionsTest.php
@@ -46,6 +46,7 @@
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\PreserveGlobalState;
 use PHPUnit\Framework\Attributes\RunInSeparateProcess;
+use PHPUnit\Framework\Attributes\WithoutErrorHandler;
 use RuntimeException;
 use stdClass;
 use Tests\Support\Models\JobModel;
@@ -274,25 +275,19 @@ public function testEscapeRecursiveArrayRaw(): void
 
  #[PreserveGlobalState(false)]
  #[RunInSeparateProcess]
+ #[WithoutErrorHandler]
  public function testSessionInstance(): void
  {
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
-
  $this->injectSessionMock();
 
  $this->assertInstanceOf(Session::class, session());
  }
 
  #[PreserveGlobalState(false)]
  #[RunInSeparateProcess]
+ #[WithoutErrorHandler]
  public function testSessionVariable(): void
  {
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
-
  $this->injectSessionMock();
 
  $_SESSION['notbogus'] = 'Hi there';
@@ -302,12 +297,9 @@ public function testSessionVariable(): void
 
  #[PreserveGlobalState(false)]
  #[RunInSeparateProcess]
+ #[WithoutErrorHandler]
  public function testSessionVariableNotThere(): void
  {
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
-
  $this->injectSessionMock();
 
  $_SESSION['bogus'] = 'Hi there';
@@ -428,12 +420,9 @@ public function testModelExistsAbsoluteClassname(): void
 
  #[PreserveGlobalState(false)]
  #[RunInSeparateProcess]
+ #[WithoutErrorHandler]
  public function testOldInput(): void
  {
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
-
  $this->injectSessionMock();
  // setup from RedirectResponseTest...
  $_SERVER['REQUEST_METHOD'] = 'GET';
@@ -465,12 +454,9 @@ public function testOldInput(): void
 
  #[PreserveGlobalState(false)]
  #[RunInSeparateProcess]
+ #[WithoutErrorHandler]
  public function testOldInputSerializeData(): void
  {
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
-
  $this->injectSessionMock();
  // setup from RedirectResponseTest...
  $_SERVER['REQUEST_METHOD'] = 'GET';
@@ -503,12 +489,9 @@ public function testOldInputSerializeData(): void
  */
  #[PreserveGlobalState(false)]
  #[RunInSeparateProcess]
+ #[WithoutErrorHandler]
  public function testOldInputArray(): void
  {
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
-
  $this->injectSessionMock();
  // setup from RedirectResponseTest...
  $_SERVER['REQUEST_METHOD'] = 'GET';
@@ -622,12 +605,9 @@ public function testRedirectResponseCookies1(): void
 
  #[PreserveGlobalState(false)]
  #[RunInSeparateProcess]
+ #[WithoutErrorHandler]
  public function testTrace(): void
  {
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
-
  ob_start();
  trace();
  $content = ob_get_clean();
@@ -647,12 +627,9 @@ public function testViewNotSaveData(): void
 
  #[PreserveGlobalState(false)]
  #[RunInSeparateProcess]
+ #[WithoutErrorHandler]
  public function testForceHttpsNullRequestAndResponse(): void
  {
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
-
  $this->assertNull(Services::response()->header('Location'));
 
  Services::response()->setCookie('force', 'cookie');
@@ -763,12 +740,9 @@ public function testDWithCSP(): void
 
  #[PreserveGlobalState(false)]
  #[RunInSeparateProcess]
+ #[WithoutErrorHandler]
  public function testTraceWithCSP(): void
  {
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
-
  $this->resetServices();
 
  /** @var App $config */
@@ -780,11 +754,6 @@ public function testTraceWithCSP(): void
 
  Kint::$cli_detection = false;
 
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- // `$app->initialize()` sets error handler.
- restore_error_handler();
-
  $this->expectOutputRegex('/<style class="kint-rich-style" nonce="[0-9a-z]{24}">/u');
  trace();
  }

diff --git a/tests/system/Config/BaseConfigTest.php b/tests/system/Config/BaseConfigTest.php
@@ -20,6 +20,7 @@
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\PreserveGlobalState;
 use PHPUnit\Framework\Attributes\RunInSeparateProcess;
+use PHPUnit\Framework\Attributes\WithoutErrorHandler;
 use PHPUnit\Framework\MockObject\MockObject;
 use RegistrarConfig;
 use RuntimeException;
@@ -35,6 +36,7 @@ final class BaseConfigTest extends CIUnitTestCase
 {
  private string $fixturesFolder;
 
+ #[WithoutErrorHandler]
  protected function setUp(): void
  {
  parent::setUp();
@@ -54,10 +56,6 @@ protected function setUp(): void
  }
 
  BaseConfig::reset();
-
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
  }
 
  protected function tearDown(): void

diff --git a/tests/system/Config/DotEnvTest.php b/tests/system/Config/DotEnvTest.php
@@ -21,6 +21,7 @@
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\PreserveGlobalState;
 use PHPUnit\Framework\Attributes\RunInSeparateProcess;
+use PHPUnit\Framework\Attributes\WithoutErrorHandler;
 use TypeError;
 
 /**
@@ -34,6 +35,7 @@ final class DotEnvTest extends CIUnitTestCase
  private string $path;
  private string $fixturesFolder;
 
+ #[WithoutErrorHandler]
  protected function setUp(): void
  {
  parent::setUp();
@@ -46,10 +48,6 @@ protected function setUp(): void
  $file = 'unreadable.env';
  $path = rtrim($this->fixturesFolder, DIRECTORY_SEPARATOR) . DIRECTORY_SEPARATOR . $file;
  chmod($path, 0644);
-
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
  }
 
  protected function tearDown(): void

diff --git a/tests/system/Config/ServicesTest.php b/tests/system/Config/ServicesTest.php
@@ -52,6 +52,7 @@
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\PreserveGlobalState;
 use PHPUnit\Framework\Attributes\RunInSeparateProcess;
+use PHPUnit\Framework\Attributes\WithoutErrorHandler;
 use RuntimeException;
 use Tests\Support\Config\Services;
 
@@ -63,15 +64,12 @@ final class ServicesTest extends CIUnitTestCase
 {
  private array $original;
 
+ #[WithoutErrorHandler]
  protected function setUp(): void
  {
  parent::setUp();
 
  $this->original = $_SERVER;
-
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
  }
 
  protected function tearDown(): void

diff --git a/tests/system/Events/EventsTest.php b/tests/system/Events/EventsTest.php
@@ -19,6 +19,7 @@
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\PreserveGlobalState;
 use PHPUnit\Framework\Attributes\RunInSeparateProcess;
+use PHPUnit\Framework\Attributes\WithoutErrorHandler;
 
 /**
  * @internal
@@ -31,17 +32,14 @@ final class EventsTest extends CIUnitTestCase
  */
  private Events $manager;
 
+ #[WithoutErrorHandler]
  protected function setUp(): void
  {
  parent::setUp();
 
  $this->manager = new MockEvents();
 
  Events::removeAllListeners();
-
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
  }
 
  protected function tearDown(): void

diff --git a/tests/system/Filters/HoneypotTest.php b/tests/system/Filters/HoneypotTest.php
@@ -24,6 +24,7 @@
 use PHPUnit\Framework\Attributes\Group;
 use PHPUnit\Framework\Attributes\PreserveGlobalState;
 use PHPUnit\Framework\Attributes\RunInSeparateProcess;
+use PHPUnit\Framework\Attributes\WithoutErrorHandler;
 
 /**
  * @internal
@@ -42,6 +43,7 @@ final class HoneypotTest extends CIUnitTestCase
 
  private ?Response $response = null;
 
+ #[WithoutErrorHandler]
  protected function setUp(): void
  {
  parent::setUp();
@@ -51,10 +53,6 @@ protected function setUp(): void
  unset($_POST[$this->honey->name]);
  $_SERVER['REQUEST_METHOD'] = 'POST';
  $_POST[$this->honey->name] = 'hey';
-
- // Workaround for errors on PHPUnit 10 and PHP 8.3.
- // See https://github.com/sebastianbergmann/phpunit/issues/5403#issuecomment-1906810619
- restore_error_handler();
  }
 
  public function testBeforeTriggered(): void