Imparting changes mentioned by nitt10prashant
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/a4829f4d Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/a4829f4d Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/a4829f4d Branch: refs/heads/master Commit: a4829f4d444c78e721ad105f47a8be8b04734fb8 Parents: 7bcf848 Author: pavan018 <[email protected]> Authored: Wed Aug 12 16:17:02 2015 +0530 Committer: pavan018 <[email protected]> Committed: Thu Aug 13 11:39:06 2015 +0530 ---------------------------------------------------------------------- .../component/test_simultaneous_volume_attach.py | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/a4829f4d/test/integration/component/test_simultaneous_volume_attach.py ---------------------------------------------------------------------- diff --git a/test/integration/component/test_simultaneous_volume_attach.py b/test/integration/component/test_simultaneous_volume_attach.py index 0c7e308..12f9677 100644 --- a/test/integration/component/test_simultaneous_volume_attach.py +++ b/test/integration/component/test_simultaneous_volume_attach.py @@ -193,7 +193,7 @@ class TestMultipleVolumeAttach(cloudstackTestCase): str(e)) return FAILED - @attr(tags = ["advanced", "advancedns", "basic","bpk"], required_hardware="true") + @attr(tags = ["advanced", "advancedns", "basic"], required_hardware="true") def test_attach_multiple_volumes(self): """Attach multiple Volumes simultaneously to a Running VM """ @@ -237,6 +237,7 @@ class TestMultipleVolumeAttach(cloudstackTestCase): list_volume_response = Volume.list( self.apiClient, virtualmachineid=self.virtual_machine.id, + type="DATADISK", account=self.account.name, domainid=self.account.domainid ) @@ -246,17 +247,8 @@ class TestMultipleVolumeAttach(cloudstackTestCase): "Check list response returns a valid list" ) - # Extract only the data disks attached to the instance. - data_disks_list = [ disk for disk in list_volume_response if disk.type == 'DATADISK'] - - self.assertEqual( - validateList(data_disks_list)[0], - PASS, - "Check data disk check returns a valid list" - ) - self.assertEqual( - len(data_disks_list), + len(list_volume_response), 4, "All 4 data disks are not attached to VM Successfully" )
