diff --git a/tests/scripts/thread-cert/border_router/test_dnssd_instance_name_with_space.py b/tests/scripts/thread-cert/border_router/test_dnssd_instance_name_with_space.py index b752ecbc2..edd14abac 100755 --- a/tests/scripts/thread-cert/border_router/test_dnssd_instance_name_with_space.py +++ b/tests/scripts/thread-cert/border_router/test_dnssd_instance_name_with_space.py @@ -178,6 +178,10 @@ class TestDnssdInstanceNameWithSpace(thread_cert.TestCase): print('not match: ', dig_answer, record, list(a == b or (callable(b) and b(a)) for a, b in zip(dig_answer, record))) + # Additional records are optional. Ignore if missing. + if additional: + return + self.fail((record, dig_result)) def _match_record(self, record, match): @@ -205,8 +209,6 @@ class TestDnssdInstanceNameWithSpace(thread_cert.TestCase): self._assert_have_answer(dig_result, record, additional=False) if 'ADDITIONAL' in expected_result: - self.assertGreaterEqual(len(dig_result['ADDITIONAL']), len(expected_result['ADDITIONAL']), dig_result) - for record in expected_result['ADDITIONAL']: self._assert_have_answer(dig_result, record, additional=True) diff --git a/tests/scripts/thread-cert/border_router/test_dnssd_server.py b/tests/scripts/thread-cert/border_router/test_dnssd_server.py index 82bc34beb..178ead171 100755 --- a/tests/scripts/thread-cert/border_router/test_dnssd_server.py +++ b/tests/scripts/thread-cert/border_router/test_dnssd_server.py @@ -302,6 +302,10 @@ class TestDnssdServerOnBr(thread_cert.TestCase): print('not match: ', dig_answer, record, list(a == b or (callable(b) and b(a)) for a, b in zip(dig_answer, record))) + # Additional records are optional. Ignore if missing. + if additional: + return + self.fail((record, dig_result)) def _match_record(self, record, match): @@ -332,8 +336,6 @@ class TestDnssdServerOnBr(thread_cert.TestCase): self._assert_have_answer(dig_result, record, additional=False) if 'ADDITIONAL' in expected_result: - self.assertGreaterEqual(len(dig_result['ADDITIONAL']), len(expected_result['ADDITIONAL']), dig_result) - for record in expected_result['ADDITIONAL']: self._assert_have_answer(dig_result, record, additional=True) diff --git a/tests/scripts/thread-cert/border_router/test_dnssd_server_multi_border_routers.py b/tests/scripts/thread-cert/border_router/test_dnssd_server_multi_border_routers.py index f220ff77b..783f7fa0b 100755 --- a/tests/scripts/thread-cert/border_router/test_dnssd_server_multi_border_routers.py +++ b/tests/scripts/thread-cert/border_router/test_dnssd_server_multi_border_routers.py @@ -382,6 +382,10 @@ class TestDnssdServerOnMultiBr(thread_cert.TestCase): print('not match: ', dig_answer, record, list(a == b or (callable(b) and b(a)) for a, b in zip(dig_answer, record))) + # Additional records are optional. Ignore if missing. + if additional: + return + self.fail((record, dig_result)) def _match_record(self, record, match): @@ -412,8 +416,6 @@ class TestDnssdServerOnMultiBr(thread_cert.TestCase): self._assert_have_answer(dig_result, record, additional=False) if 'ADDITIONAL' in expected_result: - self.assertGreaterEqual(len(dig_result['ADDITIONAL']), len(expected_result['ADDITIONAL']), dig_result) - for record in expected_result['ADDITIONAL']: self._assert_have_answer(dig_result, record, additional=True)