plugin: do not run pytest_setup and pytest_teardown with --colect-only

pytest_mh/_private/plugin.py

@@ -53,6 +53,9 @@ def __init__(self, pytest_config: pytest.Config) -> None:
         if self.mh_collect_logs is None:
             self.mh_collect_logs = self.mh_collect_artifacts
 
+        # pytest options
+        self.pytest_opt_collect_only: bool = pytest_config.getoption("collectonly")
+
     @classmethod
     def GetLogger(cls) -> logging.Logger:
         """
@@ -161,7 +164,8 @@ def pytest_sessionfinish(self, session: pytest.Session, exitstatus: int | pytest
             return
 
         # Run pytest_teardown on all hosts required by selected tests
-        self._teardown_hosts(self.required_hosts)
+        if not self.pytest_opt_collect_only:
+            self._teardown_hosts(self.required_hosts)
 
     @pytest.hookimpl(hookwrapper=True)
     def pytest_make_collect_report(self, collector: pytest.Collector) -> Generator[None, pytest.CollectReport, None]:
@@ -265,8 +269,8 @@ def pytest_collection_modifyitems(self, config: pytest.Config, items: list[pytes
         # Sort required host by name to provide deterministic runs
         self.required_hosts = sorted(required_hosts_set, key=lambda x: x.hostname)
 
-    @pytest.hookimpl(tryfirst=True)
-    def pytest_collection_finish(self, session: pytest.Session) -> None:
+    @pytest.hookimpl(hookwrapper=True)
+    def pytest_collection_finish(self, session: pytest.Session) -> Generator:
         # Log required hosts
         self.logger.info("")
         self.logger.info("")
@@ -275,6 +279,12 @@ def pytest_collection_finish(self, session: pytest.Session) -> None:
             self.logger.info(f"  {host.role}: {host.hostname}")
         self.logger.info("")
 
+        yield
+
+        # Run pytest_setup on all hosts required by selected tests
+        if not self.pytest_opt_collect_only:
+            self._setup_hosts(self.required_hosts)
+
     @pytest.hookimpl(tryfirst=True)
     def pytest_runtest_setup(self, item: pytest.Item) -> None:
         """
@@ -295,9 +305,6 @@ def pytest_runtest_setup(self, item: pytest.Item) -> None:
             return
         mark: TopologyMark = data.topology_mark
 
-        # Run pytest_setup on all hosts required by selected tests
-        self._setup_hosts(self.required_hosts)
-
         # Execute per-topology setup if topology is switched.
         if self._topology_switch(None, item):
             self.current_topology = mark.name