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

Check HTTP version and status first, then other asserts #1088

Merged

Conversation

jcamiel
Copy link
Collaborator

@jcamiel jcamiel commented Dec 10, 2022

If HTTP version or status are not correct, we fail the test without running others asserts.

@jcamiel jcamiel linked an issue Dec 10, 2022 that may be closed by this pull request
If HTTP version or status are not correct, we fail the test without running others asserts.
@jcamiel jcamiel force-pushed the 1072-order-of-operations-for-capture-vs-asserts branch from 60ac64b to 6b7753e Compare December 10, 2022 13:44
@fabricereix
Copy link
Collaborator

/accept

@hurl-bot
Copy link
Collaborator

🕗 /accept is running, please wait for completion.

@hurl-bot
Copy link
Collaborator

✅ Pull request accepted and closed by fabricereix with fast forward merge..

# List of commits merged from Orange-OpenSource/hurl/1072-order-of-operations-for-capture-vs-asserts branch into Orange-OpenSource/hurl/master branch:

  • 6b7753e Check HTTP version and status first, then other asserts.

@hurl-bot hurl-bot closed this Dec 10, 2022
@github-actions github-actions bot merged commit 6b7753e into master Dec 10, 2022
@hurl-bot hurl-bot deleted the 1072-order-of-operations-for-capture-vs-asserts branch December 10, 2022 21:22
@jcamiel jcamiel changed the title Check HTTP version and status first, then other asserts. Check HTTP version and status first, then other asserts Dec 26, 2022
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.

Order of operations for Capture vs Asserts
3 participants