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.

Sign up for GitHub

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

Jump to bottom

runtests.pl no longer stops at first error #11225

Closed
bagder opened this issue May 30, 2023 · 0 comments · Fixed by #11227
Closed

runtests.pl no longer stops at first error #11225

bagder opened this issue May 30, 2023 · 0 comments · Fixed by #11227
Assignees
@dfandrich
Labels
tests

Comments

@bagder
Copy link
Member

bagder commented May 30, 2023

reproduce

~~
$ cd tests
$ ./runtests.pl
~~

This used to stop the tests at first fail (unless -a is used) but now it will run through them all as if -a is always used.

This is with current master on Linux.
@bagder bagder added the tests label May 30, 2023
@dfandrich dfandrich mentioned this issue May 30, 2023
Merged
bch pushed a commit to bch/curl that referenced this issue Jul 19, 2023
@dfandrich @bch
runtests: abort test run after failure without -a
859d500 
This was broken in a recent refactor and test runs would not stop.

Follow-up to d4a1b5b

Reported-by: Daniel Stenberg
Fixes curl#11225
Closes curl#11227
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dfandrich dfandrich

Labels
tests
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

runtests: abort test run after failure without -a curl/curl
2 participants
@bagder @dfandrich