diff --git a/tests/unit/queues/v1/test_core.py b/tests/unit/queues/v1/test_core.py index b7cfb852..0abe2567 100644 --- a/tests/unit/queues/v1/test_core.py +++ b/tests/unit/queues/v1/test_core.py @@ -96,7 +96,7 @@ class TestV1Core(base.TestBase): req = request.Request() result = core.queue_get_stats(self.transport, req, 'test') - self.assertEqual(result, {}) + self.assertEqual({}, result) def test_message_post_one(self): messages = {'ttl': 30, 'body': 'Post one!'} @@ -110,8 +110,7 @@ class TestV1Core(base.TestBase): core.message_post(self.transport, req, 'test', messages) self.assertIn('queue_name', req.params) - self.assertEqual(json.loads(req.content), - messages) + self.assertEqual(messages, json.loads(req.content)) def test_message_post_many(self): messages = [{'ttl': 30, 'body': 'Post one!'}, @@ -127,8 +126,7 @@ class TestV1Core(base.TestBase): core.message_post(self.transport, req, 'test', messages) self.assertIn('queue_name', req.params) - self.assertEqual(json.loads(req.content), - messages) + self.assertEqual(messages, json.loads(req.content)) def test_message_list(self): with mock.patch.object(self.transport, 'send', @@ -264,4 +262,4 @@ class TestV1_1Core(TestV1Core): self.assertIn('queue_name', req.params) self.assertIn('pop', req.params) - self.assertEqual(req.params['pop'], 5) + self.assertEqual(5, req.params['pop']) diff --git a/tests/unit/queues/v1/test_message.py b/tests/unit/queues/v1/test_message.py index 1d7a0b45..5a9bbc34 100644 --- a/tests/unit/queues/v1/test_message.py +++ b/tests/unit/queues/v1/test_message.py @@ -42,7 +42,7 @@ class TestMessageIterator(base.QueuesTestBase): 'messages', message.create_object(self.queue)) iterated = [msg for msg in iterator] - self.assertEqual(len(iterated), 1) + self.assertEqual(1, len(iterated)) def test_stream(self): messages = {'links': [], @@ -73,7 +73,7 @@ class TestMessageIterator(base.QueuesTestBase): 'messages', message.create_object(self.queue)) iterated = [msg for msg in iterator.stream()] - self.assertEqual(len(iterated), 2) + self.assertEqual(2, len(iterated)) def test_iterator_respect_paging(self): messages = {'links': [], @@ -101,7 +101,7 @@ class TestMessageIterator(base.QueuesTestBase): 'messages', message.create_object(self.queue)) iterated = [msg for msg in iterator] - self.assertEqual(len(iterated), 1) + self.assertEqual(1, len(iterated)) class QueuesV1MessageHttpUnitTest(test_message.QueuesV1MessageUnitTest):