diff options
author | Tomáš Janoušek <tomi@nomi.cz> | 2023-08-15 11:20:47 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-08-15 11:20:47 +0100 |
commit | fe38101db7253e450f67893583d1a5dfeefd9ee5 (patch) | |
tree | b9a1fef7622f9a70a4687b3c2645f924bb8f480d /test | |
parent | 951b280bd5665eadd00e22bef7c7ef23cdc57854 (diff) | |
download | ale-fe38101db7253e450f67893583d1a5dfeefd9ee5.zip |
Fix error loading "text" syntax for hover (#4574)
rust-analyzer sometimes returns a hover result with language being
"text", but there's no syntax/text.vim, so this would fail with:
Error detected while processing function <SNR>150_VimOutputCallback[6]..<lambda>8[1]..ale#lsp#HandleMessage[30]..ale#hover#HandleLSPResponse[42]..ale#floating_preview#Show[13]..<SNR>161_VimShow:
line 13:
E484: Cannot open file syntax/text.vim
Only including the file when it actually exists fixes this.
Diffstat (limited to 'test')
-rw-r--r-- | test/test_hover.vader | 6 | ||||
-rw-r--r-- | test/test_hover_parsing.vader | 36 |
2 files changed, 29 insertions, 13 deletions
diff --git a/test/test_hover.vader b/test/test_hover.vader index 7a9c8d91..580b8846 100644 --- a/test/test_hover.vader +++ b/test/test_hover.vader @@ -168,7 +168,7 @@ Execute(LSP hover response with lists of strings should be handled): Execute(LSP hover response with lists of strings and marked strings should be handled): call HandleValidLSPResult({'contents': [ - \ {'language': 'rust', 'value': 'foo'}, + \ {'language': 'python', 'value': 'foo'}, \ "bar\n", \]}) @@ -178,8 +178,8 @@ Execute(LSP hover response with lists of strings and marked strings should be ha \ { \ 'commands': [ \ 'unlet! b:current_syntax', - \ 'syntax include @ALE_hover_rust syntax/rust.vim', - \ 'syntax region ALE_hover_1 start=/\%1l/ end=/\%2l/ contains=@ALE_hover_rust', + \ 'syntax include @ALE_hover_python syntax/python.vim', + \ 'syntax region ALE_hover_1 start=/\%1l/ end=/\%2l/ contains=@ALE_hover_python', \ ], \ }, \ ], diff --git a/test/test_hover_parsing.vader b/test/test_hover_parsing.vader index 4129c26a..221b42ac 100644 --- a/test/test_hover_parsing.vader +++ b/test/test_hover_parsing.vader @@ -35,12 +35,12 @@ Execute(A string with a code fence should be handled): \ [ \ [ \ 'unlet! b:current_syntax', - \ 'syntax include @ALE_hover_python syntax/python.vim', + \ 'syntax include @ALE_hover_javascript syntax/javascript.vim', \ 'unlet! b:current_syntax', - \ 'syntax include @ALE_hover_typescript syntax/typescript.vim', + \ 'syntax include @ALE_hover_python syntax/python.vim', \ 'syntax region ALE_hover_1 start=/\%1l/ end=/\%3l/ contains=@ALE_hover_python', \ 'syntax region ALE_hover_2 start=/\%5l/ end=/\%8l/ contains=@ALE_hover_python', - \ 'syntax region ALE_hover_3 start=/\%8l/ end=/\%10l/ contains=@ALE_hover_typescript', + \ 'syntax region ALE_hover_3 start=/\%8l/ end=/\%10l/ contains=@ALE_hover_javascript', \ ], \ [ \ 'def foo():', @@ -64,7 +64,7 @@ Execute(A string with a code fence should be handled): \ 'def bar():', \ ' pass', \ '```', - \ '```typescript', + \ '```javascript', \ 'const baz = () => undefined', \ '```', \ ], "\n")) @@ -74,12 +74,12 @@ Execute(Multiple strings with fences should be handled): \ [ \ [ \ 'unlet! b:current_syntax', - \ 'syntax include @ALE_hover_python syntax/python.vim', + \ 'syntax include @ALE_hover_javascript syntax/javascript.vim', \ 'unlet! b:current_syntax', - \ 'syntax include @ALE_hover_typescript syntax/typescript.vim', + \ 'syntax include @ALE_hover_python syntax/python.vim', \ 'syntax region ALE_hover_1 start=/\%1l/ end=/\%3l/ contains=@ALE_hover_python', \ 'syntax region ALE_hover_2 start=/\%5l/ end=/\%8l/ contains=@ALE_hover_python', - \ 'syntax region ALE_hover_3 start=/\%8l/ end=/\%10l/ contains=@ALE_hover_typescript', + \ 'syntax region ALE_hover_3 start=/\%8l/ end=/\%10l/ contains=@ALE_hover_javascript', \ ], \ [ \ 'def foo():', @@ -106,7 +106,7 @@ Execute(Multiple strings with fences should be handled): \ 'def bar():', \ ' pass', \ '```', - \ '```typescript', + \ '```javascript', \ 'const baz = () => undefined', \ '```', \ ], "\n"), @@ -124,7 +124,7 @@ Execute(Objects with kinds should be handled): \ 'def foo():', \ ' pass', \ '', - \ '```typescript', + \ '```javascript', \ 'const baz = () => undefined', \ '```', \ ], @@ -142,7 +142,7 @@ Execute(Objects with kinds should be handled): \ { \ 'kind': 'plaintext', \ 'value': join([ - \ '```typescript', + \ '```javascript', \ 'const baz = () => undefined', \ '```', \ ], "\n"), @@ -171,3 +171,19 @@ Execute(Simple markdown formatting should be handled): \ '```', \ 'formatted \_ line \_', \ ], "\n")) + +Execute(Non-existent syntax files shouldn't be loaded): + AssertEqual + \ [ + \ [ + \ 'syntax region ALE_hover_1 start=/\%1l/ end=/\%2l/ contains=@ALE_hover_text', + \ ], + \ [ + \ 'hello', + \ ], + \ ], + \ ale#hover#ParseLSPResult(join([ + \ '```text', + \ 'hello', + \ '```', + \ ], "\n")) |