Add sql-lint as linter
This commit is contained in:
parent
4afbf2f25d
commit
07080066e4
3 changed files with 67 additions and 0 deletions
32
ale_linters/sql/sqllint.vim
Normal file
32
ale_linters/sql/sqllint.vim
Normal file
|
@ -0,0 +1,32 @@
|
|||
" ale_linters/sql/sqllint.vim
|
||||
" Author: Joe Reynolds <joereynolds952@gmail.co>
|
||||
" Description: sql-lint for SQL files.
|
||||
" sql-lint can be found at
|
||||
" https://www.npmjs.com/package/sql-lint
|
||||
" https://github.com/joereynolds/sql-lint
|
||||
|
||||
function! ale_linters#sql#sqllint#Handle(buffer, lines) abort
|
||||
" Matches patterns like the following:
|
||||
"
|
||||
" stdin:1 [ER_NO_DB_ERROR] No database selected
|
||||
let l:pattern = '\v^[^:]+:(\d+) (.*)'
|
||||
let l:output = []
|
||||
|
||||
for l:match in ale#util#GetMatches(a:lines, l:pattern)
|
||||
call add(l:output, {
|
||||
\ 'lnum': l:match[1] + 0,
|
||||
\ 'col': l:match[2] + 0,
|
||||
\ 'type': l:match[3][0],
|
||||
\ 'text': l:match[0],
|
||||
\})
|
||||
endfor
|
||||
|
||||
return l:output
|
||||
endfunction
|
||||
|
||||
call ale#linter#Define('sql', {
|
||||
\ 'name': 'sqllint',
|
||||
\ 'executable': 'sql-lint',
|
||||
\ 'command': 'sql-lint',
|
||||
\ 'callback': 'ale_linters#sql#sqllint#Handle',
|
||||
\})
|
12
test/command_callback/test_sqllint_command_callback.vader
Normal file
12
test/command_callback/test_sqllint_command_callback.vader
Normal file
|
@ -0,0 +1,12 @@
|
|||
Before:
|
||||
" Load the linter and set up a series of commands, reset linter variables,
|
||||
" clear caches, etc.
|
||||
"
|
||||
" Vader's 'Save' command will be called here for linter variables.
|
||||
call ale#assert#SetUpLinterTest('sql', 'sqllint')
|
||||
|
||||
After:
|
||||
call ale#assert#TearDownLinterTest()
|
||||
|
||||
Execute(The default command should be correct):
|
||||
AssertLinter 'sql-lint', ['sql-lint']
|
23
test/handler/test_sqllint_handler.vader
Normal file
23
test/handler/test_sqllint_handler.vader
Normal file
|
@ -0,0 +1,23 @@
|
|||
Before:
|
||||
" Load the file which defines the linter.
|
||||
runtime ale_linters/sql/sqllint.vim
|
||||
|
||||
After:
|
||||
" Unload all linters again.
|
||||
call ale#linter#Reset()
|
||||
|
||||
Execute (The output should be correct):
|
||||
|
||||
" Test that the right loclist items are parsed from the handler.
|
||||
AssertEqual
|
||||
\ [
|
||||
\ {
|
||||
\ 'lnum': 1,
|
||||
\ 'col': 0,
|
||||
\ 'type': '',
|
||||
\ 'text': 'stdin:1 [ER_NO_DB_ERROR] No database selected'
|
||||
\ },
|
||||
\ ],
|
||||
\ ale_linters#sql#sqllint#Handle(bufnr(''), [
|
||||
\ 'stdin:1 [ER_NO_DB_ERROR] No database selected'
|
||||
\ ])
|
Reference in a new issue