fix(plugins): Collect-only parameter

Author: marioevz

docs/scripts/gen_test_case_reference.py

@@ -45,7 +45,6 @@
     "pytest_plugins.filler.gen_test_doc.gen_test_doc",
     "-p",
     "pytest_plugins.filler.eip_checklist",
-    "--collect-only",
     "--gen-docs",
     f"--gen-docs-target-fork={TARGET_FORK}",
     f"--until={GENERATE_UNTIL_FORK}",

src/cli/pytest_commands/checklist.py

@@ -21,9 +21,6 @@ def process_arguments(self, pytest_args: List[str]) -> List[str]:
         processed_args = super().process_arguments(pytest_args)
 
         # Add collect-only flag to avoid running tests
-        if "--collect-only" not in processed_args:
-            processed_args.append("--collect-only")
-
         processed_args.extend(["-p", "pytest_plugins.filler.eip_checklist"])
 
         return processed_args
@@ -66,8 +63,7 @@ def checklist(output: str, eip: tuple, **kwargs) -> None:
 
     """
     # Add output directory to pytest args
-    args = ["-p", "pytest_plugins.filler.eip_checklist"]
-    args.extend(["--checklist-output", output])
+    args = ["--checklist-output", output]
 
     # Add EIP filter if specified
     for eip_num in eip:

src/pytest_plugins/filler/eip_checklist.py

@@ -227,13 +227,19 @@ def generate_filled_checklist(self, eip: int, output_dir: Path) -> Path:
 
         return output_dir
 
+    @pytest.hookimpl(tryfirst=True)
+    def pytest_runtestloop(self, session):
+        """Skip test execution, only generate checklists."""
+        session.testscollected = 0
+        return True
+
     def pytest_collection_modifyitems(self, config: pytest.Config, items: List[pytest.Item]):
         """Collect checklist markers during test collection."""
         for item in items:
             eip, eip_path = self.extract_eip_from_path(Path(item.location[0]))
             if eip_path is not None and eip is not None:
                 self.eip_paths[eip] = eip_path
-            if item.get_closest_marker("derived_test"):
+            if item.get_closest_marker("derived_test") or item.get_closest_marker("skip"):
                 continue
             self.collect_from_item(item, eip)
 
@@ -247,6 +253,9 @@ def pytest_collection_modifyitems(self, config: pytest.Config, items: List[pytes
 
         checklist_props = {}
 
+        if not self.eip_checklist_items:
+            pytest.exit("\nNo EIPs found with checklist markers.", returncode=pytest.ExitCode.OK)
+
         # Generate a checklist for each EIP
         for eip in sorted(self.eip_checklist_items.keys()):
             # Skip if specific EIPs were requested and this isn't one of them
@@ -267,7 +276,7 @@ def pytest_collection_modifyitems(self, config: pytest.Config, items: List[pytes
                 )
             else:
                 checklist_path = self.generate_filled_checklist(eip, checklist_output)
-                print(f"Generated EIP-{eip} checklist: {checklist_path}")
+                print(f"\nGenerated EIP-{eip} checklist: {checklist_path}")
 
         if checklist_doc_gen:
             config.checklist_props = checklist_props  # type: ignore

src/pytest_plugins/shared/helpers.py

@@ -19,6 +19,8 @@ def is_help_or_collectonly_mode(config: pytest.Config) -> bool:
         or config.getoption("show_ported_from")
         or config.getoption("links_as_filled")
         or config.getoption("help")
+        or config.pluginmanager.has_plugin("pytest_plugins.filler.eip_checklist")
+        or config.pluginmanager.has_plugin("pytest_plugins.filler.gen_test_doc.gen_test_doc")
     )