gffranco
a489bea480
User reported the editor-keymaps CI job hung for 30 minutes before "Has been cancelled" — twice in a row. Two changes to keep that from happening again: 1. Wrap both harness invocations with `timeout` (60s Neovim, 30s Vim). A misbehaving LSP attach, a stuck `vim.defer_fn`, or ex-mode hung on stdin can no longer hold the runner indefinitely; `timeout` kills the process and the wrapper exits non-zero so CI shows the real failure shape. 2. `</dev/null` on `vim -e -s` so ex-mode doesn't sit waiting for input on CI's non-TTY stdin (the local zsh run inherits the user's tty, masking this). 3. CI: the keymaps job now has `needs: test` and `timeout-minutes: 10`. `needs: test` serialises behind the cargo jobs so we don't fight for the single Gitea runner, and the workflow-level timeout is a belt to the script-level `timeout`'s braces — if for some reason both fail the same way, the runner reclaims itself in 10 minutes instead of 30. Local timing: `test-keymaps.sh` 10.4s, `test-keymaps-vim.sh` 0.1s. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
63 lines
1.8 KiB
Bash
Executable File
63 lines
1.8 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
#
|
|
# scripts/test-keymaps-vim.sh — Vim counterpart of test-keymaps.sh.
|
|
#
|
|
# Pure-VimL portion only. The LSP-roundtrip keymaps (`<C-Space>`,
|
|
# `=`, `-`, …) talk to the server via vim-lsp's timer-driven async
|
|
# layer, which doesn't fire reliably inside `vim -e -s`. Those have
|
|
# Neovim coverage in scripts/test-keymaps.lua, exercising the same
|
|
# Lua command layer + server binary.
|
|
|
|
set -euo pipefail
|
|
|
|
REPO_ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
|
|
TMP="$(mktemp -d -t nuwiki-keymap-vim-test-XXXXXX)"
|
|
trap 'rm -rf "$TMP"' EXIT
|
|
|
|
log() { printf '\033[1;34m[keymap-vim]\033[0m %s\n' "$*"; }
|
|
|
|
if ! command -v vim >/dev/null 2>&1; then
|
|
log 'vim not installed — skipping'
|
|
exit 0
|
|
fi
|
|
|
|
# Don't need the LSP binary for the pure-VimL cases, but make sure
|
|
# the autoload file `expand('<sfile>')` resolves correctly when the
|
|
# script-load-time plugin_root capture runs.
|
|
|
|
mkdir -p "$TMP/wiki"
|
|
SEED="$TMP/wiki/index.wiki"
|
|
echo "= seed =" > "$SEED"
|
|
|
|
VIMRC="$TMP/vimrc"
|
|
cat > "$VIMRC" <<EOF
|
|
set nocompatible
|
|
let &runtimepath = '$REPO_ROOT' . ',' . &runtimepath
|
|
filetype plugin indent on
|
|
syntax enable
|
|
" ftdetect/nuwiki.vim already maps *.wiki → vimwiki via \`set filetype\`.
|
|
EOF
|
|
|
|
RESULTS="$TMP/results.txt"
|
|
|
|
log "running harness…"
|
|
# `</dev/null` so Vim's ex-mode doesn't hang waiting on stdin;
|
|
# `timeout` so a misbehaving script can't stall CI for 30 minutes.
|
|
NUWIKI_KEYMAP_RESULTS="$RESULTS" \
|
|
timeout 30 vim -e -s -u "$VIMRC" -c "edit $SEED" -c "source $REPO_ROOT/scripts/test-keymaps-vim.vim" \
|
|
</dev/null >"$TMP/vim.log" 2>&1 || true
|
|
|
|
if [[ ! -f "$RESULTS" ]]; then
|
|
echo 'keymap test harness produced no output' >&2
|
|
echo '--- vim log ---' >&2
|
|
cat "$TMP/vim.log" >&2 || true
|
|
exit 1
|
|
fi
|
|
|
|
cat "$RESULTS"
|
|
|
|
if grep -qE '^SUMMARY: [0-9]+ passed, 0 failed' "$RESULTS"; then
|
|
exit 0
|
|
fi
|
|
exit 1
|