Skip to content

Commit 240817c

Browse files
ppikhwangxin
authored andcommitted
Fixed "TypeError: __init__() takes exactly 8 arguments (7 given)" (#7322)
Fixed "TypeError: __init__() takes exactly 8 arguments (7 given)" in test platform_tests/test_cont_warm_reboot.py which happen after merge community code. Issue introduced in PR: #6968 - [advanced-reboot] IO path verification fixes for dualtor - What is the motivation for this PR? Fix issue "TypeError: init() takes exactly 8 arguments (7 given)" - How did you do it? Added argument in method call - How did you verify/test it? Executed test which has been modified Signed-off-by: Petro Pikh <[email protected]>
1 parent 819dd4f commit 240817c

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

tests/platform_tests/test_cont_warm_reboot.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -273,7 +273,7 @@ def create_test_report(self):
273273
format(self.test_failures, self.reboot_count))
274274

275275

276-
def start_continuous_reboot(self, request, duthost, ptfhost, localhost, tbinfo, creds):
276+
def start_continuous_reboot(self, request, duthosts, duthost, ptfhost, localhost, tbinfo, creds):
277277
self.test_set_up()
278278
# Start continuous warm/fast reboot on the DUT
279279
for count in range(self.continuous_reboot_count):
@@ -284,7 +284,7 @@ def start_continuous_reboot(self, request, duthost, ptfhost, localhost, tbinfo,
284284
.format(self.reboot_count, self.continuous_reboot_count, self.reboot_type))
285285
reboot_type = self.reboot_type + "-reboot"
286286
try:
287-
self.advancedReboot = AdvancedReboot(request, duthost, ptfhost, localhost, tbinfo, creds,\
287+
self.advancedReboot = AdvancedReboot(request, duthosts, duthost, ptfhost, localhost, tbinfo, creds,\
288288
rebootType=reboot_type, moduleIgnoreErrors=True)
289289
except Exception:
290290
self.sub_test_result = False
@@ -349,5 +349,5 @@ def test_continuous_reboot(request, duthosts, enum_rand_one_per_hwsku_frontend_h
349349
"""
350350
duthost = duthosts[enum_rand_one_per_hwsku_frontend_hostname]
351351
continuous_reboot = ContinuousReboot(request, duthost, ptfhost, localhost, conn_graph_facts)
352-
continuous_reboot.start_continuous_reboot(request, duthost, ptfhost, localhost, tbinfo, creds)
352+
continuous_reboot.start_continuous_reboot(request, duthosts, duthost, ptfhost, localhost, tbinfo, creds)
353353
continuous_reboot.test_teardown()

0 commit comments

Comments
 (0)