summaryrefslogtreecommitdiff
path: root/cinderclient/tests/test_client.py
diff options
context:
space:
mode:
authorKIYOHIRO ADACHI <adachi@mxs.nes.nec.co.jp>2014-02-26 15:59:14 +0900
committerKIYOHIRO ADACHI <adachi@mxs.nes.nec.co.jp>2014-08-18 13:41:21 +0900
commitf3ec0814971112068e933e343ef7ab8986f438c3 (patch)
treec334c7d005d948d1a36850f1ffaffb171e1b3974 /cinderclient/tests/test_client.py
parentb8e6ad0108a7e26f424edf79143ccd97db8cf478 (diff)
downloadpython-cinderclient-f3ec0814971112068e933e343ef7ab8986f438c3.tar.gz
Fix order of arguments in assertEqual
Some tests used incorrect order assertEqual(observed, expected). The correct order expected by testtools is... def assertEqual(self, expected, observed, message=''): """Assert that 'expected' is equal to 'observed'. :param expected: The expected value. :param observed: The observed value. :param message: An optional message to include in the error. """ The string length of the sum of the results of repr(expected) and repr(observed) is greater than 70, then, MismatchError message is changed, as below. Ex.: raise mismatch_error MismatchError: !=: reference = '_123456789_123456789_bar' actual = '_123456789_123456789_123456789_123456789_123456789' Change-Id: Id02ddfb2ca03df7f432cff67a7bed182cccc4924 Closes-Bug: #1259292
Diffstat (limited to 'cinderclient/tests/test_client.py')
-rw-r--r--cinderclient/tests/test_client.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/cinderclient/tests/test_client.py b/cinderclient/tests/test_client.py
index f81cf3d..bc70f1b 100644
--- a/cinderclient/tests/test_client.py
+++ b/cinderclient/tests/test_client.py
@@ -25,11 +25,11 @@ class ClientTest(utils.TestCase):
def test_get_client_class_v1(self):
output = cinderclient.client.get_client_class('1')
- self.assertEqual(output, cinderclient.v1.client.Client)
+ self.assertEqual(cinderclient.v1.client.Client, output)
def test_get_client_class_v2(self):
output = cinderclient.client.get_client_class('2')
- self.assertEqual(output, cinderclient.v2.client.Client)
+ self.assertEqual(cinderclient.v2.client.Client, output)
def test_get_client_class_unknown(self):
self.assertRaises(cinderclient.exceptions.UnsupportedVersion,