chore(test-tools): re-enable eip checklist docs unit tests (#1740)

* Unskip checklist template consistency tests

Removed skip markers from checklist template tests.

* Update test_eip_checklist.py

* Update test_checklist_template_consistency.py

* Fix template path in test_checklist_template_consistency

* Define TEMPLATE_PATH for checklist templates

Add TEMPLATE_PATH for accessing checklist templates.

* Fix path to EIP checklist template

* Import Path from pathlib in test_eip_checklist.py

* Fix TEMPLATE_PATH by adjusting parent directory count

* Update TEMPLATE_PATH to correct parent directory

* Remove unused TEMPLATE_PATH from test_eip_checklist.py

Removed unused TEMPLATE_PATH definition from test file.

* Fix template path for EIP testing checklist

* Update test_eip_checklist.py

* Update test_eip_checklist.py

* Fix indentation in test_eip_checklist.py

* Simplify TEMPLATE_PATH using Path.parents

* Update assert_outcomes to include passed, failed, skipped

* chore: remove unused pytest import

* style: apply automatic code formatting and fix lint

Author: Praniti1594

packages/testing/src/execution_testing/checklists/tests/test_checklist_template_consistency.py

@@ -9,7 +9,7 @@
 from execution_testing.checklists.eip_checklist import EIPChecklist
 
 TEMPLATE_PATH = (
-    Path(__file__).parent.parent.parent.parent
+    Path(__file__).parents[6]
     / "docs"
     / "writing_tests"
     / "checklist_templates"
@@ -60,7 +60,6 @@ def get_all_checklist_ids(obj: Any) -> Set[str]:
     return ids
 
 
-@pytest.mark.skip(reason="Skipping test until ./docs/ folder is subtree'd")
 def test_checklist_template_consistency() -> None:
     """
     Test that all IDs in markdown template match EIPChecklist class exactly.
@@ -101,7 +100,6 @@ def test_checklist_template_consistency() -> None:
         pytest.fail(error_message)
 
 
-@pytest.mark.skip(reason="Skipping test until ./docs/ folder is subtree'd")
 def test_checklist_template_exists() -> None:
     """Test that the checklist template file exists."""
     assert TEMPLATE_PATH.exists(), (

packages/testing/src/execution_testing/cli/pytest_commands/plugins/filler/eip_checklist.py

@@ -51,8 +51,8 @@ def pytest_addoption(parser: pytest.Parser) -> None:
 PERCENTAGE_LINE = (
     "| TOTAL_CHECKLIST_ITEMS | COVERED_CHECKLIST_ITEMS | PERCENTAGE |"
 )
-TEMPLATE_PATH = Path(
-    # TODO: add better repo root detection
+
+TEMPLATE_PATH = (
     Path(__file__).parents[8]
     / "docs"
     / "writing_tests"

packages/testing/src/execution_testing/cli/pytest_commands/plugins/filler/tests/test_eip_checklist.py

@@ -4,10 +4,7 @@
 import textwrap
 from typing import Any
 
-import pytest
 
-
-@pytest.mark.skip(reason="S kipping test until ./docs/ folder is subtree'd")
 def test_eip_checklist_collection(testdir: Any) -> None:
     """Test that checklist markers are collected correctly."""
     # Create the test in an EIP-specific directory