From 8bc3757e80d20575d1eeee530309ab61c02857d8 Mon Sep 17 00:00:00 2001 From: Prasanna Santhanam Date: Thu, 18 Jul 2013 14:59:35 +0530 Subject: [PATCH] CLOUDSTACK-3619: Cleanup of volume is not required as gc takes care. We don't need to delete the volume resource as part of cleanup as the storage gc is expected to cleanup orphaned volumes in deleted accounts/projects. Signed-off-by: Prasanna Santhanam --- test/integration/component/test_projects.py | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/test/integration/component/test_projects.py b/test/integration/component/test_projects.py index 68964e8ee48..510542a15b8 100644 --- a/test/integration/component/test_projects.py +++ b/test/integration/component/test_projects.py @@ -1203,10 +1203,9 @@ class TestProjectResources(cloudstackTestCase): raise Exception("Warning: Exception during cleanup : %s" % e) return - @attr(tags = ["advanced", "basic", "sg", "eip", "advancedns", "simulator"]) + @attr(tags = ["advanced", "basic", "sg", "eip", "advancedns", "simulator", "needle"]) def test_07_project_resources_account_delete(self): - """ Test Verify after an account is removed from the project, all his - resources stay with the project. + """ Test Verify after an account is removed from the project, all its resources stay with the project. """ # Validate the following # 1. Create a project. @@ -1261,13 +1260,13 @@ class TestProjectResources(cloudstackTestCase): ) # listProjectAccount to verify the user is added to project or not - accounts_reponse = Project.listAccounts( + accounts_response = Project.listAccounts( self.apiclient, projectid=project.id, account=self.user.name, ) self.assertEqual( - isinstance(accounts_reponse, list), + isinstance(accounts_response, list), True, "Check for a valid list accounts response" ) @@ -1277,7 +1276,7 @@ class TestProjectResources(cloudstackTestCase): 0, "Check list project response returns a valid project" ) - account = accounts_reponse[0] + account = accounts_response[0] self.assertEqual( account.role, @@ -1292,9 +1291,8 @@ class TestProjectResources(cloudstackTestCase): diskofferingid=self.disk_offering.id, projectid=project.id ) - self.cleanup.append(volume) - # Delete the project user + # Delete the project user ie the account self.user.delete(self.apiclient) volumes = Volume.list(self.apiclient, id=volume.id)