]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
test: make the variable names slightly more descriptive 30587/head
authorFrantisek Sumsal <frantisek@sumsal.cz>
Fri, 22 Dec 2023 13:16:56 +0000 (14:16 +0100)
committerFrantisek Sumsal <frantisek@sumsal.cz>
Sat, 23 Dec 2023 15:37:30 +0000 (16:37 +0100)
Follow-up for 5ca8d2474ca8b8b3f42999fb2f6e5e1498b22aa9.

test/units/testsuite-74.run.sh

index 69907d4f385ca3cb1d3609a38253961fda16d239..38cbca54ce3447ab486b0a4c2a652caa055a0558 100755 (executable)
@@ -213,19 +213,18 @@ for opt in nice on-{active,boot,calendar,startup,unit-active,unit-inactive} prop
 done
 
 # Let's make sure that ProtectProc= properly moves submounts of the original /proc over to the new proc
-
-A=$(cat /proc/sys/kernel/random/boot_id)
-B=$(systemd-run -q --wait --pipe -p ProtectProc=invisible cat /proc/sys/kernel/random/boot_id)
-assert_eq "$A" "$B"
-
-V="/tmp/version.$RANDOM"
-A="$(cat /proc/version).piff"
-echo "$A" > "$V"
-mount --bind "$V" /proc/version
-
-B=$(systemd-run -q --wait --pipe -p ProtectProc=invisible cat /proc/version)
-
-assert_eq "$A" "$B"
+BOOT_ID="$(</proc/sys/kernel/random/boot_id)"
+UNIT_BOOT_ID="$(systemd-run -q --wait --pipe -p ProtectProc=invisible cat /proc/sys/kernel/random/boot_id)"
+assert_eq "$BOOT_ID" "$UNIT_BOOT_ID"
+
+TMP_KVER="/tmp/version.$RANDOM"
+KVER="$(</proc/version).piff"
+echo "$KVER" >"$TMP_KVER"
+mount --bind "$TMP_KVER" /proc/version
+UNIT_KVER="$(systemd-run -q --wait --pipe -p ProtectProc=invisible cat /proc/version)"
+assert_eq "$KVER" "$UNIT_KVER"
+umount /proc/version
+rm -f "$TMP_KVER"
 
 # Check that invoking the tool under the uid0 alias name works
 uid0 ls /
@@ -239,6 +238,3 @@ done
 # Let's chain a couple of uid0 calls together, for fun
 readarray -t cmdline < <(printf "%.0suid0\n" {0..31})
 assert_eq "$("${cmdline[@]}" bash -c 'echo $SUDO_USER')" "$USER"
-
-umount /proc/version
-rm "$V"