test(pacquet/cli): assert recursive run/exec don't dispatch on a rejected selector

claude · claude · commit 77369a26145e · 2026-06-29T16:29:04.000Z
Strengthen the `[<since>]` diff-selector regression tests so they prove the command short-circuits before dispatch, not merely that stderr names the error. `run` asserts the build script's `ran.txt` marker is absent, and `exec` now runs a marker-writing command (`touch ran.txt`) and asserts the marker is absent. Per REVIEW_GUIDE.md, regression assertions must observe the changed behavior rather than only the error message. Refs #12723
diff --git a/pacquet/crates/cli/tests/exec_recursive.rs b/pacquet/crates/cli/tests/exec_recursive.rs
@@ -147,7 +147,8 @@ fn recursive_exec_diff_selector_is_unsupported() {
         .with_arg("--filter")
         .with_arg("[main]")
         .with_arg("exec")
-        .with_arg("true")
+        .with_arg("touch")
+        .with_arg("ran.txt")
         .output()
         .expect("spawn pacquet");
     assert!(!output.status.success(), "a [<since>] diff selector is unsupported and must fail");
@@ -156,6 +157,10 @@ fn recursive_exec_diff_selector_is_unsupported() {
         stderr.contains("Changed-package filter selectors"),
         "stderr should explain the diff selector is unsupported, got: {stderr}",
     );
+    assert!(
+        !workspace.join("project-1").join("ran.txt").exists(),
+        "exec must reject the selector before dispatching the command, so no marker is written",
+    );
 
     drop(root);
 }
diff --git a/pacquet/crates/cli/tests/run_recursive.rs b/pacquet/crates/cli/tests/run_recursive.rs
@@ -473,6 +473,10 @@ fn recursive_run_diff_selector_is_unsupported() {
         stderr.contains("Changed-package filter selectors"),
         "stderr should explain the diff selector is unsupported, got: {stderr}",
     );
+    assert!(
+        !workspace.join("project-1").join("ran.txt").exists(),
+        "run must reject the selector before invoking the build script, so no marker is written",
+    );
 
     drop(root);
 }
