Skip to content

Commit e5f0bca

Browse files
committed
fix(test): check err returned by os.Setenv
1 parent 8923ac0 commit e5f0bca

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

internal/pkg/services/ske/utils/utils_test.go

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -723,8 +723,14 @@ func TestGetDefaultKubeconfigPathWithEnvVar(t *testing.T) {
723723
for _, tt := range tests {
724724
t.Run(tt.description, func(t *testing.T) {
725725
// Setup environment variables
726-
os.Setenv("KUBECONFIG", tt.kubeconfigEnvVar)
727-
os.Setenv("HOME", tt.userHome)
726+
err := os.Setenv("KUBECONFIG", tt.kubeconfigEnvVar)
727+
if err != nil {
728+
t.Errorf("could not set KUBECONFIG environment variable: %s", err)
729+
}
730+
err = os.Setenv("HOME", tt.userHome)
731+
if err != nil {
732+
t.Errorf("could not set HOME environment variable: %s", err)
733+
}
728734

729735
output, err := GetDefaultKubeconfigPath()
730736

0 commit comments

Comments
 (0)