]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
test-memstream-util.c: Migrate to new assertion macros
authorYaping Li <202858510+YapingLi04@users.noreply.github.com>
Wed, 23 Jul 2025 21:16:03 +0000 (14:16 -0700)
committerYu Watanabe <watanabe.yu+github@gmail.com>
Thu, 24 Jul 2025 11:45:35 +0000 (20:45 +0900)
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-memstream-util.c

index b3b1e813b136eb837e5b069e103cbf0c6c391aac..b63e1695b6d02f6ced86a884f1e36cab736eb563 100644 (file)
@@ -6,7 +6,7 @@
 TEST(memstream_done) {
         _cleanup_(memstream_done) MemStream m = {};
 
-        assert_se(memstream_init(&m));
+        ASSERT_NOT_NULL(memstream_init(&m));
 }
 
 TEST(memstream_empty) {
@@ -14,10 +14,10 @@ TEST(memstream_empty) {
         _cleanup_free_ char *buf = NULL;
         size_t sz;
 
-        assert_se(memstream_init(&m));
-        assert_se(memstream_finalize(&m, &buf, &sz) >= 0);
+        ASSERT_NOT_NULL(memstream_init(&m));
+        ASSERT_OK(memstream_finalize(&m, &buf, &sz));
         ASSERT_STREQ(buf, "");
-        assert_se(sz == 0);
+        ASSERT_EQ(sz, 0u);
 }
 
 TEST(memstream) {
@@ -26,7 +26,7 @@ TEST(memstream) {
         size_t sz;
         FILE *f;
 
-        assert_se(f = memstream_init(&m));
+        ASSERT_NOT_NULL(f = memstream_init(&m));
         fputs("hoge", f);
         fputs("おはよう!", f);
         fputs(UTF8("😀😀😀"), f);
@@ -36,24 +36,24 @@ TEST(memstream) {
 
         buf = mfree(buf);
 
-        assert_se(f = memstream_init(&m));
+        ASSERT_NOT_NULL(f = memstream_init(&m));
         fputs("second", f);
-        assert_se(memstream_finalize(&m, &buf, &sz) >= 0);
+        ASSERT_OK(memstream_finalize(&m, &buf, &sz));
         ASSERT_STREQ(buf, "second");
-        assert_se(sz == strlen("second"));
+        ASSERT_EQ(sz, strlen("second"));
 }
 
 TEST(memstream_dump) {
         _cleanup_(memstream_done) MemStream m = {};
         FILE *f;
 
-        assert_se(f = memstream_init(&m));
+        ASSERT_NOT_NULL(f = memstream_init(&m));
         fputs("first", f);
-        assert_se(memstream_dump(LOG_DEBUG, &m) >= 0);
+        ASSERT_OK(memstream_dump(LOG_DEBUG, &m));
 
-        assert_se(f = memstream_init(&m));
+        ASSERT_NOT_NULL(f = memstream_init(&m));
         fputs("second", f);
-        assert_se(memstream_dump(LOG_DEBUG, &m) >= 0);
+        ASSERT_OK(memstream_dump(LOG_DEBUG, &m));
 }
 
 DEFINE_TEST_MAIN(LOG_DEBUG);