Handle failing to connect to eslint_d
This commit is contained in:
parent
a4a4bba884
commit
ddb3e6d57a
4 changed files with 27 additions and 1 deletions
|
@ -25,7 +25,7 @@ endfunction
|
|||
function! ale#fixers#eslint#ProcessEslintDOutput(buffer, output) abort
|
||||
" If the output is an error message, don't use it.
|
||||
for l:line in a:output[:10]
|
||||
if l:line =~# '^Error:'
|
||||
if l:line =~# '\v^Error:|^Could not connect'
|
||||
return []
|
||||
endif
|
||||
endfor
|
||||
|
|
|
@ -99,6 +99,13 @@ function! ale#handlers#eslint#Handle(buffer, lines) abort
|
|||
\}]
|
||||
endif
|
||||
|
||||
if a:lines == ['Could not connect']
|
||||
return [{
|
||||
\ 'lnum': 1,
|
||||
\ 'text': 'Could not connect to eslint_d. Try updating eslint_d or killing it.',
|
||||
\}]
|
||||
endif
|
||||
|
||||
" Matches patterns line the following:
|
||||
"
|
||||
" /path/to/some-filename.js:47:14: Missing trailing comma. [Warning/comma-dangle]
|
||||
|
|
|
@ -170,3 +170,10 @@ Execute(The eslint_d post-processor should handle error messages correctly):
|
|||
\ ale#fixers#eslint#ProcessEslintDOutput(bufnr(''), [
|
||||
\ 'Error: No ESLint configuration found.',
|
||||
\ ])
|
||||
|
||||
Execute(The eslint_d post-processor should handle failing to connect properly):
|
||||
AssertEqual
|
||||
\ [],
|
||||
\ ale#fixers#eslint#ProcessEslintDOutput(bufnr(''), [
|
||||
\ 'Could not connect',
|
||||
\ ])
|
||||
|
|
|
@ -365,3 +365,15 @@ Execute(eslint should handle react errors correctly):
|
|||
\ ale#handlers#eslint#Handle(bufnr(''), [
|
||||
\ '/path/editor-help.jsx:59:9: Property should be placed on the same line as the component declaration [Error/react/jsx-first-prop-new-line]',
|
||||
\ ])
|
||||
|
||||
Execute(Failing to connect to eslint_d should be handled correctly):
|
||||
AssertEqual
|
||||
\ [
|
||||
\ {
|
||||
\ 'lnum': 1,
|
||||
\ 'text': 'Could not connect to eslint_d. Try updating eslint_d or killing it.',
|
||||
\ },
|
||||
\ ],
|
||||
\ ale#handlers#eslint#Handle(bufnr(''), [
|
||||
\ 'Could not connect',
|
||||
\ ])
|
||||
|
|
Reference in a new issue