diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-12 00:40:22 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-12 00:40:22 -0500 |
| commit | b45638194cc19b026c49e0cb4787024c73723954 (patch) | |
| tree | 777c56061b36369ee7c887f2b8bc1168049f69fb /tests/test-prog-lsp--add-file-watch-ignored-extras.el | |
| parent | 203560396ea67bac7a7375cd2fbfa99433b672a9 (diff) | |
| download | dotemacs-b45638194cc19b026c49e0cb4787024c73723954.tar.gz dotemacs-b45638194cc19b026c49e0cb4787024c73723954.zip | |
refactor(lsp): rename cj/lsp--disable-eldoc-hover for accuracy
The helper removes lsp-mode's entry from `eldoc-documentation-functions' in the current buffer. It never touched hover display, so the old name was misleading. It's now `cj/lsp--remove-eldoc-provider', the two tests rename to match, and the docstring drops the "hover" wording.
Diffstat (limited to 'tests/test-prog-lsp--add-file-watch-ignored-extras.el')
| -rw-r--r-- | tests/test-prog-lsp--add-file-watch-ignored-extras.el | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/tests/test-prog-lsp--add-file-watch-ignored-extras.el b/tests/test-prog-lsp--add-file-watch-ignored-extras.el index 6213b58c..142cd2ea 100644 --- a/tests/test-prog-lsp--add-file-watch-ignored-extras.el +++ b/tests/test-prog-lsp--add-file-watch-ignored-extras.el @@ -87,21 +87,21 @@ (should-error (cj/lsp--add-file-watch-ignored-extras) :type 'wrong-type-argument))) -(ert-deftest test-prog-lsp--disable-eldoc-hover-removes-lsp-provider-locally () +(ert-deftest test-prog-lsp--remove-eldoc-provider-removes-lsp-provider-locally () "Normal: remove lsp-mode's Eldoc provider from the buffer-local hook." (with-temp-buffer (setq-local eldoc-documentation-functions '(lsp-eldoc-function eldoc-documentation-default)) - (cj/lsp--disable-eldoc-hover) + (cj/lsp--remove-eldoc-provider) (should-not (memq #'lsp-eldoc-function eldoc-documentation-functions)) (should (memq 'eldoc-documentation-default eldoc-documentation-functions)))) -(ert-deftest test-prog-lsp--disable-eldoc-hover-does-not-touch-default-value () - "Boundary: disabling LSP Eldoc in one buffer leaves the default hook alone." +(ert-deftest test-prog-lsp--remove-eldoc-provider-does-not-touch-default-value () + "Boundary: removing the LSP provider in one buffer leaves the default hook alone." (let ((eldoc-documentation-functions '(lsp-eldoc-function))) (with-temp-buffer (setq-local eldoc-documentation-functions '(lsp-eldoc-function)) - (cj/lsp--disable-eldoc-hover) + (cj/lsp--remove-eldoc-provider) (should-not (memq #'lsp-eldoc-function eldoc-documentation-functions))) (should (memq #'lsp-eldoc-function eldoc-documentation-functions)))) |