diff --git a/tests/common/helpers/platform_api/fan_drawer_fan.py b/tests/common/helpers/platform_api/fan_drawer_fan.py index c897835e220..baebec68b30 100644 --- a/tests/common/helpers/platform_api/fan_drawer_fan.py +++ b/tests/common/helpers/platform_api/fan_drawer_fan.py @@ -77,5 +77,5 @@ def set_status_led(conn, fan_drawer_idx, fan_idx, color): return fan_drawer_fan_api(conn, fan_drawer_idx, fan_idx, 'set_status_led', [color]) -def get_status_led(conn, fan_idx): +def get_status_led(conn, fan_drawer_idx, fan_idx): return fan_drawer_fan_api(conn, fan_drawer_idx, fan_idx, 'get_status_led') diff --git a/tests/common/helpers/platform_api/psu_fan.py b/tests/common/helpers/platform_api/psu_fan.py index ff5e2a2b815..8ddbc57be3e 100644 --- a/tests/common/helpers/platform_api/psu_fan.py +++ b/tests/common/helpers/platform_api/psu_fan.py @@ -47,6 +47,7 @@ def get_position_in_parent(conn, psu_idx, fan_idx): def is_replaceable(conn, psu_idx, fan_idx): return psu_fan_api(conn, psu_idx, fan_idx, 'is_replaceable') + # # Methods defined in fanBase class # @@ -76,5 +77,5 @@ def set_status_led(conn, psu_idx, fan_idx, color): return psu_fan_api(conn, psu_idx, fan_idx, 'set_status_led', [color]) -def get_status_led(conn, fan_idx): +def get_status_led(conn, psu_idx, fan_idx): return psu_fan_api(conn, psu_idx, fan_idx, 'get_status_led') diff --git a/tests/platform_tests/api/test_fan_drawer_fans.py b/tests/platform_tests/api/test_fan_drawer_fans.py index 853ab2e8fa5..abba94b26db 100644 --- a/tests/platform_tests/api/test_fan_drawer_fans.py +++ b/tests/platform_tests/api/test_fan_drawer_fans.py @@ -169,6 +169,7 @@ def test_is_replaceable(self, platform_api_conn): replaceable = fan_drawer_fan.is_replaceable(platform_api_conn, j, i) if self.expect(replaceable is not None, "Failed to perform is_replaceable for drawer {} fan {}".format(j, i)): self.expect(isinstance(replaceable, bool), "Replaceable value must be a bool value for drawer {} fan {}".format(j, i)) + self.assert_expectations() # diff --git a/tests/platform_tests/api/test_module.py b/tests/platform_tests/api/test_module.py index cdfd4f00516..49477e00703 100644 --- a/tests/platform_tests/api/test_module.py +++ b/tests/platform_tests/api/test_module.py @@ -133,6 +133,7 @@ def test_is_replaceable(self, platform_api_conn): replaceable = module.is_replaceable(platform_api_conn, i) if self.expect(replaceable is not None, "Failed to perform is_replaceable for module {}".format(i)): self.expect(isinstance(replaceable, bool), "Replaceable value must be a bool value for module {}".format(i)) + self.assert_expectations() # diff --git a/tests/platform_tests/api/test_psu_fans.py b/tests/platform_tests/api/test_psu_fans.py index 577e2c961d8..724254cfe9a 100644 --- a/tests/platform_tests/api/test_psu_fans.py +++ b/tests/platform_tests/api/test_psu_fans.py @@ -169,6 +169,7 @@ def test_is_replaceable(self, platform_api_conn): replaceable = psu_fan.is_replaceable(platform_api_conn, j, i) if self.expect(replaceable is not None, "Failed to perform is_replaceable for PSU {} fan {}".format(j, i)): self.expect(isinstance(replaceable, bool), "Replaceable value must be a bool value for PSU {} fan {}".format(j, i)) + self.assert_expectations() #