diff options
| author |   xudan <xudan16@huawei.com> | 2019-05-17 04:13:30 -0400 |
|---|---|---|
| committer | Dan Xu <xudan16@huawei.com> | 2019-06-10 03:24:16 +0000 |
| commit | fe1a78666f215a568866c4b5eb28c8feda0ad278 (patch) | |
| tree | b2c93c3f7f3f28b1d91bcb5247fc2b0e4b6d715c /dovetail/tests/unit | |
| parent | dc5264ec9b19a8673204af11dbba68bdb50cfde2 (diff) | |
Bugfix: Fix Dovetail 2 bugs found with CI results
There are 2 bugs found with CI results of functest.tempest.neutron_tempest_plugin_api
and all HA test cases.
Yardtsick conf file does't mapping the keyfile to Yardstick container, so all
HA test cases got SSHERROR.
Also remove some useless debug msg to make dovetail.log more concise.
Change-Id: I63c12d1b91ed5f8f8a893689384685ca9215f367
Signed-off-by: xudan <xudan16@huawei.com>
Diffstat (limited to 'dovetail/tests/unit')
| -rw-r--r-- | dovetail/tests/unit/test_report.py | 10 | ||||
| -rw-r--r-- | dovetail/tests/unit/test_testcase.py | 7 |
2 files changed, 2 insertions, 15 deletions
diff --git a/dovetail/tests/unit/test_report.py b/dovetail/tests/unit/test_report.py index 4840f2b5..849b31d8 100644 --- a/dovetail/tests/unit/test_report.py +++ b/dovetail/tests/unit/test_report.py @@ -1440,16 +1440,6 @@ class ReportTesting(unittest.TestCase): checker.check(testcase_obj, db_result) testcase_obj.sub_testcase.assert_called_once_with() - logger_obj.debug.assert_has_calls([ - call('Check sub_testcase: subt_a'), - call('Check sub_testcase: subt_b'), - call('Check sub_testcase: subt_c'), - call('Check sub_testcase: subt_d')]) - testcase_obj.sub_testcase_passed.assert_has_calls([ - call('subt_a', 'PASS'), - call('subt_b', 'SKIP'), - call('subt_c', 'FAIL'), - call('subt_d', 'FAIL')]) testcase_obj.passed.assert_has_calls([call('PASS'), call('FAIL')]) @patch('dovetail.report.dt_logger') diff --git a/dovetail/tests/unit/test_testcase.py b/dovetail/tests/unit/test_testcase.py index b915556c..06a23025 100644 --- a/dovetail/tests/unit/test_testcase.py +++ b/dovetail/tests/unit/test_testcase.py @@ -145,8 +145,6 @@ class TestcaseTesting(unittest.TestCase): result = testcase.sub_testcase_passed('name', 'passed') - logger_obj.debug.assert_called_once_with( - 'sub_testcase_passed: name passed') self.assertEquals('passed', result) def test_validate_type(self): @@ -265,9 +263,8 @@ class TestcaseTesting(unittest.TestCase): mock_path.join.assert_called_once_with('value', 'tempest_custom.txt') mock_open.assert_called_once_with(file_path, 'w+') file_obj.write.assert_called_once_with(sub_test + '\n') - logger_obj.debug.assert_has_calls([ - call('Save test cases {}'.format(sub_test)), - call('Save test cases to {}'.format(file_path))]) + logger_obj.debug.assert_called_once_with( + 'Save test cases to {}'.format(file_path)) self.assertEquals(file_path, result) @patch('__builtin__.open') |