Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
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 ourterms of service andprivacy 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
Uh oh!
There was an error while loading.Please reload this page.
Conversation
derrabus commentedAug 8, 2020
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 tod1ffb3cCompared1ffb3c tod642d85Comparestof commentedAug 10, 2020
Errors related to PHPUnit 9.3:
Errors related to PHP 8:
|
derrabus commentedAug 10, 2020
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 inb912af9
|
stof commentedAug 10, 2020
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. |
derrabus commentedAug 10, 2020
That's because PhpUnitBridge patches the trait directly into the |
derrabus commentedAug 10, 2020
Also related to php 8: We still see the message |
fabpot commentedAug 12, 2020
Thank you@derrabus. |
stof commentedAug 12, 2020 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
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.