We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The sequence in Electron100/butane@c5761cc#diff-b70d3a47c15879d308451b54821682f1d63518db732881b434c4110d9ca7a767 assumes that the user is in the directory containing the .md file. I think that is a reasonable default for trycmd. I don't see any set_current_dir in this project.
set_current_dir
It would also be useful to add something to https://docs.rs/trycmd/latest/trycmd/struct.TestCases.html to allow setting the CWD for that set of cases.
The text was updated successfully, but these errors were encountered:
Current behavior is
.toml
path.with_ext(".in")
See also https://github.com/assert-rs/trycmd/blob/main/src/schema.rs#L85
Most of those cases listed came out specific use cases but the CWD never came up in a case without a .in, so I hadn't really considered.
.in
Since we treat everything else relative to path.parent(), I think it makes sense to make this change.
path.parent()
This will be a breaking change though
Sorry, something went wrong.
No branches or pull requests
The sequence in Electron100/butane@c5761cc#diff-b70d3a47c15879d308451b54821682f1d63518db732881b434c4110d9ca7a767 assumes that the user is in the directory containing the .md file. I think that is a reasonable default for trycmd. I don't see any
set_current_dir
in this project.It would also be useful to add something to https://docs.rs/trycmd/latest/trycmd/struct.TestCases.html to allow setting the CWD for that set of cases.
The text was updated successfully, but these errors were encountered: