Fix #518 Fix handling of spaces in filenames for various linters

This commit is contained in:
w0rp 2017-04-29 17:33:18 +01:00
parent cbb01e32b9
commit 0b4acd6453
10 changed files with 200 additions and 48 deletions

View File

@ -25,35 +25,32 @@ function! ale_linters#go#gobuild#GetCommand(buffer, goenv_output) abort
\ . ' && go test -c -o /dev/null ./' \ . ' && go test -c -o /dev/null ./'
endfunction endfunction
function! ale_linters#go#gobuild#Handler(buffer, lines) abort function! ale_linters#go#gobuild#GetMatches(lines) abort
return ale_linters#go#gobuild#HandleGoBuildErrors(a:buffer, bufname(a:buffer), a:lines) " Matches patterns like the following:
endfunction
function! ale_linters#go#gobuild#HandleGoBuildErrors(buffer, full_filename, lines) abort
" Matches patterns line the following:
" "
" file.go:27: missing argument for Printf("%s"): format reads arg 2, have only 1 args " file.go:27: missing argument for Printf("%s"): format reads arg 2, have only 1 args
" file.go:53:10: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) " file.go:53:10: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary)
" file.go:5:2: expected declaration, found 'STRING' "log" " file.go:5:2: expected declaration, found 'STRING' "log"
" go test returns relative paths so use tail of filename as part of pattern matcher " go test returns relative paths so use tail of filename as part of pattern matcher
let l:filename = fnamemodify(a:full_filename, ':t') let l:pattern = '\v^([a-zA-Z]?:?[^:]+):(\d+):?(\d+)?:? (.+)$'
let l:path_pattern = '[a-zA-Z]\?\\\?:\?[[:alnum:]/\.\-_]\+'
let l:pattern = '^' . l:path_pattern . ':\(\d\+\):\?\(\d\+\)\?:\? \(.\+\)$' return ale#util#GetMatches(a:lines, l:pattern)
endfunction
function! ale_linters#go#gobuild#Handler(buffer, lines) abort
let l:output = [] let l:output = []
for l:line in a:lines for l:match in ale_linters#go#gobuild#GetMatches(a:lines)
let l:match = matchlist(l:line, l:pattern)
" Omit errors from imported go packages " Omit errors from imported go packages
if len(l:match) == 0 || l:line !~ l:filename if ale#path#IsBufferPath(a:buffer, l:match[0])
continue continue
endif endif
call add(l:output, { call add(l:output, {
\ 'lnum': l:match[1] + 0, \ 'lnum': l:match[2] + 0,
\ 'col': l:match[2] + 0, \ 'col': l:match[3] + 0,
\ 'text': l:match[3], \ 'text': l:match[4],
\ 'type': 'E', \ 'type': 'E',
\}) \})
endfor endfor

View File

@ -11,32 +11,26 @@ function! ale_linters#go#gometalinter#GetCommand(buffer) abort
\ . ' ' . fnameescape(fnamemodify(bufname(a:buffer), ':p:h')) \ . ' ' . fnameescape(fnamemodify(bufname(a:buffer), ':p:h'))
endfunction endfunction
function! ale_linters#go#gometalinter#Handler(buffer, lines) abort function! ale_linters#go#gometalinter#GetMatches(lines) abort
" Matches patterns line the following: let l:pattern = '\v^([a-zA-Z]?:?[^:]+):(\d+):?(\d+)?:?:?(warning|error):?\s\*?(.+)$'
"
" file.go:27: missing argument for Printf("%s"): format reads arg 2, have only 1 args
" file.go:53:10: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary)
" file.go:5:2: expected declaration, found 'STRING' "log"
" gometalinter returns relative paths so use tail of filename as part of pattern matcher return ale#util#GetMatches(a:lines, l:pattern)
let l:filename = fnamemodify(bufname(a:buffer), ':t') endfunction
let l:path_pattern = '[a-zA-Z]\?\\\?:\?[[:alnum:]/\.\-_]\+'
let l:pattern = '^' . l:path_pattern . ':\(\d\+\):\?\(\d\+\)\?:\?:\?\(warning\|error\):\?\s\*\?\(.\+\)$' function! ale_linters#go#gometalinter#Handler(buffer, lines) abort
let l:output = [] let l:output = []
for l:line in a:lines for l:match in ale_linters#go#gometalinter#GetMatches(a:lines)
let l:match = matchlist(l:line, l:pattern)
" Omit errors from files other than the one currently open " Omit errors from files other than the one currently open
if len(l:match) == 0 || l:line !~ l:filename if ale#path#IsBufferPath(a:buffer, l:match[0])
continue continue
endif endif
call add(l:output, { call add(l:output, {
\ 'lnum': l:match[1] + 0, \ 'lnum': l:match[2] + 0,
\ 'col': l:match[2] + 0, \ 'col': l:match[3] + 0,
\ 'text': l:match[4], \ 'type': tolower(l:match[4]) ==# 'warning' ? 'W' : 'E',
\ 'type': tolower(l:match[3]) ==# 'warning' ? 'W' : 'E', \ 'text': l:match[5],
\}) \})
endfor endfor

View File

@ -15,8 +15,6 @@ function! ale_linters#python#mypy#GetCommand(buffer) abort
\ . ' %t' \ . ' %t'
endfunction endfunction
let s:path_pattern = '[a-zA-Z]\?\\\?:\?[[:alnum:]/\.\-_]\+'
function! ale_linters#python#mypy#Handle(buffer, lines) abort function! ale_linters#python#mypy#Handle(buffer, lines) abort
" Look for lines like the following: " Look for lines like the following:
" "
@ -25,7 +23,7 @@ function! ale_linters#python#mypy#Handle(buffer, lines) abort
" Lines like these should be ignored below: " Lines like these should be ignored below:
" "
" file.py:4: note: (Stub files are from https://github.com/python/typeshed) " file.py:4: note: (Stub files are from https://github.com/python/typeshed)
let l:pattern = '^' . s:path_pattern . ':\(\d\+\):\?\(\d\+\)\?: \([^:]\+\): \(.\+\)$' let l:pattern = '\v^[a-zA-Z]?:?[^:]+:(\d+):?(\d+)?: ([^:]+): (.+)$'
let l:output = [] let l:output = []
for l:match in ale#util#GetMatches(a:lines, l:pattern) for l:match in ale#util#GetMatches(a:lines, l:pattern)

View File

@ -1,8 +1,6 @@
" Author: w0rp <devw0rp@gmail.com> " Author: w0rp <devw0rp@gmail.com>
" Description: Error handling for flake8, etc. " Description: Error handling for flake8, etc.
let s:path_pattern = '[a-zA-Z]\?\\\?:\?[[:alnum:]/\.\-_]\+'
function! ale#handlers#python#HandlePEP8Format(buffer, lines) abort function! ale#handlers#python#HandlePEP8Format(buffer, lines) abort
" Matches patterns line the following: " Matches patterns line the following:
" "
@ -10,7 +8,7 @@ function! ale#handlers#python#HandlePEP8Format(buffer, lines) abort
" "
" stdin:6:6: E111 indentation is not a multiple of four " stdin:6:6: E111 indentation is not a multiple of four
" test.yml:35: [EANSIBLE0002] Trailing whitespace " test.yml:35: [EANSIBLE0002] Trailing whitespace
let l:pattern = '^' . s:path_pattern . ':\(\d\+\):\?\(\d\+\)\?: \[\?\(\([[:alpha:]]\)[[:alnum:]]\+\)\]\? \(.*\)$' let l:pattern = '\v^[a-zA-Z]?:?[^:]+:(\d+):?(\d+)?: \[?([[:alnum:]]+)\]? (.*)$'
let l:output = [] let l:output = []
for l:match in ale#util#GetMatches(a:lines, l:pattern) for l:match in ale#util#GetMatches(a:lines, l:pattern)
@ -30,8 +28,8 @@ function! ale#handlers#python#HandlePEP8Format(buffer, lines) abort
call add(l:output, { call add(l:output, {
\ 'lnum': l:match[1] + 0, \ 'lnum': l:match[1] + 0,
\ 'col': l:match[2] + 0, \ 'col': l:match[2] + 0,
\ 'text': l:code . ': ' . l:match[5], \ 'text': l:code . ': ' . l:match[4],
\ 'type': l:match[4] ==# 'E' ? 'E' : 'W', \ 'type': l:code[:0] ==# 'E' ? 'E' : 'W',
\}) \})
endfor endfor

View File

@ -1,10 +1,8 @@
" Author: w0rp <devw0rp@gmail.com> " Author: w0rp <devw0rp@gmail.com>
" Description: Error handling for errors in a Unix format. " Description: Error handling for errors in a Unix format.
let s:path_pattern = '[a-zA-Z]\?\\\?:\?[[:alnum:]/\.\-_]\+'
function! s:HandleUnixFormat(buffer, lines, type) abort function! s:HandleUnixFormat(buffer, lines, type) abort
let l:pattern = '^' . s:path_pattern . ':\(\d\+\):\?\(\d\+\)\?:\? \?\(.\+\)$' let l:pattern = '\v^[a-zA-Z]?:?[^:]+:(\d+):?(\d+)?:? ?(.+)$'
let l:output = [] let l:output = []
for l:match in ale#util#GetMatches(a:lines, l:pattern) for l:match in ale#util#GetMatches(a:lines, l:pattern)

View File

@ -55,3 +55,30 @@ endfunction
function! ale#path#BufferCdString(buffer) abort function! ale#path#BufferCdString(buffer) abort
return ale#path#CdString(fnamemodify(bufname(a:buffer), ':p:h')) return ale#path#CdString(fnamemodify(bufname(a:buffer), ':p:h'))
endfunction endfunction
" Return 1 if a path is an absolute path.
function! ale#path#IsAbsolute(filename) abort
return match(a:filename, '^\v/|^[a-zA-Z]:\\') == 0
endfunction
" Given a directory and a filename, resolve the path, which may be relative
" or absolute, and get an absolute path to the file, following symlinks.
function! ale#path#Resolve(directory, filename) abort
return resolve(
\ ale#path#IsAbsolute(a:filename)
\ ? a:filename
\ : a:directory . '/' . a:filename
\)
endfunction
" Given a buffer number and a relative or absolute path, return 1 if the
" two paths represent the same file on disk.
function! ale#path#IsBufferPath(buffer, filename) abort
let l:buffer_filename = expand('#' . a:buffer . ':p')
let l:resolved_filename = ale#path#Resolve(
\ fnamemodify(l:buffer_filename, ':h'),
\ a:filename
\)
return resolve(l:buffer_filename) == l:resolved_filename
endfunction

View File

@ -40,6 +40,20 @@ Execute (HandlePEP8Format should handle the correct lines of output):
\ "test.yml:35: [EANSIBLE0002] Trailing whitespace", \ "test.yml:35: [EANSIBLE0002] Trailing whitespace",
\ ]) \ ])
Execute (HandlePEP8Format should handle names with spaces):
AssertEqual
\ [
\ {
\ 'lnum': 6,
\ 'col': 6,
\ 'type': 'E',
\ 'text': 'E111: indentation is not a multiple of four',
\ },
\ ],
\ ale#handlers#python#HandlePEP8Format(42, [
\ 'C:\something\with spaces.py:6:6: E111 indentation is not a multiple of four',
\ ])
Execute (HandleGCCFormat should handle the correct lines of output): Execute (HandleGCCFormat should handle the correct lines of output):
AssertEqual AssertEqual
\ [ \ [
@ -122,6 +136,20 @@ Execute (HandleUnixFormatAsError should handle lines with no space after the col
\ 'some_file.xyz:53:10:bar', \ 'some_file.xyz:53:10:bar',
\ ]) \ ])
Execute (HandleUnixFormatAsError should handle names with spaces):
AssertEqual
\ [
\ {
\ 'lnum': 13,
\ 'col': 90,
\ 'type': 'E',
\ 'text': 'leonard.exclamation.30ppm More than 30 ppm of exclamations. Keep them under control.',
\ },
\ ],
\ ale#handlers#unix#HandleAsError(42, [
\ '/Users/rrj/Notes/Astro/Taurus December SM.txt:13:90: leonard.exclamation.30ppm More than 30 ppm of exclamations. Keep them under control.',
\ ])
Execute (HandleUnixFormatAsWarning should handle some example lines of output): Execute (HandleUnixFormatAsWarning should handle some example lines of output):
AssertEqual AssertEqual
\ [ \ [

View File

@ -0,0 +1,44 @@
Before:
runtime ale_linters/go/gobuild.vim
After:
call ale#linter#Reset()
Execute (The gobuild handler should handle names with spaces):
" We can't test Windows paths with the path resovling on Linux, but we can
" test the regex.
AssertEqual
\ [
\ [
\ 'C:\something\file with spaces.go',
\ '27',
\ '',
\ 'missing argument for Printf("%s"): format reads arg 2, have only 1 args',
\ ],
\ [
\ 'C:\something\file with spaces.go',
\ '5',
\ '2',
\ 'expected declaration, found ''STRING'' "log"',
\ ],
\ ],
\ map(ale_linters#go#gobuild#GetMatches([
\ 'C:\something\file with spaces.go:27: missing argument for Printf("%s"): format reads arg 2, have only 1 args',
\ 'C:\something\file with spaces.go:5:2: expected declaration, found ''STRING'' "log"',
\ ]), 'v:val[1:4]')
Execute (The gobuild handler should handle relative paths correctly):
:file! /foo/bar/baz.go
AssertEqual
\ [
\ {
\ 'lnum': 27,
\ 'col': 0,
\ 'text': 'missing argument for Printf("%s"): format reads arg 2, have only 1 args',
\ 'type': 'E',
\ },
\ ],
\ ale_linters#go#gobuild#Handler(42, [
\ 'baz.go:27: missing argument for Printf("%s"): format reads arg 2, have only 1 args',
\ ])

View File

@ -0,0 +1,53 @@
Before:
runtime ale_linters/go/gometalinter.vim
After:
call ale#linter#Reset()
Execute (The gometalinter handler should handle names with spaces):
" We can't test Windows paths with the path resovling on Linux, but we can
" test the regex.
AssertEqual
\ [
\ [
\ 'C:\something\file with spaces.go',
\ '12',
\ '3',
\ 'warning',
\ 'expected ''package'', found ''IDENT'' gibberish (staticcheck)',
\ ],
\ [
\ 'C:\something\file with spaces.go',
\ '37',
\ '5',
\ 'error',
\ 'expected ''package'', found ''IDENT'' gibberish (golint)',
\ ],
\ ],
\ map(ale_linters#go#gometalinter#GetMatches([
\ 'C:\something\file with spaces.go:12:3:warning: expected ''package'', found ''IDENT'' gibberish (staticcheck)',
\ 'C:\something\file with spaces.go:37:5:error: expected ''package'', found ''IDENT'' gibberish (golint)',
\ ]), 'v:val[1:5]')
Execute (The gometalinter handler should handle relative paths correctly):
:file! /foo/bar/baz.go
AssertEqual
\ [
\ {
\ 'lnum': 12,
\ 'col': 3,
\ 'text': 'expected ''package'', found ''IDENT'' gibberish (staticcheck)',
\ 'type': 'W',
\ },
\ {
\ 'lnum': 37,
\ 'col': 5,
\ 'text': 'expected ''package'', found ''IDENT'' gibberish (golint)',
\ 'type': 'E',
\ },
\ ],
\ ale_linters#go#gometalinter#Handler(42, [
\ 'baz.go:12:3:warning: expected ''package'', found ''IDENT'' gibberish (staticcheck)',
\ 'baz.go:37:5:error: expected ''package'', found ''IDENT'' gibberish (golint)',
\ ])

View File

@ -1,6 +1,10 @@
Execute(The mypy handler should parse lines correctly): Before:
runtime ale_linters/python/mypy.vim runtime ale_linters/python/mypy.vim
After:
call ale#linter#Reset()
Execute(The mypy handler should parse lines correctly):
AssertEqual AssertEqual
\ [ \ [
\ { \ {
@ -22,5 +26,16 @@ Execute(The mypy handler should parse lines correctly):
\ "file.py:40:5: error: Some other problem", \ "file.py:40:5: error: Some other problem",
\ ]) \ ])
After: Execute(The mypy handler should handle Windows names with spaces):
call ale#linter#Reset() AssertEqual
\ [
\ {
\ 'lnum': 4,
\ 'col': 0,
\ 'text': "No library stub file for module 'django.db'",
\ 'type': 'E',
\ },
\ ],
\ ale_linters#python#mypy#Handle(347, [
\ "C:\something\with spaces.py:4: error: No library stub file for module 'django.db'",
\ ])