From 0ccc31d964e83b2ff2b8c25aa297b2581f9d51a2 Mon Sep 17 00:00:00 2001
From: Yaping Li <202858510+YapingLi04@users.noreply.github.com>
Date: Tue, 22 Jul 2025 15:23:48 -0700
Subject: [PATCH] test-user-record.c: Migrate to new assertion MACROs

We recently added a new set of assertion macros such as ASSERT_GE, ASSERT_OK, ASSERT_EQ, ... which show not
only the expression that failed but also the values of the arguments of the expression. Let's use them.
---
 src/test/test-user-record.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/test/test-user-record.c b/src/test/test-user-record.c
index dd5d7360070..c807ffe83af 100644
--- a/src/test/test-user-record.c
+++ b/src/test/test-user-record.c
@@ -9,7 +9,7 @@
         ({                                      \
                 typeof(ret) _r = (ret);         \
                 user_record_unref(*_r);         \
-                assert_se(user_record_build((ret), SD_JSON_BUILD_OBJECT(SD_JSON_BUILD_PAIR_STRING("disposition", "regular"), __VA_ARGS__)) >= 0); \
+                ASSERT_OK(user_record_build((ret), SD_JSON_BUILD_OBJECT(SD_JSON_BUILD_PAIR_STRING("disposition", "regular"), __VA_ARGS__))); \
                 0;                              \
         })
 
@@ -23,7 +23,7 @@ TEST(self_changes) {
         USER(&new,
              SD_JSON_BUILD_PAIR_STRING("userName", "test"),
              SD_JSON_BUILD_PAIR_UNSIGNED("notInHardCodedList", 99999));
-        assert_se(!user_record_self_changes_allowed(curr, new));
+        ASSERT_FALSE(user_record_self_changes_allowed(curr, new));
 
         /* manually allowlisted */
         USER(&curr,
@@ -35,7 +35,7 @@ TEST(self_changes) {
              SD_JSON_BUILD_PAIR_ARRAY("selfModifiableFields", SD_JSON_BUILD_STRING("notInHardCodedList")),
              /* change in order shouldn't affect things */
              SD_JSON_BUILD_PAIR_UNSIGNED("notInHardCodedList", 99999));
-        assert_se(user_record_self_changes_allowed(curr, new));
+        ASSERT_TRUE(user_record_self_changes_allowed(curr, new));
 
         /* default allowlisted */
         USER(&curr,
@@ -44,7 +44,7 @@ TEST(self_changes) {
         USER(&new,
              SD_JSON_BUILD_PAIR_STRING("userName", "test"),
              SD_JSON_BUILD_PAIR_STRING("realName", "New Name"));
-        assert_se(user_record_self_changes_allowed(curr, new));
+        ASSERT_TRUE(user_record_self_changes_allowed(curr, new));
 
         /* introduced new default allowlisted */
         USER(&curr,
@@ -52,7 +52,7 @@ TEST(self_changes) {
         USER(&new,
              SD_JSON_BUILD_PAIR_STRING("userName", "test"),
              SD_JSON_BUILD_PAIR_STRING("realName", "New Name"));
-        assert_se(user_record_self_changes_allowed(curr, new));
+        ASSERT_TRUE(user_record_self_changes_allowed(curr, new));
 
         /* introduced new not allowlisted */
         USER(&curr,
@@ -60,7 +60,7 @@ TEST(self_changes) {
         USER(&new,
              SD_JSON_BUILD_PAIR_STRING("userName", "test"),
              SD_JSON_BUILD_PAIR_UNSIGNED("notInHardCodedList", 99999));
-        assert_se(!user_record_self_changes_allowed(curr, new));
+        ASSERT_FALSE(user_record_self_changes_allowed(curr, new));
 
         /* privileged section: default allowlisted */
         USER(&curr,
@@ -71,7 +71,7 @@ TEST(self_changes) {
              SD_JSON_BUILD_PAIR_STRING("userName", "test"),
              SD_JSON_BUILD_PAIR_OBJECT("privileged",
                                     SD_JSON_BUILD_PAIR_STRING("passwordHint", "New Hint")));
-        assert_se(user_record_self_changes_allowed(curr, new));
+        ASSERT_TRUE(user_record_self_changes_allowed(curr, new));
 
         /* privileged section: not allowlisted */
         USER(&curr,
@@ -82,7 +82,7 @@ TEST(self_changes) {
              SD_JSON_BUILD_PAIR_STRING("userName", "test"),
              SD_JSON_BUILD_PAIR_OBJECT("privileged",
                                     SD_JSON_BUILD_PAIR_UNSIGNED("notInHardCodedList", 99999)));
-        assert_se(!user_record_self_changes_allowed(curr, new));
+        ASSERT_FALSE(user_record_self_changes_allowed(curr, new));
 
         /* privileged section: manually allowlisted */
         USER(&curr,
@@ -95,7 +95,7 @@ TEST(self_changes) {
              SD_JSON_BUILD_PAIR_ARRAY("selfModifiablePrivileged", SD_JSON_BUILD_STRING("notInHardCodedList")),
              SD_JSON_BUILD_PAIR_OBJECT("privileged",
                                     SD_JSON_BUILD_PAIR_UNSIGNED("notInHardCodedList", 99999)));
-        assert_se(user_record_self_changes_allowed(curr, new));
+        ASSERT_TRUE(user_record_self_changes_allowed(curr, new));
 }
 
 DEFINE_TEST_MAIN(LOG_INFO);
-- 
2.47.3