No Code Attached Yet ?
avatar PhilETaylor
PhilETaylor
24 Dec 2021

Opening an issue because currently this is just being ignored.

This issue is not about DEPRECATIONS... enough traction is already being made on that.

This issue is about PHPUnit 8 not being reliable for the future and some investment work is needed to move to PHPUnit 9 asap.

Every single build fails (silently) because the unit tests for PHP 8.1 fail.

Screenshot 2021-12-24 at 17 38 53

This is because PHP unit 8 doesn't support PHP 8/8.1 Union Types (Expected behaviour)

Screenshot 2021-12-24 at 17 40 20

PHP 8.1.x is the current PHP version. All other prior versions are now "old, dead dying". PHP 8.0 only has 11 months of life left.

Joomla needs to migrate to PHPUnit 9 and refactor any failing tests to ensure that the test suite passes again.

Simply upgrading the PHPUnit to PHPUnit 9 is NOT ENOUGH.

At the moment JOOMLA IS NOT RELYING ON UNIT TESTS ON PHP 8.1 as they are just failing. But this is an untenable position to be in. This needs fixing the sooner the better.

Yes I know there is only limited test coverage anyway, but some tests are better than no tests.

PHPUnit 9 requires PHP 7.3 (or newer), Joomla 4 requires PHP 7.2 - OR NEWER, without tests passing on PHP 8.1 Joomla cannot categorically state that Joomla runs on PHP 8.1

https://thephp.cc/articles/migrating-to-phpunit-9

avatar PhilETaylor PhilETaylor - open - 24 Dec 2021
avatar joomla-cms-bot joomla-cms-bot - change - 24 Dec 2021
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 24 Dec 2021
avatar PhilETaylor PhilETaylor - change - 24 Dec 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 24 Dec 2021
avatar PhilETaylor PhilETaylor - change - 24 Dec 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 24 Dec 2021
avatar alikon alikon - change - 27 Dec 2021
Labels Added: ?
avatar alikon alikon - labeled - 27 Dec 2021
avatar PhilETaylor PhilETaylor - change - 27 Dec 2021
Title
PHP 8.1 Unit Testing
Unit Testing, PHPUnit 8 -> 9 Migration
avatar PhilETaylor PhilETaylor - edited - 27 Dec 2021
avatar PhilETaylor PhilETaylor - change - 28 Dec 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 28 Dec 2021
avatar PhilETaylor
PhilETaylor - comment - 28 Dec 2021

Closed as no interest in moving to 9 yet

avatar PhilETaylor PhilETaylor - change - 28 Dec 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-12-28 16:08:58
Closed_By PhilETaylor
avatar PhilETaylor PhilETaylor - close - 28 Dec 2021

Add a Comment

Login with GitHub to post a comment