diff options
| author | Elias Martinez Cohen <eliasc@nvidia.com> | 2019-04-10 22:25:06 -0400 |
|---|---|---|
| committer | Elias Martinez Cohen <eliasc@nvidia.com> | 2019-07-01 11:04:33 -0400 |
| commit | 49db8210f68637a2af14b21940f3cbbaf7361047 (patch) | |
| tree | 366a5e30c22c40b2cf508b2edca6bf8ac25e6072 /test/fixers/test_goimports_fixer_callback.vader | |
| parent | 89f7292138087e1410b2430ae6d359d42e7fe31f (diff) | |
| download | ale-49db8210f68637a2af14b21940f3cbbaf7361047.zip | |
Support $GO111MODULE with Go tooling
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`
Diffstat (limited to 'test/fixers/test_goimports_fixer_callback.vader')
| -rw-r--r-- | test/fixers/test_goimports_fixer_callback.vader | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/test/fixers/test_goimports_fixer_callback.vader b/test/fixers/test_goimports_fixer_callback.vader index cec06354..a1b72425 100644 --- a/test/fixers/test_goimports_fixer_callback.vader +++ b/test/fixers/test_goimports_fixer_callback.vader @@ -5,6 +5,7 @@ Before: " Use an invalid global executable, so we don't match it. let g:ale_go_goimports_executable = 'xxxinvalid' let g:ale_go_goimports_options = '' + let g:ale_go_go111module = '' call ale#test#SetDirectory('/testplugin/test/fixers') call ale#test#SetFilename('../go_files/testfile.go') @@ -39,3 +40,16 @@ Execute(The goimports callback should include extra options): \ 'command': ale#Escape(g:ale_go_goimports_executable) . ' -l -w -srcdir %s --xxx %t' \ }, \ ale#fixers#goimports#Fix(bufnr('')) + +Execute(The goimports callback should support Go environment variables): + let g:ale_go_goimports_executable = has('win32') ? 'cmd' : 'echo' + let g:ale_go_go111module = 'on' + + AssertEqual + \ { + \ 'read_temporary_file': 1, + \ 'command': ale#Env('GO111MODULE', 'on') + \ . ale#Escape(g:ale_go_goimports_executable) + \ . ' -l -w -srcdir %s %t' + \ }, + \ ale#fixers#goimports#Fix(bufnr('')) |