test: when skipping tests, always print something

It is quite confusing if the test "fails" without printing anything at all.

A typo in an 'if' statement is also fixed.

diff --git a/src/libsystemd-network/test-dhcp-server.c b/src/libsystemd-network/test-dhcp-server.c
index 815b11e9974a5cb0b859c10efa5d477c4fb1f3d8..3de25b02c25a584e177e40f00f27c0f8d82332c2 100644 (file)
--- a/src/libsystemd-network/test-dhcp-server.c
+++ b/src/libsystemd-network/test-dhcp-server.c
@@ -54,9 +54,8 @@ static int test_basic(sd_event *event) {
         test_pool(&address_lo, 1, 0);
 
         r = sd_dhcp_server_start(server);
-
         if (r == -EPERM)
-                return EXIT_TEST_SKIP;
+                return log_info_errno(r, "sd_dhcp_server_start failed: %m");
         assert_se(r >= 0);
 
         assert_se(sd_dhcp_server_start(server) == -EBUSY);
@@ -236,8 +235,10 @@ int main(int argc, char *argv[]) {
         assert_se(sd_event_new(&e) >= 0);
 
         r = test_basic(e);
-        if (r != 0)
-                return r;
+        if (r != 0) {
+                log_notice("%s: skipping tests.", program_invocation_short_name);
+                return EXIT_TEST_SKIP;
+        }
 
         test_message_handler();
         test_client_id_hash();

diff --git a/src/test/test-boot-timestamps.c b/src/test/test-boot-timestamps.c
index ef39304b9fb7a7c590554225988f7c49fbdc434c..e7deee5d5fafbd466e7d3a497381e37ff55e260b 100644 (file)
--- a/src/test/test-boot-timestamps.c
+++ b/src/test/test-boot-timestamps.c
@@ -91,5 +91,10 @@ int main(int argc, char* argv[]) {
         r = test_boot_timestamps();
         assert(r >= 0);
 
-        return (p > 0 || q > 0 || r >> 0) ? EXIT_SUCCESS : EXIT_TEST_SKIP;
+        bool any = p > 0 || q > 0 || r > 0;
+        if (!any)
+                log_notice("%s: access to firmware variable not possible, skipping tests.",
+                           program_invocation_short_name);
+
+        return any ? EXIT_SUCCESS : EXIT_TEST_SKIP;
 }

diff --git a/src/test/test-capability.c b/src/test/test-capability.c
index af6d808b6dabcb5b13e67345defee1b72eaa1d0e..5e2d21bee9c441d520d20f644d97ae9bd5b14c47 100644 (file)
--- a/src/test/test-capability.c
+++ b/src/test/test-capability.c
@@ -91,10 +91,9 @@ static int setup_tests(bool *run_ambient) {
         int r;
 
         nobody = getpwnam(NOBODY_USER_NAME);
-        if (!nobody) {
-                log_error_errno(errno, "Could not find nobody user: %m");
-                return -EXIT_TEST_SKIP;
-        }
+        if (!nobody)
+                return log_error_errno(errno, "Could not find nobody user: %m");
+
         test_uid = nobody->pw_uid;
         test_gid = nobody->pw_gid;
 
@@ -229,12 +228,16 @@ int main(int argc, char *argv[]) {
 
         log_info("have ambient caps: %s", yes_no(ambient_capabilities_supported()));
 
-        if (getuid() != 0)
+        if (getuid() != 0) {
+                log_notice("%s: not root, skipping tests.", program_invocation_short_name);
                 return EXIT_TEST_SKIP;
+        }
 
         r = setup_tests(&run_ambient);
-        if (r < 0)
-                return -r;
+        if (r < 0) {
+                log_notice("%s: skipping tests.", program_invocation_short_name);
+                return EXIT_TEST_SKIP;
+        }
 
         show_capabilities();