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

Add OutputAssertionBuilder::empty() #82

Closed
RazrFalcon opened this issue Jan 21, 2018 · 3 comments
Closed

Add OutputAssertionBuilder::empty() #82

RazrFalcon opened this issue Jan 21, 2018 · 3 comments

Comments

@RazrFalcon
Copy link

Currently, the only way to check that there was no output is to use .stdout().is("").

I think that .stdout().empty() will be more elegant.

@epage
Copy link
Collaborator

epage commented Jan 22, 2018

I personally am a little unsure if the case is common enough to warrant its own check when there is a discoverable check that isn't much longer.

FYI because of #70, this will either turn into .stdout(Output::empty) or .stdout(empty).

@epage
Copy link
Collaborator

epage commented Apr 7, 2018

If we move forward with #98 then this will be covered by assert-rs/predicates-rs#25.

@epage
Copy link
Collaborator

epage commented May 29, 2018

Addressed in assert_cmd
https://github.com/assert-rs/assert_cmd
assert_cli is going to morph into a wider-focused tool, including things like assert_fs. See #41

@epage epage closed this as completed May 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants