ListImagesTest.test_index_no_params is checking by the list length, not the images identity
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
tempest |
Fix Released
|
Medium
|
Jaroslav Henner |
Bug Description
I'm running tests in parallel. I met an ERROR that appears it happened because some parallel test added some image. Tests should be comparing not by collection sizes, but by ids to avoid such race conditions.
16:09:12 #4 Simple test to see all fixture images returned ... FAIL
16:09:16
16:09:16 =======
16:09:16 FAIL: Simple test to see all fixture images returned
16:09:16 -------
16:09:16 Traceback (most recent call last):
16:09:16 File "tempest/
16:09:16 self = <tempest.
16:09:16 198 """
16:09:16 199 Simple test to see all fixture images returned
16:09:16 200 """
16:09:16 201 images = list(self.
16:09:16 --> 202 self.assertEqua
16:09:16 images = [
...
16:09:16 <Image {u'status': u'active', u'name': u'f16-x86_
16:09:16 ]
16:09:16 AssertionError: 10 != 11
16:09:16 '10 != 11' = '%s != %s' % (safe_repr(10), safe_repr(11))
16:09:16 '10 != 11' = self._formatMes
16:09:16 >> raise self.failureExc
Changed in tempest: | |
importance: | Undecided → Low |
importance: | Low → Medium |
Fix proposed to branch: master /review. openstack. org/18035
Review: https:/