Conversation
eb431f7 to
9040637
Compare
|
Thanks but it breaks 5.3 though.. so I guess I'll keep this open for now but it'll not be mergeable until we drop old PHP versions.. Given how little this library change I am not really keen on doing this right now. |
|
Hi @Seldaek, thanks for fixing the test suite. I can now also see that this breaks PHP 5.3, which was never my intention! I have cherry-picked the commit that marks all data providers as static to a new PR: #163. Can you please check that one? I think I will close this PR as soon as the other PR is merged; that way, it's still documented how the test suite can be made compatible with PHPUnit 10. Thanks! Aad |
|
Yup makes sense thanks for that. But I'd still keep this open as if it's closed it'll get lost imo. |
… PHPUnit 10 This is done by renaming the inherited `setUp` method to a new `setUpTestCase` method and adding an `@before` annotation.
2a52e3c to
2ed85d9
Compare
…n available
This resolves the following error when running the test suite using PHPUnit 10:
Call to undefined method PHPUnit\Framework\MockObject\MockBuilder::setMethods()
2ed85d9 to
3ad9b77
Compare
|
Hi @Seldaek, I have managed to change the PR such that all changes still work with PHP 5.3 and PHPUnit 4.8. Can you please check this PR again? Thanks! Aad |
|
Thanks yes looks good. |
Prepare test classes for use with PHPUnit 10 while maintaining compatibility with older versions of PHPUnit:
onlyMethodsinstead ofsetMethodsonMockBuilderinstance when available, assetMethodshas been removed in PHPUnit 10.voidreturn type of the inheritedsetUpmethod in PHPUnit 10.Mark all data providers as static as non-static data providers are deprecated in PHPUnit 10(this is already merged via Mark all data providers in test suite as static #163).Note that the
symfony/phpunit-bridgepackage does not support PHPunit 10. So in order to run the test suite with this version, the 10.x version of the tool should be installed as a PHAR or via Composer (composer require --dev phpunit/phpunit:~10.5).