From fbe9726ac76d5797fe5c3115b8f996d343c57596 Mon Sep 17 00:00:00 2001 From: aaltshuler Date: Wed, 10 Jun 2026 22:34:54 +0300 Subject: [PATCH] test(cli): stop the S3 e2e scaffolding omnigraph.yaml into the crate dir MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit local_cli_s3_end_to_end_init_load_read_flow ran `omnigraph init` without a current_dir, so init's project scaffold landed in crates/omnigraph-cli/ — poisoning any later test that resolves a graph target from the cwd config (query_lint_requires_schema_or_resolvable_graph_target fails determinis- tically once the file exists). Only manifests when OMNIGRAPH_S3_TEST_BUCKET is set, which is why local FS runs and CI's scoped rustfs job never caught it. The init and load calls now run inside the test's tempdir. Co-Authored-By: Claude Fable 5 --- crates/omnigraph-cli/tests/system_local.rs | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/crates/omnigraph-cli/tests/system_local.rs b/crates/omnigraph-cli/tests/system_local.rs index b2afdab..adb5dc8 100644 --- a/crates/omnigraph-cli/tests/system_local.rs +++ b/crates/omnigraph-cli/tests/system_local.rs @@ -595,8 +595,12 @@ policy: {{}} ), ); + // current_dir matters: `init` scaffolds an omnigraph.yaml into its cwd, + // and without this it pollutes the crate dir, breaking unrelated tests + // (anything resolving a graph target from the cwd config). output_success( cli() + .current_dir(query_root) .arg("init") .arg("--schema") .arg(fixture("test.pg")) @@ -604,6 +608,7 @@ policy: {{}} ); output_success( cli() + .current_dir(query_root) .arg("load") .arg("--data") .arg(fixture("test.jsonl"))