cli
test should use local devenv module
#1432
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
One change from #1421 that wasn't part of #1427. (Both PRs addressed the same bug in
devenv container build shell
, and the latter was merged.)While working on #1421, I noticed
tests/cli
, which runsdevenv container build shell
as part of its test script, wouldn't fail as it should've when the bug was present. I believe it's becausetests/cli/devenv.yaml
doesn't override thedevenv
input likedevenv-run-tests
ortests/clean/devenv.yaml
does.devenv/devenv-run-tests/src/main.rs
Lines 80 to 85 in 9ff4999
devenv/tests/clean/devenv.yaml
Lines 1 to 3 in 9ff4999