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 <tsp@apache.org>
This commit is contained in:
Prasanna Santhanam 2013-07-18 14:59:35 +05:30
parent 52fa8532bb
commit 8bc3757e80
1 changed files with 6 additions and 8 deletions

View File

@ -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)