diff options
author | Shivanand Tendulker <stendulker@gmail.com> | 2019-07-16 05:41:36 -0400 |
---|---|---|
committer | Shivanand Tendulker <stendulker@gmail.com> | 2019-07-17 01:10:30 -0400 |
commit | 0d6ee78f5bb07f84d42318ce657161eb02e36297 (patch) | |
tree | 781337025deaaf1069319652b6a1abf88a77f037 /ironic/tests/unit/drivers/modules/redfish/test_management.py | |
parent | 691d3e4992a2c6dd8d71aadbacf47d19cddbca87 (diff) | |
download | ironic-0d6ee78f5bb07f84d42318ce657161eb02e36297.tar.gz |
Failure in get_sensor_data() of 'redfish' management interface
This patch fixes following issues in get_sensor_data():
- Uses 'identity' field instead of unsupported field 'member_id' in
Sushy's PowerSupplyListField
- get_members() is not supported for Sushy's list fields like
FansListField and TemperaturesListField
Change-Id: I534d78fa9c9fc2573c7b6e15c3356da234f063ec
Story: 2006229
Task: 35830
Diffstat (limited to 'ironic/tests/unit/drivers/modules/redfish/test_management.py')
-rw-r--r-- | ironic/tests/unit/drivers/modules/redfish/test_management.py | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/ironic/tests/unit/drivers/modules/redfish/test_management.py b/ironic/tests/unit/drivers/modules/redfish/test_management.py index 0807afa57..1a057ec28 100644 --- a/ironic/tests/unit/drivers/modules/redfish/test_management.py +++ b/ironic/tests/unit/drivers/modules/redfish/test_management.py @@ -234,11 +234,10 @@ class RedfishManagementTestCase(db_base.DbTestCase): mock_chassis = mock.MagicMock(identity='ZZZ-YYY-XXX') - mock_fans = mock_chassis.thermal.fans mock_fan = mock.MagicMock(**attributes) mock_fan.name = attributes['name'] mock_fan.status = mock.MagicMock(**attributes['status']) - mock_fans.get_members.return_value = [mock_fan] + mock_chassis.thermal.fans = [mock_fan] with task_manager.acquire(self.context, self.node.uuid, shared=True) as task: @@ -279,11 +278,11 @@ class RedfishManagementTestCase(db_base.DbTestCase): } mock_chassis = mock.MagicMock(identity='ZZZ-YYY-XXX') - mock_temperatures = mock_chassis.thermal.temperatures + mock_temperature = mock.MagicMock(**attributes) mock_temperature.name = attributes['name'] mock_temperature.status = mock.MagicMock(**attributes['status']) - mock_temperatures.get_members.return_value = [mock_temperature] + mock_chassis.thermal.temperatures = [mock_temperature] with task_manager.acquire(self.context, self.node.uuid, shared=True) as task: @@ -307,7 +306,7 @@ class RedfishManagementTestCase(db_base.DbTestCase): def test__get_sensors_power(self): attributes = { - 'member_id': 0, + 'identity': 0, 'name': 'Power Supply 0', 'power_capacity_watts': 1450, 'last_power_output_watts': 650, |