openstack/zaqar
Code Issues Proposed changes

Remove race condition in test_claim_lifecycle

Browse Source 
THe age returned by claims can differ between 2 asserts, so let's make
sure we don't uselessly check it.

Change-Id: Ide47526290bce36d939bd8a26e3bc320dc32c9fa
Closes-Bug: #1477564
This commit is contained in:
Thomas Herve 2015-09-04 16:07:31 +02:00
parent 48bc1aa0a5
commit 8e8adb16c4
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
zaqar/tests/unit/storage/base.py
View File

@ -790,7 +790,11 @@ class ClaimControllerTest(ControllerBaseTest):
messages2 = list(messages2)
self.assertEqual(len(messages2), 15)
self.assertEqual(messages, messages2)
for msg1, msg2 in zip(messages, messages2):
self.assertEqual(msg1['body'], msg2['body'])
self.assertEqual(msg1['claim_id'], msg2['claim_id'])
self.assertEqual(msg1['id'], msg2['id'])
self.assertEqual(msg1['ttl'], msg2['ttl'])
self.assertEqual(claim['ttl'], 70)
self.assertEqual(claim['id'], claim_id)