Merge pull request #3553 from vkarak/bugfix/dry-run-outputdir

[bugfix] Do not wipe the output dir of a test if run with `--dry-run`

Author: vkarak

reframe/core/pipeline.py

@@ -1737,10 +1737,11 @@ def _setup_paths(self):
                 self.current_system.name, self._current_partition.name,
                 self._current_environ.name, self.short_name
             )
-            self._outputdir = runtime.make_outputdir(
-                self.current_system.name, self._current_partition.name,
-                self._current_environ.name, self.short_name
-            )
+            if not self.is_dry_run():
+                self._outputdir = runtime.make_outputdir(
+                    self.current_system.name, self._current_partition.name,
+                    self._current_environ.name, self.short_name
+                )
         except OSError as e:
             raise PipelineError('failed to set up paths') from e
 

unittests/test_cli.py

@@ -162,6 +162,19 @@ def test_check_success(run_reframe, tmp_path, run_action):
                           'reports' / 'run-report-0.json')
 
 
+def test_outputdir_dry_run(run_reframe, tmp_path):
+    # We need to make sure that outputdir is maintained if the same test is
+    # rerun with `--dry-run`
+
+    run_reframe(action='run', more_options=['-n', '^HelloTest'])
+    run_reframe(action='dry_run', more_options=['-n', '^HelloTest'])
+    prefix = (tmp_path / 'output' / 'generic' / 'default' /
+              'builtin' / 'HelloTest')
+    for path in ('rfm_build.err', 'rfm_build.out', 'rfm_build.sh',
+                 'rfm_job.err', 'rfm_job.out', 'rfm_job.sh'):
+        assert (prefix / path).exists()
+
+
 def test_check_restore_session_failed(run_reframe, tmp_path):
     run_reframe(
         checkpath=['unittests/resources/checks_unlisted/deps_complex.py']