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

Allow to filter for multiple test suites #2351

Conversation

lcobucci
Copy link
Contributor

@lcobucci lcobucci commented Nov 7, 2016

This change allows users to run pass multiple test suites on the option testsuite.
e.g.: --testsuite one,two.

I didn't add any unit test since I couldn't find any test that uses the $testSuiteFilter, definitely you guys can help me to do it so.

@lcobucci lcobucci force-pushed the allow-multiple-testsuites-execution branch from e4241ae to 8846853 Compare November 7, 2016 17:53
This change allows users to run pass multiple test suites on the
option `testsuite`. e.g.: `--testsuite one,two`
@lcobucci lcobucci force-pushed the allow-multiple-testsuites-execution branch from 8846853 to b072c85 Compare November 22, 2016 13:50
@lcobucci
Copy link
Contributor Author

@sebastianbergmann @whatthejeff I don't know if you had time to check this but this little patch is being really useful for us, what's your opinion?

@jmauerhan
Copy link

I would love this functionality. :D

@llamicron
Copy link

+1

@sebastianbergmann
Copy link
Owner

Merged manually, thanks.

@lcobucci
Copy link
Contributor Author

@sebastianbergmann thanks!

@sebastianbergmann sebastianbergmann added this to the PHPUnit 6.0 milestone Jan 21, 2017
@sebastianbergmann sebastianbergmann self-assigned this Jan 21, 2017
@sebastianbergmann sebastianbergmann added the type/enhancement A new idea that should be implemented label Jan 21, 2017
@lcobucci lcobucci deleted the allow-multiple-testsuites-execution branch January 23, 2017 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement A new idea that should be implemented
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants