[bugfix] Fix multiple test retries for tests with fixtures

reframe/frontend/executors/__init__.py

@@ -804,6 +804,10 @@ def _failed_or_deps():
             self._runall(failed_cases)
             failures = _failed_or_deps()
 
+            # Reset the run cases before re-entering the loop to make sure
+            # dependencies will be built correctly
+            cases = failed_cases
+
     def _runall(self, testcases):
         def print_separator(check, prefix):
             self._printer.separator(

unittests/resources/checks_unlisted/fixtures_retries.py

@@ -0,0 +1,32 @@
+import reframe as rfm
+import reframe.utility.sanity as sn
+from reframe.core.builtins import sanity_function, fixture
+from reframe.core.runtime import runtime
+
+
+_FIXT_PASS_THRES = 1
+_TEST_PASS_THRES = 2
+
+
+class FixtA(rfm.RunOnlyRegressionTest):
+    executable = 'true'
+
+    @sanity_function
+    def validate(self):
+        return runtime().current_run >= _FIXT_PASS_THRES
+
+
+@rfm.simple_test
+class TestA(rfm.RunOnlyRegressionTest):
+    fixtA = fixture(FixtA)
+    executable = 'true'
+    valid_systems = ['*']
+    valid_prog_environs = ['*']
+
+    @sanity_function
+    def validate(self):
+        if runtime().current_run >= _TEST_PASS_THRES:
+            return sn.assert_true(self.fixtA.stagedir.endswith('_retry1'),
+                                  msg=f'wrong stagedir: {self.fixtA.stagedir}')
+
+        return False

unittests/test_policies.py

@@ -311,6 +311,16 @@ def test_pass_in_retries(make_runner, make_cases, tmp_path, common_exec_ctx):
     assert 0 == len(runner.stats.failed())
 
 
+def test_retries_fixtures(make_runner, make_loader, common_exec_ctx):
+    runner = make_runner(max_retries=2)
+    runner.runall(executors.generate_testcases(
+        make_loader([
+            'unittests/resources/checks_unlisted/fixtures_retries.py'
+        ]).load_all()
+    ))
+    assert len(runner.stats.failed()) == 0
+
+
 @pytest.fixture(params=[signal.SIGTERM, signal.SIGHUP])
 def signum(request):
     return request.param