🌱 use testing.Setenv instead of os.Setenv (#10919)

* use testing.Setenv instead of os.Setenv

Signed-off-by: sivchari <[email protected]>

* fix: build failed error

Signed-off-by: sivchari <[email protected]>

---------

Signed-off-by: sivchari <[email protected]>

Author: sivchari

cmd/clusterctl/client/config/cert_manager_client_test.go

@@ -17,7 +17,6 @@ limitations under the License.
 package config
 
 import (
-	"os"
 	"testing"
 
 	. "github.com/onsi/gomega"
@@ -77,13 +76,8 @@ func TestCertManagerGet(t *testing.T) {
 			g := NewWithT(t)
 
 			for k, v := range tt.envVars {
-				g.Expect(os.Setenv(k, v)).To(Succeed())
+				t.Setenv(k, v)
 			}
-			defer func() {
-				for k := range tt.envVars {
-					g.Expect(os.Unsetenv(k)).To(Succeed())
-				}
-			}()
 			p := &certManagerClient{
 				reader: tt.fields.reader,
 			}

cmd/clusterctl/client/config/providers_client_test.go

@@ -18,7 +18,6 @@ package config
 
 import (
 	"fmt"
-	"os"
 	"sort"
 	"testing"
 
@@ -148,13 +147,8 @@ func Test_providers_List(t *testing.T) {
 			g := NewWithT(t)
 
 			for k, v := range tt.envVars {
-				g.Expect(os.Setenv(k, v)).To(Succeed())
+				t.Setenv(k, v)
 			}
-			defer func() {
-				for k := range tt.envVars {
-					g.Expect(os.Unsetenv(k)).To(Succeed())
-				}
-			}()
 			p := &providersClient{
 				reader: tt.fields.configGetter,
 			}

cmd/clusterctl/client/repository/overrides_test.go

@@ -17,7 +17,6 @@ limitations under the License.
 package repository
 
 import (
-	"os"
 	"path/filepath"
 	"testing"
 
@@ -74,13 +73,8 @@ func TestOverrides(t *testing.T) {
 			g := NewWithT(t)
 
 			for k, v := range tt.envVars {
-				g.Expect(os.Setenv(k, v)).To(Succeed())
+				t.Setenv(k, v)
 			}
-			defer func() {
-				for k := range tt.envVars {
-					g.Expect(os.Unsetenv(k)).To(Succeed())
-				}
-			}()
 			provider := config.NewProvider("myinfra", "", clusterctlv1.InfrastructureProviderType)
 			override := newOverride(&newOverrideInput{
 				configVariablesClient: tt.configVarClient,