with_stdin will take the ownership of command

[mssun]

The with_stdin will take the ownership of command. Therefore you cannot write chained functions like this:

Command::new("cat")
    .arg("-A")
    .with_stdin("42")
    .assert()
    .success();

Error:

error[E0507]: cannot move out of borrowed content
  --> /xxx.rs:43:5
   |
43 | /     Command::new("cat")
44 | |         .arg("-A")
   | |__________________^ cannot move out of borrowed content

[epage]

Since spawn takes in a &mut, I could probably loosen this requirement to &mut.

[mssun]

Good, thanks.

I'm trying to use assert-cmd for our integration tests: mesalock-linux/mesabox#24.

It will be very helpful if this issue can be quickly fixed. Thank you for the amazing project.

[epage]

Since this was a breaking change, I'm publishing it as v0.5.0 right now.

Did you test your conversion with #23? Should have probably checked with that first in case you find more issues :)

[mssun]

It works fine for my current usage.

The only concern is that with_stdin have to be the last function before assert since it returns StdInCommand instead of Command.

[epage]

Yup, didn't have an idea for a better way of doing that.

[mssun]

Thanks anyway. I probably will come up some other features or bugs later during my refactoring :)