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 some more tests when running phpunit with args #11153

Merged
merged 2 commits into from
Feb 16, 2025

Conversation

MoonE
Copy link
Contributor

@MoonE MoonE commented Nov 10, 2024

Related to #11122

Running this failed because of the argv values

./vendor/bin/phpunit --filter '/^Psalm\\Tests\\VariadicTest/'

@MoonE MoonE force-pushed the fix-variadic-test-when-run-with-args branch from 469077b to db4b0d9 Compare November 11, 2024 22:44
@MoonE MoonE changed the title Fix variadic test when run with args Fix some more tests when running phpunit with args Nov 11, 2024
e.g:
./vendor/bin/phpunit --filter 'Psalm\\Tests\\VariadicTest'
Call parent setUpBeforeClaass method instead of duplicating code.

Use the VersionUtils class also in AsyncTest instead of hard-coded values.
@MoonE MoonE changed the base branch from 5.x to 6.x February 16, 2025 19:36
@MoonE MoonE force-pushed the fix-variadic-test-when-run-with-args branch from db4b0d9 to e104983 Compare February 16, 2025 19:36
@orklah orklah added the release:fix The PR will be included in 'Fixes' section of the release notes label Feb 16, 2025
@orklah
Copy link
Collaborator

orklah commented Feb 16, 2025

Thanks!

@orklah orklah merged commit d24ebc5 into vimeo:6.x Feb 16, 2025
50 of 52 checks passed
@MoonE MoonE deleted the fix-variadic-test-when-run-with-args branch February 16, 2025 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:fix The PR will be included in 'Fixes' section of the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants