CLOUDSTACK-7998: test_escalations_instances.py - Correcting code related to detach volume

Signed-off-by: SrikanteswaraRao Talluri <talluri@apache.org>

test/integration/component/test_escalations_instances.py

@@ -2850,17 +2850,33 @@ class TestInstances(cloudstackTestCase):
             len(list_volumes_page1),
             "VM's volume count is not matching in page 1"
         )
-        # stopping VM before detaching volumes
-        vm_created.stop(self.userapiclient)
 
-        # Detaching root volume is allowed on XenServer only
+        list_data_disks = Volume.list(
-        if self.hypervisor.lower() == 'xenserver':
+            self.userapiclient,
-            # Detaching all the volumes attached from VM
+            listall=self.services["listall"],
-            for i in range(0, len(list_volumes_page1)):
+            virtualmachineid=vm_created.id,
-                vm_created.detach_volume(
+            page=1,
-                    self.userapiclient,
+            pagesize=self.services["pagesize"],
-                    list_volumes_page1[i]
+            type="DATADISK"
-                )
+        )
+
+        for volume in list_data_disks:
+            vm_created.detach_volume(self.userapiclient, volume)
+
+        volumes = Volume.list(
+            self.userapiclient,
+            listall=self.services["listall"],
+            virtualmachineid=vm_created.id,
+            page=1,
+            pagesize=self.services["pagesize"]
+        )
+
+        self.assertEqual(
+                len(volumes),
+                len(list_volumes_page1) - len(list_data_disks),
+                "The volumes number should match with (volumes initially\
+                        present minus volumes detached")
+
         return
 
     @attr(tags=["advanced", "basic"], required_hardware="true")