Added tests for issue #126

Author: Sylvain MARIE

pytest_cases/tests/cases/issues/test_issue_126.py

@@ -0,0 +1,76 @@
+import pytest
+from pytest_cases import parametrize_with_cases
+
+
+@pytest.fixture()
+def dependent_fixture():
+    return 0
+
+
+class Foo:
+    def case_requirement_1(self, dependent_fixture):
+        return Foo, dependent_fixture + 1
+
+    def case_requirement_2(self, dependent_fixture):
+        return Foo, dependent_fixture - 1
+
+
+def case_requirement_1(dependent_fixture):
+    return case_requirement_1.__module__, dependent_fixture + 2
+
+
+def case_requirement_2(dependent_fixture):
+    return case_requirement_1.__module__, dependent_fixture - 2
+
+
+@parametrize_with_cases("a,b", cases=(Foo, "."), prefix="case", debug=True)
+def test_functionality(a, b):
+    do_assert(test_functionality, a, b)
+
+
+@parametrize_with_cases("a,b", cases=(".", Foo), prefix="case", debug=True)
+def test_functionality_again(a, b):
+    do_assert(test_functionality_again, a, b)
+
+
+class TestNested:
+    @parametrize_with_cases("a,b", cases=(Foo, "."), prefix="case", debug=True)
+    def test_functionality_again2(self, a, b):
+        do_assert(TestNested.test_functionality_again2, a, b)
+
+
+# init our markers
+markers_dict = {}
+for host in (test_functionality, test_functionality_again, TestNested.test_functionality_again2):
+    markers_dict[host] = ({-1, 1}, {-2, 2})  # [0] is for cases in Foo, [1] is for cases in module
+
+
+def do_assert(host, a, b):
+    """used in tests below to make sure that all cases are used"""
+    if a is Foo:
+        markers_dict[host][0].remove(b)
+    elif a == case_requirement_1.__module__:
+        markers_dict[host][1].remove(b)
+    else:
+        raise ValueError()
+
+
+def test_synthesis(module_results_dct):
+    # assert that all fixtures have been used once in all tests
+    for host in (test_functionality, test_functionality_again, TestNested.test_functionality_again2):
+        assert markers_dict[host] == (set(), set())
+
+    assert list(module_results_dct) == [
+        'test_functionality[a_b_is__requirement_1]',
+        'test_functionality[a_b_is__requirement_2]',
+        'test_functionality[a_b_is__requirement_1_]',
+        'test_functionality[a_b_is__requirement_2_]',
+        'test_functionality_again[a_b_is__requirement_1_]',  # <- note: same fixtures than previously
+        'test_functionality_again[a_b_is__requirement_2_]',  # idem
+        'test_functionality_again[a_b_is__requirement_1]',   # idem
+        'test_functionality_again[a_b_is__requirement_2]',   # idem
+        'test_functionality_again2[a_b_is__requirement_1]',  # idem
+        'test_functionality_again2[a_b_is__requirement_2]',  # idem
+        'test_functionality_again2[a_b_is__requirement_1_]',  # idem
+        'test_functionality_again2[a_b_is__requirement_2_]'  # idem
+    ]

pytest_cases/tests/cases/issues/test_issue_126_2.py

@@ -0,0 +1,47 @@
+import pytest
+
+from pytest_cases.common_pytest_marks import has_pytest_param
+from pytest_cases import parametrize_with_cases
+
+
+# only do this for pytest version 3+
+if has_pytest_param:
+    @pytest.fixture
+    def b():
+        return -1
+
+
+    @pytest.fixture(name='a')
+    def a_in_module():
+        return 1
+
+
+    class TestA:
+        @pytest.fixture(name='a')
+        def a_nested(self):
+            return 2
+
+        def test_a(self, a):
+            assert a == 2
+
+        @parametrize_with_cases('o', debug=True)
+        def test_foo_nested(self, o):
+            assert o == 'case!'
+
+
+    def test_bar(a):
+        assert a == 1
+
+
+    @parametrize_with_cases('o', debug=True)
+    def test_foo(o):
+        assert o == 'case!'
+
+
+    def test_synthesis(module_results_dct):
+         assert list(module_results_dct) == [
+             'test_a',
+             'test_foo_nested[o_is_a_]',
+             'test_bar',
+             'test_foo[o_is_a_]'
+        ]

pytest_cases/tests/cases/issues/test_issue_126_2_cases.py

@@ -0,0 +1,6 @@
+def case_a(b, a):
+    # a and b are fixtures defined in caller module/class
+    # note that case id is also 'a'. The goal is to check that no conflict happens here
+    assert a in (1, 2)
+    assert b == -1
+    return 'case!'