From: Cédric Ollivier <cedric.ollivier@orange.com>
Date: Sun, 15 Jul 2018 09:22:05 +0000 (+0200)
Subject: Fix results when only one skipped test
X-Git-Tag: 0.53~1
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F23%2F59823%2F1;p=functest-xtesting.git

Fix results when only one skipped test

Change-Id: Id6c8029b8be1e3a01979feb45daebad08710fe20
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
---

diff --git a/xtesting/ci/run_tests.py b/xtesting/ci/run_tests.py
index 6ecd0cfc..0471732f 100644
--- a/xtesting/ci/run_tests.py
+++ b/xtesting/ci/run_tests.py
@@ -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