From f7a3884f174b57261394548bec32dbd2fec58d13 Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Sun, 29 Mar 2026 20:16:55 +0000 Subject: [PATCH] Fix integration test recompilation and add interactivity toggle - Update `fill_data` binary invocation to use `env!("CARGO_BIN_EXE_fill_data")` directly, bypassing Cargo to prevent unnecessary rebuilds. - Wrap Unix stdin mocking in a conditional check based on the `INTERACTIVE` environment variable to allow manual input. Co-authored-by: soniapi <396009+soniapi@users.noreply.github.com> --- tests/integration_test.rs | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/tests/integration_test.rs b/tests/integration_test.rs index 77e81af..b3f32e4 100644 --- a/tests/integration_test.rs +++ b/tests/integration_test.rs @@ -62,9 +62,9 @@ async fn test_end_to_end_sequence() { .execute(&mut connection) .unwrap(); - // Mock stdin for Unix systems + // Mock stdin for Unix systems, unless running interactively #[cfg(unix)] - { + if std::env::var("INTERACTIVE").is_err() { let temp_file_path = "/tmp/mock_stdin.txt"; let mut mock_file = std::fs::File::create(temp_file_path).expect("Failed to create mock file"); mock_file.write_all(b"1\n180000.0\n").expect("Failed to write mock input"); @@ -133,10 +133,7 @@ async fn test_end_to_end_sequence() { println!("Filling partitions with 200,000 rows. This might take a bit..."); - let mut fill_proc = Command::new("cargo") - .arg("run") - .arg("--bin") - .arg("fill_data") + let mut fill_proc = Command::new(env!("CARGO_BIN_EXE_fill_data")) .stdin(Stdio::piped()) .stdout(Stdio::inherit()) .spawn()