David Heinemeier HanssonandClaude Fable 5
83c6dd230c
The update conflict tests stub sudo and pacman, but omarchy-update-pacman now puts systemd-run between them, so on a systemd-booted host the tests would reach for the real system manager. Stub systemd-run to drop the wrapper's options and run the command, and cover the helper's own invocation composition in a new test. Raised by codex review. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015zcqENR1UbhuwC1v5u3Wop
40 lines
1.2 KiB
Bash
Executable File
40 lines
1.2 KiB
Bash
Executable File
#!/bin/bash
|
|
|
|
set -euo pipefail
|
|
|
|
source "$(dirname "$0")/base-test.sh"
|
|
|
|
test_tmp=$(mktemp -d)
|
|
trap 'rm -rf "$test_tmp"' EXIT
|
|
|
|
stub_bin="$test_tmp/bin"
|
|
mkdir -p "$stub_bin"
|
|
|
|
cat >"$stub_bin/sudo" <<'STUB'
|
|
#!/bin/bash
|
|
printf '%s\n' "$*" >"$SUDO_CALL_LOG"
|
|
STUB
|
|
chmod +x "$stub_bin/sudo"
|
|
|
|
run_helper() {
|
|
PATH="$stub_bin:$PATH" SUDO_CALL_LOG="$test_tmp/call" "$ROOT/bin/omarchy-update-pacman" "$@"
|
|
}
|
|
|
|
# The scope wrapper only applies on a systemd-booted host, so expect what the
|
|
# helper's own booted check would decide for this machine.
|
|
if [[ -d /run/systemd/system && ! -L /run/systemd/system ]]; then
|
|
expected_scope="systemd-run --scope --quiet --collect "
|
|
else
|
|
expected_scope=""
|
|
fi
|
|
|
|
run_helper -Syu --noconfirm
|
|
[[ $(cat "$test_tmp/call") == "env OMARCHY_UPDATE_PACMAN=1 ${expected_scope}pacman -Syu --noconfirm" ]] ||
|
|
fail "helper composes the guarded pacman invocation" "$(cat "$test_tmp/call")"
|
|
pass "helper composes the guarded pacman invocation"
|
|
|
|
LC_ALL=C run_helper -Syu
|
|
[[ $(cat "$test_tmp/call") == "env OMARCHY_UPDATE_PACMAN=1 LC_ALL=C ${expected_scope}pacman -Syu" ]] ||
|
|
fail "helper forwards LC_ALL to the transaction" "$(cat "$test_tmp/call")"
|
|
pass "helper forwards LC_ALL to the transaction"
|