fix the team code review comments

Author: caberos

SoftLayer/CLI/hardware/create_options.py

@@ -21,7 +21,7 @@ def cli(env, prices, location=None):
     hardware_manager = hardware.HardwareManager(env.client)
     account_manager = account.AccountManager(env.client)
     options = hardware_manager.get_create_options(location)
-    routers = account_manager.get_routers(location)
+    routers = account_manager.get_routers(location=location)
     tables = []
 
     # Datacenters

SoftLayer/managers/account.py

@@ -277,7 +277,7 @@ def get_account_all_billing_orders(self, limit=100, mask=None):
         return self.client.call('Billing_Order', 'getAllObjects',
                                 limit=limit, mask=mask)
 
-    def get_routers(self, location=None, mask=None):
+    def get_routers(self, mask=None, location=None):
         """Gets all the routers currently active on the account
 
         :param string mask: Object Mask

tests/managers/account_tests.py

@@ -163,6 +163,6 @@ def test_get_active_virtual_licenses(self):
         self.assert_called_with("SoftLayer_Account", "getActiveVirtualLicenses")
 
     def test_get_routers_with_datacenter(self):
-        self.manager.get_routers('dal13')
+        self.manager.get_routers(location='dal13')
         object_filter = {'routers': {'topLevelLocation': {'name': {'operation': 'dal13'}}}}
         self.assert_called_with("SoftLayer_Account", "getRouters", filter=object_filter)