diff options
| author | Craig Jennings <c@cjennings.net> | 2026-06-25 01:54:06 -0400 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-06-25 01:54:06 -0400 |
| commit | 9562ace44c686004fd9906b2bc8ffae404cac3c4 (patch) | |
| tree | 44de1d5cb7c83358caf893d7a49e0faa2d7b3086 /todo.org | |
| parent | 66d0d1a661f51c4fdb59042c9c27461a981e9404 (diff) | |
| download | dotemacs-9562ace44c686004fd9906b2bc8ffae404cac3c4.tar.gz dotemacs-9562ace44c686004fd9906b2bc8ffae404cac3c4.zip | |
chore(todo): close LSP smoke-test task (done)
Diffstat (limited to 'todo.org')
| -rw-r--r-- | todo.org | 9 |
1 files changed, 2 insertions, 7 deletions
@@ -1916,13 +1916,8 @@ Pitfalls: - Do not accidentally re-enable UI/doc/sideline behavior that was explicitly disabled for performance. -***** TODO [#B] Add a startup smoke test for LSP config resolution :quick:solo: - -Keep this narrow. A useful test can require the LSP-related modules with mocked -=use-package= side effects and assert that: -- generic defaults are set in one place, -- no duplicate hook entries are installed for the same mode, -- =lsp-enable-remote= remains nil. +***** 2026-06-25 Thu @ 01:53:48 -0400 Added the LSP config-resolution smoke test +=tests/test-prog-lsp.el= (5 ERT tests) pins prog-lsp's load-time invariants: =lsp-enable-remote= stays nil (no auto-start on TRAMP files), the file-watch-ignore defaults live in one idempotent helper (=cj/lsp--add-file-watch-ignored-extras=), the eldoc provider is stripped from the global hook, and no mode holds a duplicate =lsp-deferred= entry. Tests the top-level =:init= + helper surface rather than the =:config= defaults, which defer to lsp-mode's own load under =make test= (the no-package-initialize constraint). Hit and fixed the lexical-binding special-var trap on =lsp-file-watch-ignored-directories= in the test. **** TODO [#B] Gate tree-sitter grammar auto-install behind an explicit policy |