From 15bd3d06337e3a45e229f799590d15ec5108e3c6 Mon Sep 17 00:00:00 2001 From: Sankur Kundu Date: Thu, 5 Nov 2026 01:44:10 +0000 Subject: [PATCH] Test that generated tests report the file of the original test Tests generated with `--distribute`, `--repeat` and `-P` used to report ReFrame's own `testgenerators.py` as their file, for example in the failure report (#3669). This was fixed by passing the module of the original test to `make_test()`; check it in the test generator unit tests. --- unittests/test_testgenerators.py | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/unittests/test_testgenerators.py b/unittests/test_testgenerators.py index 649f867741..6da1336f25 100644 --- a/unittests/test_testgenerators.py +++ b/unittests/test_testgenerators.py @@ -3,6 +3,7 @@ # # SPDX-License-Identifier: BSD-3-Clause +import inspect import pytest import reframe as rfm @@ -62,6 +63,10 @@ def sys0p0_nodes(): with pytest.raises(StopIteration): next(nodelist_iter) + # The generated tests are reported as defined in the original file + for tc in new_cases: + assert tc.check._filename == testcases[0].check._filename + def test_repeat_testcases(): loader = RegressionCheckLoader([ @@ -70,8 +75,11 @@ def test_repeat_testcases(): testcases = executors.generate_testcases(loader.load_all()) assert len(testcases) == 2 + filename = testcases[0].check._filename testcases = repeat_tests(testcases, 10) assert len(testcases) == 20 + for tc in testcases: + assert tc.check._filename == filename @pytest.fixture @@ -110,3 +118,5 @@ def test_parameterize_tests(HelloTest): 'unknown': 1} ) assert len(testcases) == 18 + for tc in testcases: + assert tc.check._filename == inspect.getfile(HelloTest)