-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Enable testing with PHPUnit 9.x #5314
Conversation
6d17764
to
1a24bfd
Compare
0ecce90
to
4dc8aae
Compare
@@ -62,7 +62,7 @@ protected function doTest($expected, $input = null, \SplFileInfo $file = null) | |||
protected function configureFixerWithAliasedOptions(array $configuration) | |||
{ | |||
if (!$this->fixer instanceof ConfigurationDefinitionFixerInterface) { | |||
throw new \LogicException('Fixer is not configurable'); | |||
throw new \LogicException('Fixer is not configurable.'); |
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.
Fabbot asked for this and below.
I will try to get this up to date with the head branch next Monday. |
@@ -40,9 +40,10 @@ | |||
"php-cs-fixer/accessible-object": "^1.0", | |||
"php-cs-fixer/phpunit-constraint-isidenticalstring": "^1.2", | |||
"php-cs-fixer/phpunit-constraint-xmlmatchesxsd": "^1.2.1", | |||
"phpunit/phpunit": "^5.7.27 || ^6.5.14 || ^7.1", | |||
"phpunit/phpunit": "^5.7.27 || ^6.5.14 || ^7.5.20 || ^8.5.13 || ^9.4.4 <9.5", |
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.
PHPUnit-Bridge is not yet compatible with PHPUnit 9.5. Looks like this.
Reported as symfony/symfony#39329
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.
This should be fixed now.
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 is, but not in a release. Right?
3bc09d8
to
34f663e
Compare
19bf9b2
to
4db3664
Compare
4db3664
to
3d75d69
Compare
3d75d69
to
c8acb1b
Compare
@@ -74,12 +74,27 @@ public function testDeploymentJobsRunOnLatestStablePhpThatIsSupportedByTool() | |||
} | |||
} | |||
|
|||
private static function assertUpcomingPhpVersionIsCoveredByCiJob($lastSupportedVersion, array $ciVersions) | |||
private static function ensureTraversableContainsIsAvailable() |
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.
What is the purpose here?
It is named "ensure TraversableContains
is available" and when it does not exist we mark it as skipped.
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.
Yup, that's what it does. Is there a problem?
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.
Contrary to what the diff shows, assertUpcomingPhpVersionIsCoveredByCiJob
is still here.
9d3044e
to
6712f4e
Compare
Diff should be much smaller now. Please check it out. |
bd6ed51
to
40cef3a
Compare
- Replace setUp/tearDown. - Refactor propery-accessing static methods. - Fix fabbot.io notices. - Bumps timeout for small tests. - Update composer.json. Co-authored-by: Alexey Kopytko <alexey@kopytko.com>
40cef3a
to
4dee7ec
Compare
Thank you @sanmai. |
thanks for all the work @sanmai 👍 |
Replaces #5262
Uses PHPUnit 9.4 on PHP 8: https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/5314/checks?check_run_id=1491483701#step:9:9