From 95e6fbcf569b4fdce47375fb98a8127d7853421f Mon Sep 17 00:00:00 2001 From: Andrea Bondavalli Date: Mon, 15 Jun 2020 19:54:26 +0200 Subject: [PATCH] Fixed add_remove_check_mdns_browser_update_all regression test. Added note on regression tests execution. --- README.md | 4 +++- daemon/tests/daemon_test.cpp | 1 - 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 17244e2..0e5b378 100644 --- a/README.md +++ b/README.md @@ -120,7 +120,9 @@ setup the kernel parameters with: make sure that no instances of the aes67-daemon are running, enter the [tests](daemon/tests) subdirectory and run: - ./daemon-test + ./daemon-test -p + +**_NOTE:_** when running regression tests make sure that no other Ravenna mDNS sources are advertised on the network because this will affect the results. Regression tests run on loopback interface but Avahi ignores the interface parameter set and will forward to the daemon the sources found on all network interfaces. ## Run the demo ## diff --git a/daemon/tests/daemon_test.cpp b/daemon/tests/daemon_test.cpp index c9e97f1..7edf11a 100644 --- a/daemon/tests/daemon_test.cpp +++ b/daemon/tests/daemon_test.cpp @@ -866,7 +866,6 @@ BOOST_AUTO_TEST_CASE(add_remove_check_mdns_browser_update_all) { } } } - retry--; } while (retry-- && found < g_stream_num_max); BOOST_REQUIRE_MESSAGE(retry > 0, "all remote mdns source updated"); for (int id = 0; id < g_stream_num_max; id++) {