Update global ip assign/unassign to use new API

Author: caberos

SoftLayer/CLI/globalip/assign.py

@@ -5,17 +5,17 @@
 
 import SoftLayer
 from SoftLayer.CLI import environment
-from SoftLayer.CLI import helpers
 
 
-@click.command()
+@click.command(epilog="More information about types and ")
 @click.argument('identifier')
-@click.argument('target')
+@click.option('--target',
+              help='See SLDN docs. '
+                   'E.g SoftLayer_Network_Subnet_IpAddress, SoftLayer_Hardware_Server,SoftLayer_Virtual_Guest')
+@click.option('--router', help='An appropriate identifier for the specified $type. Some types have multiple identifier')
 @environment.pass_env
-def cli(env, identifier, target):
+def cli(env, identifier, target, router):
     """Assigns the global IP to a target."""
 
     mgr = SoftLayer.NetworkManager(env.client)
-    global_ip_id = helpers.resolve_id(mgr.resolve_global_ip_ids, identifier,
-                                      name='global ip')
-    mgr.assign_global_ip(global_ip_id, target)
+    mgr.route(identifier, target, router)

SoftLayer/fixtures/SoftLayer_Network_Subnet.py

@@ -44,3 +44,4 @@
 editNote = True
 setTags = True
 cancel = True
+route = True

SoftLayer/managers/network.py

@@ -824,3 +824,13 @@ def get_closed_pods(self):
         mask = """mask[name, datacenterLongName, frontendRouterId, capabilities, datacenterId, backendRouterId,
                 backendRouterName, frontendRouterName]"""
         return self.client.call('SoftLayer_Network_Pod', 'getAllObjects', mask=mask, filter=closing_filter)
+
+    def route(self, subnet_id, type_serv, target):
+        """Assigns a global IP address to a specified target.
+
+        :param int subnet_id: The ID of the global IP being assigned
+        :param string type_serv: The type service to assign
+        :param string target: The instance to assign
+        """
+        return self.client.call('SoftLayer_Network_Subnet', 'route',
+                                type_serv, target, id=subnet_id, )

tests/CLI/modules/globalip_tests.py

@@ -15,7 +15,7 @@
 class DnsTests(testing.TestCase):
 
     def test_ip_assign(self):
-        result = self.run_command(['globalip', 'assign', '1', '127.0.0.1'])
+        result = self.run_command(['globalip', 'assign', '1'])
 
         self.assert_no_fail(result)
         self.assertEqual(result.output, "")

tests/managers/network_tests.py

@@ -628,3 +628,7 @@ def test_vlan_edit(self):
     def test_get_all_pods(self):
         self.network.get_pods()
         self.assert_called_with('SoftLayer_Network_Pod', 'getAllObjects')
+
+    def test_route(self):
+        self.network.route('SoftLayer_Hardware_Server', 123456, 100)
+        self.assert_called_with('SoftLayer_Network_Subnet', 'route')