eea6832c03972ae312b5a5d69ce43f8d6e281a40,unittests/test_loader.py,TestRegressionCheckLoader,test_extend_decorator,#TestRegressionCheckLoader#,76

Before Change

        assert 0 == len(tests)

    def test_extend_decorator(self):
        with pytest.warns(ReframeDeprecationWarning) as record:
            tests = self.loader.load_from_file(
                "unittests/resources/checks_unlisted/extend_class.py")

        assert len(record) == 3

After Change

        assert 0 == len(tests)

    def test_extend_decorator(self):
        with pytest.warns(ReframeDeprecationWarning) as record:
            // tests = self.loader.load_from_file(
            //     "unittests/resources/checks_unlisted/extend_class.py")
            @rfm.simple_test
            class TestSimple(rfm.RegressionTest):
                // The test should not raise a deprecation warning even though
                // it overrides __init__
                def __init__(self):
                    pass


            @rfm.simple_test
            class TestDeprecated(rfm.RegressionTest):
                // Should raise a warning
                def setup(self, partition, environ, **job_opts):
                    super().setup(system, environ, **job_opts)


            @rfm.simple_test
            class TestDeprecatedRunOnly(rfm.RunOnlyRegressionTest):
                // Should raise a warning
                def setup(self, partition, environ, **job_opts):
                    super().setup(system, environ, **job_opts)


            @rfm.simple_test
            class TestDeprecatedCompileOnly(rfm.CompileOnlyRegressionTest):
                // Should raise a warning
                def setup(self, partition, environ, **job_opts):
                    super().setup(system, environ, **job_opts)


            @rfm.simple_test
            class TestDeprecatedCompileOnlyDerived(TestDeprecatedCompileOnly):
                // Should not raise a warning because the setup of the parent was not set as final
                def setup(self, partition, environ, **job_opts):
                    super().setup(system, environ, **job_opts)


            @rfm.simple_test
            class TestExtended(rfm.RegressionTest, extended_test=True):
                def __init__(self):
                    pass

                // Should not raise a warning
                def setup(self, partition, environ, **job_opts):
                    super().setup(system, environ, **job_opts)


            @rfm.simple_test
            class TestExtendedDerived(TestExtended):
                def __init__(self):
                    pass

                // Should not raise a warning
                def setup(self, partition, environ, **job_opts):
                    super().setup(system, environ, **job_opts)

                // Should raise a warning
                def run(self):
                    super().run()


            @rfm.simple_test
            class TestExtendedRunOnly(rfm.RunOnlyRegressionTest, extended_test=True):
                def __init__(self):
                    pass

                // Should not raise a warning
                def setup(self, partition, environ, **job_opts):
                    super().setup(system, environ, **job_opts)

                // Should not raise a warning
                def run(self):
                    super().run()


            @rfm.simple_test
            class TestExtendedCompileOnly(rfm.CompileOnlyRegressionTest, extended_test=True):
                def __init__(self):
                    pass

                // Should not raise a warning
                def setup(self, partition, environ, **job_opts):
                    super().setup(system, environ, **job_opts)

                // Should not raise a warning
                def run(self):
                    super().run()

        assert len(record) == 4

Instances