Address review: age sweep for orphaned build dirs, logger warnings, strict marker literal guard

This commit is contained in:
J. Nick Koston
2026-09-03 16:08:38 +02:00
parent cb7cfa5340
commit 7b17651837
2 changed files with 34 additions and 8 deletions
+9 -1
View File
@@ -1125,7 +1125,15 @@ def get_fixture_to_test_files() -> dict[str, frozenset[str]]:
base_name = func.replace("test_", "").partition("[")[0]
result.setdefault(base_name, set()).add(rel_path)
# Shared fixtures are named by marker, not by a test function
for name in _SHARED_YAML_RE.findall(content):
names = _SHARED_YAML_RE.findall(content)
if len(names) != content.count("shared_yaml("):
# A wrapped or non-literal marker would silently drop the mapping
# and CI would select no tests for that fixture
raise ValueError(
f"{rel_path}: every shared_yaml marker must be a single-line "
"string literal so CI test selection can map its fixture"
)
for name in names:
result.setdefault(name, set()).add(rel_path)
return {k: frozenset(v) for k, v in result.items()}