diff options
| author | Kan <kansks@cn.ibm.com> | 2016-01-12 09:03:19 +0000 |
|---|---|---|
| committer | Kan <kansks@cn.ibm.com> | 2016-01-13 02:03:11 +0000 |
| commit | 2b07976b948d0e768e2b46c5b63ee454e5f8aeb2 (patch) | |
| tree | 45b4953bfa264e3c4f93d3b1027f02d7ec6d6616 /ironic_python_agent/tests/unit/test_utils.py | |
| parent | 43a149d5eaff672e1755d683864d0acbd43bfa8c (diff) | |
| download | ironic-python-agent-2b07976b948d0e768e2b46c5b63ee454e5f8aeb2.tar.gz | |
Fix params order in assertEqual
Fix params order to correspond to real signature:
assertEqual(expected, actual)
Change-Id: I129aad5d8bb7ca0541c07913cb5edd8eef9e90ba
Closes-Bug: #1277104
Diffstat (limited to 'ironic_python_agent/tests/unit/test_utils.py')
| -rw-r--r-- | ironic_python_agent/tests/unit/test_utils.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ironic_python_agent/tests/unit/test_utils.py b/ironic_python_agent/tests/unit/test_utils.py index cb36f160..bf853373 100644 --- a/ironic_python_agent/tests/unit/test_utils.py +++ b/ironic_python_agent/tests/unit/test_utils.py @@ -139,7 +139,7 @@ class GetAgentParamsTestCase(test_base.BaseTestCase): def test__read_params_from_file_fail(self, logger_mock, open_mock): open_mock.side_effect = Exception params = utils._read_params_from_file('file-path') - self.assertEqual(params, {}) + self.assertEqual({}, params) @mock.patch(OPEN_FUNCTION_NAME) def test__read_params_from_file(self, open_mock): @@ -151,8 +151,8 @@ class GetAgentParamsTestCase(test_base.BaseTestCase): params = utils._read_params_from_file('file-path') open_mock.assert_called_once_with('file-path') read_mock.assert_called_once_with() - self.assertEqual(params['api-url'], 'http://localhost:9999') - self.assertEqual(params['foo'], 'bar') + self.assertEqual('http://localhost:9999', params['api-url']) + self.assertEqual('bar', params['foo']) self.assertFalse('baz' in params) @mock.patch.object(utils, '_set_cached_params') |
