summaryrefslogtreecommitdiff
path: root/ironic_python_agent/tests/unit/test_utils.py
diff options
context:
space:
mode:
authorKan <kansks@cn.ibm.com>2016-01-12 09:03:19 +0000
committerKan <kansks@cn.ibm.com>2016-01-13 02:03:11 +0000
commit2b07976b948d0e768e2b46c5b63ee454e5f8aeb2 (patch)
tree45b4953bfa264e3c4f93d3b1027f02d7ec6d6616 /ironic_python_agent/tests/unit/test_utils.py
parent43a149d5eaff672e1755d683864d0acbd43bfa8c (diff)
downloadironic-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.py6
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')