glenux-archive/spf13--cobra
1
0
Fork 0
mirror of https://github.com/spf13/cobra synced 2025-04-27 17:17:20 +00:00
Code Issues Projects Releases Packages Wiki Activity

Defer unsetting test env vars properly

Browse source
This commit is contained in:
Ville Skyttä 2023-12-11 18:06:40 +02:00
parent 138b0ab964
commit c96f1a622a
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
completions_test.go
View file

@ -3580,10 +3580,14 @@ func TestGetEnvConfig(t *testing.T) {
// Could make env handling cleaner with t.Setenv with Go >= 1.17 // Could make env handling cleaner with t.Setenv with Go >= 1.17
func() { func() {
err := os.Setenv(tc.cmdVar, tc.cmdVal) err := os.Setenv(tc.cmdVar, tc.cmdVal)
defer assertNoErr(t, os.Unsetenv(tc.cmdVar)) defer func() {
assertNoErr(t, os.Unsetenv(tc.cmdVar))
}()
assertNoErr(t, err) assertNoErr(t, err)
err = os.Setenv(tc.globalVar, tc.globalVal) err = os.Setenv(tc.globalVar, tc.globalVal)
defer assertNoErr(t, os.Unsetenv(tc.globalVar)) defer func() {
assertNoErr(t, os.Unsetenv(tc.globalVar))
}()
assertNoErr(t, err) assertNoErr(t, err)
cmd := &Command{Use: tc.use} cmd := &Command{Use: tc.use}
got := getEnvConfig(cmd, tc.suffix) got := getEnvConfig(cmd, tc.suffix)