diff options
| -rw-r--r-- | xtesting/ci/run_tests.py | 4 | ||||
| -rw-r--r-- | xtesting/core/testcase.py | 10 |
2 files changed, 8 insertions, 6 deletions
diff --git a/xtesting/ci/run_tests.py b/xtesting/ci/run_tests.py index 71e8cfd4..0f43210c 100644 --- a/xtesting/ci/run_tests.py +++ b/xtesting/ci/run_tests.py @@ -172,14 +172,14 @@ class Runner(): test_case.run(**kwargs) except KeyError: test_case.run() - if self.report_flag: - test_case.push_to_db() result = test_case.is_successful() LOGGER.info("Test result:\n\n%s\n", test_case) if self.clean_flag: test_case.clean() if self.push_flag: test_case.publish_artifacts() + if self.report_flag: + test_case.push_to_db() except ImportError: LOGGER.exception("Cannot import module %s", run_dict['module']) except AttributeError: diff --git a/xtesting/core/testcase.py b/xtesting/core/testcase.py index a385651f..73a7371d 100644 --- a/xtesting/core/testcase.py +++ b/xtesting/core/testcase.py @@ -295,6 +295,7 @@ class TestCase(): six.reraise(typ, value, traceback) path = urllib.parse.urlparse(dst_s3_url).path.strip("/") output_str = "\n" + self.details["links"] = [] for root, _, files in os.walk(self.dir_results): for pub_file in files: # pylint: disable=no-member @@ -304,10 +305,11 @@ class TestCase(): os.path.join(root, pub_file), start=self.dir_results))) dst_http_url = os.environ["HTTP_DST_URL"] - output_str += "\n{}".format( - os.path.join(dst_http_url, os.path.relpath( - os.path.join(root, pub_file), - start=self.dir_results))) + link = os.path.join(dst_http_url, os.path.relpath( + os.path.join(root, pub_file), + start=self.dir_results)) + output_str += "\n{}".format(link) + self.details["links"].append(link) self.__logger.info( "All artifacts were successfully published: %s\n", output_str) return TestCase.EX_OK |