From 36e72ff87fc46da22ed9d41a36d28446ef78ccf8 Mon Sep 17 00:00:00 2001 From: Aharon Malkin Date: Tue, 20 Sep 2022 14:44:13 +0300 Subject: [PATCH] Added a fix to advanced reboot script Details: Added the other_vendor_nos parameter as False by default to the AdvancedReboot class, and updated the reboot test cases correspondingly. --- tests/common/fixtures/advanced_reboot.py | 3 ++- tests/platform_tests/test_advanced_reboot.py | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/common/fixtures/advanced_reboot.py b/tests/common/fixtures/advanced_reboot.py index 1751cbd3f32..a5878e4358c 100644 --- a/tests/common/fixtures/advanced_reboot.py +++ b/tests/common/fixtures/advanced_reboot.py @@ -80,7 +80,8 @@ def __init__(self, request, duthost, ptfhost, localhost, tbinfo, creds, **kwargs self.moduleIgnoreErrors = kwargs["allow_fail"] if "allow_fail" in kwargs else False self.allowMacJump = kwargs["allow_mac_jumping"] if "allow_mac_jumping" in kwargs else False self.advanceboot_loganalyzer = kwargs["advanceboot_loganalyzer"] if "advanceboot_loganalyzer" in kwargs else None - self.__dict__.update(kwargs) + self.other_vendor_nos = kwargs['other_vendor_nos'] if 'other_vendor_nos' in kwargs else False + self.__dict__.update(kwargs) self.__extractTestParam() self.rebootData = {} self.hostMaxLen = 0 diff --git a/tests/platform_tests/test_advanced_reboot.py b/tests/platform_tests/test_advanced_reboot.py index 3928f197079..ade8ea5f4a6 100644 --- a/tests/platform_tests/test_advanced_reboot.py +++ b/tests/platform_tests/test_advanced_reboot.py @@ -39,7 +39,7 @@ def test_fast_reboot(request, get_advanced_reboot, verify_dut_health, @param get_advanced_reboot: advanced reboot test fixture ''' advancedReboot = get_advanced_reboot(rebootType='fast-reboot',\ - advanceboot_loganalyzer=advanceboot_loganalyzer, other_vendor_nos = False) + advanceboot_loganalyzer=advanceboot_loganalyzer) advancedReboot.runRebootTestcase()