From 70867692894aa99e9ef450f19f5ff7b571b98aad Mon Sep 17 00:00:00 2001
From: w0rp <devw0rp@gmail.com>
Date: Thu, 6 Sep 2018 17:05:05 +0100
Subject: #1891 Fix the go-langserver tests

---
 .../test_golangserver_command_callback.vader       | 37 ++++++++++++++++------
 1 file changed, 27 insertions(+), 10 deletions(-)

(limited to 'test')

diff --git a/test/command_callback/test_golangserver_command_callback.vader b/test/command_callback/test_golangserver_command_callback.vader
index a5893cee..ee88e1a4 100644
--- a/test/command_callback/test_golangserver_command_callback.vader
+++ b/test/command_callback/test_golangserver_command_callback.vader
@@ -1,13 +1,21 @@
 Before:
   Save $GOPATH
-  call ale#assert#SetUpLinterTest('go', 'golangserver')
-  let sep = has('win32') ? ';' : ':'
-  let $GOPATH=ale#path#Simplify(g:dir . '/go_paths/go1')
-      \ . sep
-      \ . ale#path#Simplify(g:dir . '/go_paths/go2')
+  Save g:ale_completion_enabled
+
+  let g:ale_completion_enabled = 0
+  let g:sep = has('win32') ? ';' : ':'
+
+  call ale#assert#SetUpLinterTest('go', 'langserver')
+  let $GOPATH = ale#path#Simplify(g:dir . '/go_paths/go1')
+  \ . g:sep
+  \ . ale#path#Simplify(g:dir . '/go_paths/go2')
 
 After:
   Restore
+
+  unlet! b:ale_completion_enabled
+  unlet! g:sep
+
   call ale#assert#TearDownLinterTest()
 
 Execute(should set correct defaults):
@@ -15,36 +23,45 @@ Execute(should set correct defaults):
 
 Execute(should configure go-langserver callback executable):
   let b:ale_go_langserver_executable = 'boo'
+
   AssertLinter 'boo', ale#Escape('boo')
-  unlet b:ale_go_langserver_executable
 
 Execute(should set go-langserver options):
   call ale#test#SetFilename('go_paths/go1/prj1/file.go')
   let b:ale_completion_enabled = 1
   let b:ale_go_langserver_options = ''
-  AssertEqual ale_linters#go#golangserver#GetCommand(bufnr('%')), ale#Escape('go-langserver') . ' -gocodecompletion'
+
+  AssertLinter 'go-langserver',
+  \ ale#Escape('go-langserver') . ' -gocodecompletion'
 
   let b:ale_go_langserver_options = '-trace'
-  AssertEqual ale_linters#go#golangserver#GetCommand(bufnr('%')), ale#Escape('go-langserver') . ' -gocodecompletion -trace'
+
+  AssertLinter 'go-langserver',
+  \ ale#Escape('go-langserver') . ' -gocodecompletion -trace'
 
 Execute(should ignore go-langserver -gocodecompletion option):
   call ale#test#SetFilename('go_paths/go1/prj1/file.go')
 
   let b:ale_go_langserver_options = '-trace -gocodecompletion'
   let b:ale_completion_enabled = 1
-  AssertEqual ale_linters#go#golangserver#GetCommand(bufnr('%')), ale#Escape('go-langserver') . ' -gocodecompletion -trace'
+
+  AssertLinter 'go-langserver',
+  \ ale#Escape('go-langserver') . ' -gocodecompletion -trace'
 
   let b:ale_completion_enabled = 0
-  AssertEqual ale_linters#go#golangserver#GetCommand(bufnr('%')), ale#Escape('go-langserver') . ' -trace'
+
+  AssertLinter 'go-langserver', ale#Escape('go-langserver') . ' -trace'
 
 Execute(should set go-langserver for go app1):
   call ale#test#SetFilename('go_paths/go1/prj1/file.go')
+
   AssertLSPLanguage 'go'
   AssertLSPOptions {}
   AssertLSPProject ale#path#Simplify(g:dir . '/go_paths/go1')
 
 Execute(should set go-langserver for go app2):
   call ale#test#SetFilename('go_paths/go2/prj1/file.go')
+
   AssertLSPLanguage 'go'
   AssertLSPOptions {}
   AssertLSPProject ale#path#Simplify(g:dir . '/go_paths/go2')
-- 
cgit v1.2.3