From 393d437b2620b6fe49a501db010b87316d689e32 Mon Sep 17 00:00:00 2001 From: Girish Shilamkar Date: Wed, 8 Jan 2014 12:00:09 +0530 Subject: [PATCH] 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. --- test/integration/component/test_vpc_offerings.py | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/test/integration/component/test_vpc_offerings.py b/test/integration/component/test_vpc_offerings.py index 4f5802285f2..699cef9d8fa 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'