teknoraver / rpms / systemd

Forked from rpms/systemd 2 months ago
Clone
Blob Blame History Raw
From 37b398e2a3d6cd225a3121843540a033f4415b80 Mon Sep 17 00:00:00 2001
From: Frantisek Sumsal <frantisek@sumsal.cz>
Date: Fri, 17 Jun 2022 17:29:22 +0200
Subject: [PATCH] test: check journal directly instead of capturing console
 output

(cherry picked from commit 8cda7b91ff267fc21325d4886980e243389a7566)

Related: #2087652
---
 test/TEST-52-HONORFIRSTSHUTDOWN/test.sh | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/test/TEST-52-HONORFIRSTSHUTDOWN/test.sh b/test/TEST-52-HONORFIRSTSHUTDOWN/test.sh
index 936e801c51..6a1ec9ca29 100755
--- a/test/TEST-52-HONORFIRSTSHUTDOWN/test.sh
+++ b/test/TEST-52-HONORFIRSTSHUTDOWN/test.sh
@@ -14,15 +14,10 @@ TEST_NO_QEMU=1
 # could turn into a reboot if the test fails.
 NSPAWN_TIMEOUT=60
 
-# Remove this file if it exists. This is used along with
-# the make target "finish". Since concrete confirmation is
-# only found from the console during the poweroff.
-rm -f /tmp/honorfirstshutdown.log >/dev/null
-
 check_result_nspawn_hook() {
-    grep -q "Shutdown is already active. Skipping emergency action request" /tmp/honorfirstshutdown.log
+    local workspace="${1:?}"
+
+    "${JOURNALCTL:?}" -D "${workspace:?}/var/log/journal" --grep "Shutdown is already active. Skipping emergency action request" --no-pager
 }
 
-# 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)
+do_test "$@"