tiennm99/ccs
1
0
Fork 0
mirror of https://github.com/tiennm99/ccs.git synced 2026-09-02 08:19:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(uninstall): update tests for per-profile hook behavior

Browse Source 
- verify uninstall does NOT touch global settings.json

- verify migration marker is cleaned up

- verify user hooks in global settings preserved
This commit is contained in:
kaitranntt
2026-01-25 20:26:40 -05:00
parent ba1fb7eeb3
commit ce59eb6269
1 changed files with 22 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/native/unix/uninstall.sh
+22 -35
View File
@@ -282,34 +282,15 @@ test_case "Edge case: Missing parent directories" "Handles missing .claude direc
" "
# ============================================================================ # ============================================================================
# TEST SECTION 7: SETTINGS.JSON HOOK CLEANUP # TEST SECTION 7: PER-PROFILE HOOK BEHAVIOR (Global settings untouched)
# ============================================================================ # ============================================================================
echo "" echo ""
echo "========================================" echo "========================================"
echo "SECTION 7: SETTINGS.JSON HOOK CLEANUP" echo "SECTION 7: PER-PROFILE HOOK BEHAVIOR"
echo "========================================" echo "========================================"
test_case "Hook cleanup: Removes CCS WebSearch hook from settings.json" "Hook removed" bash -c " test_case "Per-profile: Uninstall does NOT touch global settings.json" "Global settings unchanged" bash -c "
# Setup: Create settings.json with CCS hook
mkdir -p '$TEST_CLAUDE_DIR'
cat > '$TEST_CLAUDE_DIR/settings.json' << 'EOFINNER'
{
\"hooks\": {
\"PreToolUse\": [
{ \"matcher\": \"WebSearch\", \"hooks\": [{ \"command\": \"node ~/.ccs/hooks/websearch-transformer.cjs\" }] }
]
}
}
EOFINNER
# Install then uninstall
HOME='$TEST_HOME' '$CCS_PATH' --install > /dev/null 2>&1
HOME='$TEST_HOME' '$CCS_PATH' --uninstall > /dev/null 2>&1
# Check hook removed
! grep -q 'websearch-transformer' '$TEST_CLAUDE_DIR/settings.json'
"
test_case "Hook cleanup: Preserves user-defined WebSearch hooks" "User hooks kept" bash -c "
# Setup: Create settings.json with user hook # Setup: Create settings.json with user hook
mkdir -p '$TEST_CLAUDE_DIR' mkdir -p '$TEST_CLAUDE_DIR'
cat > '$TEST_CLAUDE_DIR/settings.json' << 'EOFINNER' cat > '$TEST_CLAUDE_DIR/settings.json' << 'EOFINNER'
@@ -323,35 +304,41 @@ test_case "Hook cleanup: Preserves user-defined WebSearch hooks" "User hooks kep
EOFINNER EOFINNER
# Uninstall # Uninstall
HOME='$TEST_HOME' '$CCS_PATH' --uninstall > /dev/null 2>&1 HOME='$TEST_HOME' '$CCS_PATH' --uninstall > /dev/null 2>&1
# Check user hook preserved # Verify global settings unchanged
grep -q 'my-custom-hook' '$TEST_CLAUDE_DIR/settings.json' grep -q 'my-custom-hook' '$TEST_CLAUDE_DIR/settings.json'
" "
test_case "Hook cleanup: Handles mixed CCS and user hooks" "Only CCS hook removed" bash -c " test_case "Per-profile: Uninstall preserves user hooks in global settings" "User hooks in global preserved" bash -c "
# Setup: Create settings.json with both CCS and user hooks # Setup: Create settings.json with CCS hook (old format - shouldn't be touched)
mkdir -p '$TEST_CLAUDE_DIR' mkdir -p '$TEST_CLAUDE_DIR'
cat > '$TEST_CLAUDE_DIR/settings.json' << 'EOFINNER' cat > '$TEST_CLAUDE_DIR/settings.json' << 'EOFINNER'
{ {
\"hooks\": { \"hooks\": {
\"PreToolUse\": [ \"PreToolUse\": [
{ \"matcher\": \"WebSearch\", \"hooks\": [{ \"command\": \"node ~/.ccs/hooks/websearch-transformer.cjs\" }] }, { \"matcher\": \"WebSearch\", \"hooks\": [{ \"command\": \"node ~/.ccs/hooks/websearch-transformer.cjs\" }] },
{ \"matcher\": \"WebSearch\", \"hooks\": [{ \"command\": \"my-custom-hook.sh\" }] }, { \"matcher\": \"WebSearch\", \"hooks\": [{ \"command\": \"my-custom-hook.sh\" }] }
{ \"matcher\": \"Bash\", \"hooks\": [{ \"command\": \"bash-hook.sh\" }] }
] ]
} }
} }
EOFINNER EOFINNER
# Uninstall # Uninstall - should NOT modify global settings
HOME='$TEST_HOME' '$CCS_PATH' --uninstall > /dev/null 2>&1 HOME='$TEST_HOME' '$CCS_PATH' --uninstall > /dev/null 2>&1
# Check CCS hook removed # Both hooks should still be in global settings (not touched)
! grep -q 'websearch-transformer' '$TEST_CLAUDE_DIR/settings.json' && grep -q 'websearch-transformer' '$TEST_CLAUDE_DIR/settings.json' &&
# Check user WebSearch hook preserved grep -q 'my-custom-hook' '$TEST_CLAUDE_DIR/settings.json'
grep -q 'my-custom-hook' '$TEST_CLAUDE_DIR/settings.json' &&
# Check other hooks preserved
grep -q 'bash-hook' '$TEST_CLAUDE_DIR/settings.json'
" "
test_case "Hook cleanup: Handles missing settings.json" "No error when settings.json missing" bash -c " test_case "Per-profile: Migration marker cleanup on uninstall" "Marker file removed" bash -c "
# Setup: Create ~/.ccs/.hook-migrated marker
mkdir -p '$TEST_HOME/.ccs'
echo '2026-01-25T00:00:00.000Z' > '$TEST_HOME/.ccs/.hook-migrated'
# Uninstall
HOME='$TEST_HOME' '$CCS_PATH' --uninstall > /dev/null 2>&1
# Verify marker removed
[[ ! -f '$TEST_HOME/.ccs/.hook-migrated' ]]
"
test_case "Per-profile: Handles missing settings.json" "No error when settings.json missing" bash -c "
# Ensure settings.json doesn't exist # Ensure settings.json doesn't exist
rm -f '$TEST_CLAUDE_DIR/settings.json' rm -f '$TEST_CLAUDE_DIR/settings.json'
HOME='$TEST_HOME' '$CCS_PATH' --uninstall > /dev/null 2>&1 HOME='$TEST_HOME' '$CCS_PATH' --uninstall > /dev/null 2>&1