Skip to content
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 compatibility with PHPUnit 11 #493

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Nov 3, 2024

This PR fixes a couple of compatibility issues with PHPUnit 11:

  • Data provider methods must be static.
  • The mock builder does not have a setMethods() method anymore.
  • Abstract test classes must not use the suffix Test.

Note: The tests are still run with PHPUnit 8.5 and 9.6 though. The PR merely enables us to use PHPUnit 11 in the future.

@derrabus derrabus requested a review from xabbuh November 3, 2024 16:01
@derrabus derrabus force-pushed the improvement/phpunit-11-compat branch from af88e9d to 1e2cbbc Compare November 6, 2024 11:58
derrabus added a commit that referenced this pull request Nov 6, 2024
This PR was merged into the 3.x-dev branch.

Discussion
----------

do not mock Symfony DI component classes

see #493 (comment)

Commits
-------

5142756 do not mock Symfony DI component classes
@derrabus derrabus force-pushed the improvement/phpunit-11-compat branch from 1e2cbbc to 7c93761 Compare November 6, 2024 12:02
@derrabus derrabus merged commit 33bf5ca into symfony:master Nov 6, 2024
19 checks passed
@xabbuh
Copy link
Member

xabbuh commented Nov 6, 2024

Thank you Alexander.

@derrabus derrabus deleted the improvement/phpunit-11-compat branch November 6, 2024 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants