From 2b07976b948d0e768e2b46c5b63ee454e5f8aeb2 Mon Sep 17 00:00:00 2001 From: Kan Date: Tue, 12 Jan 2016 09:03:19 +0000 Subject: Fix params order in assertEqual Fix params order to correspond to real signature: assertEqual(expected, actual) Change-Id: I129aad5d8bb7ca0541c07913cb5edd8eef9e90ba Closes-Bug: #1277104 --- ironic_python_agent/tests/unit/test_utils.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'ironic_python_agent/tests/unit/test_utils.py') 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') -- cgit v1.2.1