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

Use PHPUnit's --filter option #5

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

mcorteel-harel
Copy link

@mcorteel-harel mcorteel-harel commented Nov 5, 2018

The current implementation executes the phpunit command once per test.
In our development context, we use bootstrapping (to reset our database and clear cache in a Symfony project), which adds an important overhead to this command.

So this change makes use of the filter option of PHPUnit to execute only one command. Because only one command is executed, I also removed the result parsing and launch the new command with passthru.

It might prove useful to other projects.

Also, I bumped the version of the PHPUnit dependency which fixes #4!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update PHPUnit dependency to >=7.0
1 participant