Fixed minor bug with parametrized @pytest_fixture_plus: spaces are now correctly removed when multiple parameter names are provided in the same parametrize call. Fixes #27

Author: Sylvain MARIE

pytest_cases/main.py

@@ -308,13 +308,15 @@ def pytest_fixture_plus(scope="function",
                              "name in a @pytest.mark.parametrize mark")
 
         else:
+            paramnames = [name.strip() for name in m.param_names]
+
             # create a fixture function for this parameter
             def _param_fixture(request):
                 """a dummy fixture that simply returns the parameter"""
                 return request.param
 
             # generate a fixture name (find an available name if already used)
-            gen_name = fixture_func.__name__ + "__" + 'X'.join(m.param_names)  # + "__gen"
+            gen_name = fixture_func.__name__ + "__" + 'X'.join(paramnames)  # + "__gen"
             i = 0
             _param_fixture.__name__ = gen_name
             while _param_fixture.__name__ in dir(module):
@@ -334,7 +336,7 @@ def _param_fixture(request):
                                  "".format(_param_fixture.__name__, module))
 
             # remember
-            params_map[_param_fixture.__name__] = m.param_names
+            params_map[_param_fixture.__name__] = paramnames
 
     # wrap the fixture function so that each of its parameter becomes the associated fixture name
     new_parameter_names = tuple(params_map.keys())

pytest_cases/tests/simple/test_fixtures_params.py

@@ -16,3 +16,14 @@ def myfix(arg1, arg2):
 def test_one(myfix):
     print(myfix)
 
+
+@pytest_fixture_plus
+@pytest.mark.parametrize("arg1, arg2", [
+    (1, 2),
+    (3, 4),
+])
+def myfix2(arg1, arg2):
+    return arg1, arg2
+
+def test_two(myfix2):
+    print(myfix)