wehub-resource-sync
55 lines
1.7 KiB
Python
55 lines
1.7 KiB
Python
"""CI guard: the decision-log aggregate library must stay importable.
|
|
|
|
`observability.decision_log_aggregate` is consumed by two paths that
|
|
sit in different parts of the tree:
|
|
|
|
1. `skills/bundled/history-explorer/scripts/explore.py` — subprocess
|
|
entrypoint; imports via the `sys.path.insert` bootstrap.
|
|
2. `skills/creator/auto_propose.py` — in-tree library code.
|
|
|
|
If somebody renames or relocates the module, this test fails loudly
|
|
so the duplicated definitions don't drift back into the bundled
|
|
script.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
|
|
def test_module_importable_with_expected_public_api() -> None:
|
|
from opensquilla.observability import decision_log_aggregate as agg
|
|
|
|
for name in (
|
|
"parse_log_line",
|
|
"within_window",
|
|
"aggregate_co_occurrences",
|
|
"aggregate_meta_usage",
|
|
):
|
|
assert hasattr(agg, name), f"public API drifted: missing {name!r}"
|
|
|
|
assert name in agg.__all__ # last name from the loop is enough as smoke
|
|
|
|
|
|
def test_history_explorer_script_imports_from_aggregate_module() -> None:
|
|
"""The bundled script must not redefine the lifted functions."""
|
|
|
|
from pathlib import Path
|
|
|
|
script = (
|
|
Path(__file__).resolve().parents[2]
|
|
/ "src"
|
|
/ "opensquilla"
|
|
/ "skills"
|
|
/ "bundled"
|
|
/ "history-explorer"
|
|
/ "scripts"
|
|
/ "explore.py"
|
|
)
|
|
text = script.read_text(encoding="utf-8")
|
|
assert "from opensquilla.observability.decision_log_aggregate import" in text
|
|
assert "def aggregate_co_occurrences" not in text, (
|
|
"explore.py must import aggregate_co_occurrences, not redefine it"
|
|
)
|
|
assert "def aggregate_meta_usage" not in text, (
|
|
"explore.py must import aggregate_meta_usage, not redefine it"
|
|
)
|