summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorhexin <hxhehx@cn.ibm.com>2016-05-18 15:43:38 +0800
committerhexin <hxhehx@cn.ibm.com>2016-05-19 17:14:07 +0800
commit87c30d90a70f6e017fdff937897a9db676c9b578 (patch)
tree9386c28051b897badc2a656c8612a1071cadd43f
parentb067575fb9591f1b04955943fb24faa6a13c8acc (diff)
downloadpython-ceilometerclient-87c30d90a70f6e017fdff937897a9db676c9b578.tar.gz
Correct the usage of the method assertEqual()
The first argument of the method assertEqual() should be the expected value. Change-Id: I8ea18641e7563e6d7b2779888759d761c9a843f8
-rw-r--r--ceilometerclient/tests/unit/v2/test_events.py20
1 files changed, 10 insertions, 10 deletions
diff --git a/ceilometerclient/tests/unit/v2/test_events.py b/ceilometerclient/tests/unit/v2/test_events.py
index 6cb1c1e..4a9270f 100644
--- a/ceilometerclient/tests/unit/v2/test_events.py
+++ b/ceilometerclient/tests/unit/v2/test_events.py
@@ -135,10 +135,10 @@ class EventManagerTest(utils.BaseTestCase):
'GET', '/v2/events'
]
self.http_client.assert_called(*expect)
- self.assertEqual(len(events), 3)
- self.assertEqual(events[0].event_type, 'Foo')
- self.assertEqual(events[1].event_type, 'Foo')
- self.assertEqual(events[2].event_type, 'Bar')
+ self.assertEqual(3, len(events))
+ self.assertEqual('Foo', events[0].event_type)
+ self.assertEqual('Foo', events[1].event_type)
+ self.assertEqual('Bar', events[2].event_type)
def test_list_one(self):
event = self.mgr.get(2)
@@ -147,7 +147,7 @@ class EventManagerTest(utils.BaseTestCase):
]
self.http_client.assert_called(*expect)
self.assertIsNotNone(event)
- self.assertEqual(event.event_type, 'Foo')
+ self.assertEqual('Foo', event.event_type)
def test_list_with_query(self):
events = list(self.mgr.list(q=[{"field": "hostname",
@@ -158,8 +158,8 @@ class EventManagerTest(utils.BaseTestCase):
'&q.value=localhost'
]
self.http_client.assert_called(*expect)
- self.assertEqual(len(events), 2)
- self.assertEqual(events[0].event_type, 'Foo')
+ self.assertEqual(2, len(events))
+ self.assertEqual('Foo', events[0].event_type)
def test_list_with_query_no_type(self):
events = list(self.mgr.list(q=[{"field": "hostname",
@@ -169,8 +169,8 @@ class EventManagerTest(utils.BaseTestCase):
'&q.type=&q.value=foreignhost'
]
self.http_client.assert_called(*expect)
- self.assertEqual(len(events), 2)
- self.assertEqual(events[0].event_type, 'Foo')
+ self.assertEqual(2, len(events))
+ self.assertEqual('Foo', events[0].event_type)
def test_list_with_multiple_filters(self):
events = list(self.mgr.list(q=[{"field": "hostname",
@@ -184,7 +184,7 @@ class EventManagerTest(utils.BaseTestCase):
'&q.type=&q.type=integer&q.value=localhost&q.value=5'
]
self.http_client.assert_called(*expect)
- self.assertEqual(len(events), 1)
+ self.assertEqual(1, len(events))
def test_get_from_event_class(self):
event = self.mgr.get(2)