You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ts...@apache.org on 2013/08/20 13:59:04 UTC

[2/2] git commit: updated refs/heads/4.2 to a82f66e

CLOUDSTACK-4257: [Automation] test_storage_motion test cases failed during with unexpected result in listStoragePool API call

Signed-off-by: Prasanna Santhanam <ts...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/a9e45bf7
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/a9e45bf7
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/a9e45bf7

Branch: refs/heads/4.2
Commit: a9e45bf7766728c8439fcc0e3a1a6e3af074b2b7
Parents: 319a7bf
Author: Saksham Srivastava <sa...@citrix.com>
Authored: Mon Aug 19 16:28:00 2013 +0530
Committer: Prasanna Santhanam <ts...@apache.org>
Committed: Tue Aug 20 17:24:56 2013 +0530

----------------------------------------------------------------------
 test/integration/component/test_storage_motion.py | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/a9e45bf7/test/integration/component/test_storage_motion.py
----------------------------------------------------------------------
diff --git a/test/integration/component/test_storage_motion.py b/test/integration/component/test_storage_motion.py
index eda77d1..8ddbebe 100644
--- a/test/integration/component/test_storage_motion.py
+++ b/test/integration/component/test_storage_motion.py
@@ -259,16 +259,8 @@ class TestStorageMotion(cloudstackTestCase):
                               self.apiclient,
                               id=volume.id
                               )
-            self.assertEqual(
-                         isinstance(pools, list),
-                         True,
-                         "Check list storage pools response for valid list"
-                        )
-            self.assertNotEqual(
-                        pools,
-                        None,
-                        "Check if pools  exists in ListStoragePools"
-                        )
+            if not pools or pools is None:
+                self.skipTest("No suitable storage pools found for volume migration.Skipping")
 
             pool = pools[0]
             self.debug("Migrating Volume-ID: %s to Pool: %s" % (