You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by sa...@apache.org on 2015/09/23 11:55:46 UTC

[2/4] git commit: updated refs/heads/master to a601ae4

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 <pa...@gmail.com>
Authored: Wed Aug 12 16:17:02 2015 +0530
Committer: pavan018 <pa...@gmail.com>
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"
                             )