pFad - Phone/Frame/Anonymizer/Declutterfier! Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

URL: http://github.com/tarides/ocaml-devcontainer/commit/52d2ed2f61b1ebeb470afeb20156c0354686bd7f

Install neovim and add vim/neovim test coverage · tarides/ocaml-devcontainer@52d2ed2 · GitHub
Skip to content

Commit 52d2ed2

Browse files
cuihtlauacclaude
andcommitted
Install neovim and add vim/neovim test coverage
Neovim was configured (init.lua) but not installed. Add the neovim package to base/Dockerfile, harden the neovim test to fail if missing, and add a new test-vim.sh for vim installation and config verification. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent d679619 commit 52d2ed2

File tree

4 files changed

+53
-3
lines changed

4 files changed

+53
-3
lines changed

CLAUDE.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@ See [google/sanitizers#1716](https://github.com/google/sanitizers/issues/1716).
4949
./test/test-profiling.sh # landmarks, memtrace, olly, bisect_ppx
5050
./test/test-dune-pkg.sh # Dune package management workflow
5151
./test/test-vscode.sh # VS Code devcontainer integration
52+
./test/test-vim.sh # Vim installation + config
5253
./test/test-neovim.sh # Neovim exec pathway + LSP
5354
./test/test-emacs.sh # Emacs TRAMP + eglot integration
5455
./test/test-claude.sh # Claude Code installation

base/Dockerfile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
3838
unzip \
3939
# Editors
4040
vim \
41+
neovim \
4142
emacs-nox \
4243
# Debugging & profiling tools
4344
gdb \

test/test-neovim.sh

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,8 @@ if command -v nvim &> /dev/null; then
1717
echo "Neovim: $VERSION"
1818
echo "PASS: Neovim available"
1919
else
20-
echo "WARN: Neovim not installed in container"
21-
echo "Neovim can be installed or used via devcontainer exec from host"
22-
exit 0
20+
echo "FAIL: Neovim not installed"
21+
exit 1
2322
fi
2423

2524
# Test 2: Check Neovim config exists

test/test-vim.sh

Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,49 @@
1+
#!/bin/bash
2+
# Test Vim integration
3+
# Usage: test-vim.sh
4+
5+
set -e
6+
7+
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
8+
EXAMPLES_DIR="$SCRIPT_DIR/../examples"
9+
10+
echo "=== Testing Vim integration ==="
11+
12+
# Test 1: Check if vim is available
13+
echo ""
14+
echo "--- Test 1: Check Vim availability ---"
15+
if command -v vim &> /dev/null; then
16+
VERSION=$(vim --version | head -1)
17+
echo "Vim: $VERSION"
18+
echo "PASS: Vim available"
19+
else
20+
echo "FAIL: Vim not installed"
21+
exit 1
22+
fi
23+
24+
# Test 2: Check Vim config exists
25+
echo ""
26+
echo "--- Test 2: Check Vim config ---"
27+
CONFIG_PATH="$HOME/.vimrc"
28+
if [ -f "$CONFIG_PATH" ]; then
29+
grep -q "syntax on" "$CONFIG_PATH" && echo "PASS: Vim syntax highlighting enabled" || echo "WARN: syntax highlighting not configured"
30+
grep -q "filetype plugin indent on" "$CONFIG_PATH" && echo "PASS: Vim filetype detection enabled" || echo "WARN: filetype detection not configured"
31+
else
32+
echo "WARN: Vim config not found at $CONFIG_PATH"
33+
fi
34+
35+
# Test 3: Test Vim can start and exit cleanly
36+
echo ""
37+
echo "--- Test 3: Test Vim startup ---"
38+
vim -es -c 'quit' 2>&1 || { echo "FAIL: Vim failed to start"; exit 1; }
39+
echo "PASS: Vim starts and exits cleanly"
40+
41+
# Test 4: Test opening an OCaml file
42+
echo ""
43+
echo "--- Test 4: Test opening OCaml file ---"
44+
cd "$EXAMPLES_DIR/hello"
45+
vim -es hello.ml -c 'quit' 2>&1 || { echo "FAIL: Vim failed to open OCaml file"; exit 1; }
46+
echo "PASS: Vim can open OCaml files"
47+
48+
echo ""
49+
echo "=== Vim tests completed ==="

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad © 2024 Your Company Name. All rights reserved.





Check this box to remove all script contents from the fetched content.



Check this box to remove all images from the fetched content.


Check this box to remove all CSS styles from the fetched content.


Check this box to keep images inefficiently compressed and original size.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy