]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
test-format-table: convert to ASSERT_* macros
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Tue, 3 Dec 2024 13:52:15 +0000 (14:52 +0100)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Tue, 3 Dec 2024 13:57:01 +0000 (14:57 +0100)
src/test/test-format-table.c

index f074c5b6f7a660af484ae2591c26d879a448952c..a9c41da511c74efb36e5779cc6e8e99970467d86 100644 (file)
@@ -15,135 +15,134 @@ TEST(issue_9549) {
         _cleanup_(table_unrefp) Table *table = NULL;
         _cleanup_free_ char *formatted = NULL;
 
-        assert_se(table = table_new("name", "type", "ro", "usage", "created", "modified"));
-        assert_se(table_set_align_percent(table, TABLE_HEADER_CELL(3), 100) >= 0);
-        assert_se(table_add_many(table,
+        ASSERT_NOT_NULL(table = table_new("name", "type", "ro", "usage", "created", "modified"));
+        ASSERT_OK(table_set_align_percent(table, TABLE_HEADER_CELL(3), 100));
+        ASSERT_OK(table_add_many(table,
                                  TABLE_STRING, "foooo",
                                  TABLE_STRING, "raw",
                                  TABLE_BOOLEAN, false,
                                  TABLE_SIZE, (uint64_t) (673.7*1024*1024),
                                  TABLE_STRING, "Wed 2018-07-11 00:10:33 JST",
-                                 TABLE_STRING, "Wed 2018-07-11 00:16:00 JST") >= 0);
+                                 TABLE_STRING, "Wed 2018-07-11 00:16:00 JST"));
 
         table_set_width(table, 75);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
 
         printf("%s\n", formatted);
-        assert_se(streq(formatted,
-                        "NAME  TYPE RO  USAGE CREATED                    MODIFIED\n"
-                        "foooo raw  no 673.6M Wed 2018-07-11 00:10:33 J… Wed 2018-07-11 00:16:00 JST\n"
-                        ));
+        ASSERT_STREQ(formatted,
+                     "NAME  TYPE RO  USAGE CREATED                    MODIFIED\n"
+                     "foooo raw  no 673.6M Wed 2018-07-11 00:10:33 J… Wed 2018-07-11 00:16:00 JST\n");
 }
 
 TEST(multiline) {
         _cleanup_(table_unrefp) Table *table = NULL;
         _cleanup_free_ char *formatted = NULL;
 
-        assert_se(table = table_new("foo", "bar"));
+        ASSERT_NOT_NULL(table = table_new("foo", "bar"));
 
-        assert_se(table_set_align_percent(table, TABLE_HEADER_CELL(1), 100) >= 0);
+        ASSERT_OK(table_set_align_percent(table, TABLE_HEADER_CELL(1), 100));
 
-        assert_se(table_add_many(table,
+        ASSERT_OK(table_add_many(table,
                                  TABLE_STRING, "three\ndifferent\nlines",
-                                 TABLE_STRING, "two\nlines\n") >= 0);
+                                 TABLE_STRING, "two\nlines\n"));
 
         table_set_cell_height_max(table, 1);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO     BAR\n"
-                        "three… two…\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO     BAR\n"
+                     "three… two…\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 2);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO          BAR\n"
-                        "three        two\n"
-                        "different… lines\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO          BAR\n"
+                     "three        two\n"
+                     "different… lines\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 3);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO         BAR\n"
-                        "three       two\n"
-                        "different lines\n"
-                        "lines     \n"));
+        ASSERT_STREQ(formatted,
+                     "FOO         BAR\n"
+                     "three       two\n"
+                     "different lines\n"
+                     "lines     \n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, SIZE_MAX);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO         BAR\n"
-                        "three       two\n"
-                        "different lines\n"
-                        "lines     \n"));
+        ASSERT_STREQ(formatted,
+                     "FOO         BAR\n"
+                     "three       two\n"
+                     "different lines\n"
+                     "lines     \n");
         formatted = mfree(formatted);
 
-        assert_se(table_add_many(table,
+        ASSERT_OK(table_add_many(table,
                                  TABLE_STRING, "short",
-                                 TABLE_STRING, "a\npair") >= 0);
+                                 TABLE_STRING, "a\npair"));
 
-        assert_se(table_add_many(table,
+        ASSERT_OK(table_add_many(table,
                                  TABLE_STRING, "short2\n",
-                                 TABLE_STRING, "a\nfour\nline\ncell") >= 0);
+                                 TABLE_STRING, "a\nfour\nline\ncell"));
 
         table_set_cell_height_max(table, 1);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO     BAR\n"
-                        "three… two…\n"
-                        "short    a…\n"
-                        "short2   a…\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO     BAR\n"
+                     "three… two…\n"
+                     "short    a…\n"
+                     "short2   a…\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 2);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO          BAR\n"
-                        "three        two\n"
-                        "different… lines\n"
-                        "short          a\n"
-                        "            pair\n"
-                        "short2         a\n"
-                        "           four…\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO          BAR\n"
+                     "three        two\n"
+                     "different… lines\n"
+                     "short          a\n"
+                     "            pair\n"
+                     "short2         a\n"
+                     "           four…\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 3);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO         BAR\n"
-                        "three       two\n"
-                        "different lines\n"
-                        "lines     \n"
-                        "short         a\n"
-                        "           pair\n"
-                        "short2        a\n"
-                        "           four\n"
-                        "          line…\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO         BAR\n"
+                     "three       two\n"
+                     "different lines\n"
+                     "lines     \n"
+                     "short         a\n"
+                     "           pair\n"
+                     "short2        a\n"
+                     "           four\n"
+                     "          line…\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, SIZE_MAX);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO         BAR\n"
-                        "three       two\n"
-                        "different lines\n"
-                        "lines     \n"
-                        "short         a\n"
-                        "           pair\n"
-                        "short2        a\n"
-                        "           four\n"
-                        "           line\n"
-                        "           cell\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO         BAR\n"
+                     "three       two\n"
+                     "different lines\n"
+                     "lines     \n"
+                     "short         a\n"
+                     "           pair\n"
+                     "short2        a\n"
+                     "           four\n"
+                     "           line\n"
+                     "           cell\n");
         formatted = mfree(formatted);
 }
 
@@ -151,111 +150,111 @@ TEST(strv) {
         _cleanup_(table_unrefp) Table *table = NULL;
         _cleanup_free_ char *formatted = NULL;
 
-        assert_se(table = table_new("foo", "bar"));
+        ASSERT_NOT_NULL(table = table_new("foo", "bar"));
 
-        assert_se(table_set_align_percent(table, TABLE_HEADER_CELL(1), 100) >= 0);
+        ASSERT_OK(table_set_align_percent(table, TABLE_HEADER_CELL(1), 100));
 
-        assert_se(table_add_many(table,
+        ASSERT_OK(table_add_many(table,
                                  TABLE_STRV, STRV_MAKE("three", "different", "lines"),
-                                 TABLE_STRV, STRV_MAKE("two", "lines")) >= 0);
+                                 TABLE_STRV, STRV_MAKE("two", "lines")));
 
         table_set_cell_height_max(table, 1);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO     BAR\n"
-                        "three… two…\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO     BAR\n"
+                     "three… two…\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 2);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO          BAR\n"
-                        "three        two\n"
-                        "different… lines\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO          BAR\n"
+                     "three        two\n"
+                     "different… lines\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 3);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO         BAR\n"
-                        "three       two\n"
-                        "different lines\n"
-                        "lines     \n"));
+        ASSERT_STREQ(formatted,
+                     "FOO         BAR\n"
+                     "three       two\n"
+                     "different lines\n"
+                     "lines     \n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, SIZE_MAX);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO         BAR\n"
-                        "three       two\n"
-                        "different lines\n"
-                        "lines     \n"));
+        ASSERT_STREQ(formatted,
+                     "FOO         BAR\n"
+                     "three       two\n"
+                     "different lines\n"
+                     "lines     \n");
         formatted = mfree(formatted);
 
-        assert_se(table_add_many(table,
+        ASSERT_OK(table_add_many(table,
                                  TABLE_STRING, "short",
-                                 TABLE_STRV, STRV_MAKE("a", "pair")) >= 0);
+                                 TABLE_STRV, STRV_MAKE("a", "pair")));
 
-        assert_se(table_add_many(table,
+        ASSERT_OK(table_add_many(table,
                                  TABLE_STRV, STRV_MAKE("short2"),
-                                 TABLE_STRV, STRV_MAKE("a", "four", "line", "cell")) >= 0);
+                                 TABLE_STRV, STRV_MAKE("a", "four", "line", "cell")));
 
         table_set_cell_height_max(table, 1);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
+        ASSERT_STREQ(formatted,
                         "FOO     BAR\n"
                         "three… two…\n"
                         "short    a…\n"
-                        "short2   a…\n"));
+                        "short2   a…\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 2);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO          BAR\n"
-                        "three        two\n"
-                        "different… lines\n"
-                        "short          a\n"
-                        "            pair\n"
-                        "short2         a\n"
-                        "           four…\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO          BAR\n"
+                     "three        two\n"
+                     "different… lines\n"
+                     "short          a\n"
+                     "            pair\n"
+                     "short2         a\n"
+                     "           four…\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 3);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO         BAR\n"
-                        "three       two\n"
-                        "different lines\n"
-                        "lines     \n"
-                        "short         a\n"
-                        "           pair\n"
-                        "short2        a\n"
-                        "           four\n"
-                        "          line…\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO         BAR\n"
+                     "three       two\n"
+                     "different lines\n"
+                     "lines     \n"
+                     "short         a\n"
+                     "           pair\n"
+                     "short2        a\n"
+                     "           four\n"
+                     "          line…\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, SIZE_MAX);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO         BAR\n"
-                        "three       two\n"
-                        "different lines\n"
-                        "lines     \n"
-                        "short         a\n"
-                        "           pair\n"
-                        "short2        a\n"
-                        "           four\n"
-                        "           line\n"
-                        "           cell\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO         BAR\n"
+                     "three       two\n"
+                     "different lines\n"
+                     "lines     \n"
+                     "short         a\n"
+                     "           pair\n"
+                     "short2        a\n"
+                     "           four\n"
+                     "           line\n"
+                     "           cell\n");
         formatted = mfree(formatted);
 }
 
@@ -263,101 +262,101 @@ TEST(strv_wrapped) {
         _cleanup_(table_unrefp) Table *table = NULL;
         _cleanup_free_ char *formatted = NULL;
 
-        assert_se(table = table_new("foo", "bar"));
+        ASSERT_NOT_NULL(table = table_new("foo", "bar"));
 
-        assert_se(table_set_align_percent(table, TABLE_HEADER_CELL(1), 100) >= 0);
+        ASSERT_OK(table_set_align_percent(table, TABLE_HEADER_CELL(1), 100));
 
-        assert_se(table_add_many(table,
+        ASSERT_OK(table_add_many(table,
                                  TABLE_STRV_WRAPPED, STRV_MAKE("three", "different", "lines"),
-                                 TABLE_STRV_WRAPPED, STRV_MAKE("two", "lines")) >= 0);
+                                 TABLE_STRV_WRAPPED, STRV_MAKE("two", "lines")));
 
         table_set_cell_height_max(table, 1);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO                         BAR\n"
-                        "three different lines two lines\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO                         BAR\n"
+                     "three different lines two lines\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 2);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO                         BAR\n"
-                        "three different lines two lines\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO                         BAR\n"
+                     "three different lines two lines\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 3);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO                         BAR\n"
-                        "three different lines two lines\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO                         BAR\n"
+                     "three different lines two lines\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, SIZE_MAX);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO                         BAR\n"
-                        "three different lines two lines\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO                         BAR\n"
+                     "three different lines two lines\n");
         formatted = mfree(formatted);
 
-        assert_se(table_add_many(table,
+        ASSERT_OK(table_add_many(table,
                                  TABLE_STRING, "short",
-                                 TABLE_STRV_WRAPPED, STRV_MAKE("a", "pair")) >= 0);
+                                 TABLE_STRV_WRAPPED, STRV_MAKE("a", "pair")));
 
-        assert_se(table_add_many(table,
+        ASSERT_OK(table_add_many(table,
                                  TABLE_STRV_WRAPPED, STRV_MAKE("short2"),
                                  TABLE_STRV_WRAPPED, STRV_MAKE("a", "eight", "line", "ćęłł",
-                                                               "___5___", "___6___", "___7___", "___8___")) >= 0);
+                                                               "___5___", "___6___", "___7___", "___8___")));
 
         table_set_cell_height_max(table, 1);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO                             BAR\n"
-                        "three different…          two lines\n"
-                        "short                        a pair\n"
-                        "short2           a eight line ćęłł…\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO                             BAR\n"
+                     "three different…          two lines\n"
+                     "short                        a pair\n"
+                     "short2           a eight line ćęłł…\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 2);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO                           BAR\n"
-                        "three different         two lines\n"
-                        "lines           \n"
-                        "short                      a pair\n"
-                        "short2          a eight line ćęłł\n"
-                        "                 ___5___ ___6___…\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO                           BAR\n"
+                     "three different         two lines\n"
+                     "lines           \n"
+                     "short                      a pair\n"
+                     "short2          a eight line ćęłł\n"
+                     "                 ___5___ ___6___…\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, 3);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO                           BAR\n"
-                        "three different         two lines\n"
-                        "lines           \n"
-                        "short                      a pair\n"
-                        "short2          a eight line ćęłł\n"
-                        "                  ___5___ ___6___\n"
-                        "                  ___7___ ___8___\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO                           BAR\n"
+                     "three different         two lines\n"
+                     "lines           \n"
+                     "short                      a pair\n"
+                     "short2          a eight line ćęłł\n"
+                     "                  ___5___ ___6___\n"
+                     "                  ___7___ ___8___\n");
         formatted = mfree(formatted);
 
         table_set_cell_height_max(table, SIZE_MAX);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
         fputs(formatted, stdout);
-        assert_se(streq(formatted,
-                        "FOO                           BAR\n"
-                        "three different         two lines\n"
-                        "lines           \n"
-                        "short                      a pair\n"
-                        "short2          a eight line ćęłł\n"
-                        "                  ___5___ ___6___\n"
-                        "                  ___7___ ___8___\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO                           BAR\n"
+                     "three different         two lines\n"
+                     "lines           \n"
+                     "short                      a pair\n"
+                     "short2          a eight line ćęłł\n"
+                     "                  ___5___ ___6___\n"
+                     "                  ___7___ ___8___\n");
         formatted = mfree(formatted);
 }
 
@@ -365,31 +364,31 @@ TEST(json) {
         _cleanup_(sd_json_variant_unrefp) sd_json_variant *v = NULL, *w = NULL;
         _cleanup_(table_unrefp) Table *t = NULL;
 
-        assert_se(t = table_new_raw(4));
+        ASSERT_NOT_NULL(t = table_new_raw(4));
 
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_HEADER, "foo bar",
                                  TABLE_HEADER, "quux",
                                  TABLE_HEADER, "piep miau",
                                  TABLE_HEADER, "asdf",
-                                 TABLE_SET_JSON_FIELD_NAME, "asdf-custom") >= 0);
-        assert_se(table_set_json_field_name(t, 2, "zzz") >= 0);
+                                 TABLE_SET_JSON_FIELD_NAME, "asdf-custom"));
+        ASSERT_OK(table_set_json_field_name(t, 2, "zzz"));
 
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_STRING, "v1",
                                  TABLE_UINT64, UINT64_C(4711),
                                  TABLE_BOOLEAN, true,
-                                 TABLE_EMPTY) >= 0);
+                                 TABLE_EMPTY));
 
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_STRV, STRV_MAKE("a", "b", "c"),
                                  TABLE_EMPTY,
                                  TABLE_MODE, 0755,
-                                 TABLE_EMPTY) >= 0);
+                                 TABLE_EMPTY));
 
-        assert_se(table_to_json(t, &v) >= 0);
+        ASSERT_OK(table_to_json(t, &v));
 
-        assert_se(sd_json_build(&w,
+        ASSERT_OK(sd_json_build(&w,
                              SD_JSON_BUILD_ARRAY(
                                              SD_JSON_BUILD_OBJECT(
                                                              SD_JSON_BUILD_PAIR("foo_bar", JSON_BUILD_CONST_STRING("v1")),
@@ -400,9 +399,9 @@ TEST(json) {
                                                              SD_JSON_BUILD_PAIR("foo_bar", SD_JSON_BUILD_STRV(STRV_MAKE("a", "b", "c"))),
                                                              SD_JSON_BUILD_PAIR("quux", SD_JSON_BUILD_NULL),
                                                              SD_JSON_BUILD_PAIR("zzz", SD_JSON_BUILD_UNSIGNED(0755)),
-                                                             SD_JSON_BUILD_PAIR("asdf-custom", SD_JSON_BUILD_NULL)))) >= 0);
+                                                             SD_JSON_BUILD_PAIR("asdf-custom", SD_JSON_BUILD_NULL)))));
 
-        assert_se(sd_json_variant_equal(v, w));
+        ASSERT_TRUE(sd_json_variant_equal(v, w));
 }
 
 TEST(json_mangling) {
@@ -431,9 +430,9 @@ TEST(json_mangling) {
 
         FOREACH_ELEMENT(i, cases) {
                 _cleanup_free_ char *ret = NULL;
-                assert_se(ret = table_mangle_to_json_field_name(i->arg));
+                ASSERT_NOT_NULL(ret = table_mangle_to_json_field_name(i->arg));
                 printf("\"%s\" -> \"%s\"\n", i->arg, ret);
-                assert_se(streq(ret, i->exp));
+                ASSERT_STREQ(ret, i->exp);
         }
 }
 
@@ -441,192 +440,192 @@ TEST(table) {
         _cleanup_(table_unrefp) Table *t = NULL;
         _cleanup_free_ char *formatted = NULL;
 
-        assert_se(t = table_new("one", "two", "three", "four"));
+        ASSERT_NOT_NULL(t = table_new("one", "two", "three", "four"));
 
-        assert_se(table_set_align_percent(t, TABLE_HEADER_CELL(3), 100) >= 0);
+        ASSERT_OK(table_set_align_percent(t, TABLE_HEADER_CELL(3), 100));
 
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_STRING, "xxx",
                                  TABLE_STRING, "yyy",
                                  TABLE_BOOLEAN, true,
-                                 TABLE_INT, -1) >= 0);
+                                 TABLE_INT, -1));
 
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_STRING, "a long field",
                                  TABLE_STRING, "yyy",
                                  TABLE_SET_UPPERCASE, 1,
                                  TABLE_BOOLEAN, false,
-                                 TABLE_INT, -999999) >= 0);
+                                 TABLE_INT, -999999));
 
-        assert_se(table_format(t, &formatted) >= 0);
+        ASSERT_OK(table_format(t, &formatted));
         printf("%s\n", formatted);
 
-        assert_se(streq(formatted,
-                        "ONE          TWO THREE    FOUR\n"
-                        "xxx          yyy yes        -1\n"
-                        "a long field YYY no    -999999\n"));
+        ASSERT_STREQ(formatted,
+                     "ONE          TWO THREE    FOUR\n"
+                     "xxx          yyy yes        -1\n"
+                     "a long field YYY no    -999999\n");
 
         formatted = mfree(formatted);
 
         table_set_width(t, 40);
 
-        assert_se(table_format(t, &formatted) >= 0);
+        ASSERT_OK(table_format(t, &formatted));
         printf("%s\n", formatted);
 
-        assert_se(streq(formatted,
-                        "ONE            TWO   THREE          FOUR\n"
-                        "xxx            yyy   yes              -1\n"
-                        "a long field   YYY   no          -999999\n"));
+        ASSERT_STREQ(formatted,
+                     "ONE            TWO   THREE          FOUR\n"
+                     "xxx            yyy   yes              -1\n"
+                     "a long field   YYY   no          -999999\n");
 
         formatted = mfree(formatted);
 
         table_set_width(t, 15);
-        assert_se(table_format(t, &formatted) >= 0);
+        ASSERT_OK(table_format(t, &formatted));
         printf("%s\n", formatted);
 
-        assert_se(streq(formatted,
-                        "ONE TWO TH… FO…\n"
-                        "xxx yyy yes  -1\n"
-                        "a … YYY no  -9…\n"));
+        ASSERT_STREQ(formatted,
+                     "ONE TWO TH… FO…\n"
+                     "xxx yyy yes  -1\n"
+                     "a … YYY no  -9…\n");
 
         formatted = mfree(formatted);
 
         table_set_width(t, 5);
-        assert_se(table_format(t, &formatted) >= 0);
+        ASSERT_OK(table_format(t, &formatted));
         printf("%s\n", formatted);
 
-        assert_se(streq(formatted,
-                        "… … … …\n"
-                        "… … … …\n"
-                        "… … … …\n"));
+        ASSERT_STREQ(formatted,
+                     "… … … …\n"
+                     "… … … …\n"
+                     "… … … …\n");
 
         formatted = mfree(formatted);
 
         table_set_width(t, 3);
-        assert_se(table_format(t, &formatted) >= 0);
+        ASSERT_OK(table_format(t, &formatted));
         printf("%s\n", formatted);
 
-        assert_se(streq(formatted,
-                        "… … … …\n"
-                        "… … … …\n"
-                        "… … … …\n"));
+        ASSERT_STREQ(formatted,
+                     "… … … …\n"
+                     "… … … …\n"
+                     "… … … …\n");
 
         formatted = mfree(formatted);
 
         table_set_width(t, SIZE_MAX);
-        assert_se(table_set_sort(t, (size_t) 0, (size_t) 2, SIZE_MAX) >= 0);
+        ASSERT_OK(table_set_sort(t, (size_t) 0, (size_t) 2, SIZE_MAX));
 
-        assert_se(table_format(t, &formatted) >= 0);
+        ASSERT_OK(table_format(t, &formatted));
         printf("%s\n", formatted);
 
-        assert_se(streq(formatted,
-                        "ONE          TWO THREE    FOUR\n"
-                        "a long field YYY no    -999999\n"
-                        "xxx          yyy yes        -1\n"));
+        ASSERT_STREQ(formatted,
+                     "ONE          TWO THREE    FOUR\n"
+                     "a long field YYY no    -999999\n"
+                     "xxx          yyy yes        -1\n");
 
         formatted = mfree(formatted);
 
         table_set_header(t, false);
 
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_STRING, "fäää",
                                  TABLE_STRING, "uuu",
                                  TABLE_BOOLEAN, true,
-                                 TABLE_INT, 42) >= 0);
+                                 TABLE_INT, 42));
 
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_STRING, "fäää",
                                  TABLE_STRING, "zzz",
                                  TABLE_BOOLEAN, false,
-                                 TABLE_INT, 0) >= 0);
+                                 TABLE_INT, 0));
 
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_EMPTY,
                                  TABLE_SIZE, (uint64_t) 4711,
                                  TABLE_TIMESPAN, (usec_t) 5*USEC_PER_MINUTE,
-                                 TABLE_INT64, (uint64_t) -123456789) >= 0);
+                                 TABLE_INT64, (uint64_t) -123456789));
 
-        assert_se(table_format(t, &formatted) >= 0);
+        ASSERT_OK(table_format(t, &formatted));
         printf("%s\n", formatted);
 
-        assert_se(streq(formatted,
-                        "a long field YYY  no      -999999\n"
-                        "fäää         zzz  no            0\n"
-                        "fäää         uuu  yes          42\n"
-                        "xxx          yyy  yes          -1\n"
-                        "             4.6K 5min -123456789\n"));
+        ASSERT_STREQ(formatted,
+                     "a long field YYY  no      -999999\n"
+                     "fäää         zzz  no            0\n"
+                     "fäää         uuu  yes          42\n"
+                     "xxx          yyy  yes          -1\n"
+                     "             4.6K 5min -123456789\n");
 
         formatted = mfree(formatted);
 
-        assert_se(table_set_display(t, (size_t) 2, (size_t) 0, (size_t) 2, (size_t) 0, (size_t) 0, SIZE_MAX) >= 0);
+        ASSERT_OK(table_set_display(t, (size_t) 2, (size_t) 0, (size_t) 2, (size_t) 0, (size_t) 0, SIZE_MAX));
 
-        assert_se(table_format(t, &formatted) >= 0);
+        ASSERT_OK(table_format(t, &formatted));
         printf("%s\n", formatted);
 
         if (isatty_safe(STDOUT_FILENO))
-                assert_se(streq(formatted,
-                                "no   a long f… no   a long f… a long fi…\n"
-                                "no   fäää      no   fäää      fäää\n"
-                                "yes  fäää      yes  fäää      fäää\n"
-                                "yes  xxx       yes  xxx       xxx\n"
-                                "5min           5min           \n"));
+                ASSERT_STREQ(formatted,
+                             "no   a long f… no   a long f… a long fi…\n"
+                             "no   fäää      no   fäää      fäää\n"
+                             "yes  fäää      yes  fäää      fäää\n"
+                             "yes  xxx       yes  xxx       xxx\n"
+                             "5min           5min           \n");
         else
-                assert_se(streq(formatted,
-                                "no   a long field no   a long field a long field\n"
-                                "no   fäää         no   fäää         fäää\n"
-                                "yes  fäää         yes  fäää         fäää\n"
-                                "yes  xxx          yes  xxx          xxx\n"
-                                "5min              5min              \n"));
+                ASSERT_STREQ(formatted,
+                             "no   a long field no   a long field a long field\n"
+                             "no   fäää         no   fäää         fäää\n"
+                             "yes  fäää         yes  fäää         fäää\n"
+                             "yes  xxx          yes  xxx          xxx\n"
+                             "5min              5min              \n");
 }
 
 TEST(vertical) {
         _cleanup_(table_unrefp) Table *t = NULL;
         _cleanup_free_ char *formatted = NULL;
 
-        assert_se(t = table_new_vertical());
+        ASSERT_NOT_NULL(t = table_new_vertical());
 
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_FIELD, "pfft aa", TABLE_STRING, "foo",
                                  TABLE_FIELD, "uuu o", TABLE_SIZE, UINT64_C(1024),
                                  TABLE_FIELD, "quux", TABLE_STRING, "asdf", TABLE_SET_JSON_FIELD_NAME, "custom-quux",
-                                 TABLE_FIELD, "lllllllllllo", TABLE_STRING, "jjjjjjjjjjjjjjjjj") >= 0);
+                                 TABLE_FIELD, "lllllllllllo", TABLE_STRING, "jjjjjjjjjjjjjjjjj"));
 
-        assert_se(table_set_json_field_name(t, 1, "dimpfelmoser") >= 0);
+        ASSERT_OK(table_set_json_field_name(t, 1, "dimpfelmoser"));
 
-        assert_se(table_format(t, &formatted) >= 0);
+        ASSERT_OK(table_format(t, &formatted));
 
-        assert_se(streq(formatted,
-                        "     pfft aa: foo\n"
-                        "       uuu o: 1K\n"
-                        "        quux: asdf\n"
-                        "lllllllllllo: jjjjjjjjjjjjjjjjj\n"));
+        ASSERT_STREQ(formatted,
+                     "     pfft aa: foo\n"
+                     "       uuu o: 1K\n"
+                     "        quux: asdf\n"
+                     "lllllllllllo: jjjjjjjjjjjjjjjjj\n");
 
         _cleanup_(sd_json_variant_unrefp) sd_json_variant *a = NULL, *b = NULL;
-        assert_se(table_to_json(t, &a) >= 0);
+        ASSERT_OK(table_to_json(t, &a));
 
-        assert_se(sd_json_build(&b, SD_JSON_BUILD_OBJECT(
+        ASSERT_OK(sd_json_build(&b, SD_JSON_BUILD_OBJECT(
                                              SD_JSON_BUILD_PAIR("pfft_aa", SD_JSON_BUILD_STRING("foo")),
                                              SD_JSON_BUILD_PAIR("dimpfelmoser", SD_JSON_BUILD_UNSIGNED(1024)),
                                              SD_JSON_BUILD_PAIR("custom-quux", SD_JSON_BUILD_STRING("asdf")),
-                                             SD_JSON_BUILD_PAIR("lllllllllllo", SD_JSON_BUILD_STRING("jjjjjjjjjjjjjjjjj")))) >= 0);
+                                             SD_JSON_BUILD_PAIR("lllllllllllo", SD_JSON_BUILD_STRING("jjjjjjjjjjjjjjjjj")))));
 
-        assert_se(sd_json_variant_equal(a, b));
+        ASSERT_TRUE(sd_json_variant_equal(a, b));
 }
 
 TEST(path_basename) {
         _cleanup_(table_unrefp) Table *t = NULL;
         _cleanup_free_ char *formatted = NULL;
 
-        assert_se(t = table_new("x"));
+        ASSERT_NOT_NULL(t = table_new("x"));
 
         table_set_header(t, false);
 
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_PATH_BASENAME, "/foo/bar",
                                  TABLE_PATH_BASENAME, "/quux/bar",
-                                 TABLE_PATH_BASENAME, "/foo/baz") >= 0);
+                                 TABLE_PATH_BASENAME, "/foo/baz"));
 
-        assert_se(table_format(t, &formatted) >= 0);
+        ASSERT_OK(table_format(t, &formatted));
 
         ASSERT_STREQ(formatted, "bar\nbar\nbaz\n");
 }
@@ -635,10 +634,10 @@ TEST(dup_cell) {
         _cleanup_(table_unrefp) Table *t = NULL;
         _cleanup_free_ char *formatted = NULL;
 
-        assert_se(t = table_new("foo", "bar", "x", "baz", ".", "%", "!", "~", "+"));
+        ASSERT_NOT_NULL(t = table_new("foo", "bar", "x", "baz", ".", "%", "!", "~", "+"));
         table_set_width(t, 75);
 
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_STRING, "hello",
                                  TABLE_UINT8, UINT8_C(42),
                                  TABLE_UINT16, UINT16_C(666),
@@ -647,38 +646,38 @@ TEST(dup_cell) {
                                  TABLE_PATH_BASENAME, "/foo/bar",
                                  TABLE_STRING, "aaa",
                                  TABLE_STRING, "bbb",
-                                 TABLE_STRING, "ccc") >= 0);
+                                 TABLE_STRING, "ccc"));
 
         /* Add the second row by duping cells */
         for (size_t i = 0; i < table_get_columns(t); i++)
-                assert_se(table_dup_cell(t, table_get_cell(t, 1, i)) >= 0);
+                ASSERT_OK(table_dup_cell(t, table_get_cell(t, 1, i)));
 
         /* Another row, but dupe the last three strings from the same cell */
-        assert_se(table_add_many(t,
+        ASSERT_OK(table_add_many(t,
                                  TABLE_STRING, "aaa",
                                  TABLE_UINT8, UINT8_C(0),
                                  TABLE_UINT16, UINT16_C(65535),
                                  TABLE_UINT32, UINT32_C(4294967295),
                                  TABLE_PERCENT, 100,
-                                 TABLE_PATH_BASENAME, "../") >= 0);
+                                 TABLE_PATH_BASENAME, "../"));
 
         for (size_t i = 6; i < table_get_columns(t); i++)
-                assert_se(table_dup_cell(t, table_get_cell(t, 2, 0)) >= 0);
+                ASSERT_OK(table_dup_cell(t, table_get_cell(t, 2, 0)));
 
-        assert_se(table_format(t, &formatted) >= 0);
+        ASSERT_OK(table_format(t, &formatted));
         printf("%s\n", formatted);
-        assert_se(streq(formatted,
-                        "FOO     BAR   X       BAZ          .      %      !        ~        +\n"
-                        "hello   42    666     253          0%     bar    aaa      bbb      ccc\n"
-                        "hello   42    666     253          0%     bar    aaa      bbb      ccc\n"
-                        "aaa     0     65535   4294967295   100%   ../    hello    hello    hello\n"));
+        ASSERT_STREQ(formatted,
+                     "FOO     BAR   X       BAZ          .      %      !        ~        +\n"
+                     "hello   42    666     253          0%     bar    aaa      bbb      ccc\n"
+                     "hello   42    666     253          0%     bar    aaa      bbb      ccc\n"
+                     "aaa     0     65535   4294967295   100%   ../    hello    hello    hello\n");
 }
 
 TEST(table_bps) {
         _cleanup_(table_unrefp) Table *table = NULL;
         _cleanup_free_ char *formatted = NULL;
 
-        assert_se(table = table_new("uint64", "size", "bps"));
+        ASSERT_NOT_NULL(table = table_new("uint64", "size", "bps"));
         uint64_t v;
         FOREACH_ARGUMENT(v,
                          2500,
@@ -688,30 +687,29 @@ TEST(table_bps) {
                          1000000000,
                          2000000000,
                          2500000000)
-                assert_se(table_add_many(table,
+                ASSERT_OK(table_add_many(table,
                                          TABLE_UINT64, v,
                                          TABLE_SIZE, v,
-                                         TABLE_BPS, v) >= 0);
+                                         TABLE_BPS, v));
 
         table_set_width(table, 50);
-        assert_se(table_format(table, &formatted) >= 0);
+        ASSERT_OK(table_format(table, &formatted));
 
         printf("%s", formatted);
-        assert_se(streq(formatted,
-                        "UINT64             SIZE           BPS\n"
-                        "2500               2.4K           2.5Kbps\n"
-                        "10000000           9.5M           10Mbps\n"
-                        "20000000           19M            20Mbps\n"
-                        "25000000           23.8M          25Mbps\n"
-                        "1000000000         953.6M         1Gbps\n"
-                        "2000000000         1.8G           2Gbps\n"
-                        "2500000000         2.3G           2.5Gbps\n"
-                  ));
+        ASSERT_STREQ(formatted,
+                     "UINT64             SIZE           BPS\n"
+                     "2500               2.4K           2.5Kbps\n"
+                     "10000000           9.5M           10Mbps\n"
+                     "20000000           19M            20Mbps\n"
+                     "25000000           23.8M          25Mbps\n"
+                     "1000000000         953.6M         1Gbps\n"
+                     "2000000000         1.8G           2Gbps\n"
+                     "2500000000         2.3G           2.5Gbps\n");
 }
 
 static int intro(void) {
-        assert_se(setenv("SYSTEMD_COLORS", "0", 1) >= 0);
-        assert_se(setenv("COLUMNS", "40", 1) >= 0);
+        ASSERT_OK(setenv("SYSTEMD_COLORS", "0", 1));
+        ASSERT_OK(setenv("COLUMNS", "40", 1));
         return EXIT_SUCCESS;
 }