49db8210f6
Allows the user to override $GO111MODULE environment variable through ale options. This gives control over the default behavior of Go module resolution. Golang documentation: https://github.com/golang/go/wiki/Modules#how-to-use-modules Add `ale#Go#EnvString()` function to make it easy to add similar Go environment variables in the future. Use the new `EnvString` function in all available Go tools callbacks & update tests Also add test of linter command callback for `gofmt`
41 lines
1 KiB
Text
41 lines
1 KiB
Text
Before:
|
|
Save g:ale_go_go_executable
|
|
|
|
call ale#assert#SetUpLinterTest('go', 'gobuild')
|
|
|
|
GivenCommandOutput ['/foo/bar', '/foo/baz']
|
|
|
|
After:
|
|
Restore
|
|
call ale#assert#TearDownLinterTest()
|
|
|
|
Execute(The default commands should be correct):
|
|
AssertLinter 'go',
|
|
\ ale#path#CdString(expand('%:p:h'))
|
|
\ . 'go test -c -o /dev/null ./'
|
|
|
|
Execute(Go environment variables should be supported):
|
|
let b:ale_go_go111module = 'on'
|
|
|
|
AssertLinter 'go',
|
|
\ ale#path#CdString(expand('%:p:h'))
|
|
\ . ale#Env('GO111MODULE', 'on')
|
|
\ . 'go test -c -o /dev/null ./'
|
|
|
|
unlet! b:ale_go_go111module
|
|
|
|
Execute(Extra options should be supported):
|
|
let g:ale_go_gobuild_options = '--foo-bar'
|
|
|
|
AssertLinter 'go',
|
|
\ ale#path#CdString(expand('%:p:h'))
|
|
\ . 'go test --foo-bar -c -o /dev/null ./'
|
|
|
|
let g:ale_go_gobuild_options = ''
|
|
|
|
Execute(The executable should be configurable):
|
|
let g:ale_go_go_executable = 'foobar'
|
|
|
|
AssertLinter 'foobar',
|
|
\ ale#path#CdString(expand('%:p:h'))
|
|
\ . 'foobar test -c -o /dev/null ./'
|