claude-dev-env 2.27.0 → 2.28.1

package/hooks/blocking/test_code_rules_enforcer_docstring_delegation_summary.py

@@ -1,385 +0,0 @@
-"""Tests for check_docstring_delegation_summary_enumeration_drift (O6).
-
-A thin wrapper docstring that enumerates its actions and points at the home
-of the real body drifts when the same-named function in that named neighbor
-carries a summary enumeration omitting one of those actions. The gate fires
-from both sides: saving the wrapper compares it against the named neighbor on
-disk, and saving the delegated body compares it against every neighboring
-wrapper docstring pointing at it.
-"""
-
-from __future__ import annotations
-
-import importlib.util
-import sys
-from pathlib import Path
-from types import ModuleType
-
-
-def _load_enforcer_module() -> ModuleType:
-    module_path = Path(__file__).parent / "code_rules_enforcer.py"
-    spec = importlib.util.spec_from_file_location("code_rules_enforcer", module_path)
-    assert spec is not None
-    assert spec.loader is not None
-    module = importlib.util.module_from_spec(spec)
-    spec.loader.exec_module(module)
-    return module
-
-
-code_rules_enforcer = _load_enforcer_module()
-
-
-def check_docstring_delegation_summary_enumeration_drift(
-    content: str, file_path: str
-) -> list[str]:
-    return code_rules_enforcer.check_docstring_delegation_summary_enumeration_drift(
-        content, file_path
-    )
-
-
-def validate_content(content: str, file_path: str, old_content: str) -> list[str]:
-    return code_rules_enforcer.validate_content(content, file_path, old_content)
-
-
-HOOK_INFRASTRUCTURE_PATH = "/home/user/.claude/hooks/blocking/example.py"
-TARGET_FLOW_STEM = "listing_edit_flow"
-PROCESSOR_FILE_NAME = "portal_processor.py"
-
-
-def _drifted_processor_source() -> str:
-    return (
-        "class PortalProcessor:\n"
-        "    async def _refresh_store_sections(self, page: object) -> bool:\n"
-        '        """Apply App Info, Russia, review note, publication edits;'
-        " full doc on ``listing_edit_flow``.\"\"\"\n"
-        "        return await _refresh_store_sections(self, page)\n"
-    )
-
-
-def _matching_processor_source() -> str:
-    return (
-        "class PortalProcessor:\n"
-        "    async def _refresh_store_sections(self, page: object) -> bool:\n"
-        '        """Apply Russia, review note, publication edits;'
-        " full doc on ``listing_edit_flow``.\"\"\"\n"
-        "        return await _refresh_store_sections(self, page)\n"
-    )
-
-
-def _target_flow_source() -> str:
-    return (
-        "async def _refresh_store_sections(processor: object, page: object) -> bool:\n"
-        '    """Apply Russia uncheck, review note, and Publication edits.\n'
-        "\n"
-        "    The App Info edit runs earlier in the binary phase and never\n"
-        "    repeats here.\n"
-        '    """\n'
-        "    return True\n"
-    )
-
-
-def _target_flow_plain_summary() -> str:
-    return (
-        "async def _refresh_store_sections(processor: object, page: object) -> bool:\n"
-        '    """Drive one theme end to end."""\n'
-        "    return True\n"
-    )
-
-
-def _create_target_flow(directory: Path, source: str) -> Path:
-    created_path = directory / (TARGET_FLOW_STEM + ".py")
-    created_path.write_text(source, encoding="utf-8")
-    return created_path
-
-
-def test_should_flag_listed_action_absent_from_the_named_neighbor(tmp_path: Path) -> None:
-    _create_target_flow(tmp_path, _target_flow_source())
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _drifted_processor_source(), checked_path
-    )
-    assert any("App Info" in each for each in issues), (
-        f"Expected 'App Info' to be flagged, got: {issues!r}"
-    )
-
-
-def test_should_pass_matching_summaries(tmp_path: Path) -> None:
-    _create_target_flow(tmp_path, _target_flow_source())
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _matching_processor_source(), checked_path
-    )
-    assert issues == [], f"Aligned summaries pass, got: {issues!r}"
-
-
-def test_should_flag_saved_body_that_strands_a_pointing_neighbor(tmp_path: Path) -> None:
-    checked_path = tmp_path / PROCESSOR_FILE_NAME
-    checked_path.write_text(_drifted_processor_source(), encoding="utf-8")
-    created_path = str(tmp_path / (TARGET_FLOW_STEM + ".py"))
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _target_flow_source(), created_path
-    )
-    assert any(
-        "App Info" in each and PROCESSOR_FILE_NAME in each for each in issues
-    ), f"Expected the stranded caller to be flagged, got: {issues!r}"
-
-
-def test_should_pass_plain_one_purpose_summary(tmp_path: Path) -> None:
-    _create_target_flow(tmp_path, _target_flow_plain_summary())
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _drifted_processor_source(), checked_path
-    )
-    assert issues == [], (
-        f"A comma-free summary compares against nothing, got: {issues!r}"
-    )
-
-
-def test_should_pass_when_the_named_neighbor_does_not_exist(tmp_path: Path) -> None:
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _drifted_processor_source(), checked_path
-    )
-    assert issues == [], f"Nothing on disk to compare against, got: {issues!r}"
-
-
-def test_should_pass_single_listed_action(tmp_path: Path) -> None:
-    _create_target_flow(tmp_path, _target_flow_source())
-    solo_source = (
-        "class PortalProcessor:\n"
-        "    async def _refresh_store_sections(self, page: object) -> bool:\n"
-        '        """Apply the App Info edit;'
-        " full doc on ``listing_edit_flow``.\"\"\"\n"
-        "        return await _refresh_store_sections(self, page)\n"
-    )
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        solo_source, checked_path
-    )
-    assert issues == [], f"A lone named action never fires, got: {issues!r}"
-
-
-def test_should_skip_strict_test_paths(tmp_path: Path) -> None:
-    _create_target_flow(tmp_path, _target_flow_source())
-    skipped_path = str(tmp_path / "test_portal_processor.py")
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _drifted_processor_source(), skipped_path
-    )
-    assert issues == [], f"Exempt, got: {issues!r}"
-
-
-def test_should_skip_hook_infrastructure() -> None:
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _drifted_processor_source(), HOOK_INFRASTRUCTURE_PATH
-    )
-    assert issues == [], f"Exempt, got: {issues!r}"
-
-
-def test_should_handle_syntax_error_gracefully(tmp_path: Path) -> None:
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        "def fetch(\n", checked_path
-    )
-    assert issues == [], f"A syntax error yields no issues, got: {issues!r}"
-
-
-def test_should_handle_unparseable_neighbor(tmp_path: Path) -> None:
-    _create_target_flow(tmp_path, "def broken(\n")
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _drifted_processor_source(), checked_path
-    )
-    assert issues == [], f"Unparseable neighbors yield no issues, got: {issues!r}"
-
-
-def test_validate_content_surfaces_the_drift(tmp_path: Path) -> None:
-    _create_target_flow(tmp_path, _target_flow_source())
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = validate_content(_drifted_processor_source(), checked_path, old_content="")
-    matching_issues = [each for each in issues if "App Info" in each and "O6" in each]
-    assert matching_issues, (
-        f"Expected validate_content to surface the O6 finding, got: {issues!r}"
-    )
-
-
-def _drifted_processor_source_with_wrapped_summary() -> str:
-    return (
-        "class PortalProcessor:\n"
-        "    async def _refresh_store_sections(self, page: object) -> bool:\n"
-        '        """Apply App Info, Russia, review note, publication\n'
-        " edits; full doc on ``listing_edit_flow``.\"\"\"\n"
-        "        return await _refresh_store_sections(self, page)\n"
-    )
-
-
-def test_should_flag_listed_action_when_the_summary_wraps_a_physical_line(
-    tmp_path: Path,
-) -> None:
-    _create_target_flow(tmp_path, _target_flow_source())
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _drifted_processor_source_with_wrapped_summary(), checked_path
-    )
-    assert any("App Info" in each for each in issues), (
-        f"A summary wrapped onto a second physical line still names 'App Info', "
-        f"got: {issues!r}"
-    )
-
-
-def _drifted_processor_source_with_capitalized_and_uneven_whitespace() -> str:
-    return (
-        "class PortalProcessor:\n"
-        "    async def _refresh_store_sections(self, page: object) -> bool:\n"
-        '        """Apply App Info, Russia, review note,  And  publication'
-        " edits; full doc on ``listing_edit_flow``.\"\"\"\n"
-        "        return await _refresh_store_sections(self, page)\n"
-    )
-
-
-def test_should_flag_listed_action_when_the_conjunction_is_capitalized_with_uneven_whitespace(
-    tmp_path: Path,
-) -> None:
-    _create_target_flow(tmp_path, _target_flow_source())
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _drifted_processor_source_with_capitalized_and_uneven_whitespace(), checked_path
-    )
-    assert any("App Info" in each for each in issues), (
-        f"A capitalized 'And' with uneven whitespace still splits into its own "
-        f"listed entry, got: {issues!r}"
-    )
-
-
-def _write_filler_neighbor_modules(directory: Path, filler_count: int) -> None:
-    for each_index in range(filler_count):
-        filler_path = directory / f"filler_module_{each_index:03d}.py"
-        filler_path.write_text("PLACEHOLDER_VALUE = 1\n", encoding="utf-8")
-
-
-def test_should_stop_scanning_neighbors_past_the_configured_limit(
-    tmp_path: Path,
-) -> None:
-    neighbor_scan_limit = sys.modules["code_rules_docstrings"].NEIGHBOR_SCAN_FILE_LIMIT
-    _write_filler_neighbor_modules(tmp_path, filler_count=neighbor_scan_limit + 5)
-    stranded_wrapper_path = tmp_path / "zzz_stranded_processor.py"
-    stranded_wrapper_path.write_text(_drifted_processor_source(), encoding="utf-8")
-    created_path = str(tmp_path / (TARGET_FLOW_STEM + ".py"))
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _target_flow_source(), created_path
-    )
-    assert issues == [], (
-        "The alphabetically-last wrapper sits past the neighbor scan limit, so "
-        f"the scan must stop before reaching it, got: {issues!r}"
-    )
-
-
-def test_should_still_flag_a_stranded_wrapper_within_the_scan_limit(
-    tmp_path: Path,
-) -> None:
-    neighbor_scan_limit = sys.modules["code_rules_docstrings"].NEIGHBOR_SCAN_FILE_LIMIT
-    stranded_wrapper_path = tmp_path / "aaa_stranded_processor.py"
-    stranded_wrapper_path.write_text(_drifted_processor_source(), encoding="utf-8")
-    _write_filler_neighbor_modules(tmp_path, filler_count=neighbor_scan_limit + 5)
-    created_path = str(tmp_path / (TARGET_FLOW_STEM + ".py"))
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _target_flow_source(), created_path
-    )
-    assert any(
-        "App Info" in each and "aaa_stranded_processor.py" in each for each in issues
-    ), (
-        "The alphabetically-first wrapper sits within the neighbor scan limit "
-        f"and stays flagged even with extra filler neighbors, got: {issues!r}"
-    )
-
-
-def _target_flow_source_with_nested_duplicate_name() -> str:
-    return (
-        "async def _refresh_store_sections(processor: object, page: object) -> bool:\n"
-        '    """Apply Russia uncheck, review note, and Publication edits.\n'
-        "\n"
-        "    The App Info edit runs earlier in the binary phase and never\n"
-        "    repeats here.\n"
-        '    """\n'
-        "    return True\n"
-        "\n"
-        "\n"
-        "def _outer_helper() -> bool:\n"
-        "    def _refresh_store_sections() -> bool:\n"
-        '        """A same-named nested helper that must not shadow the'
-        ' top-level entry."""\n'
-        "        return True\n"
-        "\n"
-        "    return _refresh_store_sections()\n"
-    )
-
-
-def test_should_not_let_a_nested_duplicate_name_hide_the_top_level_entry(
-    tmp_path: Path,
-) -> None:
-    _create_target_flow(tmp_path, _target_flow_source_with_nested_duplicate_name())
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _drifted_processor_source(), checked_path
-    )
-    assert any("App Info" in each for each in issues), (
-        "A nested helper sharing the top-level function's name must not "
-        f"delete the top-level entry from comparison, got: {issues!r}"
-    )
-
-
-def _processor_source_with_nested_pointer_wrapper() -> str:
-    return (
-        "class PortalProcessor:\n"
-        "    async def _refresh_store_sections(self, page: object) -> bool:\n"
-        '        """Drive one theme end to end."""\n'
-        "        return True\n"
-        "\n"
-        "    def _run_nested_helper(self) -> bool:\n"
-        "        async def _refresh_store_sections() -> bool:\n"
-        '            """Apply App Info, Russia, review note, publication'
-        ' edits; full doc on ``listing_edit_flow``.\"\"\"\n'
-        "            return True\n"
-        "        return True\n"
-    )
-
-
-def test_should_not_flag_a_nested_wrapper_pointing_at_a_delegate(
-    tmp_path: Path,
-) -> None:
-    _create_target_flow(tmp_path, _target_flow_source())
-    checked_path = str(tmp_path / PROCESSOR_FILE_NAME)
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _processor_source_with_nested_pointer_wrapper(), checked_path
-    )
-    assert issues == [], (
-        "A function-local wrapper is not a top-level function or class "
-        f"method, so it must never be compared, got: {issues!r}"
-    )
-
-
-def _neighbor_source_with_nested_pointer_wrapper() -> str:
-    return (
-        "def _run_nested_helper() -> bool:\n"
-        "    async def _refresh_store_sections() -> bool:\n"
-        '        """Apply App Info, Russia, review note, publication edits;'
-        ' full doc on ``listing_edit_flow``.\"\"\"\n'
-        "        return True\n"
-        "    return True\n"
-    )
-
-
-def test_should_not_flag_a_nested_neighbor_wrapper_as_a_stranded_caller(
-    tmp_path: Path,
-) -> None:
-    neighbor_path = tmp_path / "portal_helpers.py"
-    neighbor_path.write_text(
-        _neighbor_source_with_nested_pointer_wrapper(), encoding="utf-8"
-    )
-    created_path = str(tmp_path / (TARGET_FLOW_STEM + ".py"))
-    issues = check_docstring_delegation_summary_enumeration_drift(
-        _target_flow_source(), created_path
-    )
-    assert issues == [], (
-        "A function-local wrapper in a neighbor module is not a pointing "
-        f"caller, so it must never be treated as stranded, got: {issues!r}"
-    )