From ce10323f7c58925eeb9efe0d06afcba7c8415c11 Mon Sep 17 00:00:00 2001 From: Abdessamad Derraz <3028866+Abdess@users.noreply.github.com> Date: Wed, 21 Oct 2026 05:53:01 +0200 Subject: [PATCH] chore: align two tests with the merged lock flow The lock refusal moved to stderr and pipeline labels lost their /8 in two branches that met at merge. --- tests/test_artifact_lock.py | 2 +- tests/test_audit_regressions.py | 6 +++++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/tests/test_artifact_lock.py b/tests/test_artifact_lock.py index 5caf6406..7e60fd0c 100644 --- a/tests/test_artifact_lock.py +++ b/tests/test_artifact_lock.py @@ -138,7 +138,7 @@ class PackLockCliTest(unittest.TestCase): except subprocess.TimeoutExpired: self.fail("generate_pack waited on the data tree instead of refusing") self.assertEqual(proc.returncode, 1, proc.stdout + proc.stderr) - self.assertIn("is in use by another run", proc.stdout) + self.assertIn("is in use by another run", proc.stderr) def test_every_writing_mode_refuses_a_locked_output(self): """Two of seven modes took the lock, and five wrote straight into it. diff --git a/tests/test_audit_regressions.py b/tests/test_audit_regressions.py index f20e983e..2b0501c0 100644 --- a/tests/test_audit_regressions.py +++ b/tests/test_audit_regressions.py @@ -6115,8 +6115,12 @@ class SkipPacksHelpSaysWhatRuns(unittest.TestCase): self.assertIn("Steps 7 and 8 still rewrite README.md and docs/", text) def test_every_step_it_names_is_one_it_skips(self): + sys.path.insert(0, str(ROOT / "scripts")) + import pipeline + source = (ROOT / "scripts" / "pipeline.py").read_text(encoding="utf-8") - skipped = set(re.findall(r"--- (\w+)/8 [^\n]*SKIPPED \(--skip-packs\)", source)) + keys = re.findall(r'skip_notice\("(\w+)", "--skip-packs"\)', source) + skipped = {pipeline.step_number(key) for key in keys} self.assertEqual(skipped, {"4", "4b", "4c", "6"})