From 64c568aec49d63e6aacd069695093375d0863eff Mon Sep 17 00:00:00 2001 From: Bartosz Golaszewski Date: Mon, 3 Jun 2019 18:18:53 +0200 Subject: [PATCH] tests: gpiomon: use test_trigger_event() to fix timing errors Since we don't need to trigger events from a separate thread when testing tools and we've experienced timing issues with these two test cases, let's use the new helper for triggering of single events. Signed-off-by: Bartosz Golaszewski --- tests/tests-gpiomon.c | 26 ++++++++++++++------------ 1 file changed, 14 insertions(+), 12 deletions(-) diff --git a/tests/tests-gpiomon.c b/tests/tests-gpiomon.c index b17e411..7440d65 100644 --- a/tests/tests-gpiomon.c +++ b/tests/tests-gpiomon.c @@ -131,12 +131,13 @@ static void gpiomon_watch_multiple_lines(void) { test_tool_run("gpiomon", "--format=%o", test_chip_name(0), "1", "2", "3", "4", "5", (char *)NULL); - test_set_event(0, 2, 100); - usleep(150000); - test_set_event(0, 3, 100); - usleep(150000); - test_set_event(0, 4, 100); - usleep(150000); + usleep(100000); + test_trigger_event(0, 2); + usleep(100000); + test_trigger_event(0, 3); + usleep(100000); + test_trigger_event(0, 4); + usleep(100000); test_tool_signal(SIGTERM); test_tool_wait(); @@ -155,12 +156,13 @@ static void gpiomon_watch_multiple_lines_not_in_order(void) { test_tool_run("gpiomon", "--format=%o", test_chip_name(0), "5", "2", "7", "1", "6", (char *)NULL); - test_set_event(0, 2, 100); - usleep(150000); - test_set_event(0, 1, 100); - usleep(150000); - test_set_event(0, 6, 100); - usleep(150000); + usleep(100000); + test_trigger_event(0, 2); + usleep(100000); + test_trigger_event(0, 1); + usleep(100000); + test_trigger_event(0, 6); + usleep(100000); test_tool_signal(SIGTERM); test_tool_wait(); -- 2.30.2