From a4829f4d444c78e721ad105f47a8be8b04734fb8 Mon Sep 17 00:00:00 2001 From: pavan018 Date: Wed, 12 Aug 2015 16:17:02 +0530 Subject: [PATCH] Imparting changes mentioned by nitt10prashant --- .../component/test_simultaneous_volume_attach.py | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/test/integration/component/test_simultaneous_volume_attach.py b/test/integration/component/test_simultaneous_volume_attach.py index 0c7e308474e..12f9677cef4 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" )