Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: update PHPUnit to 10 #8069

Merged
merged 17 commits into from
Apr 7, 2024
Merged

Conversation

kenjis
Copy link
Member

@kenjis kenjis commented Oct 20, 2023

Important

This PR will be merged just prior to the release of v4.5.0.

Description
PHPUnit requires PHP 8.1 or later.
https://packagist.org/packages/phpunit/phpunit#10.4.1

https://phpunit.de/announcements/phpunit-10.html

TODO

  • update nexusphp/tachycardia
  • fix errors on PHP 8.3

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@kenjis kenjis added the 4.5 label Oct 20, 2023
@kenjis kenjis marked this pull request as draft October 20, 2023 23:57
@kenjis
Copy link
Member Author

kenjis commented Oct 20, 2023

@paulbalandan Any plan to release nexusphp/tachycardia that are compatible with PHPUnit10?

@paulbalandan
Copy link
Member

Is it ready but lacking integration tests. I'll try to finish this weekend. Thanks for reminding me.

@kenjis
Copy link
Member Author

kenjis commented Oct 21, 2023

There was 1 PHPUnit test runner warning:

1) Class CodeIgniter\Cache\Handlers\AbstractHandlerTest declared in /home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Cache/Handlers/AbstractHandlerTest.php is abstract

--

There was 1 PHPUnit test runner deprecation:

1) Your XML configuration validates against a deprecated schema. Migrate your XML configuration using "--migrate-configuration"!

--

6 tests triggered 6 PHPUnit deprecations:

1) CodeIgniter\AutoReview\FrameworkCodeTest::testEachTestClassHasCorrectGroupAnnotation
Data Provider method CodeIgniter\AutoReview\FrameworkCodeTest::provideEachTestClassHasCorrectGroupAnnotation() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/AutoReview/FrameworkCodeTest.php:46

2) CodeIgniter\HTTP\SiteURITest::testConstructor
Data Provider method CodeIgniter\HTTP\SiteURITest::provideConstructor() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/HTTP/SiteURITest.php:32

3) CodeIgniter\Validation\CreditCardRulesTest::testValidCCNumber
Data Provider method CodeIgniter\Validation\CreditCardRulesTest::provideValidCCNumber() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Validation/CreditCardRulesTest.php:54

4) CodeIgniter\Validation\StrictRules\CreditCardRulesTest::testValidCCNumber
Data Provider method CodeIgniter\Validation\StrictRules\CreditCardRulesTest::provideValidCCNumber() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Validation/StrictRules/CreditCardRulesTest.php:55

5) CodeIgniter\Validation\StrictRules\ValidationTest::testSetRuleRulesFormat
Data Provider method CodeIgniter\Validation\StrictRules\ValidationTest::provideSetRuleRulesFormat() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Validation/ValidationTest.php:193

6) CodeIgniter\Validation\ValidationTest::testSetRuleRulesFormat
Data Provider method CodeIgniter\Validation\ValidationTest::provideSetRuleRulesFormat() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Validation/ValidationTest.php:193

WARNINGS!
Tests: 300, Assertions: 905, Warnings: 1, Deprecations: 7.

https://github.com/codeigniter4/CodeIgniter4/actions/runs/6593809502/job/17916840733?pr=8069

@kenjis
Copy link
Member Author

kenjis commented Oct 21, 2023

There was 1 PHPUnit test runner warning:

1) Class CodeIgniter\Cache\Handlers\AbstractHandlerTest declared in /home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Cache/Handlers/AbstractHandlerTest.php is abstract

--

There was 1 PHPUnit test runner deprecation:

1) Your XML configuration validates against a deprecated schema. Migrate your XML configuration using "--migrate-configuration"!

--

There were 41 errors:

1) CodeIgniter\Autoloader\AutoloaderTest::testLoadHelpers
Test was run in child process and ended unexpectedly

2) CodeIgniter\CodeIgniterTest::testRunEmptyDefaultRoute
Test was run in child process and ended unexpectedly

3) CodeIgniter\CodeIgniterTest::testOutputBufferingControl
Test was run in child process and ended unexpectedly

4) CodeIgniter\CodeIgniterTest::testRunEmptyDefaultRouteReturnResponse
Test was run in child process and ended unexpectedly

5) CodeIgniter\CodeIgniterTest::testRunClosureRoute
Test was run in child process and ended unexpectedly

6) CodeIgniter\CodeIgniterTest::testRun404Override
Test was run in child process and ended unexpectedly

7) CodeIgniter\CodeIgniterTest::testRun404OverrideControllerReturnsResponse
Test was run in child process and ended unexpectedly

8) CodeIgniter\CodeIgniterTest::testRun404OverrideReturnResponse
Test was run in child process and ended unexpectedly

9) CodeIgniter\CodeIgniterTest::testRun404OverrideByClosure
Test was run in child process and ended unexpectedly

1) CodeIgniter\Validation\StrictRules\ValidationTest::testPlaceholderReplacementTestFails
Failed asserting that exception message 'Failed asserting that 'filter[{id}]' [ASCII](length: 12) does not contain "{id}" [ASCII](length: 4).' contains 'Failed asserting that 'filter[{id}]' does not contain "{id}".'.

2) CodeIgniter\Validation\ValidationTest::testPlaceholderReplacementTestFails
Failed asserting that exception message 'Failed asserting that 'filter[{id}]' [ASCII](length: 12) does not contain "{id}" [ASCII](length: 4).' contains 'Failed asserting that 'filter[{id}]' does not contain "{id}".'.

--

6 tests triggered 6 PHPUnit deprecations:

1) CodeIgniter\AutoReview\FrameworkCodeTest::testEachTestClassHasCorrectGroupAnnotation
Data Provider method CodeIgniter\AutoReview\FrameworkCodeTest::provideEachTestClassHasCorrectGroupAnnotation() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/AutoReview/FrameworkCodeTest.php:46

2) CodeIgniter\HTTP\SiteURITest::testConstructor
Data Provider method CodeIgniter\HTTP\SiteURITest::provideConstructor() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/HTTP/SiteURITest.php:32

3) CodeIgniter\Validation\CreditCardRulesTest::testValidCCNumber
Data Provider method CodeIgniter\Validation\CreditCardRulesTest::provideValidCCNumber() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Validation/CreditCardRulesTest.php:54

4) CodeIgniter\Validation\StrictRules\CreditCardRulesTest::testValidCCNumber
Data Provider method CodeIgniter\Validation\StrictRules\CreditCardRulesTest::provideValidCCNumber() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Validation/StrictRules/CreditCardRulesTest.php:55

5) CodeIgniter\Validation\StrictRules\ValidationTest::testSetRuleRulesFormat
Data Provider method CodeIgniter\Validation\StrictRules\ValidationTest::provideSetRuleRulesFormat() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Validation/ValidationTest.php:193

6) CodeIgniter\Validation\ValidationTest::testSetRuleRulesFormat
Data Provider method CodeIgniter\Validation\ValidationTest::provideSetRuleRulesFormat() is not static

/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Validation/ValidationTest.php:193

ERRORS!
Tests: 5017, Assertions: 8400, Errors: 41, Failures: 2, Warnings: 1, Deprecations: 7, Skipped: 1.

https://github.com/codeigniter4/CodeIgniter4/actions/runs/6593809497/job/17916881103?pr=8069

@paulbalandan
Copy link
Member

The first warning is easy. Just change the name to AbstractHandlerTestCase. Abstract tests need to be suffixed with TestCase.

For the static issues, just run the rule php_unit_data_provider_static for php-cs-fixer

@kenjis
Copy link
Member Author

kenjis commented Oct 21, 2023

Unfortunately php-cs-fixer was not able to refactor the dataProviders because of $this.

@paulbalandan
Copy link
Member

Ah yes. The rule is risky because of uses of $this. Those cases should be modified to use static properties/methods.

@kenjis
Copy link
Member Author

kenjis commented Oct 21, 2023

Why does this test fail?

$ vendor/bin/phpunit tests/system/Validation/StrictRules/ValidationTest.php 
PHPUnit 10.4.1 by Sebastian Bergmann and contributors.

Runtime:       PHP 8.1.24
Configuration: /Users/kenji/work/codeigniter/official/CodeIgniter4/phpunit.xml

.................................................................................................... 100 / 121 ( 82%)
...........F.........                                                                                121 / 121 (100%)

Time: 00:00.930, Memory: 16.00 MB

...


--

There was 1 failure:

1) CodeIgniter\Validation\StrictRules\ValidationTest::testPlaceholderReplacementTestFails
Failed asserting that exception message 'Failed asserting that 'filter[{id}]' [ASCII](length: 12) does not contain "{id}" [ASCII](length: 4).' contains 'Failed asserting that 'filter[{id}]' does not contain "{id}".'.

FAILURES!
Tests: 121, Assertions: 173, Failures: 1, Warnings: 1.

@kenjis
Copy link
Member Author

kenjis commented Oct 21, 2023

I got the reason. The error message of PHPUnit has been changed.

Copy link
Member

@MGatner MGatner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will want to update DevKit as well - not sure when the appropriate time to do that is though, maybe when we actually drop 8.0? I know @paulbalandan is working on Tachycardia comparability - that would be great to include.

Edit: oops! I missed the existing conversation. All good.

@paulbalandan
Copy link
Member

nexusphp/tachycardia v2.0 is out

@kenjis kenjis force-pushed the update-phpunit-10 branch 2 times, most recently from 2052f1a to a524abc Compare October 27, 2023 04:39
@kenjis
Copy link
Member Author

kenjis commented Oct 27, 2023

I don't know why, but only in PHP 8.3, the NoTestCaseObjectOnCallStackException occurs.

$ vendor/bin/phpunit tests/system/Autoloader/AutoloaderTest.php 
PHPUnit 10.4.2 by Sebastian Bergmann and contributors.

Runtime:       PHP 8.3.0-dev
Configuration: /Users/kenji/work/codeigniter/official/CodeIgniter4/phpunit.xml

..........................E                                                                             27 / 27 (100%)

Time: 00:00.486, Memory: 16.00 MB

There was 1 error:

1) CodeIgniter\Autoloader\AutoloaderTest::testLoadHelpers
PHPUnit\Framework\Exception: PHP Fatal error:  Uncaught CodeIgniter\Format\Exceptions\FormatException: Failed to parse JSON string. Error: Type is not supported in /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Format/JSONFormatter.php:44
Stack trace:
#0 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Format/JSONFormatter.php(44): CodeIgniter\Format\Exceptions\FormatException::forInvalidJSON('Type is not sup...')
#1 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/API/ResponseTrait.php(345): CodeIgniter\Format\JSONFormatter->format(Array)
#2 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/API/ResponseTrait.php(98): CodeIgniter\Debug\ExceptionHandler->format(Array)
#3 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Debug/ExceptionHandler.php(81): CodeIgniter\Debug\ExceptionHandler->respond(Array, 500)
#4 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Debug/Exceptions.php(148): CodeIgniter\Debug\ExceptionHandler->handle(Object(PHPUnit\Event\Code\NoTestCaseObjectOnCallStackException), Object(CodeIgniter\HTTP\IncomingRequest), Object(CodeIgniter\HTTP\Response), 500, 1)
#5 [internal function]: CodeIgniter\Debug\Exceptions->exceptionHandler(Object(PHPUnit\Event\Code\NoTestCaseObjectOnCallStackException))
#6 {main}
  thrown in /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Format/JSONFormatter.php on line 44

ERRORS!
Tests: 27, Assertions: 45, Errors: 1.

@kenjis kenjis added the help wanted More help is needed for the proper resolution of an issue or pull request label Oct 27, 2023
@kenjis
Copy link
Member Author

kenjis commented Oct 27, 2023

--- a/system/Format/JSONFormatter.php
+++ b/system/Format/JSONFormatter.php
@@ -37,6 +37,7 @@ class JSONFormatter implements FormatterInterface
 
         $options = ENVIRONMENT === 'production' ? $options : $options | JSON_PRETTY_PRINT;
 
+        log_message('debug', var_export($data, true));
         $result = json_encode($data, $options, 512);
 
         if (! in_array(json_last_error(), [JSON_ERROR_NONE, JSON_ERROR_RECURSION], true)) {
CRITICAL - 2023-10-27 07:08:34 --> Cannot find TestCase object on call stack
in VENDORPATH/phpunit/phpunit/src/Event/Value/Test/TestMethodBuilder.php on line 63.
 1 VENDORPATH/phpunit/phpunit/src/Runner/ErrorHandler.php(83): PHPUnit\Event\Code\TestMethodBuilder::fromCallStack()
 2 [internal function]: PHPUnit\Runner\ErrorHandler->__invoke(2, 'rewind(): Stream does not support seeking', 'Standard input code', 71)
 3 Standard input code(71): rewind(resource (stream))
 4 Standard input code(118): __phpunit_run_isolated_test()
DEBUG - 2023-10-27 07:08:34 --> array (
  'title' => 'PHPUnit\\Event\\Code\\NoTestCaseObjectOnCallStackException',
  'type' => 'PHPUnit\\Event\\Code\\NoTestCaseObjectOnCallStackException',
  'code' => 500,
  'message' => 'Cannot find TestCase object on call stack',
  'file' => '/Users/kenji/work/codeigniter/official/CodeIgniter4/vendor/phpunit/phpunit/src/Event/Value/Test/TestMethodBuilder.php',
  'line' => 63,
  'trace' => 
  array (
    0 => 
    array (
      'file' => '/Users/kenji/work/codeigniter/official/CodeIgniter4/vendor/phpunit/phpunit/src/Runner/ErrorHandler.php',
      'line' => 83,
      'function' => 'fromCallStack',
      'class' => 'PHPUnit\\Event\\Code\\TestMethodBuilder',
      'type' => '::',
      'args' => 
      array (
      ),
    ),
    1 => 
    array (
      'function' => '__invoke',
      'class' => 'PHPUnit\\Runner\\ErrorHandler',
      'type' => '->',
      'args' => 
      array (
        0 => 2,
        1 => 'rewind(): Stream does not support seeking',
        2 => 'Standard input code',
        3 => 71,
      ),
    ),
    2 => 
    array (
      'file' => 'Standard input code',
      'line' => 71,
      'function' => 'rewind',
      'args' => 
      array (
        0 => NULL,
      ),
    ),
    3 => 
    array (
      'file' => 'Standard input code',
      'line' => 118,
      'function' => '__phpunit_run_isolated_test',
      'args' => 
      array (
      ),
    ),
  ),
)
CRITICAL - 2023-10-27 07:08:34 --> Uncaught CodeIgniter\Format\Exceptions\FormatException: Failed to parse JSON string. Error: Type is not supported in /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Format/JSONFormatter.php:44
Stack trace:
#0 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Format/JSONFormatter.php(44): CodeIgniter\Format\Exceptions\FormatException::forInvalidJSON('Type is not sup...')
#1 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/API/ResponseTrait.php(345): CodeIgniter\Format\JSONFormatter->format(Array)
#2 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/API/ResponseTrait.php(98): CodeIgniter\Debug\ExceptionHandler->format(Array)
#3 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Debug/ExceptionHandler.php(81): CodeIgniter\Debug\ExceptionHandler->respond(Array, 500)
#4 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Debug/Exceptions.php(148): CodeIgniter\Debug\ExceptionHandler->handle(Object(PHPUnit\Event\Code\NoTestCaseObjectOnCallStackException), Object(CodeIgniter\HTTP\IncomingRequest), Object(CodeIgniter\HTTP\Response), 500, 1)
#5 [internal function]: CodeIgniter\Debug\Exceptions->exceptionHandler(Object(PHPUnit\Event\Code\NoTestCaseObjectOnCallStackException))
#6 {main}
  thrown
【Previous Exception】
PHPUnit\Event\Code\NoTestCaseObjectOnCallStackException
Cannot find TestCase object on call stack
#0 /Users/kenji/work/codeigniter/official/CodeIgniter4/vendor/phpunit/phpunit/src/Runner/ErrorHandler.php(83): PHPUnit\Event\Code\TestMethodBuilder::fromCallStack()
#1 [internal function]: PHPUnit\Runner\ErrorHandler->__invoke(2, 'rewind(): Strea...', 'Standard input ...', 71)
#2 Standard input code(71): rewind(Resource id #2)
#3 Standard input code(118): __phpunit_run_isolated_test()
#4 {main}
in SYSTEMPATH/Format/JSONFormatter.php on line 44.
 1 [internal function]: CodeIgniter\Debug\Exceptions->shutdownHandler()
DEBUG - 2023-10-27 07:08:34 --> array (
  'title' => 'ErrorException',
  'type' => 'ErrorException',
  'code' => 500,
  'message' => 'Uncaught CodeIgniter\\Format\\Exceptions\\FormatException: Failed to parse JSON string. Error: Type is not supported in /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Format/JSONFormatter.php:44
Stack trace:
#0 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Format/JSONFormatter.php(44): CodeIgniter\\Format\\Exceptions\\FormatException::forInvalidJSON(\'Type is not sup...\')
#1 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/API/ResponseTrait.php(345): CodeIgniter\\Format\\JSONFormatter->format(Array)
#2 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/API/ResponseTrait.php(98): CodeIgniter\\Debug\\ExceptionHandler->format(Array)
#3 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Debug/ExceptionHandler.php(81): CodeIgniter\\Debug\\ExceptionHandler->respond(Array, 500)
#4 /Users/kenji/work/codeigniter/official/CodeIgniter4/system/Debug/Exceptions.php(148): CodeIgniter\\Debug\\ExceptionHandler->handle(Object(PHPUnit\\Event\\Code\\NoTestCaseObjectOnCallStackException), Object(CodeIgniter\\HTTP\\IncomingRequest), Object(CodeIgniter\\HTTP\\Response), 500, 1)
#5 [internal function]: CodeIgniter\\Debug\\Exceptions->exceptionHandler(Object(PHPUnit\\Event\\Code\\NoTestCaseObjectOnCallStackException))
#6 {main}
  thrown
【Previous Exception】
PHPUnit\\Event\\Code\\NoTestCaseObjectOnCallStackException
Cannot find TestCase object on call stack
#0 /Users/kenji/work/codeigniter/official/CodeIgniter4/vendor/phpunit/phpunit/src/Runner/ErrorHandler.php(83): PHPUnit\\Event\\Code\\TestMethodBuilder::fromCallStack()
#1 [internal function]: PHPUnit\\Runner\\ErrorHandler->__invoke(2, \'rewind(): Strea...\', \'Standard input ...\', 71)
#2 Standard input code(71): rewind(Resource id #2)
#3 Standard input code(118): __phpunit_run_isolated_test()
#4 {main}',
  'file' => '/Users/kenji/work/codeigniter/official/CodeIgniter4/system/Format/JSONFormatter.php',
  'line' => 44,
  'trace' => 
  array (
    0 => 
    array (
      'function' => 'shutdownHandler',
      'class' => 'CodeIgniter\\Debug\\Exceptions',
      'type' => '->',
      'args' => 
      array (
      ),
    ),
  ),
)

@github-actions github-actions bot added the stale Pull requests with conflicts label Oct 28, 2023
@github-actions

This comment was marked as outdated.

@kenjis kenjis removed the stale Pull requests with conflicts label Oct 30, 2023

This comment was marked as outdated.

@github-actions github-actions bot added the stale Pull requests with conflicts label Nov 30, 2023
@kenjis kenjis removed the stale Pull requests with conflicts label Nov 30, 2023
@kenjis
Copy link
Member Author

kenjis commented Nov 30, 2023

No good. Tests on PHP 8.3 fails.

There were 42 errors:

1) CodeIgniter\Autoloader\AutoloaderTest::testLoadHelpers
Test was run in child process and ended unexpectedly

2) CodeIgniter\CodeIgniterTest::testRunEmptyDefaultRoute
Test was run in child process and ended unexpectedly

3) CodeIgniter\CodeIgniterTest::testOutputBufferingControl
Test was run in child process and ended unexpectedly

https://github.com/codeigniter4/CodeIgniter4/actions/runs/7043858533/job/19170759175?pr=8069

@kenjis kenjis force-pushed the update-phpunit-10 branch 2 times, most recently from 5b2f18f to 47bf62e Compare April 2, 2024 23:26
@kenjis kenjis marked this pull request as ready for review April 2, 2024 23:45
@kenjis kenjis changed the title chore: update PHPUnit to 10 refactor: update PHPUnit to 10 Apr 2, 2024
kenjis added 17 commits April 7, 2024 12:38
vendor/bin/phpunit --migrate-configuration -c phpunit.xml.dist
Message:  file_put_contents(/home/runner/work/CodeIgniter4/CodeIgniter4/build/coverage/text/coverage.txt): Failed to open stream: No such file or directory
Location: /home/runner/work/CodeIgniter4/CodeIgniter4/vendor/phpunit/phpunit/src/Runner/CodeCoverage.php:317

Script done.
Error: Process completed with exit code 255.
1) CodeIgniter\Database\DatabaseTestCase\DatabaseTestCaseMigrationOnce1Test::testMigrationDone
TypeError: json_encode(): Argument #2 ($flags) must be of type int, bool given

/home/runner/work/CodeIgniter4/CodeIgniter4/system/Test/Constraints/SeeInDatabase.php:118
/home/runner/work/CodeIgniter4/CodeIgniter4/system/Test/DatabaseTestTrait.php:282
/home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Database/DatabaseTestCase/DatabaseTestCaseMigrationOnce1Test.php:86
There was 1 error:

 1) CodeIgniter\Security\SecurityCSRFSessionRandomizeTokenTest::testCSRFVerifyPostInvalidToken
PHPUnit\Framework\Exception: PHP Warning:  hex2bin(): Hexadecimal input string must have an even length in /home/runner/work/CodeIgniter4/CodeIgniter4/system/Security/Security.php on line 378
PHP Warning:  hex2bin(): Hexadecimal input string must have an even length in /home/runner/work/CodeIgniter4/CodeIgniter4/system/Security/Security.php on line 378
There was 1 error:

1) CodeIgniter\CommonFunctionsTest::testTraceWithCSP
Test was run in child process and ended unexpectedly
@kenjis
Copy link
Member Author

kenjis commented Apr 7, 2024

Rebased to resolve the conflict.

@kenjis kenjis merged commit 5c45ef1 into codeigniter4:4.5 Apr 7, 2024
38 checks passed
@kenjis kenjis deleted the update-phpunit-10 branch April 7, 2024 03:57
@MGatner
Copy link
Member

MGatner commented Apr 10, 2024

Happy to see this one launched!

@kenjis
Copy link
Member Author

kenjis commented Apr 10, 2024

But we still need restore_error_handler() to separate process testing.
I don't know the true cause of the errors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.5 help wanted More help is needed for the proper resolution of an issue or pull request refactor Pull requests that refactor code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants