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

Feature: Early termination on test failure #84

Open
erikh opened this issue Nov 16, 2016 · 2 comments
Open

Feature: Early termination on test failure #84

erikh opened this issue Nov 16, 2016 · 2 comments

Comments

@erikh
Copy link

erikh commented Nov 16, 2016

For long running e2e or functional tests, it would be nice if we could terminate the entire test suite at the point of first failure. I haven't found a way to resolve this yet if there is a way to already, but a flag/parameter to control this behavior internally would make this a lot easier and clearer to implement from an end-user perspective.

My earlier attempts were around trying to use the C.Failed() method to determine whether or not I could panic, but c.Failed doesn't report appropriately inside fixture hooks. Seems kinda like a hack anyway. :)

If you're willing to accept the idea, I can take a stab at a patch.

/cc @unclejack

@unclejack
Copy link
Contributor

unclejack commented Nov 16, 2016

This would probably be a good idea to set up at the suite level. This would just make it possible for the suite to bail when a test has failed without any extra code in the test itself.

@barrettj12
Copy link

c.Assert does exactly this.

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

No branches or pull requests

3 participants