diff --git a/reframe/core/decorators.py b/reframe/core/decorators.py index 0a0b745430..7f1f26f14c 100644 --- a/reframe/core/decorators.py +++ b/reframe/core/decorators.py @@ -85,8 +85,8 @@ def _validate_test(cls): 'subclass of RegressionTest') if (cls.is_abstract()): - raise ValueError(f'decorated test ({cls.__qualname__!r}) is an' - f' abstract test') + raise ValueError(f'decorated test ({cls.__qualname__!r}) has one or ' + f'more undefined parameters') def simple_test(cls): diff --git a/reframe/core/meta.py b/reframe/core/meta.py index 7cc43f6944..422997050a 100644 --- a/reframe/core/meta.py +++ b/reframe/core/meta.py @@ -233,13 +233,12 @@ def param_space(cls): return cls._rfm_param_space def is_abstract(cls): - '''Check if the test is an abstract test. + '''Check if the class is an abstract test. - If the parameter space has undefined parameters, the test is considered - an abstract test. If that is the case, the length of the parameter - space is just 0. + This is the case when some parameters are undefined, which results in + the length of the parameter space being 0. - :return: bool indicating wheteher the test is abstract or not + :return: bool indicating wheteher the test has undefined parameters. :meta private: ''' diff --git a/unittests/test_parameters.py b/unittests/test_parameters.py index e9c1a4fa50..f1605ff9c4 100644 --- a/unittests/test_parameters.py +++ b/unittests/test_parameters.py @@ -21,6 +21,7 @@ class TwoParams(NoParams): class Abstract(TwoParams): + '''An abstract test is a test with undefined parameters.''' P0 = parameter()