Fix results when only one skipped test 23/59823/1
authorCédric Ollivier <cedric.ollivier@orange.com>
Sun, 15 Jul 2018 09:22:05 +0000 (11:22 +0200)
committerCédric Ollivier <cedric.ollivier@orange.com>
Sun, 15 Jul 2018 09:22:05 +0000 (11:22 +0200)
Change-Id: Id6c8029b8be1e3a01979feb45daebad08710fe20
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
xtesting/ci/run_tests.py

index 6ecd0cf..0471732 100644 (file)
@@ -150,6 +150,7 @@ class Runner(object):
         run_dict = self.get_run_dict(test.get_name())
         if run_dict:
             try:
+                LOGGER.info("Loading test case '%s'...", test.get_name())
                 module = importlib.import_module(run_dict['module'])
                 cls = getattr(module, run_dict['class'])
                 test_dict = Runner.get_dict_by_test(test.get_name())
@@ -158,6 +159,7 @@ class Runner(object):
                 test_case.check_requirements()
                 if test_case.is_skipped:
                     LOGGER.info("Skipping test case '%s'...", test.get_name())
+                    LOGGER.info("Test result:\n\n%s\n", test_case)
                     return testcase.TestCase.EX_TESTCASE_SKIPPED
                 LOGGER.info("Running test case '%s'...", test.get_name())
                 try:
@@ -233,7 +235,7 @@ class Runner(object):
                 elif self.tiers.get_test(kwargs['test']):
                     result = self.run_test(
                         self.tiers.get_test(kwargs['test']))
-                    if result != testcase.TestCase.EX_OK:
+                    if result == testcase.TestCase.EX_TESTCASE_FAILED:
                         LOGGER.error("The test case '%s' failed.",
                                      kwargs['test'])
                         self.overall_result = Result.EX_ERROR