From: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Date: Wed, 28 Apr 2021 17:16:12 +0000 (-0400)
Subject: Tests: notification: fail on trigger unregistration failure
X-Git-Tag: v2.13.0-rc2~56
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=c78585cf832fba85289419a842d5d4e7dce1e4a1;p=lttng-tools.git

Tests: notification: fail on trigger unregistration failure

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: I474c92f34604100081676dda484513cc8a950c73
---

diff --git a/tests/regression/tools/notification/notification.c b/tests/regression/tools/notification/notification.c
index 66cef82a0..bdea55921 100644
--- a/tests/regression/tools/notification/notification.c
+++ b/tests/regression/tools/notification/notification.c
@@ -1220,7 +1220,8 @@ static void test_subscription_twice(const char *session_name,
 			domain_type, BUFFER_USAGE_TYPE_LOW, 0.99, &condition,
 			&action, &trigger);
 	if (ret) {
-		fail("Setup error on trigger registration");
+		fail("Setup error on trigger registration in %s()",
+				__FUNCTION__);
 		goto end;
 	}
 
@@ -1245,7 +1246,11 @@ static void test_subscription_twice(const char *session_name,
 			"Subscribe to a condition for which subscription was already done");
 
 end:
-	lttng_unregister_trigger(trigger);
+	ret = lttng_unregister_trigger(trigger);
+	if (ret) {
+		fail("Failed to unregister trigger in %s()", __FUNCTION__);
+	}
+
 	lttng_trigger_destroy(trigger);
 	lttng_notification_channel_destroy(notification_channel);
 	lttng_action_destroy(action);