chore: os.Setenv -> t.Setenv

Author: hugodutka

cmd/server/server_test.go

@@ -117,7 +117,7 @@ func isolateViper(t *testing.T) {
 		// Restore env vars
 		for _, key := range agentapiEnvs {
 			if val := os.Getenv(key); val != "" {
-				os.Setenv(key, val)
+				t.Setenv(key, val)
 			}
 		}
 	})
@@ -174,8 +174,7 @@ func TestServerCmd_AllEnvVars(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			isolateViper(t)
-			os.Setenv(tt.envVar, tt.envValue)
-			defer os.Unsetenv(tt.envVar)
+			t.Setenv(tt.envVar, tt.envValue)
 
 			serverCmd := CreateServerCmd()
 			cmd := &cobra.Command{}
@@ -245,8 +244,7 @@ func TestServerCmd_ArgsPrecedenceOverEnv(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			isolateViper(t)
-			os.Setenv(tt.envVar, tt.envValue)
-			defer os.Unsetenv(tt.envVar)
+			t.Setenv(tt.envVar, tt.envValue)
 
 			// Mock execution to test arg parsing without running server
 			args := append(tt.args, "--help")
@@ -264,10 +262,8 @@ func TestMixed_ConfigurationScenarios(t *testing.T) {
 		isolateViper(t)
 
 		// Set some env vars
-		os.Setenv("AGENTAPI_TYPE", "goose")
-		os.Setenv("AGENTAPI_TERM_WIDTH", "120")
-		defer os.Unsetenv("AGENTAPI_TYPE")
-		defer os.Unsetenv("AGENTAPI_TERM_WIDTH")
+		t.Setenv("AGENTAPI_TYPE", "goose")
+		t.Setenv("AGENTAPI_TERM_WIDTH", "120")
 
 		// Set some CLI args
 		serverCmd := CreateServerCmd()