fix(test): normalize benchmark names without truncating param values

Author: clean6378-max-it

scripts/check_benchmark_regression.py

@@ -15,8 +15,15 @@ class BenchmarkDataError(ValueError):
 
 
 def normalize_benchmark_name(name: str) -> str:
-    """Strip pytest node prefix so baselines match short or full benchmark names."""
-    return str(name).rsplit("::", 1)[-1]
+    """Strip pytest file node prefix so baselines match short or full benchmark names."""
+    text = str(name)
+    if "::" not in text:
+        return text
+    prefix, _, suffix = text.partition("::")
+    # Only strip module paths (…/test_foo.py::test_name); leave "::" inside [param::value] intact.
+    if prefix.endswith(".py"):
+        return suffix
+    return text
 
 
 def load_results(results_path: str | Path) -> dict[str, float]:

tests/test_check_benchmark_regression.py

@@ -37,6 +37,13 @@ def test_normalize_benchmark_name_strips_module_prefix() -> None:
     assert normalize_benchmark_name("test_summary_cache_hit") == "test_summary_cache_hit"
 
 
+def test_normalize_benchmark_name_preserves_colons_in_param_values() -> None:
+    short = "test_x[param::v]"
+    full = f"tests/benchmarks/test_x.py::{short}"
+    assert normalize_benchmark_name(short) == short
+    assert normalize_benchmark_name(full) == short
+
+
 def test_load_results_normalizes_full_node_id(tmp_path) -> None:
     path = tmp_path / "results.json"
     _write_results(