Skip to content

Support PHPUnit its --only-repeat-failed option#218

Closed
PieterDC wants to merge 18 commits intoparatestphp:masterfrom
MaartenElsen:only-repeat-failed
Closed

Support PHPUnit its --only-repeat-failed option#218
PieterDC wants to merge 18 commits intoparatestphp:masterfrom
MaartenElsen:only-repeat-failed

Conversation

@PieterDC
Copy link

PHPUnit doesn't have an option to only repeat failed tests yet. But we provided a pull request for it: sebastianbergmann/phpunit#1965

And we coded ParaTest support for --only-repeat-failed, building upon what has been done for the --repeat option: #217

Maarten Elsen and others added 18 commits April 19, 2016 10:27
# Conflicts:
#	src/ParaTest/Console/Commands/ParaTestCommand.php
#	src/ParaTest/Runners/PHPUnit/BaseRunner.php
#	src/ParaTest/Runners/PHPUnit/Options.php
…the PHPUnit library. Tests run once by default.
# Conflicts:
#	test/unit/ParaTest/Console/Commands/ParaTestCommandTest.php
…the PHPUnit library. Tests run once by default. - part 2.
# Conflicts:
#	test/unit/ParaTest/Console/Commands/ParaTestCommandTest.php
@julianseeger
Copy link
Contributor

Wow what happened to your history? That's 8 lines per commit. I guess that's what happens when you really apply commit early, commit often ;)

For the phpunit PR: I think you need to direct every PR to the master. It then get's merged to all maintained releases (if they whish so).

@julianseeger
Copy link
Contributor

@PieterDC can you please give me an update on this? Your PR for phpunit has been closed and you didn't link any new one that targeted a current phpunit version. If that's no longer an option, I would close this and #217 ... even though a repeat-option would be nice.

@julianseeger
Copy link
Contributor

Without the --only-repeat-failed option in phpunit, there is nothing to support in paratest

@PieterDC
Copy link
Author

PieterDC commented Mar 3, 2017

I haven't made progress on sebastianbergmann/phpunit#1965 Also not in a separate issue. Also not on my local machine. All code I have/had concerning that issue is pushed.

I agree that

Without the --only-repeat-failed option in phpunit, there is nothing to support in paratest

So the PHPUnit feature request ideally has to be finished first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants