-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Feature request: fixture file as input to pipe (with_stdin
)
#24
Comments
Thats a really great idea! For stdin, its just a matter of coming up with the name. For stdout, stderr, the challenge is we won't know if its intended to be binary or |
Also, something else for us to consider with |
Thanks @epage, I didn't notice the usage of In terms of the name, how about |
with_stdin
), stdout
and stderr
with_stdin
)
I've gone ahead and removed stdout/stderr from this to make our narrowed focus clear.
imo whether you use a file or in-memory string, both would be considered fixtures, right? So one option is |
Yes, you are right. I think |
There is just enough overlap between how people expect paths and strings to be used, I felt we couldn't just have an `IntoStdIn`. Long term, we should consider streaming the content from file / reader to stdin. Fixes assert-rs#24 BREAKING CHANGE: `with_stdin(buffer)` is now `with_stdin().buffer(buffer)`.
There is just enough overlap between how people expect paths and strings to be used, I felt we couldn't just have an `IntoStdIn`. Long term, we should consider streaming the content from file / reader to stdin. Fixes assert-rs#24 BREAKING CHANGE: `with_stdin(buffer)` is now `with_stdin().buffer(buffer)`.
Publishing v0.6.0 with this fix. Sorry for the delay. |
I guess I cannot use I'm a little confused about |
Yeah, I realized that on closer inspection and opened assert-rs/predicates-rs#56 My plan has been to finish assert-rs/predicates-rs#7 first because I've started it and it is more invasive but it is larger and controversial, so it might take longer to get in. |
I see. Thank you @epage. Can you ping me if it's ready to use? |
Just to check, is that another blocking issue for you?
Alternatively, you could subscribe to the issue? |
Yes, my expectation is to pipe in fixture files and compare stdout with fixture files. I have subscribed these issues you mentioned. Thanks. |
chore: Ensure pre-commit gets non-system Python
It will be very useful if I can use a fixture file as input
The text was updated successfully, but these errors were encountered: