diff --git a/tests/test_scan_workstream_terminology.py b/tests/test_scan_workstream_terminology.py index 90cc5cb..f40b36c 100644 --- a/tests/test_scan_workstream_terminology.py +++ b/tests/test_scan_workstream_terminology.py @@ -2,8 +2,6 @@ from __future__ import annotations from pathlib import Path -import pytest - from tools.scan_workstream_terminology import ( DEFAULT_ALLOWLIST_PATH, load_allowlist, @@ -38,17 +36,18 @@ def test_path_is_excluded_for_state_hub_compat_router() -> None: ) -@pytest.mark.xfail( - reason="pre-existing, unrelated to CUST-WP-0060: scan_workstream_allowlist.yaml's " - "agentic-resources entry no longer sets exclude_repo: true, so the whole-repo " - "exclusion this test asserts doesn't currently hold. Found while wiring CI for " - "tests/ (previously unrun); needs an allowlist-owner decision (README.md is " - "user-facing prose that could legitimately want the scan), not a silent fix.", - strict=True, -) -def test_path_is_excluded_for_whole_repo() -> None: +def test_path_is_excluded_for_generated_tree_not_whole_repo() -> None: + """agentic-resources' exclusion was deliberately narrowed from + exclude_repo: true to just the generated session_memory/.store/ tree + (commit e0f9330, CUST-WP-0055 T02/T06/T07, 2026-07-08) so the + terminology scan would cover the repo's real source. This replaces a + stale test that still asserted the old whole-repo exclusion (found + while wiring CI for tests/, CUST-WP-0060 test-coverage review).""" config = load_allowlist(DEFAULT_ALLOWLIST_PATH) - assert path_is_excluded("agentic-resources", "README.md", config) + assert path_is_excluded( + "agentic-resources", "session_memory/.store/foo.json", config + ) + assert not path_is_excluded("agentic-resources", "README.md", config) def test_prose_gate_violations_detect_user_facing_workstream() -> None: