X-Git-Url: http://git.meshlink.io/?a=blobdiff_plain;f=test%2Fblackbox%2Frun_blackbox_tests%2Ftest_cases_channel_set_poll_cb.c;h=5ad8f03d6c2d99de657cdc3d27f16b6f4438bfff;hb=3febbb4;hp=2234c393bb078b3c61dfbfbdcb12aa7546b26909;hpb=dc68da94af8fca91748579c84ef5ed798db7efab;p=meshlink

diff --git a/test/blackbox/run_blackbox_tests/test_cases_channel_set_poll_cb.c b/test/blackbox/run_blackbox_tests/test_cases_channel_set_poll_cb.c
index 2234c393..5ad8f03d 100644
--- a/test/blackbox/run_blackbox_tests/test_cases_channel_set_poll_cb.c
+++ b/test/blackbox/run_blackbox_tests/test_cases_channel_set_poll_cb.c
@@ -16,6 +16,11 @@
     with this program; if not, write to the Free Software Foundation, Inc.,
     51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 */
+
+#ifdef NDEBUG
+#undef NDEBUG
+#endif
+
 #include "execute_tests.h"
 #include "test_cases_channel_set_poll_cb.h"
 #include "../common/containers.h"
@@ -80,6 +85,13 @@ static void poll_cb(meshlink_handle_t *mesh, meshlink_channel_t *channel, size_t
 }
 
 static void node_status_cb(meshlink_handle_t *mesh, meshlink_node_t *source, bool reach) {
+	(void)mesh;
+	(void)source;
+
+	if(!reach) {
+		return;
+	}
+
 	pthread_mutex_lock(&reachable_lock);
 	reachable = true;
 	assert(!pthread_cond_broadcast(&reachable_cond));
@@ -102,8 +114,8 @@ static void test_case_channel_set_poll_cb_01(void **state) {
 static bool test_steps_channel_set_poll_cb_01(void) {
 	/* deleting the confbase if already exists */
 	struct timespec timeout = {0};
-	meshlink_destroy("pollconf1");
-	meshlink_destroy("pollconf2");
+	assert(meshlink_destroy("pollconf1"));
+	assert(meshlink_destroy("pollconf2"));
 	meshlink_set_log_cb(NULL, TEST_MESHLINK_LOG_LEVEL, meshlink_callback_logger);
 
 	/* Create meshlink instances */
@@ -158,8 +170,8 @@ static bool test_steps_channel_set_poll_cb_01(void) {
 	/* closing channel, meshes and destroying confbase */
 	meshlink_close(mesh1);
 	meshlink_close(mesh2);
-	meshlink_destroy("pollconf1");
-	meshlink_destroy("pollconf2");
+	assert(meshlink_destroy("pollconf1"));
+	assert(meshlink_destroy("pollconf2"));
 
 	return true;
 }
@@ -197,10 +209,10 @@ static bool test_steps_channel_set_poll_cb_02(void) {
 
 	/* Setting poll cb with NULL as mesh handler */
 	meshlink_set_channel_poll_cb(NULL, channel, poll_cb);
-	assert_int_equal(meshlink_errno, MESHLINK_EINVAL);
+	assert_int_not_equal(meshlink_errno, 0);
 
 	meshlink_close(mesh_handle);
-	meshlink_destroy("channelpollconf3");
+	assert(meshlink_destroy("channelpollconf3"));
 	return true;
 }
 
@@ -233,7 +245,7 @@ static bool test_steps_channel_set_poll_cb_03(void) {
 	assert_int_equal(meshlink_errno, MESHLINK_EINVAL);
 
 	meshlink_close(mesh_handle);
-	meshlink_destroy("channelpollconf4");
+	assert(meshlink_destroy("channelpollconf4"));
 	return true;
 }