From 5094c0e97264b801523d30125ef344a5c3d328aa Mon Sep 17 00:00:00 2001
From: spisarski <s.pisarski@cablelabs.com>
Date: Thu, 6 Jul 2017 15:03:35 -0600
Subject: Completed tests for creating networks of different types.

Change-Id: I33601da3f51d7acc60bd646e1d783a07afd8ea45
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
---
 snaps/openstack/tests/create_network_tests.py | 81 ++++++++++++---------------
 1 file changed, 37 insertions(+), 44 deletions(-)

diff --git a/snaps/openstack/tests/create_network_tests.py b/snaps/openstack/tests/create_network_tests.py
index bf6c629..c66cd48 100644
--- a/snaps/openstack/tests/create_network_tests.py
+++ b/snaps/openstack/tests/create_network_tests.py
@@ -502,26 +502,24 @@ class CreateNetworkTypeTests(OSComponentTestCase):
                     False)
             self.net_creator.clean()
 
-    # TODO - determine why this is not working on Newton
-    #        - Unable to create the network. No tenant network is available for allocation.
-    # def test_create_network_type_vlan(self):
-    #     """
-    #     Tests the creation of an OpenStack network of type vlan.
-    #     """
-    #     # Create Network
-    #     network_type = 'vlan'
-    #     net_settings = NetworkSettings(name=self.net_config.network_settings.name,
-    #                                    subnet_settings=self.net_config.network_settings.subnet_settings,
-    #                                    network_type=network_type)
-    #
-    #     # When setting the network_type, creds must be admin
-    #     self.net_creator = OpenStackNetwork(self.os_creds, net_settings)
-    #     network = self.net_creator.create()
-    #
-    #     # Validate network was created
-    #     neutron_utils_tests.validate_network(self.neutron, net_settings.name, True)
-    #
-    #     self.assertEquals(network_type, network['network']['provider:network_type'])
+    def test_create_network_type_vlan(self):
+        """
+        Tests the creation of an OpenStack network of type vlan.
+        """
+        # Create Network
+        network_type = 'vlan'
+        net_settings = NetworkSettings(name=self.net_config.network_settings.name,
+                                       subnet_settings=self.net_config.network_settings.subnet_settings,
+                                       network_type=network_type)
+
+        # When setting the network_type, creds must be admin
+        self.net_creator = OpenStackNetwork(self.os_creds, net_settings)
+        network = self.net_creator.create()
+
+        # Validate network was created
+        neutron_utils_tests.validate_network(self.neutron, net_settings.name, True)
+
+        self.assertEquals(network_type, network['network']['provider:network_type'])
 
     def test_create_network_type_vxlan(self):
         """
@@ -545,30 +543,25 @@ class CreateNetworkTypeTests(OSComponentTestCase):
         self.assertEqual(network_type,
                          network['network']['provider:network_type'])
 
-    # TODO - determine what value we need to place into physical_network
-    #        - Do not know what vaule to place into the 'physical_network' setting.
-    # def test_create_network_type_flat(self):
-    #     """
-    #     Tests the creation of an OpenStack network of type flat.
-    #     """
-    #     # Create Network
-    #     network_type = 'flat'
-    #
-    #     # Unable to find documentation on how to find a value that will work here.
-    #     # https://visibilityspots.org/vlan-flat-neutron-provider.html
-    #     # https://community.rackspace.com/products/f/45/t/4225
-    #     # It appears that this may be due to how OPNFV is configuring OpenStack.
-    #     physical_network = '???'
-    #     net_settings = NetworkSettings(name=self.net_config.network_settings.name,
-    #                                    subnet_settings=self.net_config.network_settings.subnet_settings,
-    #                                    network_type=network_type, physical_network=physical_network)
-    #     self.net_creator = OpenStackNetwork(self.os_creds, net_settings)
-    #     network = self.net_creator.create()
-    #
-    #     # Validate network was created
-    #     neutron_utils_tests.validate_network(self.neutron, net_settings.name, True)
-    #
-    #     self.assertEquals(network_type, network['network']['provider:network_type'])
+    def test_create_network_type_flat(self):
+        """
+        Tests the creation of an OpenStack network of type flat.
+        """
+        # Create Network
+        network_type = 'flat'
+
+        # TODO - this value must be variable to work on all OpenStack pods
+        physical_network = 'datacentre'
+        net_settings = NetworkSettings(name=self.net_config.network_settings.name,
+                                       subnet_settings=self.net_config.network_settings.subnet_settings,
+                                       network_type=network_type, physical_network=physical_network)
+        self.net_creator = OpenStackNetwork(self.os_creds, net_settings)
+        network = self.net_creator.create()
+
+        # Validate network was created
+        neutron_utils_tests.validate_network(self.neutron, net_settings.name, True)
+
+        self.assertEquals(network_type, network['network']['provider:network_type'])
 
     def test_create_network_type_foo(self):
         """
-- 
cgit 1.2.3-korg