diff options
author | Gregory Anders <greg@gpanders.com> | 2021-09-17 00:17:54 -0600 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-09-17 08:17:54 +0200 |
commit | c13242cf471e9bac19672798798fbd6bb9a8125f (patch) | |
tree | 272a1323e02646ac5686a7a228eba397a5561a8e /test/functional/plugin/lsp/diagnostic_spec.lua | |
parent | c2a65921d762929a1b8e740aeb92c7970d025736 (diff) | |
download | rneovim-c13242cf471e9bac19672798798fbd6bb9a8125f.tar.gz rneovim-c13242cf471e9bac19672798798fbd6bb9a8125f.tar.bz2 rneovim-c13242cf471e9bac19672798798fbd6bb9a8125f.zip |
fix(diagnostic): remove useless highlight links (#15683)
These links were actually defined backwards: the highlight groups
actually being used for display are the new "Diagnostic*" groups, so
linking the old "LspDiagnostics*" groups to these does absolutely
nothing, since there is nothing actually being highlighted with the
LspDiagnostics* groups.
These links were made in an attempt to preserve backward compatibility
with existing colorschemes. We could reverse the links to maintain this
preservation, but then that disallows us from actually defining default
values for the new highlight groups.
Instead, just remove the links and be done with the old LspDiagnostics*
highlight groups.
This is not technically a breaking change: the breaking change already
happened in #15585, but this PR just makes that explicit.
Diffstat (limited to 'test/functional/plugin/lsp/diagnostic_spec.lua')
-rw-r--r-- | test/functional/plugin/lsp/diagnostic_spec.lua | 28 |
1 files changed, 0 insertions, 28 deletions
diff --git a/test/functional/plugin/lsp/diagnostic_spec.lua b/test/functional/plugin/lsp/diagnostic_spec.lua index 2a6d7de634..353de667ea 100644 --- a/test/functional/plugin/lsp/diagnostic_spec.lua +++ b/test/functional/plugin/lsp/diagnostic_spec.lua @@ -1,6 +1,5 @@ local helpers = require('test.functional.helpers')(after_each) -local command = helpers.command local clear = helpers.clear local exec_lua = helpers.exec_lua local eq = helpers.eq @@ -945,31 +944,4 @@ describe('vim.lsp.diagnostic', function() assert(loc_list[1].lnum < loc_list[2].lnum) end) end) - - it('highlight groups', function() - command('runtime plugin/diagnostic.vim') - eq({ - 'LspDiagnosticsDefaultError', - 'LspDiagnosticsDefaultHint', - 'LspDiagnosticsDefaultInformation', - 'LspDiagnosticsDefaultWarning', - 'LspDiagnosticsFloatingError', - 'LspDiagnosticsFloatingHint', - 'LspDiagnosticsFloatingInformation', - 'LspDiagnosticsFloatingWarning', - 'LspDiagnosticsSignError', - 'LspDiagnosticsSignHint', - 'LspDiagnosticsSignInformation', - 'LspDiagnosticsSignWarning', - 'LspDiagnosticsUnderlineError', - 'LspDiagnosticsUnderlineHint', - 'LspDiagnosticsUnderlineInformation', - 'LspDiagnosticsUnderlineWarning', - 'LspDiagnosticsVirtualTextError', - 'LspDiagnosticsVirtualTextHint', - 'LspDiagnosticsVirtualTextInformation', - 'LspDiagnosticsVirtualTextWarning', - }, exec_lua([[return vim.fn.getcompletion('Lsp', 'highlight')]])) - end) - end) |