Improve error handling by using fail rather than custom exception

Author: skeletorXVI

src/pytest_parametrization_annotation/__init__.py

@@ -1,7 +1,5 @@
 from .annotation import Parametrized
-from .exceptions import ParameterValueUndefined
 
 __all__ = [
     "Parametrized",
-    "ParameterValueUndefined",
 ]

src/pytest_parametrization_annotation/exceptions.py

@@ -1,13 +1,17 @@
-from typing import Annotated, Any, Callable, Mapping, get_origin, get_type_hints
+from __future__ import annotations
 
-from _pytest.python import Metafunc
+from typing import Annotated, Any, Callable, Mapping, get_origin, get_type_hints, TYPE_CHECKING
+
+from _pytest.outcomes import fail
 
 from pytest_parametrization_annotation.annotation import (
     Parametrized,
     _Default,
     _DefaultCallable,
 )
-from pytest_parametrization_annotation.exceptions import ParameterValueUndefined
+
+if TYPE_CHECKING:
+    from _pytest.python import Metafunc
 
 
 def get_parameters_from_type_hints(func: Callable) -> dict[str, Parametrized]:
@@ -33,7 +37,7 @@ def get_parameters_from_type_hints(func: Callable) -> dict[str, Parametrized]:
 
 
 def get_parametrized_value(
-    kwargs: Mapping[str, Any], parameter: str, meta: Parametrized
+        kwargs: Mapping[str, Any], parameter: str, meta: Parametrized
 ) -> Any:
     if parameter in kwargs:
         return kwargs[parameter]
@@ -57,10 +61,10 @@ def register_parametrized_cases(metafunc: Metafunc):
 
         cases = []
         ids = []
-        for i, marker in enumerate(markers):
+        fails: list[tuple[str, str]] = []
+        for i, marker in enumerate(reversed(markers)):
             case_id = marker.args[0] if len(marker.args) > 0 else None
             ids.append(case_id)
-
             try:
                 cases.append(
                     tuple(
@@ -69,11 +73,16 @@ def register_parametrized_cases(metafunc: Metafunc):
                     )
                 )
             except KeyError as e:
-                raise ParameterValueUndefined(
-                    function_definition.nodeid,
-                    case_id or i,
-                    e.args[0],
-                ) from e
+                case_descriptor = f"named '{case_id}'" if case_id else f"number {i + 1}"
+                fails.append((case_descriptor, e.args[0]))
+
+        if fails:
+            fail_message = "\n".join(
+                f"{function_definition.nodeid} | Case {case_descriptor}: Failed to populate because the parameter '{parameter}' is not provided and default is not configured."  # noqa: E501
+                for case_descriptor, parameter in fails
+            )
+
+            fail(fail_message, pytrace=False)
 
         metafunc.parametrize(
             tuple(parameters),
@@ -82,4 +91,4 @@ def register_parametrized_cases(metafunc: Metafunc):
                 name for name, parameter in parameters.items() if parameter.indirect
             ],
             ids=ids,
-        )
+        )

src/pytest_parametrization_annotation/functionality.py

@@ -0,0 +1,20 @@
+{{ "from typing import Annotated" }}
+
+import pytest
+{{ "from pytest_parametrization_annotation import Parametrized" }}
+
+
+{% for case in cases -%}
+@pytest.mark.case(
+    {% if case.id != None %}"{{ case.id }}",{% endif %}
+    {% for parameter in case.parameters -%}
+        {{ parameter }}=1,
+    {% endfor -%}
+)
+{% endfor %}
+def test(
+    {% for parameter in parameters -%}
+        {{ parameter }}: Annotated[int, Parametrized],
+    {% endfor -%}
+) -> None:
+    assert True

tests/templates/test_case_marker/missing_parameter.py

@@ -0,0 +1,12 @@
+from typing import Annotated
+
+import pytest
+from pytest_parametrization_annotation import Parametrized
+
+
+@pytest.mark.case("example")
+def test(
+    first: Annotated[int, Parametrized],
+    second: Annotated[int, Parametrized(default=123)],
+):
+    assert first == 1

tests/templates/test_missing_parameters/test_error_messages.py.j2

@@ -0,0 +1,12 @@
+from typing import Annotated
+
+import pytest
+from pytest_parametrization_annotation import Parametrized
+
+
+@pytest.mark.case("example")
+def test(
+    first: Annotated[int, Parametrized(default_factory=lambda: 123)],
+    second: Annotated[int, Parametrized],
+):
+    assert first == 1

tests/templates/test_missing_parameters/test_with_default.py

@@ -355,20 +355,3 @@ def test_indirect_parametrization(
 
     result = pytester.runpytest()
     result.assert_outcomes(passed=expected_passed, failed=expected_failed)
-
-
-def test_missing_parameter(pytester):
-    python_code = open(
-        os.path.join(
-            os.path.dirname(os.path.realpath(__file__)),
-            "templates/test_case_marker/missing_parameter.py",
-        )
-    ).read()
-    pytester.makepyfile(python_code)
-
-    result = pytester.runpytest()
-    assert (
-        "E   pytest_parametrization_annotation.exceptions.ParameterValueUndefined: test_missing_parameter.py::test | Case 'example': Failed to populate because the parameter 'second' is not provided and default is not configured." # noqa: E501
-        in result.outlines
-    )
-    result.assert_outcomes(errors=1)