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

Add a --failfast option to stop testing after the first failure #2290

Closed
wants to merge 1 commit into from

Conversation

lamby
Copy link
Contributor

@lamby lamby commented Jan 16, 2015

Whilst misleading if run from Jenkins or from a general regression point of
view (where knowing about all failures is invariably more useful), bailing
out on the first failure is more appropriate behaviour in a "REPL", TDD or
simply a quick "hack it until the tests pass" mode of coding.

Signed-off-by: Chris Lamb chris@chris-lamb.co.uk

Whilst misleading if run from Jenkins or from a general regression point of
view (where knowing about all failures is invariably more useful), bailing
out on the first failure is more appropriate behaviour in a "REPL", TDD or
simply a quick "hack it until the tests pass" mode of coding.

Signed-off-by: Chris Lamb <chris@chris-lamb.co.uk>
@lamby
Copy link
Contributor Author

lamby commented Dec 9, 2017

Just wondering if you've had any thoughts on this after a couple of years? :)

@lamby
Copy link
Contributor Author

lamby commented Aug 25, 2018

Any update on this, NACK/ACK wise?

@antirez
Copy link
Contributor

antirez commented Aug 26, 2018

Does not apply but maybe this option is already available, even if with a different semantics, that is the new --stop option. But apparently you want it to exit with an error?

@lamby
Copy link
Contributor Author

lamby commented Aug 26, 2018

Ooh. --stop works for me; it's more about the REPL case after all.

(--stop added in 26897c0)

@lamby lamby closed this Aug 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants