diff options
| author |   Cédric Ollivier <cedric.ollivier@orange.com> | 2018-08-29 16:08:14 +0200 |
|---|---|---|
| committer | Cédric Ollivier <cedric.ollivier@orange.com> | 2018-08-29 16:11:42 +0200 |
| commit | 76f86384a93aea045645d4e17b4e74beed07fa52 (patch) | |
| tree | b9247fa71527a063e82ecbf6b82693c36d214f4c | |
| parent | 97b050fed328a94919b9251b4b2b0e413a2e7d42 (diff) | |
Stop verifying ResultWriter.write_results exit codes
ResultWriter.write_results() returns false if one test fails.
It's skipped by run() because the report is correctly generated.
[1] https://robot-framework.readthedocs.io/en/2.9.2/_modules/robot/reporting/resultwriter.html
Change-Id: If29059b272679ed89a1189e0bb52997c83401b94
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
| -rw-r--r-- | xtesting/core/robotframework.py | 2 | ||||
| -rw-r--r-- | xtesting/tests/unit/core/test_robotframework.py | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/xtesting/core/robotframework.py b/xtesting/core/robotframework.py index 9865a26f..2791b559 100644 --- a/xtesting/core/robotframework.py +++ b/xtesting/core/robotframework.py @@ -127,7 +127,7 @@ class RobotFramework(testcase.TestCase): try: self.parse_results() self.__logger.info("Results were successfully parsed") - assert self.generate_report() == 0 + self.generate_report() self.__logger.info("Results were successfully generated") except RobotError as ex: self.__logger.error("Run suites before publishing: %s", ex.message) diff --git a/xtesting/tests/unit/core/test_robotframework.py b/xtesting/tests/unit/core/test_robotframework.py index 4cda171c..398cf87a 100644 --- a/xtesting/tests/unit/core/test_robotframework.py +++ b/xtesting/tests/unit/core/test_robotframework.py @@ -283,7 +283,7 @@ class RunTesting(unittest.TestCase): def test_generate_report_err(self): with mock.patch.object(self.test, 'generate_report', return_value=1) as mmethod: - self._test_generate_report(self.test.EX_RUN_ERROR) + self._test_generate_report(self.test.EX_OK) mmethod.assert_called_once_with() def test_generate_report(self): |