-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
Use PHPUnit 9.3 on php 8 #37771
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
Use PHPUnit 9.3 on php 8 #37771
Conversation
I'm browsing through all failures on the php 8 run at the moment and see if I can fix or file separate issues for them. |
97b8005
to
d1ffb3c
Compare
d1ffb3c
to
d642d85
Compare
Errors related to PHPUnit 9.3:
Errors related to PHP 8:
|
Thanks for the overview. I can look into the individual issues in the upcoming days, but nevertheless I think that we can merge this PR already. It improves the php 8 CI from "completely broken" to "just a little broken". 😃 A few notes:
➡️ #37780
It does already: symfony/src/Symfony/Bridge/PhpUnit/Legacy/PolyfillAssertTrait.php Lines 355 to 358 in b912af9
|
then that's even easier. We can migrate to use the new method everywhere. But this polyfill is weird. It never calls the parent implementation. |
That's because PhpUnitBridge patches the trait directly into the |
Also related to php 8: We still see the message |
Thank you @derrabus. |
for reference, this is now solved. |
Our CI for PHP 8 is currently a blind spot, mainly because the PHPUnit version 8.3 that we use for the test suite is incompatible with php 8. This PR changes the PHPUnit version used on Travis for php 8 to PHPUnit 9.3.
Our test suite might not be 100% compatible with that new PHPUnit release yet, but this change should allow us to run most tests on php 8 again and enable us to iteratively migrate to PHPUnit 9.3.