summaryrefslogtreecommitdiff
path: root/auto_tests/bootstrap_test.c
diff options
context:
space:
mode:
authoriphydf <iphydf@users.noreply.github.com>2018-08-25 12:16:54 +0000
committeriphydf <iphydf@users.noreply.github.com>2018-08-25 15:25:33 +0000
commit94b06818fbd7a93387ff4c155b6eff711651aea9 (patch)
tree63365adeb88705c5c2f0cdb4d2afd00a4bd8b43c /auto_tests/bootstrap_test.c
parent0935aab92760a7c9c547faa86c29dcd0d1e03f24 (diff)
Use do-while instead of while in tests.
This forces all the loop bodies to be executed at least once, which is harmless since it just means one more tox event loop iteration. This reduces the jitter we see in coverage measurements, which is partially caused by loops sometimes being entered and sometimes not (because their condition happens to randomly already be true).
Diffstat (limited to 'auto_tests/bootstrap_test.c')
-rw-r--r--auto_tests/bootstrap_test.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/auto_tests/bootstrap_test.c b/auto_tests/bootstrap_test.c
index cbe81ebc..7d1a2f06 100644
--- a/auto_tests/bootstrap_test.c
+++ b/auto_tests/bootstrap_test.c
@@ -24,13 +24,13 @@ int main(void)
24 24
25 printf("Waiting for connection"); 25 printf("Waiting for connection");
26 26
27 while (tox_self_get_connection_status(tox_udp) == TOX_CONNECTION_NONE) { 27 do {
28 printf("."); 28 printf(".");
29 fflush(stdout); 29 fflush(stdout);
30 30
31 tox_iterate(tox_udp, nullptr); 31 tox_iterate(tox_udp, nullptr);
32 c_sleep(ITERATION_INTERVAL); 32 c_sleep(ITERATION_INTERVAL);
33 } 33 } while (tox_self_get_connection_status(tox_udp) == TOX_CONNECTION_NONE);
34 34
35 const TOX_CONNECTION status = tox_self_get_connection_status(tox_udp); 35 const TOX_CONNECTION status = tox_self_get_connection_status(tox_udp);
36 ck_assert_msg(status == TOX_CONNECTION_UDP, 36 ck_assert_msg(status == TOX_CONNECTION_UDP,