Skip to content

Commit

Permalink
Fix MacOS tests (#449)
Browse files Browse the repository at this point in the history
These didn't work due to issues with bash/echo on MacOS.
  • Loading branch information
aaronmondal committed Dec 7, 2023
1 parent 406eab7 commit befd1b6
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion native-link-worker/tests/local_worker_test.rs
Expand Up @@ -81,7 +81,7 @@ mod local_worker_tests {
"baz".to_string(),
// Note: new lines will result in two entries for same key.
#[cfg(target_family = "unix")]
WorkerProperty::query_cmd("echo -e 'hello\ngoodbye'".to_string()),
WorkerProperty::query_cmd("printf 'hello\ngoodbye'".to_string()),
#[cfg(target_family = "windows")]
WorkerProperty::query_cmd("cmd /C \"echo hello && echo goodbye\"".to_string()),
);
Expand Down
4 changes: 2 additions & 2 deletions native-link-worker/tests/running_actions_manager_test.rs
Expand Up @@ -652,7 +652,7 @@ mod running_actions_manager_tests {
let arguments = vec![
"sh".to_string(),
"-c".to_string(),
"echo -n \"123 \" > ./test.txt; echo -n \"foo-stdout \"; >&2 echo -n \"bar-stderr \"".to_string(),
"printf '123 ' > ./test.txt; printf 'foo-stdout '; >&2 printf 'bar-stderr '".to_string(),
];
#[cfg(target_family = "windows")]
let arguments = vec![
Expand Down Expand Up @@ -811,7 +811,7 @@ mod running_actions_manager_tests {
let arguments = vec![
"sh".to_string(),
"-c".to_string(),
"echo -n \"123 \" > ./test.txt; echo -n \"foo-stdout \"; >&2 echo -n \"bar-stderr \"".to_string(),
"printf '123 ' > ./test.txt; printf 'foo-stdout '; >&2 printf 'bar-stderr '".to_string(),
];
#[cfg(target_family = "windows")]
let arguments = vec![
Expand Down

0 comments on commit befd1b6

Please sign in to comment.