28dcbae376
* Restore preinstalls from the menu, and drop the Omacom apps with them Remove Preinstalls missed omacut, omacalc, and omawrite, so the three Omacom apps survived an opt-out that was supposed to clear the desk. Opting out was also one-way. Install > Preinstalls now puts everything back: the shipped .desktop launchers and mise stubs via omarchy-refresh-applications, the dropped packages via pacman, and the opt-out marker deleted so the preinstalled keybindings return on reload. The two menu entries guard on the marker, so exactly one of them is ever visible. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * Trim the preinstall lists to what quattro actually ships Remove Preinstalls was still dropping typora, spotify, 1password, 1password-cli, signal-desktop, opencode, claude-code, and github-cli. None of those are in omarchy-base.packages anymore: typora gave way to omawrite, the services moved to on-demand menu installs, and the agent CLIs are mise-managed. Removing them took out apps the user had deliberately installed, and restoring them would have put back what we no longer ship. Both lists are now the same twelve packages, all of them in omarchy-base.packages. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> * Keep the opt-out marker when a restore fails omarchy-pkg-add exits non-zero when pacman cannot install a package, but the restore ran straight past it, cleared the marker, and reloaded Hyprland. That reported success and brought back keybindings for apps that never arrived. The marker now falls last, behind a check on the transaction. The new test also pins the two lists to each other and to omarchy-base.packages, which is the drift that let retired packages linger in the removal list. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
92 lines
3.2 KiB
Bash
Executable File
92 lines
3.2 KiB
Bash
Executable File
#!/bin/bash
|
|
|
|
set -euo pipefail
|
|
|
|
source "$(cd -- "$(dirname -- "${BASH_SOURCE[0]}")" && pwd)/base-test.sh"
|
|
|
|
test_tmp=$(mktemp -d)
|
|
trap 'rm -rf "$test_tmp"' EXIT
|
|
|
|
mock_bin="$test_tmp/bin"
|
|
test_home="$test_tmp/home"
|
|
marker="$test_home/.local/state/omarchy/preinstalls-removed"
|
|
pkg_log="$test_tmp/packages"
|
|
mkdir -p "$mock_bin" "$test_home/.local/state/omarchy"
|
|
|
|
for command in omarchy-webapp-remove-all omarchy-tui-remove-all omarchy-refresh-applications hyprctl; do
|
|
printf '#!/bin/bash\nexit 0\n' >"$mock_bin/$command"
|
|
done
|
|
|
|
cat >"$mock_bin/gum" <<'SH'
|
|
#!/bin/bash
|
|
[[ $1 == confirm ]] && exit "${OMARCHY_TEST_CONFIRM:-0}"
|
|
exit 0
|
|
SH
|
|
|
|
cat >"$mock_bin/omarchy-pkg-add" <<'SH'
|
|
#!/bin/bash
|
|
printf '%s\n' "$@" >"$OMARCHY_TEST_PKG_LOG"
|
|
exit "${OMARCHY_TEST_PKG_ADD_STATUS:-0}"
|
|
SH
|
|
|
|
cat >"$mock_bin/omarchy-pkg-drop" <<'SH'
|
|
#!/bin/bash
|
|
printf '%s\n' "$@" >"$OMARCHY_TEST_PKG_LOG"
|
|
SH
|
|
|
|
chmod +x "$mock_bin"/*
|
|
|
|
export PATH="$mock_bin:$PATH"
|
|
export HOME="$test_home"
|
|
export OMARCHY_TEST_PKG_LOG="$pkg_log"
|
|
|
|
# Both scripts restore and remove the same set, and every package in it has to be
|
|
# one Omarchy actually ships, or Remove Preinstalls takes out an app the user
|
|
# chose from the menu and Install Preinstalls puts back one we retired.
|
|
mapfile -t shipped < <(sed -e 's/[[:space:]]*#.*$//' -e '/^[[:space:]]*$/d' "$ROOT/install/omarchy-base.packages")
|
|
|
|
"$ROOT/bin/omarchy-install-preinstalls" >/dev/null
|
|
mapfile -t restored <"$pkg_log"
|
|
|
|
"$ROOT/bin/omarchy-remove-preinstalls" >/dev/null
|
|
mapfile -t dropped <"$pkg_log"
|
|
|
|
[[ ${restored[*]} == "${dropped[*]}" ]] ||
|
|
fail "Install and Remove Preinstalls cover the same packages" \
|
|
"restored: ${restored[*]}
|
|
dropped: ${dropped[*]}"
|
|
pass "Install and Remove Preinstalls cover the same packages"
|
|
|
|
for package in "${restored[@]}"; do
|
|
printf '%s\n' "${shipped[@]}" | grep -qxF "$package" ||
|
|
fail "every preinstall is shipped in omarchy-base.packages" "$package is not shipped"
|
|
done
|
|
pass "every preinstall is shipped in omarchy-base.packages"
|
|
|
|
for package in omacut omacalc omawrite; do
|
|
printf '%s\n' "${restored[@]}" | grep -qxF "$package" ||
|
|
fail "preinstalls cover the Omacom apps" "$package is missing"
|
|
done
|
|
pass "preinstalls cover the Omacom apps"
|
|
|
|
# The bindings key off the marker, so clearing it before the packages land would
|
|
# point them at apps that never came back.
|
|
touch "$marker"
|
|
OMARCHY_TEST_PKG_ADD_STATUS=1 "$ROOT/bin/omarchy-install-preinstalls" >/dev/null && status=0 || status=$?
|
|
(( status == 1 )) || fail "restore reports a failed package transaction" "exit status was $status"
|
|
[[ -f $marker ]] || fail "restore keeps the opt-out marker when packages fail to install"
|
|
pass "restore keeps the opt-out marker when packages fail to install"
|
|
|
|
"$ROOT/bin/omarchy-install-preinstalls" >/dev/null
|
|
[[ ! -e $marker ]] || fail "restore clears the opt-out marker once the packages are back"
|
|
pass "restore clears the opt-out marker once the packages are back"
|
|
|
|
rm -f "$marker"
|
|
OMARCHY_TEST_CONFIRM=1 "$ROOT/bin/omarchy-remove-preinstalls" >/dev/null
|
|
[[ ! -e $marker ]] || fail "declining Remove Preinstalls changes nothing"
|
|
pass "declining Remove Preinstalls changes nothing"
|
|
|
|
"$ROOT/bin/omarchy-remove-preinstalls" >/dev/null
|
|
[[ -f $marker ]] || fail "Remove Preinstalls records the opt-out"
|
|
pass "Remove Preinstalls records the opt-out"
|