virtual detail: improve coverage

Add test for verifying host_id attribute

Author: Flavio Fernandes

SoftLayer/fixtures/SoftLayer_Virtual_DedicatedHost.py

@@ -0,0 +1,10 @@
+getObject = {
+    'id': 37401,
+    'memoryCapacity': 242,
+    'modifyDate': '',
+    'name': 'test-dedicated',
+    'diskCapacity': 1200,
+    'createDate': '2017-10-16T12:50:23-05:00',
+    'cpuCount': 56,
+    'accountId': 1199911
+}

SoftLayer/fixtures/SoftLayer_Virtual_Guest.py

@@ -44,6 +44,7 @@
     'networkVlans': [{'networkSpace': 'PUBLIC',
                       'vlanNumber': 23,
                       'id': 1}],
+    'dedicatedHost': {'id': 37401},
     'operatingSystem': {
         'passwords': [{'username': 'user', 'password': 'pass'}],
         'softwareLicense': {

tests/CLI/modules/vs_tests.py

@@ -9,6 +9,7 @@
 import mock
 
 from SoftLayer.CLI import exceptions
+from SoftLayer import SoftLayerAPIError
 from SoftLayer import testing
 
 
@@ -42,6 +43,8 @@ def test_detail_vs(self):
                           'cores': 2,
                           'created': '2013-08-01 15:23:45',
                           'datacenter': 'TEST00',
+                          'dedicated_host': 'test-dedicated',
+                          'dedicated_host_id': 37401,
                           'hostname': 'vs-test1',
                           'domain': 'test.sftlyr.ws',
                           'fqdn': 'vs-test1.test.sftlyr.ws',
@@ -88,6 +91,24 @@ def test_detail_vs_empty_tag(self):
             ['example-tag'],
         )
 
+    def test_detail_vs_dedicated_host_not_found(self):
+        ex = SoftLayerAPIError('SoftLayer_Exception', 'Not found')
+        mock = self.set_mock('SoftLayer_Virtual_DedicatedHost', 'getObject')
+        mock.side_effect = ex
+        result = self.run_command(['vs', 'detail', '100'])
+        self.assert_no_fail(result)
+        self.assertEqual(json.loads(result.output)['dedicated_host_id'], 37401)
+        self.assertIsNone(json.loads(result.output)['dedicated_host'])
+
+    def test_detail_vs_no_dedicated_host_hostname(self):
+        mock = self.set_mock('SoftLayer_Virtual_DedicatedHost', 'getObject')
+        mock.return_value = {'this_is_a_fudged_Virtual_DedicatedHost': True,
+                             'name_is_not_provided': ''}
+        result = self.run_command(['vs', 'detail', '100'])
+        self.assert_no_fail(result)
+        self.assertEqual(json.loads(result.output)['dedicated_host_id'], 37401)
+        self.assertIsNone(json.loads(result.output)['dedicated_host'])
+
     def test_create_options(self):
         result = self.run_command(['vs', 'create-options'])