test: correctly mask supporting services in tests, take #2

Due to a little misunderstanding the last patch doesn't work as
expected, since test_create_image() is called only for the first image
(usually TEST-01-BASIC), and all subsequent images are then (possibly)
modified with test_append_files().

Follow-up to 179ca4d2b1.
This commit is contained in:
Frantisek Sumsal 2021-06-24 12:13:52 +02:00 committed by Luca Boccassi
parent 86e24d608a
commit eb70d9450c
2 changed files with 7 additions and 13 deletions

View file

@ -9,18 +9,6 @@ TEST_REQUIRE_INSTALL_TESTS=0
# shellcheck source=test/test-functions
. "${TEST_BASE_DIR:?}/test-functions"
# Explicitly override the default test_create_image() function to avoid the
# call to mask_supporting_services(), since we want to run them in TEST-01-BASIC
test_create_image() {
create_empty_image_rootdir
# Create what will eventually be our root filesystem onto an overlay
(
LOG_LEVEL=5
setup_basic_environment
)
}
test_append_files() {
# install tests manually so the test is functional even when -Dinstall-tests=false
local dst="${1:?}/usr/lib/systemd/tests/testdata/units/"

View file

@ -2416,7 +2416,6 @@ test_create_image() {
(
LOG_LEVEL=5
setup_basic_environment
mask_supporting_services
)
}
@ -2467,6 +2466,13 @@ test_setup() {
fi
mount_initdir
# We want to test all services in TEST-01-BASIC, but mask them in
# all other tests
if [[ "${TESTID:?}" != "01" ]]; then
dinfo "Masking supporting services"
mask_supporting_services
fi
if [ "$hook_defined" -eq 0 ]; then
test_append_files "${initdir:?}"
fi