Skip to content

Commit

Permalink
Limit run-external --redirect-combine sh test to not(Windows) (nush…
Browse files Browse the repository at this point in the history
…ell#10905)

# Description

Limit the test `-p nu-command --test main
commands::run_external::redirect_combine` which uses `sh` to running on
`not(Windows)` like is done for other tests assuming unixy CLI items;
`sh` doesn't exist on Windows.

# User-Facing Changes

None; this is a change to tests only.

# Tests + Formatting

- :green_circle: `toolkit fmt`
- :green_circle: `toolkit clippy`
- :green_circle: `toolkit test`
- :green_circle: `toolkit test stdlib`
  • Loading branch information
CAD97 authored Nov 7, 2023
1 parent 1874082 commit 9b202d5
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions crates/nu-command/tests/commands/run_external.rs
Original file line number Diff line number Diff line change
Expand Up @@ -321,6 +321,7 @@ fn quotes_trimmed_when_shelling_out() {
assert_eq!(actual.out, "foo");
}

#[cfg(not(windows))]
#[test]
fn redirect_combine() {
Playground::setup("redirect_combine", |dirs, _| {
Expand Down

0 comments on commit 9b202d5

Please sign in to comment.