You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by gi...@apache.org on 2014/01/08 07:34:32 UTC

git commit: updated refs/heads/4.3 to 5aa0bc3

Updated Branches:
  refs/heads/4.3 cb48c9780 -> 5aa0bc3f8


CLOUDSTACK-5627: Removed a validation which expected createVPCOffering to fail if SourceNAT not provided

createVPCOffering API creates VPC with SourceNat even if not provided as param. So this validation
is not required any more.


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

Branch: refs/heads/4.3
Commit: 5aa0bc3f8349f644bf4148f2dd41972dd8e03839
Parents: cb48c97
Author: Girish Shilamkar <gi...@clogeny.com>
Authored: Wed Jan 8 12:00:09 2014 +0530
Committer: Girish Shilamkar <gi...@clogeny.com>
Committed: Wed Jan 8 12:04:15 2014 +0530

----------------------------------------------------------------------
 test/integration/component/test_vpc_offerings.py | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/5aa0bc3f/test/integration/component/test_vpc_offerings.py
----------------------------------------------------------------------
diff --git a/test/integration/component/test_vpc_offerings.py b/test/integration/component/test_vpc_offerings.py
index 4f58022..699cef9 100644
--- a/test/integration/component/test_vpc_offerings.py
+++ b/test/integration/component/test_vpc_offerings.py
@@ -820,19 +820,9 @@ class TestVPCOffering(cloudstackTestCase):
         """Test VPC offering with invalid services"""
 
         # Validate the following
-        # 1. Creating VPC Offering with no SourceNat service should FAIL.
-        # 2. Creating VPC Offering with services NOT supported by VPC
+        # 1. Creating VPC Offering with services NOT supported by VPC
         #    like Firewall should not be allowed
 
-        self.debug("Creating a VPC offering without sourceNAT")
-        self.services["vpc_offering"]["supportedservices"] = 'Dhcp,Dns,PortForwarding,Vpn,Lb,UserData,StaticNat'
-
-        with self.assertRaises(Exception):
-            VpcOffering.create(
-                                self.apiclient,
-                                self.services["vpc_offering"]
-                             )
-
         self.debug("Creating a VPC offering with Firewall")
         self.services["vpc_offering"]["supportedservices"] = 'Dhcp,Dns,PortForwarding,Firewall,Vpn,SourceNat,Lb,UserData,StaticNat'