Minor: improved _create_param_fixture now raises an exception if auto_simplify=True and the provided argvalue is marked using pytest.param.

Author: Sylvain MARIE

pytest_cases/fixture_core2.py

@@ -102,7 +102,9 @@ def _create_param_fixture(fixtures_dest,
         # Simplification: do not parametrize the fixture, it will directly return the single value
         argvalue_to_return = argvalues[0]
         if is_marked_parameter_value(argvalue_to_return):
-            argvalue_to_return = get_marked_parameter_values(argvalue_to_return)
+            # Warning in that case the argvalues should not be a pytest.param.
+            # argvalue_to_return = get_marked_parameter_values(argvalue_to_return)
+            raise ValueError("When auto_simplify=True the argvalue can not be a pytest.param")
 
         # create the fixture - set its name so that the optional hook can read it easily
         @with_signature("%s()" % argname)

pytest_cases/fixture_parametrize_plus.py

@@ -529,7 +529,7 @@ def _create_params_alt(fh, test_func_name, union_name, from_i, to_i, hook):  # n
                     print(" - Creating new fixture %r to handle parameter %s" % (p_fix_name, i))
 
                 # Create the fixture that will return the unique parameter value ("auto-simplify" flag)
-                # IMPORTANT that fixture is NOT parametrized so has no id nor marks: use argvalues not marked_argvalues
+                # IMPORTANT auto_simplify=True we create a NON-parametrized fixture. use argvalues not marked_argvalues
                 _create_param_fixture(fh, argname=p_fix_name, argvalues=argvalues[i:i + 1], hook=hook, auto_simplify=True)
 
                 # Create the alternative