test(discover): gate version-specific diagnostics tests on RF 7.0+

The diagnostics-summary tests in discover/test_errors.py assert on the
singular-header deprecation warning and arg-spec parse diagnostics, both
of which Robot Framework only emits from 7.0 onward. On RF 5.0/6.0/6.1
they don't exist, so the tests failed across all OS/Python combinations.

Skip them on RF < 7.0 via a needs_rf_70 marker. Move that marker (and
needs_rf_72) into a shared runner/cli/rf_markers.py so results/ and
discover/ reference a single definition instead of redeclaring it.

Author: d-biehl

tests/robotcode/runner/cli/discover/test_errors.py

@@ -8,6 +8,7 @@
 import json
 from pathlib import Path
 
+from ..rf_markers import needs_rf_70
 from .conftest import CliRunner
 
 
@@ -58,6 +59,7 @@ def test_parse_warnings_dont_fail_the_command(robotcode_cli: CliRunner, parse_er
     assert data.get("diagnostics"), "expected non-empty diagnostics"
 
 
+@needs_rf_70
 def test_text_default_shows_diagnostics_summary_only(text_discover: CliRunner, parse_error_suite: Path) -> None:
     """By default a compact `## Diagnostics` counts section follows the
     statistics (a separate block, not folded into them) plus a pointer to
@@ -82,6 +84,7 @@ def test_text_clean_suite_has_no_diagnostics_section(text_discover: CliRunner, f
     assert "--diagnostics" not in result.stdout
 
 
+@needs_rf_70
 def test_text_diagnostics_flag_shows_full_block_before_listing(
     text_discover: CliRunner, parse_error_suite: Path
 ) -> None:
@@ -100,6 +103,7 @@ def test_text_diagnostics_flag_shows_full_block_before_listing(
     assert str(parse_error_suite) not in result.stdout
 
 
+@needs_rf_70
 def test_text_error_severity_in_diagnostics(robotcode_cli: CliRunner, tmp_path: Path) -> None:
     """An error-level parse problem (here an invalid argument spec) shows an
     `Errors` row in the counts section and an `_(error)_` label in the full

tests/robotcode/runner/cli/results/conftest.py

@@ -38,20 +38,6 @@
 SUITES_DIR = Path(__file__).parent / "suites"
 
 
-# ---------------------------------------------------------------------------
-# RF-version markers
-# ---------------------------------------------------------------------------
-
-needs_rf_70 = pytest.mark.skipif(
-    RF_VERSION < (7, 0),
-    reason="requires Robot Framework 7.0+ (VAR, JSON output, attribute renames)",
-)
-needs_rf_72 = pytest.mark.skipif(
-    RF_VERSION < (7, 2),
-    reason="requires Robot Framework 7.2+ (GROUP block)",
-)
-
-
 # ---------------------------------------------------------------------------
 # Subprocess wiring
 # ---------------------------------------------------------------------------

tests/robotcode/runner/cli/results/test_log.py

@@ -13,8 +13,9 @@
 
 import pytest
 
+from ..rf_markers import needs_rf_70, needs_rf_72
 from ._helpers import count_entries_of_type, find_test, iter_body, strip_ansi
-from .conftest import CliRunner, JsonRunner, needs_rf_70, needs_rf_72
+from .conftest import CliRunner, JsonRunner
 
 # ---------------------------------------------------------------------------
 # Helpers local to this file

tests/robotcode/runner/cli/rf_markers.py

@@ -0,0 +1,19 @@
+"""Shared Robot-Framework-version skip markers for the `runner/cli` tests.
+
+Single source of truth so sibling packages (`results`, `discover`, ...)
+don't each redefine the same `skipif` condition.
+"""
+
+import pytest
+
+from robotcode.robot.utils import RF_VERSION
+
+needs_rf_70 = pytest.mark.skipif(
+    RF_VERSION < (7, 0),
+    reason="requires Robot Framework 7.0+ (VAR, JSON output, attribute renames, "
+    "singular-header deprecation warning, arg-spec parse diagnostics)",
+)
+needs_rf_72 = pytest.mark.skipif(
+    RF_VERSION < (7, 2),
+    reason="requires Robot Framework 7.2+ (GROUP block)",
+)