Skip to content

Commit

Permalink
Prevent dual compilation of test_binary
Browse files Browse the repository at this point in the history
  • Loading branch information
GuillaumeGomez committed Apr 6, 2024
1 parent edfbe92 commit da009f8
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions tests/process.rs
Expand Up @@ -81,11 +81,11 @@ fn test_cmd() {
}
}

fn build_test_binary() {
fn build_test_binary(file_name: &str) {
std::process::Command::new("rustc")
.arg("test_bin/main.rs")
.arg("-o")
.arg("target/test_binary")
.arg(file_name)
.stdout(std::process::Stdio::null())
.spawn()
.unwrap()
Expand All @@ -98,8 +98,9 @@ fn test_environ() {
if !sysinfo::IS_SUPPORTED_SYSTEM || cfg!(feature = "apple-sandbox") {
return;
}
build_test_binary();
let mut p = std::process::Command::new("./target/test_binary")
let file_name = "target/test_binary";
build_test_binary(file_name);
let mut p = std::process::Command::new(&format!("./{file_name}"))
.env("FOO", "BAR")
.env("OTHER", "VALUE")
.spawn()
Expand Down Expand Up @@ -810,8 +811,9 @@ fn test_parent_change() {
return;
}

build_test_binary();
let mut p = std::process::Command::new("./target/test_binary")
let file_name = "target/test_binary2";
build_test_binary(file_name);
let mut p = std::process::Command::new(&format!("./{file_name}"))
.arg("1")
.spawn()
.unwrap();
Expand Down

0 comments on commit da009f8

Please sign in to comment.