rootiest/color-tool
1
1
Fork 0
Code Issues Pull Requests Actions Releases 6 Activity

fix(tests): redirect stdin from /dev/null in uninstall test to prevent hang
Test PR / test (pull_request) Successful in 6s
Details
Auto Label PRs / label (pull_request) Successful in 3s
Details
Release on Merge / release (pull_request) Successful in 7s
Details

Browse Source 
--uninstall prompts for config removal when stdin is a TTY. Running the
test from an interactive terminal caused it to block on read. Feeding
/dev/null forces the non-interactive path without changing behaviour.
This commit is contained in:
Rootiest
2026-05-09 01:00:31 -04:00
parent ad7a1f15b3
commit 1a27dbde4e
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/run_tests.sh
+1 -1
View File
@@ -259,7 +259,7 @@ fi
# 16. Uninstall — removes installed files, config is kept in non-interactive mode # 16. Uninstall — removes installed files, config is kept in non-interactive mode
it "uninstalls installed files (keeps config)" it "uninstalls installed files (keeps config)"
uninstall_out=$("$COLOR_TOOL" --uninstall 2>&1) uninstall_out=$("$COLOR_TOOL" --uninstall </dev/null 2>&1)
if [[ ! -e "$HOME/.local/bin/color-tool" ]] && if [[ ! -e "$HOME/.local/bin/color-tool" ]] &&
[[ ! -L "$HOME/.local/bin/color-tool" ]] && [[ ! -L "$HOME/.local/bin/color-tool" ]] &&
[[ ! -d "$XDG_DATA_HOME/color-tool" ]] && [[ ! -d "$XDG_DATA_HOME/color-tool" ]] &&