Added a test for workaround in issue #211 + minor docstring edit + changelog update

Author: Sylvain MARIE

docs/changelog.md

@@ -1,5 +1,9 @@
 # Changelog
 
+### (in progress) - `currentcases` fixture cleanup
+
+ - Fixed minor issue where empty entries could be present in `currentcases`. Fixes [#213](https://github.com/smarie/python-pytest-cases/issues/213)
+
 ### 3.5.2 - bugfix with the `currentcases` fixture
 
  - Fixed issues where the `currentcases` fixture would not return the correct case function. Fixed [#212](https://github.com/smarie/python-pytest-cases/issues/212)

pytest_cases/case_parametrizer_new.py

@@ -422,7 +422,7 @@ def case_to_argvalues(host_class_or_module,    # type: Union[Type, ModuleType]
         #                             id="%s-%s" % (case_id, c.id), marks=case_marks + tuple(c.marks))
         #                  for c in meta._calls)
     else:
-        # at least a required fixture (direct req or through @pytest.mark.usefixtures ):
+        # at least 1 required fixture (direct req or through @pytest.mark.usefixtures ), OR parametrized.
 
         # if meta.is_parametrized:
         #    # nothing to do, the parametrization marks are on the fixture to create so they will be taken into account

pytest_cases/tests/cases/issues/test_issue_211.py

@@ -0,0 +1,38 @@
+import pytest
+import sys
+from pytest_cases import parametrize, parametrize_with_cases
+
+
+PY3 = sys.version_info >= (3,)
+
+
+class MyClassName:
+    @parametrize(name=("joe", "alice"))
+    def case_widget(self, name):
+        return name
+
+    def case_foo(self):
+        return "foo"
+
+
+@parametrize_with_cases('val', cases=MyClassName)
+def test_function(val, current_cases, request):
+    print(val)
+    case_id, case_func = current_cases['val']
+    print((case_id, case_func))
+
+    if (case_func is MyClassName.case_widget) if PY3 else (case_func == MyClassName.case_widget):
+        # workaround to get the parameter, but a bit dirty
+        if request.node.callspec.params['widget'].argvalues[0] == "joe":
+            assert request.node.name == 'test_function[widget-name=joe]'
+            pytest.skip("joe skipped")
+
+    assert request.node.name != 'test_function[widget-name=joe]'
+
+
+def test_synthesis(module_results_dct):
+    assert list(module_results_dct) == [
+        'test_function[widget-name=joe]',
+        'test_function[widget-name=alice]',
+        'test_function[foo]'
+    ]