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
Update TestFrameworkFinder to always look for a .bat first. #506
Conversation
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.
Thank you for contributing to Infection. We noticed you didn't add any tests. Could you please add them to make sure everything works as expected?
No need to add more tests, this PR supposed to fix an existing test. |
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.
Thank you for fixing it @sanmai
…n#506) * Update TestFrameworkFinder to always look for a .bat first. Fixes infection#503 * Phpunit_Bat_Wrapper: add cwd to path just for this test * Phpunit_Bat_Wrapper: it does not matter if Infection succeeds, only it must find phpunit.bat
…n#506) * Update TestFrameworkFinder to always look for a .bat first. Fixes infection#503 * Phpunit_Bat_Wrapper: add cwd to path just for this test * Phpunit_Bat_Wrapper: it does not matter if Infection succeeds, only it must find phpunit.bat
* Restrict installation with broken versions of symfony/console Also ignore bogus PHPStan warnings. * Update TestFrameworkFinder to always look for a .bat first. (#506) * Update TestFrameworkFinder to always look for a .bat first. Fixes #503 * Phpunit_Bat_Wrapper: add cwd to path just for this test * Phpunit_Bat_Wrapper: it does not matter if Infection succeeds, only it must find phpunit.bat
…n#506) * Update TestFrameworkFinder to always look for a .bat first. Fixes infection#503 * Phpunit_Bat_Wrapper: add cwd to path just for this test * Phpunit_Bat_Wrapper: it does not matter if Infection succeeds, only it must find phpunit.bat
Fixes #503