fix observations

Author: Brian Flores

SoftLayer/CLI/virt/host_list.py

@@ -11,10 +11,10 @@
 
 
 @click.command(cls=SLCommand, short_help="List virtual servers.")
-@click.option('--name', '-n', help='Filter by name of the dedicated host')
 @click.option('--datacenter', '-d', help='Filter by datacenter of the dedicated host')
-@click.option('--owner', help='Filter by owner of the dedicated host')
+@click.option('--name', '-n', help='Filter by name of the dedicated host')
 @click.option('--order', help='Filter by ID of the order which purchased this dedicated host', type=click.INT)
+@click.option('--owner', help='Filter by owner of the dedicated host')
 @environment.pass_env
 def cli(env, name, datacenter, owner, order,):
     """List dedicated hosts."""

SoftLayer/managers/account.py

@@ -367,7 +367,7 @@ def getBandwidthDetail(self, identifier):
         bareMetalInstances[outboundBandwidthUsage,bandwidthAllotmentDetail[allocation]]"""
         return self.client['SoftLayer_Network_Bandwidth_Version1_Allotment'].getObject(id=identifier, mask=_mask)
 
-    def get_dedicated_hosts(self, object_mask, object_filter):
+    def get_dedicated_hosts(self, object_mask="", object_filter=""):
         """Returns all associated virtual dedicated host objects.
 
         :return SoftLayer_Virtual_DedicatedHost[].

tests/managers/account_tests.py

@@ -178,5 +178,5 @@ def test_get_bandwidth_pool_counts(self):
         self.assertEqual(total, 2)
 
     def test_get_dedicated_hosts(self):
-        self.manager.get_dedicated_hosts("", "")
+        self.manager.get_dedicated_hosts()
         self.assert_called_with('SoftLayer_Account', 'getDedicatedHosts')