-
Notifications
You must be signed in to change notification settings - Fork 821
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
FIX Ensure tests pass by reducing to PHPUnit 7.1.3 #8271
FIX Ensure tests pass by reducing to PHPUnit 7.1.3 #8271
Conversation
PHPUnit 7.1.4 and higher introduces a bug with float comparisons in strings. The fix until this is resolved is to use PHPUnit 7.1.3. Reference: sebastianbergmann/phpunit#3185 Also updates the phpunit config file to match the schema and imports a missing exception namespace.
…s in Travis config See silverstripe/silverstripe-framework#8271 for info on the specific PHPUnit version (temporary)
…s in Travis config See silverstripe/silverstripe-framework#8271 for info on the specific PHPUnit version (temporary)
…s in Travis config See silverstripe/silverstripe-framework#8271 for info on the specific PHPUnit version (temporary)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's good.
The only thing I'm not sure about is that .travis
is still testing against PHP 5.6 and PHP 7.0. It's not testing against PHP 7.2 at all.
@maxime-rainville are you sure? I see this for the builds on this branch: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good. I looked directly at the travis file, but I must have been on the wrong branch.
Gosh, that's a painful regression that you'd think they'd fix a bit more quickly. :( |
PHPUnit 7.1.4 and higher introduces a bug with float comparisons in strings. The fix until this is resolved is to use PHPUnit 7.1.3.
Reference: sebastianbergmann/phpunit#3185
Also updates the phpunit config file to match the schema and imports a missing exception namespace.