From: Busayo Dada <toyinoluwabusayo@gmail.com>
Date: Mon, 24 Mar 2025 12:28:20 +0000 (+0100)
Subject: test: update to use the new ASSERT_OK() macro and friends
X-Git-Tag: v258-rc1~1014
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=86960038ef765e5fb3f164485239132dde4729e9;p=thirdparty%2Fsystemd.git

test: update to use the new ASSERT_OK() macro and friends
---

diff --git a/src/test/test-errno-util.c b/src/test/test-errno-util.c
index ab463bd1b39..a04e95223a3 100644
--- a/src/test/test-errno-util.c
+++ b/src/test/test-errno-util.c
@@ -27,8 +27,8 @@ TEST(STRERROR) {
         log_info("STRERROR(%d), STRERROR(%d) → %s, %s", 200, 201, STRERROR(200), STRERROR(201));
 
         const char *a = STRERROR(200), *b = STRERROR(201);
-        assert_se(strstr(a, "200"));
-        assert_se(strstr(b, "201"));
+        ASSERT_NOT_NULL(strstr(a, "200"));
+        ASSERT_NOT_NULL(strstr(b, "201"));
 
         /* Check with negative values */
         ASSERT_STREQ(a, STRERROR(-200));
@@ -38,7 +38,7 @@ TEST(STRERROR) {
         char buf[DECIMAL_STR_MAX(int)];
         xsprintf(buf, "%d", INT_MAX);  /* INT_MAX is hexadecimal, use printf to convert to decimal */
         log_info("STRERROR(%d) → %s", INT_MAX, c);
-        assert_se(strstr(c, buf));
+        ASSERT_NOT_NULL(strstr(c, buf));
 }
 
 TEST(STRERROR_OR_ELSE) {
@@ -53,7 +53,7 @@ TEST(PROTECT_ERRNO) {
                 PROTECT_ERRNO;
                 errno = 11;
         }
-        assert_se(errno == 12);
+        ASSERT_EQ(errno, 12);
 }
 
 static void test_unprotect_errno_inner_function(void) {
@@ -71,42 +71,41 @@ TEST(UNPROTECT_ERRNO) {
 
         UNPROTECT_ERRNO;
 
-        assert_se(errno == 4711);
+        ASSERT_EQ(errno, 4711);
 
         test_unprotect_errno_inner_function();
-
-        assert_se(errno == 4711);
+        ASSERT_EQ(errno, 4711);
 }
 
 TEST(RET_GATHER) {
         int x = 0, y = 2;
 
-        assert_se(RET_GATHER(x, 5) == 0);
-        assert_se(RET_GATHER(x, -5) == -5);
-        assert_se(RET_GATHER(x, -1) == -5);
+        ASSERT_EQ(RET_GATHER(x, 5), 0);
+        ASSERT_EQ(RET_GATHER(x, -5), -5);
+        ASSERT_EQ(RET_GATHER(x, -1), -5);
 
-        assert_se(RET_GATHER(x, y++) == -5);
-        assert_se(y == 3);
+        ASSERT_EQ(RET_GATHER(x, y++), -5);
+        ASSERT_EQ(y, 3);
 }
 
 TEST(ERRNO_IS_TRANSIENT) {
-        assert_se( ERRNO_IS_NEG_TRANSIENT(-EINTR));
-        assert_se(!ERRNO_IS_NEG_TRANSIENT(EINTR));
-        assert_se( ERRNO_IS_TRANSIENT(-EINTR));
-        assert_se( ERRNO_IS_TRANSIENT(EINTR));
+        ASSERT_TRUE(ERRNO_IS_NEG_TRANSIENT(-EINTR));
+        ASSERT_FALSE(ERRNO_IS_NEG_TRANSIENT(EINTR));
+        ASSERT_TRUE(ERRNO_IS_TRANSIENT(-EINTR));
+        ASSERT_TRUE(ERRNO_IS_TRANSIENT(EINTR));
 
         /* Test with type wider than int */
         ssize_t r = -EAGAIN;
-        assert_se( ERRNO_IS_NEG_TRANSIENT(r));
+        ASSERT_TRUE(ERRNO_IS_NEG_TRANSIENT(r));
 
         /* On 64-bit arches, now (int) r == EAGAIN */
         r = SSIZE_MAX - EAGAIN + 1;
-        assert_se(!ERRNO_IS_NEG_TRANSIENT(r));
+        ASSERT_FALSE(ERRNO_IS_NEG_TRANSIENT(r));
 
-        assert_se(!ERRNO_IS_NEG_TRANSIENT(INT_MAX));
-        assert_se(!ERRNO_IS_NEG_TRANSIENT(INT_MIN));
-        assert_se(!ERRNO_IS_NEG_TRANSIENT(INTMAX_MAX));
-        assert_se(!ERRNO_IS_NEG_TRANSIENT(INTMAX_MIN));
+        ASSERT_FALSE(ERRNO_IS_NEG_TRANSIENT(INT_MAX));
+        ASSERT_FALSE(ERRNO_IS_NEG_TRANSIENT(INT_MIN));
+        ASSERT_FALSE(ERRNO_IS_NEG_TRANSIENT(INTMAX_MAX));
+        ASSERT_FALSE(ERRNO_IS_NEG_TRANSIENT(INTMAX_MIN));
 }
 
 DEFINE_TEST_MAIN(LOG_INFO);