From 49005b324f18e19166bcbb801bcd07609e44d8db Mon Sep 17 00:00:00 2001 From: Gaurav Aradhye Date: Thu, 24 Jul 2014 15:21:28 +0530 Subject: [PATCH] CLOUDSTACK-7125: Fixed test_blocker_bugs.py, added code to wait for snapshots to be in 'BackedU' state --- .../component/test_blocker_bugs.py | 45 +++---------------- 1 file changed, 5 insertions(+), 40 deletions(-) diff --git a/test/integration/component/test_blocker_bugs.py b/test/integration/component/test_blocker_bugs.py index ba15b186c1b..f0b03c09cb0 100644 --- a/test/integration/component/test_blocker_bugs.py +++ b/test/integration/component/test_blocker_bugs.py @@ -863,26 +863,9 @@ class TestTemplates(cloudstackTestCase): account=self.account.name, domainid=self.account.domainid ) - self.debug("Created snapshot with ID: %s" % snapshot.id) - snapshots = Snapshot.list( - self.apiclient, - id=snapshot.id - ) - self.assertEqual( - isinstance(snapshots, list), - True, - "Check list response returns a valid list" - ) - self.assertNotEqual( - snapshots, - None, - "Check if result exists in list snapshots call" - ) - self.assertEqual( - snapshots[0].id, - snapshot.id, - "Check snapshot id in list resources call" - ) + + response = snapshot.validateState(self.apiclient, Snapshot.BACKED_UP) + self.assertEqual(response[0], PASS, response[1]) # Generate template from the snapshot template = Template.create_from_snapshot( @@ -945,26 +928,8 @@ class TestTemplates(cloudstackTestCase): account=self.account.name, domainid=self.account.domainid ) - self.debug("Created snapshot with ID: %s" % snapshot.id) - snapshots = Snapshot.list( - self.apiclient, - id=snapshot.id - ) - self.assertEqual( - isinstance(snapshots, list), - True, - "Check list response returns a valid list" - ) - self.assertNotEqual( - snapshots, - None, - "Check if result exists in list snapshots call" - ) - self.assertEqual( - snapshots[0].id, - snapshot.id, - "Check snapshot id in list resources call" - ) + response = snapshot.validateState(self.apiclient, Snapshot.BACKED_UP) + self.assertEqual(response[0], PASS, response[1]) # Generate template from the snapshot template = Template.create_from_snapshot(