[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

ci/check_requires_coverage.py

@@ -11,10 +11,9 @@
 
 import argparse
 import json
-import sys
 from collections import defaultdict
+from collections.abc import Iterable
 from pathlib import Path
-from typing import Iterable
 
 _SKIP_PREFIX = "Skipped: "
 
@@ -54,7 +53,9 @@ def _skip_reason(longrepr: object) -> str | None:
     return reason
 
 
-def collect_reportlog_data(reportlogs: Iterable[Path]) -> dict[str, dict[str, set[str]]]:
+def collect_reportlog_data(
+    reportlogs: Iterable[Path],
+) -> dict[str, dict[str, set[str]]]:
     """Return per-nodeid execution and skip information."""
     data: dict[str, dict[str, set[str]]] = defaultdict(
         lambda: {"call_outcomes": set(), "skip_reasons": set()}
@@ -136,7 +137,9 @@ def main(argv: list[str] | None = None) -> int:
             print(f"- {nodeid}: {', '.join(sorted(reasons))}")
         return 1
 
-    print(f"Checked {len(reportlogs)} report-log file(s); no uncovered requires tests found.")
+    print(
+        f"Checked {len(reportlogs)} report-log file(s); no uncovered requires tests found."
+    )
     return 0
 
 

ci/test_check_requires_coverage.py

@@ -15,8 +15,20 @@ def test_uncovered_requires_test_is_reported(tmp_path: Path) -> None:
     reportlog = _write_reportlog(
         tmp_path / "reportlog.jsonl",
         [
-            {"$report_type": "TestReport", "nodeid": "test_mod.py::test_gpu", "when": "setup", "outcome": "skipped", "longrepr": ["test_mod.py", 12, "Skipped: requires foo"]},
-            {"$report_type": "TestReport", "nodeid": "test_mod.py::test_gpu", "when": "teardown", "outcome": "passed", "longrepr": None},
+            {
+                "$report_type": "TestReport",
+                "nodeid": "test_mod.py::test_gpu",
+                "when": "setup",
+                "outcome": "skipped",
+                "longrepr": ["test_mod.py", 12, "Skipped: requires foo"],
+            },
+            {
+                "$report_type": "TestReport",
+                "nodeid": "test_mod.py::test_gpu",
+                "when": "teardown",
+                "outcome": "passed",
+                "longrepr": None,
+            },
         ],
     )
 
@@ -29,15 +41,39 @@ def test_requires_test_is_covered_if_it_runs_elsewhere(tmp_path: Path) -> None:
     reportlog_a = _write_reportlog(
         tmp_path / "a.jsonl",
         [
-            {"$report_type": "TestReport", "nodeid": "test_mod.py::test_optional", "when": "setup", "outcome": "skipped", "longrepr": ["test_mod.py", 12, "Skipped: requires foo"]},
+            {
+                "$report_type": "TestReport",
+                "nodeid": "test_mod.py::test_optional",
+                "when": "setup",
+                "outcome": "skipped",
+                "longrepr": ["test_mod.py", 12, "Skipped: requires foo"],
+            },
         ],
     )
     reportlog_b = _write_reportlog(
         tmp_path / "b.jsonl",
         [
-            {"$report_type": "TestReport", "nodeid": "test_mod.py::test_optional", "when": "setup", "outcome": "passed", "longrepr": None},
-            {"$report_type": "TestReport", "nodeid": "test_mod.py::test_optional", "when": "call", "outcome": "passed", "longrepr": None},
-            {"$report_type": "TestReport", "nodeid": "test_mod.py::test_optional", "when": "teardown", "outcome": "passed", "longrepr": None},
+            {
+                "$report_type": "TestReport",
+                "nodeid": "test_mod.py::test_optional",
+                "when": "setup",
+                "outcome": "passed",
+                "longrepr": None,
+            },
+            {
+                "$report_type": "TestReport",
+                "nodeid": "test_mod.py::test_optional",
+                "when": "call",
+                "outcome": "passed",
+                "longrepr": None,
+            },
+            {
+                "$report_type": "TestReport",
+                "nodeid": "test_mod.py::test_optional",
+                "when": "teardown",
+                "outcome": "passed",
+                "longrepr": None,
+            },
         ],
     )
 
@@ -48,7 +84,17 @@ def test_requires_skip_reason_allowlist_is_respected(tmp_path: Path) -> None:
     reportlog = _write_reportlog(
         tmp_path / "reportlog.jsonl",
         [
-            {"$report_type": "TestReport", "nodeid": "test_mod.py::test_ros3", "when": "setup", "outcome": "skipped", "longrepr": ["test_mod.py", 12, "Skipped: requires h5netcdf>=1.3.0 and h5py with ros3 support"]},
+            {
+                "$report_type": "TestReport",
+                "nodeid": "test_mod.py::test_ros3",
+                "when": "setup",
+                "outcome": "skipped",
+                "longrepr": [
+                    "test_mod.py",
+                    12,
+                    "Skipped: requires h5netcdf>=1.3.0 and h5py with ros3 support",
+                ],
+            },
         ],
     )