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 2013/07/22 07:12:25 UTC

git commit: updated refs/heads/4.2 to d79066d

Updated Branches:
  refs/heads/4.2 8c3f97043 -> d79066dfb


CLOUDSTACK-3636: Fix invalid condition to check for empty response for private service offerings


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

Branch: refs/heads/4.2
Commit: d79066dfb17a761a59642ae0779b4234e4cb1e63
Parents: 8c3f970
Author: sailajam <sa...@citrix.com>
Authored: Thu Jul 18 16:52:14 2013 +0530
Committer: sailajam <sa...@citrix.com>
Committed: Mon Jul 22 10:38:14 2013 +0530

----------------------------------------------------------------------
 test/integration/component/test_accounts.py | 16 ++--------------
 1 file changed, 2 insertions(+), 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/d79066df/test/integration/component/test_accounts.py
----------------------------------------------------------------------
diff --git a/test/integration/component/test_accounts.py b/test/integration/component/test_accounts.py
index 5ca0570..7d0aeb1 100644
--- a/test/integration/component/test_accounts.py
+++ b/test/integration/component/test_accounts.py
@@ -824,22 +824,10 @@ class TestServiceOfferingHierarchy(cloudstackTestCase):
                                                   domainid=self.domain_2.id
                                                   )
         self.assertEqual(
-                            isinstance(service_offerings, list),
-                            True,
+                            service_offerings,
+                            None,
                             "Check List Service Offerings for a valid response"
                         )
-        self.assertNotEqual(
-                            len(service_offerings),
-                            0,
-                            "Check List Service Offerings response"
-                            )
-
-        for service_offering in service_offerings:
-            self.assertEqual(
-               service_offering.id,
-               self.service_offering.id,
-               "Check Service offering ID for domain" + str(self.domain_2.name)
-            )
         return