From 4c34a0e4a403ca6d95c985fa0adfe471faccf32e Mon Sep 17 00:00:00 2001 From: Abtin Keshavarzian Date: Thu, 14 Mar 2024 13:43:19 -0700 Subject: [PATCH] [test] enhance `test_netdata_publisher` robustness (#9932) Updates validation logic after published route entry replacement to prevent occasional failures due to a race condition. This increases test stability. --- tests/scripts/thread-cert/test_netdata_publisher.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/scripts/thread-cert/test_netdata_publisher.py b/tests/scripts/thread-cert/test_netdata_publisher.py index 7192e6f43..7648edaaa 100755 --- a/tests/scripts/thread-cert/test_netdata_publisher.py +++ b/tests/scripts/thread-cert/test_netdata_publisher.py @@ -494,15 +494,15 @@ class NetDataPublisher(thread_cert.TestCase): # Replace the published route on leader with '::/0'. leader.netdata_publish_replace(EXTERNAL_ROUTE, '::/0', EXTERNAL_FLAGS, 'med') - self.simulator.go(0.2) + self.simulator.go(1) routes = leader.get_routes() self.assertEqual([route.split(' ')[0] == '::/0' for route in routes].count(True), 1) - self.check_num_of_routes(routes, num - 1, 1, 0) # Replace it back to the original route. leader.netdata_publish_replace('::/0', EXTERNAL_ROUTE, EXTERNAL_FLAGS, 'high') self.simulator.go(WAIT_TIME) routes = leader.get_routes() + self.assertEqual([route.split(' ')[0] == '::/0' for route in routes].count(True), 0) self.check_num_of_routes(routes, num - 1, 0, 1) # Publish the same prefix on leader as an on-mesh prefix. Make