Skip to content
This repository was archived by the owner on Dec 29, 2021. It is now read-only.

The I-don't-care-about-the-status feature #23

Closed
killercup opened this issue Mar 22, 2017 · 2 comments
Closed

The I-don't-care-about-the-status feature #23

killercup opened this issue Mar 22, 2017 · 2 comments
Labels
enhancement help wanted

Comments

@killercup
Copy link
Collaborator

People might not care whether a program succeeds or fails, but we currently default to asserting success and you can only switch to "assert failure" not to "I don't care".

We already talked about this a bit here:
@killercup killercup mentioned this issue Mar 22, 2017
Merged
4 tasks
@epage
Copy link
Collaborator

epage commented Oct 13, 2017

Maybe it could be called .ignore_status()?

@killercup
Copy link
Collaborator Author

killercup commented Oct 13, 2017 via email

@dsprenkels dsprenkels mentioned this issue Oct 17, 2017
Merged
bors bot added a commit that referenced this issue Oct 17, 2017
@bors
Merge #56
7a7fbb5 
56: Add Assert::ignore_status() for ignoring the command status r=epage a=dsprenkels

Implements `Assert::ignore_status()`, which can be used to ignore whether a command exited successfully.

Fixes #23.
@epage epage closed this as completed in #56 Oct 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@killercup @epage