hasufell/ale
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix #1216 - Suppress trailing whitespace warings for ansible-lint when the option is set

Browse Source
This commit is contained in:
w0rp 2017-12-13 11:14:31 +00:00
parent 7a88a3605c
commit d2a6d9a915
2 changed files with 18 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ale_linters/ansible/ansible_lint.vim
View File

@ -21,7 +21,7 @@ function! ale_linters#ansible#ansible_lint#Handle(buffer, lines) abort
for l:match in ale#util#GetMatches(a:lines, l:pattern)
let l:code = l:match[4]
if l:code is# 'EANSIBLE002'
if l:code is# 'EANSIBLE0002'
\&& !ale#Var(a:buffer, 'warn_about_trailing_whitespace')
" Skip warnings for trailing whitespace if the option is off.
continue

20
test/handler/test_ansible_lint_handler.vader
View File

@ -1,9 +1,13 @@
Before:
runtime ale_linters/ansible/ansible_lint.vim
call ale#test#SetFilename('main.yml')
runtime ale_linters/ansible/ansible_lint.vim
call ale#test#SetFilename('main.yml')
let b:ale_warn_about_trailing_whitespace = 1
After:
call ale#linter#Reset()
unlet! b:ale_warn_about_trailing_whitespace
call ale#linter#Reset()
Execute(The ansible-lint handler should handle basic errors):
AssertEqual
@ -20,6 +24,16 @@ Execute(The ansible-lint handler should handle basic errors):
\ fnamemodify(tempname(), ':h') . '/main.yml:35: [EANSIBLE0002] Trailing whitespace',
\ ])
Execute(The ansible-lint handler should supress trailing whitespace output when the option is used):
let b:ale_warn_about_trailing_whitespace = 0
AssertEqual
\ [
\ ],
\ ale_linters#ansible#ansible_lint#Handle(bufnr(''), [
\ fnamemodify(tempname(), ':h') . '/main.yml:35: [EANSIBLE0002] Trailing whitespace',
\ ])
Execute (The ansible-lint handler should handle names with spaces):
AssertEqual
\ [