From 0e1d18a2d0f48243fc4e0d40515bb0f224f618f0 Mon Sep 17 00:00:00 2001 From: "Dmitrij D. Czarkoff" Date: Sun, 1 May 2016 14:40:54 +0200 Subject: Make imv_navigator_add() return an integer value In case of error return non-zero value instead of cleaning up navigator. This way main loop may do some more cleanup, or maybe even act differently. --- test/navigator.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) (limited to 'test') diff --git a/test/navigator.c b/test/navigator.c index 2a8a7bb..55ab392 100644 --- a/test/navigator.c +++ b/test/navigator.c @@ -25,13 +25,13 @@ static void test_navigator_add_remove(void **state) /* Check poll_changed */ assert_false(imv_navigator_poll_changed(&nav, 0)); - /* Add 3 paths */ - imv_navigator_add(&nav, FILENAME1, 0); - imv_navigator_add(&nav, FILENAME2, 0); - imv_navigator_add(&nav, FILENAME3, 0); - imv_navigator_add(&nav, FILENAME4, 0); - imv_navigator_add(&nav, FILENAME5, 0); - imv_navigator_add(&nav, FILENAME6, 0); + /* Add 6 paths, one non-existant should fail */ + assert_false(imv_navigator_add(&nav, FILENAME1, 0)); + assert_false(imv_navigator_add(&nav, FILENAME2, 0)); + assert_false(imv_navigator_add(&nav, FILENAME3, 0)); + assert_false(imv_navigator_add(&nav, FILENAME4, 0)); + assert_false(imv_navigator_add(&nav, FILENAME5, 0)); + assert_false(imv_navigator_add(&nav, FILENAME6, 0)); assert_int_equal(nav.num_paths, 6); /* Check poll_changed */ @@ -88,7 +88,7 @@ static void test_navigator_file_changed(void **state) } assert_false(futimens(fd, times) == -1); - imv_navigator_add(&nav, FILENAME1, 0); + assert_false(imv_navigator_add(&nav, FILENAME1, 0)); assert_true(imv_navigator_poll_changed(&nav, 0)); assert_false(imv_navigator_poll_changed(&nav, 0)); -- cgit v1.2.3