refactor: delete the unused CheckResult type and the descriptor-gate script entry (#84)

Author: lesnik512

semvertag/_types.py

@@ -37,10 +37,3 @@ class Commit:
 class Tag:
     name: str
     commit_sha: str
-
-
-@dataclasses.dataclass(frozen=True, slots=True, kw_only=True)
-class CheckResult:
-    name: str
-    status: typing.Literal["passed", "failed", "skipped"]
-    cause: str

tests/_descriptor_gate.py

@@ -1,12 +1,6 @@
-"""Structural validator for templates/semvertag.yml.
-
-Shared between the `.github/workflows/ci.yml` lint step and the
-regression test fixtures in test_ci_descriptor_gate.py so the gate
-and its tests cannot drift.
-"""
+"""Structural validator for templates/semvertag.yml, run by test_ci_descriptor_gate.py."""
 
 import pathlib
-import sys
 import typing
 
 import yaml
@@ -17,7 +11,6 @@
 _SUBSTITUTION_LITERAL: typing.Final = "$[[ inputs.strategy ]]"
 _EXPECTED_OPTIONS: typing.Final = {"branch-prefix", "conventional-commits"}
 _EXPECTED_DOC_COUNT: typing.Final = 2
-_EXPECTED_ARGV_LEN: typing.Final = 2
 
 
 class DescriptorGateError(SystemExit):
@@ -106,15 +99,3 @@ def validate(path: str) -> None:
         ">=" in first or "==" in first or _VERSION_MARKER in first,
         f"job.script[0] must pin the semvertag version (contain '>=', '==', or '{_VERSION_MARKER}'), got {first!r}",
     )
-
-
-def main(argv: list[str]) -> None:
-    if len(argv) != _EXPECTED_ARGV_LEN:
-        msg = f"usage: python -m tests._descriptor_gate <path>; got {argv!r}"
-        raise DescriptorGateError(msg)
-    validate(argv[1])
-    sys.stdout.write(f"{argv[1]} shape OK\n")
-
-
-if __name__ == "__main__":
-    main(sys.argv)

tests/test_ci_descriptor_gate.py

@@ -7,14 +7,12 @@
 """
 
 import pathlib
-import runpy
-import sys
 import typing
 
 import pytest
 import yaml
 
-from tests._descriptor_gate import DescriptorGateError, main, validate
+from tests._descriptor_gate import DescriptorGateError, validate
 
 
 _REPO_ROOT: typing.Final = pathlib.Path(__file__).parent.parent
@@ -123,24 +121,3 @@ def test_unpinned_semvertag_fails(tmp_path: pathlib.Path, shipped_descriptor_doc
     bad = _write_descriptor(tmp_path, [spec, body])
     with pytest.raises(DescriptorGateError, match=r"script.* must pin the semvertag version"):
         validate(str(bad))
-
-
-def test_main_prints_shape_ok_for_valid_descriptor(capsys: pytest.CaptureFixture[str]) -> None:
-    main(["_descriptor_gate", str(_DESCRIPTOR_PATH)])
-    captured: typing.Final = capsys.readouterr()
-    assert captured.out == f"{_DESCRIPTOR_PATH} shape OK\n"
-
-
-def test_module_run_as_script_validates_argv_path(
-    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
-) -> None:
-    monkeypatch.setattr(sys, "argv", ["_descriptor_gate", str(_DESCRIPTOR_PATH)])
-    monkeypatch.delitem(sys.modules, "tests._descriptor_gate")
-    runpy.run_module("tests._descriptor_gate", run_name="__main__")
-    captured: typing.Final = capsys.readouterr()
-    assert captured.out == f"{_DESCRIPTOR_PATH} shape OK\n"
-
-
-def test_main_raises_when_argv_length_wrong() -> None:
-    with pytest.raises(DescriptorGateError, match=r"usage: python -m tests\._descriptor_gate"):
-        main(["_descriptor_gate"])