From 77cbd63e50cccd9cecb9581c70381f3e64886a5f Mon Sep 17 00:00:00 2001 From: Anusha Ramineni Date: Fri, 10 Apr 2015 21:18:11 +0530 Subject: [PATCH] Missing argument in set_http_boot_url Change-Id: Ie0566a1a9af42e2eb776891858b44f0ef3179be5 --- proliantutils/ilo/client.py | 2 +- proliantutils/tests/ilo/test_client.py | 183 +++++++++++++++++++++++++ 2 files changed, 184 insertions(+), 1 deletion(-) diff --git a/proliantutils/ilo/client.py b/proliantutils/ilo/client.py index e4cb406..fd678bc 100644 --- a/proliantutils/ilo/client.py +++ b/proliantutils/ilo/client.py @@ -84,7 +84,7 @@ class IloClient(operations.IloOperations): :raises: IloCommandNotSupportedError, if the command is not supported on the server. """ - return self._call_method('set_http_boot_url') + return self._call_method('set_http_boot_url', url) def get_one_time_boot(self): """Retrieves the current setting for the one time boot.""" diff --git a/proliantutils/tests/ilo/test_client.py b/proliantutils/tests/ilo/test_client.py index 530e878..5e8291a 100644 --- a/proliantutils/tests/ilo/test_client.py +++ b/proliantutils/tests/ilo/test_client.py @@ -46,3 +46,186 @@ class IloClientTestCase(testtools.TestCase): self.client.model = 'Gen9' self.client._call_method('reset_ilo') ilo_mock.assert_called_once_with() + + @mock.patch.object(client.IloClient, '_call_method') + def test_set_http_boot_url(self, call_mock): + self.client.set_http_boot_url('fake-url') + call_mock.assert_called_once_with('set_http_boot_url', 'fake-url') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_product_name(self, call_mock): + self.client.get_product_name() + call_mock.assert_called_once_with('get_product_name') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_all_licenses(self, call_mock): + self.client.get_all_licenses() + call_mock.assert_called_once_with('get_all_licenses') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_host_power_status(self, call_mock): + self.client.get_host_power_status() + call_mock.assert_called_once_with('get_host_power_status') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_http_boot_url(self, call_mock): + self.client.get_http_boot_url() + call_mock.assert_called_once_with('get_http_boot_url') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_one_time_boot(self, call_mock): + self.client.get_one_time_boot() + call_mock.assert_called_once_with('get_one_time_boot') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_vm_status(self, call_mock): + self.client.get_vm_status('CDROM') + call_mock.assert_called_once_with('get_vm_status', 'CDROM') + + @mock.patch.object(client.IloClient, '_call_method') + def test_press_pwr_btn(self, call_mock): + self.client.press_pwr_btn() + call_mock.assert_called_once_with('press_pwr_btn') + + @mock.patch.object(client.IloClient, '_call_method') + def test_reset_server(self, call_mock): + self.client.reset_server() + call_mock.assert_called_once_with('reset_server') + + @mock.patch.object(client.IloClient, '_call_method') + def test_hold_pwr_btn(self, call_mock): + self.client.hold_pwr_btn() + call_mock.assert_called_once_with('hold_pwr_btn') + + @mock.patch.object(client.IloClient, '_call_method') + def test_set_host_power(self, call_mock): + self.client.set_host_power('ON') + call_mock.assert_called_once_with('set_host_power', 'ON') + + @mock.patch.object(client.IloClient, '_call_method') + def test_set_one_time_boot(self, call_mock): + self.client.set_one_time_boot('CDROM') + call_mock.assert_called_once_with('set_one_time_boot', 'CDROM') + + @mock.patch.object(client.IloClient, '_call_method') + def test_insert_virtual_media(self, call_mock): + self.client.insert_virtual_media(url='fake-url', device='FLOPPY') + call_mock.assert_called_once_with('insert_virtual_media', 'fake-url', + 'FLOPPY') + + @mock.patch.object(client.IloClient, '_call_method') + def test_eject_virtual_media(self, call_mock): + self.client.eject_virtual_media(device='FLOPPY') + call_mock.assert_called_once_with('eject_virtual_media', 'FLOPPY') + + @mock.patch.object(client.IloClient, '_call_method') + def test_set_vm_status(self, call_mock): + self.client.set_vm_status(device='FLOPPY', boot_option='BOOT_ONCE', + write_protect='YES') + call_mock.assert_called_once_with('set_vm_status', 'FLOPPY', + 'BOOT_ONCE', 'YES') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_current_boot_mode(self, call_mock): + self.client.get_current_boot_mode() + call_mock.assert_called_once_with('get_current_boot_mode') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_pending_boot_mode(self, call_mock): + self.client.get_pending_boot_mode() + call_mock.assert_called_once_with('get_pending_boot_mode') + + @mock.patch.object(client.IloClient, '_call_method') + def test_set_pending_boot_mode(self, call_mock): + self.client.set_pending_boot_mode('UEFI') + call_mock.assert_called_once_with('set_pending_boot_mode', 'UEFI') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_persistent_boot(self, call_mock): + self.client.get_persistent_boot() + call_mock.assert_called_once_with('get_persistent_boot') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_persistent_boot_device(self, call_mock): + self.client.get_persistent_boot_device() + call_mock.assert_called_once_with('get_persistent_boot_device') + + @mock.patch.object(client.IloClient, '_call_method') + def test_update_persistent_boot(self, call_mock): + self.client.update_persistent_boot(['HDD']) + call_mock.assert_called_once_with('update_persistent_boot', ['HDD']) + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_secure_boot_mode(self, call_mock): + self.client.get_secure_boot_mode() + call_mock.assert_called_once_with('get_secure_boot_mode') + + @mock.patch.object(client.IloClient, '_call_method') + def test_set_secure_boot_mode(self, call_mock): + self.client.set_secure_boot_mode(True) + call_mock.assert_called_once_with('set_secure_boot_mode', True) + + @mock.patch.object(client.IloClient, '_call_method') + def test_reset_secure_boot_keys(self, call_mock): + self.client.reset_secure_boot_keys() + call_mock.assert_called_once_with('reset_secure_boot_keys') + + @mock.patch.object(client.IloClient, '_call_method') + def test_clear_secure_boot_keys(self, call_mock): + self.client.clear_secure_boot_keys() + call_mock.assert_called_once_with('clear_secure_boot_keys') + + @mock.patch.object(client.IloClient, '_call_method') + def test_reset_ilo_credential(self, call_mock): + self.client.reset_ilo_credential('password') + call_mock.assert_called_once_with('reset_ilo_credential', 'password') + + @mock.patch.object(client.IloClient, '_call_method') + def test_reset_ilo(self, call_mock): + self.client.reset_ilo() + call_mock.assert_called_once_with('reset_ilo') + + @mock.patch.object(client.IloClient, '_call_method') + def test_reset_bios_to_default(self, call_mock): + self.client.reset_bios_to_default() + call_mock.assert_called_once_with('reset_bios_to_default') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_host_uuid(self, call_mock): + self.client.get_host_uuid() + call_mock.assert_called_once_with('get_host_uuid') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_host_health_data(self, call_mock): + self.client.get_host_health_data('fake-data') + call_mock.assert_called_once_with('get_host_health_data', 'fake-data') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_host_health_present_power_reading(self, call_mock): + self.client.get_host_health_present_power_reading('fake-data') + call_mock.assert_called_once_with( + 'get_host_health_present_power_reading', 'fake-data') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_host_health_power_supplies(self, call_mock): + self.client.get_host_health_power_supplies('fake-data') + call_mock.assert_called_once_with('get_host_health_power_supplies', + 'fake-data') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_host_health_fan_sensors(self, call_mock): + self.client.get_host_health_fan_sensors('fake-data') + call_mock.assert_called_once_with('get_host_health_fan_sensors', + 'fake-data') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_host_health_temperature_sensors(self, call_mock): + self.client.get_host_health_temperature_sensors('fake-data') + call_mock.assert_called_once_with( + 'get_host_health_temperature_sensors', 'fake-data') + + @mock.patch.object(client.IloClient, '_call_method') + def test_get_host_health_at_a_glance(self, call_mock): + self.client.get_host_health_at_a_glance('fake-data') + call_mock.assert_called_once_with('get_host_health_at_a_glance', + 'fake-data')