From b11d0d22058fc097b694f2bda6cce6df9aa591e6 Mon Sep 17 00:00:00 2001 From: Abtin Keshavarzian Date: Wed, 23 Jun 2021 15:22:26 -0700 Subject: [PATCH] [test] restart SRP server (instead of reset) to validate port change (#6763) This commit updates `test_srp_server_reboot_port` to restart the SRP sever instead of performing a full device reset. The SRP restart will trigger the port number change by the server which is the behavior this test is intended to verify. This change should help address intermittent failures on this test. --- tests/scripts/thread-cert/test_srp_server_reboot_port.py | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/tests/scripts/thread-cert/test_srp_server_reboot_port.py b/tests/scripts/thread-cert/test_srp_server_reboot_port.py index dd0d7f883..4a729a368 100755 --- a/tests/scripts/thread-cert/test_srp_server_reboot_port.py +++ b/tests/scripts/thread-cert/test_srp_server_reboot_port.py @@ -35,7 +35,7 @@ import command import thread_cert # Test description: -# This test verifies when an SRP server reboots, it will listen to a UDP port +# This test verifies when an SRP server restarts, it will listen to a UDP port # that wasn't used in the last time. # # Topology: @@ -49,7 +49,7 @@ SERVER = 2 REBOOT_TIMES = 25 -class SrpAutoStartMode(thread_cert.TestCase): +class SrpServerRebootPort(thread_cert.TestCase): USE_MESSAGE_FACTORY = False SUPPORT_NCP = False @@ -99,8 +99,6 @@ class SrpAutoStartMode(thread_cert.TestCase): # old_port = server.get_srp_server_port() server.srp_server_set_enabled(False) - server.reset() - server.start() self.simulator.go(5) server.srp_server_set_enabled(True) self.simulator.go(5) @@ -124,8 +122,6 @@ class SrpAutoStartMode(thread_cert.TestCase): # old_port = server.get_srp_server_port() server.srp_server_set_enabled(False) - server.reset() - server.start() self.simulator.go(5) #