]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
test_l3: Don't assume the order of subnets
authorYAMAMOTO Takashi <yamamoto@midokura.com>
Wed, 3 Jun 2015 13:35:58 +0000 (22:35 +0900)
committerYAMAMOTO Takashi <yamamoto@midokura.com>
Fri, 5 Jun 2015 02:09:09 +0000 (11:09 +0900)
Fixes test_router_add_gateway_multiple_subnets_ipv6 failures
on my environment.

Change-Id: I44b4dd0cee393ea796f35b999cc111b60ef63db1

neutron/tests/unit/extensions/test_l3.py

index 2392adc03bbfe7ef0451a159a28bdeeb3d0ee1f5..51dee1cad5a3c4a8f6d8e4bc8fcbf05f466f9b17 100644 (file)
@@ -1432,12 +1432,20 @@ class L3NatTestCaseBase(L3NatTestCaseMixin):
                 res = self._show('routers', r['router']['id'])
                 fips = (res['router']['external_gateway_info']
                         ['external_fixed_ips'])
-                fip_subnet_ids = [fip['subnet_id'] for fip in fips]
-                self.assertIn(s1['subnet']['id'], fip_subnet_ids)
-                self.assertNotIn(s2['subnet']['id'], fip_subnet_ids)
-                self.assertIn(s3['subnet']['id'], fip_subnet_ids)
-                self.assertIn(s4['subnet']['id'], fip_subnet_ids)
-                self.assertIn(s5['subnet']['id'], fip_subnet_ids)
+                fip_subnet_ids = {fip['subnet_id'] for fip in fips}
+                # one of s1 or s2 should be in the list.
+                if s1['subnet']['id'] in fip_subnet_ids:
+                    self.assertEqual({s1['subnet']['id'],
+                                      s3['subnet']['id'],
+                                      s4['subnet']['id'],
+                                      s5['subnet']['id']},
+                                     fip_subnet_ids)
+                else:
+                    self.assertEqual({s2['subnet']['id'],
+                                      s3['subnet']['id'],
+                                      s4['subnet']['id'],
+                                      s5['subnet']['id']},
+                                     fip_subnet_ids)
                 self._remove_external_gateway_from_router(
                     r['router']['id'],
                     n['network']['id'])