Ensures that floating IP created in test is destroyed in a finally: block
Fix for the Bug 991842
Change-Id: I11c57a9c2219d783ec95b993dec16a9a707da332
diff --git a/tempest/tests/test_list_floating_ips.py b/tempest/tests/test_list_floating_ips.py
index d100c01..03ed089 100644
--- a/tempest/tests/test_list_floating_ips.py
+++ b/tempest/tests/test_list_floating_ips.py
@@ -39,23 +39,25 @@
def test_get_floating_ip_details(self):
"""Positive test:Should be able to GET the details of floatingIP"""
#Creating a floating IP for which details are to be checked
- resp, body = self.client.create_floating_ip()
- floating_ip_instance_id = body['instance_id']
- floating_ip_ip = body['ip']
- floating_ip_fixed_ip = body['fixed_ip']
- floating_ip_id = body['id']
- resp, body = \
- self.client.get_floating_ip_details(floating_ip_id)
- self.assertEqual(200, resp.status)
- #Comparing the details of floating IP
- self.assertEqual(floating_ip_instance_id,
+ try:
+ resp, body = self.client.create_floating_ip()
+ floating_ip_instance_id = body['instance_id']
+ floating_ip_ip = body['ip']
+ floating_ip_fixed_ip = body['fixed_ip']
+ floating_ip_id = body['id']
+ resp, body = \
+ self.client.get_floating_ip_details(floating_ip_id)
+ self.assertEqual(200, resp.status)
+ #Comparing the details of floating IP
+ self.assertEqual(floating_ip_instance_id,
body['instance_id'])
- self.assertEqual(floating_ip_ip, body['ip'])
- self.assertEqual(floating_ip_fixed_ip,
+ self.assertEqual(floating_ip_ip, body['ip'])
+ self.assertEqual(floating_ip_fixed_ip,
body['fixed_ip'])
- self.assertEqual(floating_ip_id, body['id'])
+ self.assertEqual(floating_ip_id, body['id'])
#Deleting the floating IP created in this method
- self.client.delete_floating_ip(floating_ip_id)
+ finally:
+ self.client.delete_floating_ip(floating_ip_id)
@attr(type='negative')
@unittest.skip('Skipping until Nova Bug 940500 is fixed')