tests: change directories before deleting

Change out of the dynamically created directory before trying to delete
that directory so that the directory change after deletion does not fail
because the current directory at the time of change does not exist.

This situation is also handled by Vim 8.2.0189, but vim-go has to
support 8.1.2269 for now, so adjust tests so that vim-go is no longer
susceptible to the condition that 8.2.0189 fixes.

autoload/go/calls_test.vim

@@ -5,6 +5,7 @@ set cpo&vim
 scriptencoding utf-8
 
 func! Test_Callers() abort
+  let l:wd = getcwd()
   try
     let l:tmp = gotest#write_file('calls/caller.go', [
           \ 'package main',
@@ -37,6 +38,7 @@ func! Test_Callers() abort
 
     call gotest#assert_quickfix(l:actual, l:expected)
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc

autoload/go/cmd_test.vim

@@ -3,6 +3,7 @@ let s:cpo_save = &cpo
 set cpo&vim
 
 func! Test_GoBuildErrors()
+  let l:wd = getcwd()
   try
     let g:go_gopls_enabled = 0
     let l:filename = 'cmd/bad.go'
@@ -27,6 +28,7 @@ func! Test_GoBuildErrors()
 
     call gotest#assert_quickfix(actual, l:expected)
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc

autoload/go/complete_test.vim

@@ -9,6 +9,7 @@ endfunction
 
 func! s:getinfo()
     let l:filename = 'complete/complete.go'
+    let l:wd = getcwd()
     let l:tmp = gotest#load_fixture(l:filename)
     try
       call cursor(8, 3)
@@ -17,6 +18,7 @@ func! s:getinfo()
       let actual = go#complete#GetInfo()
       call assert_equal(expected, actual)
     finally
+      call go#util#Chdir(l:wd)
       call delete(l:tmp, 'rf')
     endtry
 endfunction

autoload/go/config_test.vim

@@ -82,6 +82,7 @@ func! Test_GoplsEnabled_Clear() abort
     return
   endif
 
+  let l:wd = getcwd()
   try
     let g:go_gopls_enabled = 0
 
@@ -94,6 +95,7 @@ func! Test_GoplsEnabled_Clear() abort
           \ ] )
 
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc

autoload/go/debug_test.vim

@@ -23,6 +23,7 @@ function! Test_GoDebugStart_Errors() abort
     return
   endif
 
+  let l:wd = getcwd()
   try
     let g:go_gopls_enabled = 0
     let l:tmp = gotest#load_fixture('debug/compilerror/main.go')
@@ -60,6 +61,7 @@ function! Test_GoDebugStart_Errors() abort
     call assert_false(exists(':GoDebugStop'))
 
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
     " clear the quickfix lists
     call setqflist([], 'r')
@@ -71,6 +73,7 @@ function! Test_GoDebugModeRemapsAndRestoresKeys() abort
     return
   endif
 
+  let l:wd = getcwd()
   try
     let g:go_gopls_enabled = 0
     let g:go_debug_mappings = {'(go-debug-continue)': {'key': 'q', 'arguments': '<nowait>'}}
@@ -96,6 +99,7 @@ function! Test_GoDebugModeRemapsAndRestoresKeys() abort
     endwhile
     call assert_equal('', maparg('q'))
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunction
@@ -105,6 +109,7 @@ function! Test_GoDebugStopRemovesPlugMappings() abort
     return
   endif
 
+  let l:wd = getcwd()
   try
     let g:go_gopls_enabled = 0
     let l:tmp = gotest#load_fixture('debug/debugmain/debugmain.go')
@@ -129,6 +134,7 @@ function! Test_GoDebugStopRemovesPlugMappings() abort
     endwhile
     call assert_equal('', maparg('<Plug>(go-debug-stop'))
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunction
@@ -141,6 +147,7 @@ function! s:debug(...) abort
     return
   endif
 
+  let l:wd = getcwd()
   try
     let g:go_gopls_enabled = 0
     let $oldgopath = $GOPATH
@@ -179,6 +186,7 @@ function! s:debug(...) abort
 
   finally
     call go#debug#Breakpoint(6)
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunction

autoload/go/def_test.vim

@@ -5,6 +5,7 @@ set cpo&vim
 scriptencoding utf-8
 
 func! Test_jump_to_declaration_guru() abort
+  let l:wd = getcwd()
   try
     let g:go_gopls_enabled = 0
     let l:filename = 'def/jump.go'
@@ -19,11 +20,13 @@ func! Test_jump_to_declaration_guru() abort
     call assert_equal(l:lnum, getcurpos()[1])
     call assert_equal(l:col, getcurpos()[2])
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
 
 func! Test_jump_to_declaration_godef() abort
+  let l:wd = getcwd()
   try
     let g:go_gopls_enabled = 0
     let l:filename = 'def/jump.go'
@@ -38,11 +41,13 @@ func! Test_jump_to_declaration_godef() abort
     call assert_equal(l:lnum, getcurpos()[1])
     call assert_equal(l:col, getcurpos()[2])
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
 
 func! Test_Jump_leaves_lists() abort
+  let l:wd = getcwd()
   try
     let l:filename = 'def/jump.go'
     let l:tmp = gotest#load_fixture(l:filename)
@@ -74,6 +79,7 @@ func! Test_Jump_leaves_lists() abort
     let l:actual = getqflist()
     call gotest#assert_quickfix(l:actual, l:expected)
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
@@ -83,6 +89,7 @@ func! Test_DefJump_gopls_simple_first() abort
     return
   endif
 
+  let l:wd = getcwd()
   try
     let g:go_def_mode = 'gopls'
 
@@ -108,6 +115,7 @@ func! Test_DefJump_gopls_simple_first() abort
 
     call assert_equal(l:expected, getpos('.'))
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
@@ -117,6 +125,7 @@ func! Test_DefJump_gopls_simple_last() abort
     return
   endif
 
+  let l:wd = getcwd()
   try
     let g:go_def_mode = 'gopls'
 
@@ -142,6 +151,7 @@ func! Test_DefJump_gopls_simple_last() abort
 
     call assert_equal(l:expected, getpos('.'))
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
@@ -151,6 +161,7 @@ func! Test_DefJump_gopls_MultipleCodeUnit_first() abort
     return
   endif
 
+  let l:wd = getcwd()
   try
     let g:go_def_mode = 'gopls'
 
@@ -175,6 +186,7 @@ func! Test_DefJump_gopls_MultipleCodeUnit_first() abort
 
     call assert_equal(l:expected, getpos('.'))
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
@@ -185,6 +197,7 @@ func! Test_DefJump_gopls_MultipleCodeUnit_last() abort
     return
   endif
 
+  let l:wd = getcwd()
   try
     let g:go_def_mode = 'gopls'
 
@@ -209,6 +222,7 @@ func! Test_DefJump_gopls_MultipleCodeUnit_last() abort
 
     call assert_equal(l:expected, getpos('.'))
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc

autoload/go/extract_test.vim

@@ -3,6 +3,7 @@ let s:cpo_save = &cpo
 set cpo&vim
 
 func! Test_Extract() abort
+  let l:wd = getcwd()
   try
     let l:tmp = gotest#write_file('a/a.go', [
           \ 'package a',
@@ -36,6 +37,7 @@ func! Test_Extract() abort
           \ '}'])
 
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc

autoload/go/fillstruct_test.vim

@@ -3,6 +3,7 @@ let s:cpo_save = &cpo
 set cpo&vim
 
 func! Test_fillstruct() abort
+  let l:wd = getcwd()
   try
     let g:go_gopls_enabled = 0
     let g:go_fillstruct_mode = 'fillstruct'
@@ -18,11 +19,13 @@ func! Test_fillstruct() abort
           \ '\tAddress: "",',
           \ '}'])
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
 
 func! Test_fillstruct_line() abort
+  let l:wd = getcwd()
   try
     let g:go_gopls_enabled = 0
     let g:go_fillstruct_mode = 'fillstruct'
@@ -38,11 +41,13 @@ func! Test_fillstruct_line() abort
           \ '\tAddress: "",',
           \ '}'])
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
 
 func! Test_fillstruct_two_line() abort
+  let l:wd = getcwd()
   try
     let g:go_fillstruct_mode = 'fillstruct'
     let g:go_gopls_enabled = 0
@@ -68,11 +73,13 @@ func! Test_fillstruct_two_line() abort
           \ '\tAddress: "",',
           \ '}) }'])
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
 
 func! Test_fillstruct_two_cursor() abort
+  let l:wd = getcwd()
   try
     let g:go_fillstruct_mode = 'fillstruct'
     let g:go_gopls_enabled = 0
@@ -95,11 +102,13 @@ func! Test_fillstruct_two_cursor() abort
           \ '\tAddress: "",',
           \ '}) }'])
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
 
 func! Test_gopls_fillstruct() abort
+  let l:wd = getcwd()
   try
     let g:go_fillstruct_mode = 'gopls'
     let l:tmp = gotest#write_file('a/a.go', [
@@ -120,11 +129,13 @@ func! Test_gopls_fillstruct() abort
           \ '\tAddress: "",',
           \ '}'])
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
 
 func! Test_gopls_fillstruct_line() abort
+  let l:wd = getcwd()
   try
     let g:go_fillstruct_mode = 'gopls'
     let l:tmp = gotest#write_file('a/a.go', [
@@ -145,11 +156,13 @@ func! Test_gopls_fillstruct_line() abort
           \ '\tAddress: "",',
           \ '}'])
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
 
 func! Test_gopls_fillstruct_two_cursor_first() abort
+  let l:wd = getcwd()
   try
     let g:go_fillstruct_mode = 'gopls'
     let l:tmp = gotest#write_file('a/a.go', [
@@ -177,11 +190,13 @@ func! Test_gopls_fillstruct_two_cursor_first() abort
           \ '\tAddress: "",',
           \ '}, mail.Address{}) }'])
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc
 
 func! Test_gopls_fillstruct_two_cursor_second() abort
+  let l:wd = getcwd()
   try
     let g:go_fillstruct_mode = 'gopls'
     let l:tmp = gotest#write_file('a/a.go', [
@@ -209,6 +224,7 @@ func! Test_gopls_fillstruct_two_cursor_second() abort
           \ '\tAddress: "",',
           \ '}) }'])
   finally
+    call go#util#Chdir(l:wd)
     call delete(l:tmp, 'rf')
   endtry
 endfunc