-
-
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
PHPUnit - fix to assertCount #3750
Labels
Comments
I believe that this was not an configuration for fixer that we decided to not fix, but we actually not have fixer capable to do it out of the box, by just adjusting the config (like internal list of methods, not settings exposed to end-user) of it. |
just leaving my notes on this here;
|
keradus
added a commit
that referenced
this issue
Jun 2, 2018
…acePossum) This PR was squashed before being merged into the 2.12-dev branch (closes #3785). Discussion ---------- PhpUnitDedicateAssertFixer - fix to assertCount too closes #3750 ``` 'assert same' => [ '<?php $this->assertCount(1, $a);', '<?php $this->assertSame(1, count($a));', ], ``` for more example please see the tests Commits ------- cf8221b PhpUnitDedicateAssertFixer - fix to assertCount too
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
A few years back there was an issue with
assertCount
when using an iterator on HHVM (sebastianbergmann/phpunit#1413), as such we don't fix asserts in PHPUnit tests like;Since than;
Therefor I suggest to add it.
The text was updated successfully, but these errors were encountered: