Use the same formatting as other files for the dogma linter file, and cover the Handler function with tests
This commit is contained in:
parent
ba97017380
commit
82fde0ea51
@ -37,8 +37,9 @@ function! ale_linters#elixir#dogma#Handle(buffer, lines) abort
|
|||||||
endfunction
|
endfunction
|
||||||
|
|
||||||
call ale#linter#Define('elixir', {
|
call ale#linter#Define('elixir', {
|
||||||
\ 'name': 'dogma',
|
\ 'name': 'dogma',
|
||||||
\ 'executable': 'mix',
|
\ 'executable': 'mix',
|
||||||
\ 'command': 'mix dogma %s --format=flycheck',
|
\ 'command': 'mix dogma %s --format=flycheck',
|
||||||
\ 'lint_file': 1,
|
\ 'lint_file': 1,
|
||||||
\ 'callback': 'ale_linters#elixir#dogma#Handle' })
|
\ 'callback': 'ale_linters#elixir#dogma#Handle',
|
||||||
|
\})
|
||||||
|
28
test/handler/test_dogma_handler.vader
Normal file
28
test/handler/test_dogma_handler.vader
Normal file
@ -0,0 +1,28 @@
|
|||||||
|
Execute(The dogma handler should parse lines correctly):
|
||||||
|
runtime ale_linters/elixir/dogma.vim
|
||||||
|
|
||||||
|
AssertEqual
|
||||||
|
\ [
|
||||||
|
\ {
|
||||||
|
\ 'bufnr': 347,
|
||||||
|
\ 'lnum': 18,
|
||||||
|
\ 'col': 5,
|
||||||
|
\ 'text': 'Some error',
|
||||||
|
\ 'type': 'E',
|
||||||
|
\ },
|
||||||
|
\ {
|
||||||
|
\ 'bufnr': 347,
|
||||||
|
\ 'lnum': 19,
|
||||||
|
\ 'col': 7,
|
||||||
|
\ 'text': 'Some warning',
|
||||||
|
\ 'type': 'W',
|
||||||
|
\ },
|
||||||
|
\ ],
|
||||||
|
\ ale_linters#elixir#dogma#Handle(347, [
|
||||||
|
\ 'This line should be ignored completely',
|
||||||
|
\ 'lib/filename.ex:18:5: C: Some error',
|
||||||
|
\ 'lib/filename.ex:19:7: R: Some warning',
|
||||||
|
\ ])
|
||||||
|
|
||||||
|
After:
|
||||||
|
call ale#linter#Reset()
|
Loading…
Reference in New Issue
Block a user