Merge pull request #516 from zancas/bugfix.tunnels_is_a_collection_515

TunnelS --> back to --> Tunnels_s

Author: pjbreaux

f5/bigip/tm/net/__init__.py

@@ -35,7 +35,7 @@
 from f5.bigip.tm.net.route import Routes
 from f5.bigip.tm.net.route_domain import Route_Domains
 from f5.bigip.tm.net.selfip import Selfips
-from f5.bigip.tm.net.tunnels import TunnelS
+from f5.bigip.tm.net.tunnels import Tunnels_s
 from f5.bigip.tm.net.vlan import Vlans
 
 
@@ -50,6 +50,6 @@ def __init__(self, tm):
             Routes,
             Route_Domains,
             Selfips,
-            TunnelS,
+            Tunnels_s,
             Vlans
         ]

f5/bigip/tm/net/tunnels.py

@@ -28,14 +28,13 @@
 """
 
 from f5.bigip.resource import Collection
-from f5.bigip.resource import OrganizingCollection
 from f5.bigip.resource import Resource
 
 
-class TunnelS(OrganizingCollection):
+class Tunnels_s(Collection):
     """BIG-IP® network tunnels collection"""
     def __init__(self, net):
-        super(TunnelS, self).__init__(net)
+        super(Tunnels_s, self).__init__(net)
         self._meta_data['allowed_lazy_attributes'] = [
             Gres,
             Tunnels,
@@ -45,8 +44,8 @@ def __init__(self, net):
 
 class Tunnels(Collection):
     """BIG-IP® network tunnels resource (collection for GRE, Tunnel, VXLANs"""
-    def __init__(self, tunnels):
-        super(Tunnels, self).__init__(tunnels)
+    def __init__(self, tunnels_s):
+        super(Tunnels, self).__init__(tunnels_s)
         self._meta_data['allowed_lazy_attributes'] = [Gres, Tunnel, Vxlans]
         self._meta_data['attribute_registry'] =\
             {'tm:net:tunnels:tunnel:tunnelstate': Tunnel}

test/functional/tm/net/test_tunnels.py

@@ -32,7 +32,7 @@ def teardown():
         delete_resource(tunnel)
     request.addfinalizer(teardown)
 
-    tunnel = bigip.net.tunnels.tunnels.tunnel.create(
+    tunnel = bigip.net.tunnels_s.tunnels.tunnel.create(
         name=name, partition=partition, localAddress=ip, profile=profile)
     return tunnel
 
@@ -42,7 +42,7 @@ def teardown():
         delete_resource(gre)
     request.addfinalizer(teardown)
 
-    gre = bigip.net.tunnels.gres.gre.create(
+    gre = bigip.net.tunnels_s.gres.gre.create(
         name=name, partition=partition)
     return gre
 
@@ -52,14 +52,14 @@ def teardown():
         delete_resource(vxlan)
     request.addfinalizer(teardown)
 
-    vxlan = bigip.net.tunnels.vxlans.vxlan.create(
+    vxlan = bigip.net.tunnels_s.vxlans.vxlan.create(
         name=name, partition=partition)
     return vxlan
 
 
 class TestTunnels(object):
     def test_tunnel_list(self, bigip):
-        tunnels = bigip.net.tunnels.tunnels.get_collection()
+        tunnels = bigip.net.tunnels_s.tunnels.get_collection()
         assert len(tunnels)
         for tunnel in tunnels:
             assert tunnel.generation
@@ -74,7 +74,7 @@ def test_tunnel_CURDL(self, request, bigip):
         )
 
         # Load
-        t2 = bigip.net.tunnels.tunnels.tunnel.load(
+        t2 = bigip.net.tunnels_s.tunnels.tunnel.load(
             name='tunnel-test', partition='Common')
         assert t1.name == 'tunnel-test'
         assert t1.name == t2.name
@@ -99,7 +99,7 @@ def test_gre_CURDL(self, request, bigip):
         assert g1.name == 'gre-test'
 
         # Load
-        g2 = bigip.net.tunnels.gres.gre.load(
+        g2 = bigip.net.tunnels_s.gres.gre.load(
             name='gre-test', partition='Common')
         assert g1.name == g2.name
         assert g1.generation == g2.generation
@@ -123,7 +123,7 @@ def test_vxlan_CURDL(self, request, bigip):
         assert vx1.name == 'vxlan-test'
 
         # Load
-        vx2 = bigip.net.tunnels.vxlans.vxlan.load(
+        vx2 = bigip.net.tunnels_s.vxlans.vxlan.load(
             name='vxlan-test', partition='Common')
         assert vx1.name == vx2.name
         assert vx1.generation == vx2.generation