]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
test: use set -eux and set -o pipefail everywhere
authorFrantisek Sumsal <frantisek@sumsal.cz>
Fri, 9 Apr 2021 17:39:41 +0000 (19:39 +0200)
committerFrantisek Sumsal <frantisek@sumsal.cz>
Tue, 13 Apr 2021 10:08:01 +0000 (12:08 +0200)
This should make the scripts more robust.

57 files changed:
test/units/testsuite-02.sh
test/units/testsuite-03.sh
test/units/testsuite-04.sh
test/units/testsuite-05.sh
test/units/testsuite-06.sh
test/units/testsuite-07.sh
test/units/testsuite-11.sh
test/units/testsuite-12.sh
test/units/testsuite-13.sh
test/units/testsuite-14.sh
test/units/testsuite-15.sh
test/units/testsuite-16.sh
test/units/testsuite-17.sh
test/units/testsuite-18.sh
test/units/testsuite-19.sh
test/units/testsuite-20.sh
test/units/testsuite-22.01.sh
test/units/testsuite-22.02.sh
test/units/testsuite-22.03.sh
test/units/testsuite-22.04.sh
test/units/testsuite-22.05.sh
test/units/testsuite-22.06.sh
test/units/testsuite-22.07.sh
test/units/testsuite-22.08.sh
test/units/testsuite-22.09.sh
test/units/testsuite-22.10.sh
test/units/testsuite-22.sh
test/units/testsuite-23.sh
test/units/testsuite-25.sh
test/units/testsuite-26.sh
test/units/testsuite-27.sh
test/units/testsuite-29.sh
test/units/testsuite-30.sh
test/units/testsuite-31.sh
test/units/testsuite-32.sh
test/units/testsuite-33.sh
test/units/testsuite-34.sh
test/units/testsuite-36.sh
test/units/testsuite-37.sh
test/units/testsuite-38.sh
test/units/testsuite-39.sh
test/units/testsuite-40.sh
test/units/testsuite-41.sh
test/units/testsuite-42.sh
test/units/testsuite-43.sh
test/units/testsuite-44.sh
test/units/testsuite-46.sh
test/units/testsuite-47.sh
test/units/testsuite-48.sh
test/units/testsuite-49.sh
test/units/testsuite-50.sh
test/units/testsuite-51.sh
test/units/testsuite-53.sh
test/units/testsuite-54.sh
test/units/testsuite-55-slowgrowth.sh
test/units/testsuite-55.sh
test/units/testsuite-56.sh

index 1ff1c3347b2c04c7b78591a3fd58b76bf7733151..b0807c9d8aaaf71098b0bd8bd6ef2cc3cc9ea460 100755 (executable)
@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
-#set -ex
-#set -o pipefail
+set -eux
+set -o pipefail
 
 NPROC=$(nproc)
 MAX_QUEUE_SIZE=${NPROC:-2}
index 5a2dadd6b83d73e8759f63c0253717aae2de1ecf..82858f0c8e7f8826cbc655a33025754d4b780499 100755 (executable)
@@ -1,5 +1,6 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
+set -o pipefail
 
 # Test merging of a --job-mode=ignore-dependencies job into a previously
 # installed job.
index a2a1292380818f67c726472a1e325856778eed83..0969a8313e8295bfbce5782bb53f8f5ce56469e5 100755 (executable)
@@ -1,6 +1,5 @@
 #!/usr/bin/env bash
-set -x
-set -e
+set -eux
 set -o pipefail
 
 # Test stdout stream
index 9168e7279972cbc8efe408d21da842994e14d605..f899fcaa4f628bfc72c9b0848d58009331983646 100755 (executable)
@@ -1,6 +1,5 @@
 #!/usr/bin/env bash
-set -x
-set -e
+set -eux
 set -o pipefail
 
 P=/run/systemd/system.conf.d
index f9b106da7bdeaaa23f12d8982ca07cd117082c93..f89698ded4171e4ae1b0a45c12b028cce38ce73a 100755 (executable)
@@ -1,6 +1,5 @@
 #!/usr/bin/env bash
-set -x
-set -e
+set -eux
 set -o pipefail
 
 echo 1 >/sys/fs/selinux/enforce || {
index fbb2d1d30a17a813f6ad68f9962bdf25dbc8fa3c..c63394e68cdb45341b49448391a6395684b92d2b 100755 (executable)
@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
-set -x
-set -e
+set -eux
+set -o pipefail
 
 >/failed
 
index 708c7cebb7dea6458206fb8dbbd3543c958b07a3..3ec391f6e651581fcb61de3f7f24c8de33a7b967 100755 (executable)
@@ -1,5 +1,6 @@
 #!/usr/bin/env bash
-set -x
+set -eux
+set -o pipefail
 
 systemctl start fail-on-restart.service
 active_state=$(systemctl show --value --property ActiveState fail-on-restart.service)
index 7e7b0ec25a0f9566989f0023204a66376c0bec17..a02f38c488d090161e0f4f9f1dfd15b1b7913ec2 100755 (executable)
@@ -1,6 +1,5 @@
 #!/usr/bin/env bash
-set -x
-set -e
+set -eux
 set -o pipefail
 
 U=/run/systemd/system/test12.socket
index 3e66aa9d547db6ad3bac36e890c6b6fcf770703c..03b1bc39712145583112d038ee7d173e12bfdab1 100755 (executable)
@@ -1,7 +1,6 @@
 #!/usr/bin/env bash
-set -x
-set -e
-set -u
+# shellcheck disable=SC2016
+set -eux
 set -o pipefail
 
 export SYSTEMD_LOG_LEVEL=debug
index 95ac9b65aee779cf684724183a8f2e1049abeed5..6eecf916a205fd12d2e0140436eb98b87363ad9a 100755 (executable)
@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
-set -e
-set -x
+set -eux
+set -o pipefail
 
 function setup_root {
     local _root="$1"
index fc392c584dcff186f5ce90c7d51000a9e22ac857..018ae8aab9b2957f9f5fc7721cda0e542a96ef47 100755 (executable)
@@ -1,6 +1,6 @@
-#! /bin/bash
-set -e
-set -x
+#!/bin/bash
+set -eux
+set -o pipefail
 
 _clear_service () {
     systemctl stop $1.service 2>/dev/null || :
index 8388ef7a5d2e3682032235dda5d4e9fc280cdec7..8d2117b4cf973d325b34a4168bc372e5d8c3e9bf 100755 (executable)
@@ -1,5 +1,6 @@
 #!/usr/bin/env bash
-set -v -x
+set -eux
+set -o pipefail
 
 rm -f /test.log
 
index afce85aceb9bcf11cc3f671538ef4fdf93daf3f1..3cf57550fc3528747f8f4492d829558cbc613156 100755 (executable)
@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
-set -x
-set -e
+set -eux
+set -o pipefail
 
 >/failed
 
index c4e3ca211081aeadcbc03e601b4fd40b2e5c3883..fe4f9ce5e0eda055d8fb4f5519a2bb204dac2adc 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 systemd-run --wait -p FailureAction=poweroff true
index dea2f4ba1f7f834863ff75197fe443895bad0360..8ea89e6cfe8d5ef09f68c99cf9166813e9646aa7 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 if grep -q cgroup2 /proc/filesystems ; then
index 6003e4f1813be4b865df6efd62be25d9d5b1d196..56f07933a91676241c0aec2505f632b9166e72f8 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 systemd-analyze log-level debug
index b10b5aaa1aa7c0ce8b999c37475960628ba241ca..7975980799611bf3167f8810b8ae5631db0fa41b 100755 (executable)
@@ -1,10 +1,11 @@
-#! /bin/bash
+#!/bin/bash
 #
 # With "e" don't attempt to set permissions when file doesn't exist, see
 # https://github.com/systemd/systemd/pull/6682.
 #
 
-set -e
+set -eux
+set -o pipefail
 
 rm -fr /tmp/test
 
index c337cd6e05b829125d595e0cfa1b137091380a86..5d1254db03ea893a54d1ec46a8f963fcf4684862 100755 (executable)
@@ -1,10 +1,10 @@
-#! /bin/bash
+#!/bin/bash
 #
 # Basic tests for types creating directories
 #
 
-set -e
-set -x
+set -eux
+set -o pipefail
 
 rm -fr /tmp/{C,d,D,e}
 mkdir  /tmp/{C,d,D,e}
index c905784bee4a4ead748177ff1647d86279b2861a..bb321c1c2f8968cfdea2cefa1f8d28541da630d4 100755 (executable)
@@ -1,10 +1,10 @@
-#! /bin/bash
+#!/bin/bash
 #
 # Basic tests for types creating/writing files
 #
 
-set -e
-set -x
+set -eux
+set -o pipefail
 
 rm -fr /tmp/{f,F,w}
 mkdir  /tmp/{f,F,w}
index fc90ab4dc7fcce8464d226d12a6e16bf6afe612b..3298aa04ed909684d5a764464fbe7b5a6ff302a0 100755 (executable)
@@ -1,10 +1,10 @@
-#! /bin/bash
+#!/bin/bash
 #
 # Basic tests for types creating fifos
 #
 
-set -e
-set -x
+set -eux
+set -o pipefail
 
 rm -fr /tmp/p
 mkdir  /tmp/p
index 13c4ac80fc4f15c1ab1b5f214a573467519e5f84..a1820089c01211ae9ac0e57c7172f67b1bbef6d5 100755 (executable)
@@ -1,7 +1,7 @@
 #! /bin/bash
 
-set -e
-set -x
+set -eux
+set -o pipefail
 
 rm -fr /tmp/{z,Z}
 mkdir  /tmp/{z,Z}
index 8ce0a08f68a8f442c5844a20495383b68c6c578e..ce5b1bf69813d0bdf1727d7ffbbe2654b884eb26 100755 (executable)
@@ -1,9 +1,10 @@
-#! /bin/bash
+#!/bin/bash
 #
 # Inspired by https://github.com/systemd/systemd/issues/9508
 #
 
-set -e
+set -eux
+set -o pipefail
 
 test_snippet() {
         systemd-tmpfiles "$@" - <<EOF
index 74286a33057b2668ae9f274743ed823e7cb0780f..8d61032d277bf7d3866d3d69f4b59c7f152fb713 100755 (executable)
@@ -1,10 +1,10 @@
-#! /bin/bash
+#!/bin/bash
 #
 # Verifies the issues described by https://github.com/systemd/systemd/issues/10191
 #
 
-set -e
-set -x
+set -eux
+set -o pipefail
 
 rm -rf /tmp/test-prefix
 
index 01d3bdb50cfbe72b46222966f9287ed1bbfa4ccb..e4272f8ea3ae7e4de75f9deb088ff38088bb8316 100755 (executable)
@@ -1,4 +1,4 @@
-#! /bin/bash
+#!/bin/bash
 #
 # Verify tmpfiles can run in a root directory under a path prefix that contains
 # directories owned by unprivileged users, for example when a root file system
@@ -7,7 +7,8 @@
 # https://github.com/systemd/systemd/pull/11820
 #
 
-set -e
+set -eux
+set -o pipefail
 
 rm -fr /tmp/root /tmp/user
 mkdir -p /tmp/root /tmp/user/root
index c558dfd4db370855b5fd5d3c66e31bad16f86421..c852f778efaa1e8dc8982a58a9f5924f66d17fc2 100755 (executable)
@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
-set -e
-set -x
+set -eux
+set -o pipefail
 
 # Make sure that the "stat" output is not locale dependent.
 export LANG=C LC_ALL=C
index 5ec297fbf87c7f90ea43bbdd6eb192bdd3a96790..d650b2145c20b61342a5fc3831bb4554a4eed08d 100755 (executable)
@@ -1,7 +1,6 @@
 #!/usr/bin/env bash
 
-set -e
-set -x
+set -eux
 set -o pipefail
 
 systemd-tmpfiles --create - <<EOF
index afce85aceb9bcf11cc3f671538ef4fdf93daf3f1..3cf57550fc3528747f8f4492d829558cbc613156 100755 (executable)
@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 
-set -x
-set -e
+set -eux
+set -o pipefail
 
 >/failed
 
index a21523756230184e2507a0d2c49d2ff3bb572325..c66ed26229682bf6590b107401b78459a8081179 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 systemd-analyze log-level debug
index d4d2e0f9532b30a53e05a346a04f68e4f2e0a023..05448efb0ac77453ed4cf2c5a16c469ed42a82a8 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 export SYSTEMD_PAGER=cat
index 9c1abc5916186b83639dd96ca26158c68174d3ad..79820993074eccbe6cf118640479c0e748c2e8e6 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 # Make sure PATH is set
index 0e9ffe118927aa2c7ed6554123e907e9febc29d3..2248380351a7a99ad88ddb59264c3972718207b7 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 systemd-analyze log-level debug
index 4e91aa73363ed2ff776c035e281e547f2a8c728c..3408e6d71acb86167fae93e97caf84667035a920 100755 (executable)
@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 # -*- mode: shell-script; indent-tabs-mode: nil; sh-basic-offset: 4; -*-
 # ex: ts=8 sw=4 sts=4 et filetype=sh
-set -ex
+set -eux
 set -o pipefail
 
 export SYSTEMD_LOG_LEVEL=debug
index cb3b46d2cbea58432c711a0f20ef9199008a2849..ac8a3e06cf561863847e9f843f1ba318e277487b 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 systemd-analyze log-level debug
index 42873bfd833a88166bdcb84f667ae13ea7b73057..a9a50e1470b02a6764b1d6c1ccb79e284c1f3858 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -e
+set -eux
 set -o pipefail
 
 if journalctl -b -t systemd --grep '\.device: Changed plugged -> dead'; then
index c1704ab34e31f20d9c531ae292eddae987dca22f..d7827090bbfda2b40919f7aa6e0687038e299303 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 # Let's run this test only if the "memory.oom.group" cgroupfs attribute
index 929c76062be8f1d6fd2bbd5e56301dbf6e94eb3c..6e750f63a3817a2f1b2a70cf817ecf4bfaaabd5a 100755 (executable)
@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 # -*- mode: shell-script; indent-tabs-mode: nil; sh-basic-offset: 4; -*-
 # ex: ts=8 sw=4 sts=4 et filetype=sh
-set -ex
+set -eux
 set -o pipefail
 
 cat >/etc/systemd/system/testservice.service <<EOF
index ca54a01f403894defed71ba567db2f187f469b66..394b220d60a31f02d5b4b6f6b9b1d9d6cd6c6a9c 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 systemd-analyze log-level debug
index 3decf4b3f33ddc84aee5ea027d3b721f2ea0dfca..46c08017d1be8dbeece6843db7a4e38e2413b6e6 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 at_exit() {
index 3ae0cb32fe166084805706ef53f98dd790fa3d6a..c3eb5344d1163808005b2bf46de9d1c1d8e261c7 100755 (executable)
@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 # -*- mode: shell-script; indent-tabs-mode: nil; sh-basic-offset: 4; -*-
 # ex: ts=8 sw=4 sts=4 et filetype=sh
-set -ex
+set -eux
 set -o pipefail
 
 systemd-mount -p RuntimeDirectory=hoge -p RuntimeDirectoryPreserve=yes -t tmpfs tmpfs /tmp/aaa
index 707fd8a86466105a59c2de360c1ea9a4c8f903df..baf52fd4991af3ef626122c9c7bc8d2b60ce49c7 100755 (executable)
@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
 
-set -ex
+set -eux
 set -o pipefail
 
 systemd-analyze log-level debug
index 5c0680878f352c2329dce336eeb066f58ccb7ded..90da2bf5e18aea02cd40b3eeff2cfd08832922c9 100755 (executable)
@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
 
-set -ex
+set -eux
 set -o pipefail
 
 systemd-analyze log-level debug
index 7593401369eef8d07b47021bf995a407b9032646..fdb052c39ca7a48ba121fea5fd5ac73d58fb2423 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 systemd-analyze log-level debug
index 0a43f8ba57412a0ed908f93c2c500ec72c449c49..527213fcf8c72826631c999bb9512b851ff09d25 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 # wait this many secs for each test service to succeed in what is being tested
index ca13c5dec5cab036070318298af3db78e9e98300..e47fab46e6e3cef2c73705a30fe7c61baba54479 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 
 systemd-analyze log-level debug
 
index f3c0ca4fd223dccc9827f4557a1a28cc88f0f0b1..dcbe32def6194cb74d19141b2babf00acc344987 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 systemd-analyze log-level debug
index 0badb7112591ec322c92349280fce5814e3df789..50b5c3046f2c9e6bc9e6aadb8fa304554f826452 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 
 systemd-analyze log-level debug
 
index d3f0f710434f54314155e8c1e27893580ce45ce2..686c7f5bc9bd24349615ec6644a20324da6c2bcc 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 # Check if homectl is installed, and if it isn't bail out early instead of failing
index f8529083a2b23fdfcc7359ac157d218dc261063b..c1714d14d0e39a9b91291a0bae801f1ac9e2b5dd 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 systemd-analyze log-level debug
index ea84b1d72325d25ae214eb67651dcacebd3fb76a..0ce8e9c067b637f19839e50dcc733f6cd6eea8e7 100755 (executable)
@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 # -*- mode: shell-script; indent-tabs-mode: nil; sh-basic-offset: 4; -*-
 # ex: ts=8 sw=4 sts=4 et filetype=sh
-set -ex
+set -eux
 
 cat >/run/systemd/system/testservice-48.target <<EOF
 [Unit]
index f92280f884bedabca0dbaeb22caf512af91f8f95..8e360b4951c4900ef0b28f2e2fdcbdebd9d0ae3b 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 
 echo "MARKER_FIXED" >/run/testservice-49-fixed
 mkdir -p /run/inaccessible
index b3c120d9a6491d9324ff478d89b0880db28b3685..8067997a9dfcbd2202623725e75d6a91b5708b20 100755 (executable)
@@ -1,7 +1,7 @@
 #!/usr/bin/env bash
 # -*- mode: shell-script; indent-tabs-mode: nil; sh-basic-offset: 4; -*-
 # ex: ts=8 sw=4 sts=4 et filetype=sh
-set -ex
+set -eux
 set -o pipefail
 
 export SYSTEMD_LOG_LEVEL=debug
index 246412a07986bbd31fff1ff405d9114637307820..06bc1602f874bfe69e487dea615f810fc460f976 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 systemctl start testsuite-51-repro-1
index 3536c2427125ac4b7e931aac9d6bbcb29dda0e23..298d5d50eb7b3e04119119e6432a6bd168eb86d6 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 >/failed
index d145d7e33ec498628fa08d2d4758ea3b39fca54e..f8ddff4a30567f437ceeec6ee7805a7560c4d74f 100755 (executable)
@@ -1,5 +1,6 @@
 #!/usr/bin/env bash
-set -ex
+# shellcheck disable=SC2016
+set -eux
 
 systemd-analyze log-level debug
 
index ff5a74734827e494b5bb8a3b17ff7a642b1d9c55..ed4ec6d6e4a59dfbf6147952363c9a1b7b919498 100755 (executable)
@@ -1,5 +1,8 @@
 #!/usr/bin/env bash
-set -eu -o pipefail
+# Don't use set -x here, since it generates a lot of output and slows
+# the script down, causing unexpected test fails.
+set -eu
+set -o pipefail
 
 PAGE_SIZE=$(getconf PAGE_SIZE)
 BLOAT_ITERATION_TARGET=$(( 100 << 20 )) # 100 MB
index d73f4b1e7ae21b76e9a5000e345c1116a61b6bf4..25c7f8cc63a460e355784a201dd3b058791b50b2 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 set -o pipefail
 
 systemd-analyze log-level debug
index 079da072015a666c9e20c90332c77a22d3c6e511..27bd3ca8f73ca2c170546f5a2fd0ae7fd9adfa57 100755 (executable)
@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -ex
+set -eux
 
 systemd-analyze log-level debug