From 0a6a0ba527ef677e446b2571c266eada8ca77f99 Mon Sep 17 00:00:00 2001 From: Teknium <127238744+teknium1@users.noreply.github.com> Date: Mon, 25 May 2026 06:03:58 -0700 Subject: [PATCH] test(skills): widen assertion in PR#6656 regression to accept new validator msg MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The new install-path validator from this PR raises 'Unsafe install path: ...' earlier in the pipeline than the previous resolve-then-check path. Behavior is identical (ok=False, victim untouched, refused before rmtree) — only the error string changed. --- tests/tools/test_pr_6656_regressions.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/tests/tools/test_pr_6656_regressions.py b/tests/tools/test_pr_6656_regressions.py index 9429a804135..48f53e65a30 100644 --- a/tests/tools/test_pr_6656_regressions.py +++ b/tests/tools/test_pr_6656_regressions.py @@ -99,7 +99,12 @@ class TestUninstallPathTraversal: ok, msg = uninstall_skill("evil") assert ok is False - assert "outside" in msg or "resolves" in msg or "skills directory" in msg + assert ( + "outside" in msg + or "resolves" in msg + or "skills directory" in msg + or "Unsafe install path" in msg + ) # The victim directory MUST still exist. assert victim.exists() assert (victim / "important.txt").exists()