src/shared/bus-print-properties.c

@@ -166,7 +166,6 @@ static int bus_print_property(const char *name, const char *expected_value, sd_b
                         if (!result)
                                 return -EINVAL;
 
-                        if (all || !isempty(result))
                         bus_print_property_value(name, expected_value, value, result);
 
                 } else if (STR_IN_SET(name, "CapabilityBoundingSet", "AmbientCapabilities")) {
@@ -176,7 +175,6 @@ static int bus_print_property(const char *name, const char *expected_value, sd_b
                         if (r < 0)
                                 return r;
 
-                        if (all || !isempty(s))
                         bus_print_property_value(name, expected_value, value, s);
 
                 } else if ((STR_IN_SET(name, "CPUWeight", "StartupCPUWeight", "IOWeight", "StartupIOWeight") && u == CGROUP_WEIGHT_INVALID) ||

src/systemctl/systemctl-show.c

@@ -1425,7 +1425,6 @@ static int print_property(const char *name, const char *expected_value, sd_bus_m
                         if (n < 0)
                                 return log_oom();
 
-                        if (all || !isempty(h))
                         bus_print_property_value(name, expected_value, value, h);
 
                         return 1;
@@ -1627,7 +1626,6 @@ static int print_property(const char *name, const char *expected_value, sd_bus_m
                         if (!affinity)
                                 return log_oom();
 
-                        if (all || !isempty(affinity))
                         bus_print_property_value(name, expected_value, value, affinity);
 
                         return 1;

test/TEST-01-BASIC/test.sh

@@ -16,4 +16,4 @@ test_append_files() {
     cp -v "$TEST_UNITS_DIR"/{testsuite-01,end}.service "$TEST_UNITS_DIR/testsuite.target" "$dst"
 }
 
-do_test "$@"
+do_test "$@" 01

test/TEST-02-UNITTESTS/test.sh

@@ -71,4 +71,4 @@ check_result_qemu() {
     return $ret
 }
 
-do_test "$@"
+do_test "$@" 02

test/TEST-03-JOBS/test.sh

@@ -8,4 +8,4 @@ IMAGE_NAME="default"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 03

test/TEST-04-JOURNAL/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="Journal-related tests"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 04

test/TEST-05-RLIMITS/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="Resource limits-related tests"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 05

test/TEST-06-SELINUX/test.sh

@@ -54,4 +54,4 @@ test_append_files() {
     )
 }
 
-do_test "$@"
+do_test "$@" 06

test/TEST-07-ISSUE-1981/test.sh

@@ -9,4 +9,4 @@ TEST_NO_QEMU=1
 
 NSPAWN_TIMEOUT=30
 
-do_test "$@"
+do_test "$@" 07

test/TEST-08-ISSUE-2730/test.sh

@@ -12,4 +12,4 @@ QEMU_TIMEOUT=300
 FSTYPE=ext4
 TEST_FORCE_NEWIMAGE=1
 
-do_test "$@"
+do_test "$@" 08

test/TEST-09-ISSUE-2691/test.sh

@@ -9,4 +9,4 @@ TEST_NO_NSPAWN=1
 
 QEMU_TIMEOUT=300
 
-do_test "$@"
+do_test "$@" 09

test/TEST-10-ISSUE-2467/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="https://github.com/systemd/systemd/issues/2467"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 10

test/TEST-11-ISSUE-3166/test.sh

@@ -7,4 +7,4 @@ TEST_NO_NSPAWN=1
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 11

test/TEST-12-ISSUE-3171/test.sh

@@ -7,4 +7,4 @@ TEST_NO_QEMU=1
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 12

test/TEST-13-NSPAWN-SMOKE/test.sh

@@ -17,4 +17,4 @@ test_append_files() {
     )
 }
 
-do_test "$@"
+do_test "$@" 13

test/TEST-14-MACHINE-ID/test.sh

@@ -12,4 +12,4 @@ test_append_files() {
     printf "556f48e837bc4424a710fa2e2c9d3e3c\ne3d\n" >"${1:?}/etc/machine-id"
 }
 
-do_test "$@"
+do_test "$@" 14

test/TEST-15-DROPIN/test.sh

@@ -7,4 +7,4 @@ TEST_NO_QEMU=1
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 15

test/TEST-16-EXTEND-TIMEOUT/test.sh

@@ -8,4 +8,4 @@ TEST_NO_QEMU=1
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 16

test/TEST-17-UDEV/test.sh

@@ -17,4 +17,4 @@ test_append_files() {
     )
 }
 
-do_test "$@"
+do_test "$@" 17

test/TEST-18-FAILUREACTION/test.sh

@@ -8,4 +8,4 @@ TEST_DESCRIPTION="FailureAction= operation"
 
 QEMU_TIMEOUT=600
 
-do_test "$@"
+do_test "$@" 18

test/TEST-19-DELEGATE/test.sh

@@ -10,4 +10,4 @@ TEST_NO_NSPAWN=1
 QEMU_TIMEOUT=600
 UNIFIED_CGROUP_HIERARCHY=yes
 
-do_test "$@"
+do_test "$@" 19

test/TEST-20-MAINPIDGAMES/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test changing main PID"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 20

test/TEST-22-TMPFILES/test.sh

@@ -18,4 +18,4 @@ test_append_files() {
     fi
 }
 
-do_test "$@"
+do_test "$@" 22

test/TEST-23-TYPE-EXEC/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test Type=exec"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 23

test/TEST-24-CRYPTSETUP/test.sh

@@ -86,4 +86,4 @@ test_setup_cleanup() {
     cleanup_initdir
 }
 
-do_test "$@"
+do_test "$@" 24

test/TEST-25-IMPORT/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test importd"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 25

test/TEST-26-SETENV/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test setenv"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 26

test/TEST-27-STDOUTFILE/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test StandardOutput=file:"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 27

test/TEST-28-PERCENTJ-WANTEDBY/test.sh

@@ -7,4 +7,4 @@ RUN_IN_UNPRIVILEGED_CONTAINER=yes
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 28

test/TEST-29-PORTABLE/test.sh

@@ -27,4 +27,4 @@ test_append_files() {
     )
 }
 
-do_test "$@"
+do_test "$@" 29

test/TEST-30-ONCLOCKCHANGE/test.sh

@@ -7,4 +7,4 @@ TEST_NO_NSPAWN=1
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 30

test/TEST-31-DEVICE-ENUMERATION/test.sh

@@ -9,4 +9,4 @@ TEST_NO_NSPAWN=1
 
 QEMU_TIMEOUT=300
 
-do_test "$@"
+do_test "$@" 31

test/TEST-32-OOMPOLICY/test.sh

@@ -9,4 +9,4 @@ TEST_NO_NSPAWN=1
 
 UNIFIED_CGROUP_HIERARCHY=yes
 
-do_test "$@"
+do_test "$@" 32

test/TEST-33-CLEAN-UNIT/test.sh

@@ -8,4 +8,4 @@ TEST_DESCRIPTION="test CleanUnit"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 33

test/TEST-34-DYNAMICUSERMIGRATE/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test migrating state directory from DynamicUser=1 to DynamicUs
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 34

test/TEST-36-NUMAPOLICY/test.sh

@@ -13,4 +13,4 @@ else
     QEMU_OPTIONS="-numa node,nodeid=0"
 fi
 
-do_test "$@"
+do_test "$@" 36

test/TEST-37-RUNTIMEDIRECTORYPRESERVE/test.sh

@@ -8,4 +8,4 @@ TEST_DESCRIPTION="test RuntimeDirectoryPreserve=yes"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 37

test/TEST-38-FREEZER/test.sh

@@ -7,4 +7,4 @@ TEST_NO_NSPAWN=1
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 38

test/TEST-39-EXECRELOAD/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="Test ExecReload= (PR #13098)"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 39

test/TEST-40-EXEC-COMMAND-EX/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test ExecXYZEx= service unit dbus hookups"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 40

test/TEST-41-ONESHOT-RESTART/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="Test oneshot unit restart on failure"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 41

test/TEST-42-EXECSTOPPOST/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test that ExecStopPost= is always run"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 42

test/TEST-43-PRIVATEUSER-UNPRIV/test.sh

@@ -8,4 +8,4 @@ TEST_DESCRIPTION="Test PrivateUsers=yes on user manager"
 
 has_user_dbus_socket || exit 0
 
-do_test "$@"
+do_test "$@" 43

test/TEST-44-LOG-NAMESPACE/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test log namespaces"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 44

test/TEST-46-HOMED/test.sh

@@ -7,4 +7,4 @@ TEST_NO_QEMU=1
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 46

test/TEST-47-ISSUE-14566/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="Test that KillMode=mixed does not leave left over processes wi
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 47

test/TEST-48-START-STOP-NO-RELOAD/test.sh

@@ -8,4 +8,4 @@ TEST_DESCRIPTION="test StartStopNoReload"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 48

test/TEST-49-RUNTIME-BIND-PATHS/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test adding new BindPaths while unit is already running"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 49

test/TEST-50-DISSECT/test.sh

@@ -30,4 +30,4 @@ test_append_files() {
     )
 }
 
-do_test "$@"
+do_test "$@" 50

test/TEST-51-ISSUE-16115/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="Test ExecCondition= does not restart on abnormal or failure"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 51

test/TEST-52-HONORFIRSTSHUTDOWN/Makefile

@@ -1 +0,0 @@
-../TEST-01-BASIC/Makefile

test/TEST-52-HONORFIRSTSHUTDOWN/Makefile

@@ -0,0 +1,14 @@
+all setup run clean clean-again:
+	@TEST_BASE_DIR=../ ./test.sh --$@
+
+# finish option is used to run checks that can only be run outside of
+# the test execution. Example case, honor first shutdown, proof is obtained
+# from the console output as the image shuts down. This does not show up in
+# the journal so the output from the do_test is captured in a file in /tmp.
+# Without the use of finish the test will still pass because if it fails
+# the test will loop and will be terminated via a command timeout.
+# This just provides concrete confirmation.
+finish:
+	@TEST_BASE_DIR=../ ./fini.sh --$@
+
+.PHONY: all setup run clean clean-again finish

test/TEST-52-HONORFIRSTSHUTDOWN/fini.sh

@@ -0,0 +1,10 @@
+#!/bin/bash
+TEST_DESCRIPTION="test honor first shutdown"
+
+if grep -q "Shutdown is already active. Skipping emergency action request" /tmp/honorfirstshutdown.log; then
+    echo "$TEST_DESCRIPTION [pass]"
+    exit 0
+else
+    echo "$TEST_DESCRIPTION [fail]"
+    exit 1
+fi

test/TEST-52-HONORFIRSTSHUTDOWN/test.sh

@@ -18,10 +18,4 @@ NSPAWN_TIMEOUT=20
 # only found from the console during the poweroff.
 rm -f /tmp/honorfirstshutdown.log >/dev/null
 
-check_result_nspawn_hook() {
+do_test "$@" 52 >/tmp/honorfirstshutdown.log
-    grep -q "Shutdown is already active. Skipping emergency action request" /tmp/honorfirstshutdown.log
-}
-
-# Note: don't use a pipe in the following expression, as it breaks the trap
-#       handlers we have defined in test/test-functions.
-do_test "$@" > >(tee /tmp/honorfirstshutdown.log)

test/TEST-53-ISSUE-16347/test.sh

@@ -9,4 +9,4 @@ QEMU_OPTIONS="-rtc base=$(date -u +%Y-%m-%dT%H:%M:%S -d '+3 days')"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 53

test/TEST-54-CREDS/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test credentials"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 54

test/TEST-56-EXIT-TYPE/test.sh

@@ -6,4 +6,4 @@ TEST_DESCRIPTION="test ExitType=cgroup"
 # shellcheck source=test/test-functions
 . "${TEST_BASE_DIR:?}/test-functions"
 
-do_test "$@"
+do_test "$@" 56

test/TEST-58-REPART/test.sh

@@ -4,4 +4,4 @@ TEST_DESCRIPTION="test systemd-repart"
 TEST_NO_NSPAWN=1
 . $TEST_BASE_DIR/test-functions
 
-do_test "$@"
+do_test "$@" 58

test/TEST-59-RELOADING-RESTART/test.sh

@@ -6,4 +6,4 @@ TEST_NO_QEMU=1
 
 . $TEST_BASE_DIR/test-functions
 
-do_test "$@"
+do_test "$@" 59

test/test-functions

@@ -57,7 +57,7 @@ fi
 
 # The calling test.sh scripts have TEST_BASE_DIR set via their Makefile, but we don't need them to provide it
 TEST_BASE_DIR=${TEST_BASE_DIR:-$(realpath "$(dirname "${BASH_SOURCE[0]}")")}
-TEST_UNITS_DIR="$(realpath "$TEST_BASE_DIR/units")"
+TEST_UNITS_DIR="$TEST_BASE_DIR/units"
 SOURCE_DIR=$(realpath "$TEST_BASE_DIR/..")
 TOOLS_DIR="$SOURCE_DIR/tools"
 # These variables are used by test scripts
@@ -91,17 +91,6 @@ STATEFILE="$STATEDIR/.testdir"
 IMAGESTATEDIR="$STATEDIR/.."
 TESTLOG="$STATEDIR/test.log"
 
-if ! [[ "$TESTNAME" =~ ^TEST\-([0-9]+)\-.+$ ]]; then
-    echo "ERROR: Test name '$TESTNAME' is not in the expected format: TEST-[0-9]+-*" >&2
-    exit 1
-fi
-TESTID="${BASH_REMATCH[1]:?}"
-
-if [[ ! -f "$TEST_UNITS_DIR/testsuite-$TESTID.service" ]]; then
-    echo "ERROR: Test '$TESTNAME' is missing its service file '$TEST_UNITS_DIR/testsuite-$TESTID.service" >&2
-    exit 1
-fi
-
 BASICTOOLS=(
     awk
     basename
@@ -1174,14 +1163,6 @@ check_result_nspawn() {
     check_result_common "${workspace}"
     ret=$?
 
-    # Run additional test-specific checks if defined by check_result_nspawn_hook()
-    if declare -F check_result_nspawn_hook >/dev/null; then
-        if ! check_result_nspawn_hook; then
-            derror "check_result_nspawn_hook() returned with EC > 0"
-            ret=4
-        fi
-    fi
-
     _umount_dir "${initdir:?}"
 
     return $ret
@@ -1197,14 +1178,6 @@ check_result_qemu() {
 
     _umount_dir "${initdir:?}"
 
-    # Run additional test-specific checks if defined by check_result_qemu_hook()
-    if declare -F check_result_qemu_hook >/dev/null; then
-        if ! check_result_qemu_hook; then
-            derror "check_result_qemu_hook() returned with EC > 0"
-            ret=4
-        fi
-    fi
-
     return $ret
 }
 
@@ -2471,7 +2444,7 @@ do_test() {
         case $1 in
             --run)
                 echo "${testname} RUN: $TEST_DESCRIPTION"
-                test_run "$TESTID"
+                test_run "$2"
                 ret=$?
                 if [ $ret -eq 0 ]; then
                     echo "${testname} RUN: $TEST_DESCRIPTION [OK]"
@@ -2502,7 +2475,7 @@ do_test() {
                     test_setup_cleanup </dev/null >>"$TESTLOG" 2>&1 || ret=$?
                 fi
                 if [ $ret -eq 0 ]; then
-                    test_run "$TESTID" </dev/null >>"$TESTLOG" 2>&1 || ret=$?
+                    test_run "$2" </dev/null >>"$TESTLOG" 2>&1 || ret=$?
                 fi
                 test_cleanup
                 if [ $ret -eq 0 ]; then

test/testsuite-52.units/testsuite-52.service

@@ -2,5 +2,5 @@
 Description=Testsuite service
 
 [Service]
-ExecStart=/usr/lib/systemd/tests/testdata/units/%N.sh
+ExecStart=/usr/lib/systemd/tests/testdata/%N.units/%N.sh
 Type=oneshot

test/testsuite-52.units/testsuite-52.sh

@@ -2,6 +2,11 @@
 set -ex
 set -o pipefail
 
+if ! test -x /usr/lib/systemd/tests/testdata/units/test-honor-first-shutdown.sh ; then
+        echo "honor-first-shutdown script not found - FAIL" >/testok
+        exit 0
+fi
+
 systemd-analyze log-level debug
 systemd-analyze log-target console
 

test/units/test-honor-first-shutdown.service

@@ -3,7 +3,7 @@ Description=Honor First Shutdown feature
 After=multi-user.target
 
 [Service]
-ExecStart=/usr/lib/systemd/tests/testdata/testsuite-52.units/%N.sh
+ExecStart=/usr/lib/systemd/tests/testdata/units/%N.sh
 ExecStop=sh -c 'kill -KILL $MAINPID'
 FailureAction=reboot