marc/ale
Archived
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix prospector empty string error

Browse source 
Prospector linter is raising error when no warnings are present in file
(#1680). Copied fix from #779.
This commit is contained in:
Fenner Macrae 2018-06-26 16:58:34 -07:00
parent c2ab885384
commit 1ca2334846
2 changed files with 9 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
ale_linters/python/prospector.vim
View file

@ -23,6 +23,10 @@ endfunction
function! ale_linters#python#prospector#Handle(buffer, lines) abort function! ale_linters#python#prospector#Handle(buffer, lines) abort
let l:output = [] let l:output = []
if empty(a:lines)
return []
endif
let l:prospector_error = json_decode(join(a:lines, '')) let l:prospector_error = json_decode(join(a:lines, ''))
for l:error in l:prospector_error.messages for l:error in l:prospector_error.messages

5
test/handler/test_prospector_handler.vader
View file

@ -156,3 +156,8 @@ Execute(Ignoring trailing whitespace messages should work):
\ ' ]', \ ' ]',
\ '}', \ '}',
\ ]) \ ])
Execute(The prospector handler should handle empty output):
AssertEqual
\ [],
\ ale_linters#python#prospector#Handle(bufnr(''), [])