format(TestReconcileInClusterSAToken): indent the code by two tabs

Author: ingvagabund

pkg/descheduler/descheduler_test.go

@@ -2372,89 +2372,89 @@ func TestReconcileInClusterSAToken(t *testing.T) {
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			ctrl := &promClientController{
-				currentPrometheusAuthToken: tc.currentAuthToken,
-				metricsProviders: map[api.MetricsSource]*api.MetricsProvider{
-					api.PrometheusMetrics: {
-						Source: api.PrometheusMetrics,
-						Prometheus: &api.Prometheus{
-							URL: prometheusURL,
+					ctrl := &promClientController{
+						currentPrometheusAuthToken: tc.currentAuthToken,
+						metricsProviders: map[api.MetricsSource]*api.MetricsProvider{
+							api.PrometheusMetrics: {
+								Source: api.PrometheusMetrics,
+								Prometheus: &api.Prometheus{
+									URL: prometheusURL,
+								},
+							},
 						},
-					},
-				},
-				inClusterConfig: tc.inClusterConfigFunc,
-			}
+						inClusterConfig: tc.inClusterConfigFunc,
+					}
 
-			// Set previous transport and client if test expects them to be cleared
-			if tc.expectPreviousTransportCleared {
-				ctrl.previousPrometheusClientTransport = &http.Transport{}
-			}
-			if tc.expectPromClientCleared {
-				ctrl.promClient = &mockPrometheusClient{name: "old-client"}
-			}
+					// Set previous transport and client if test expects them to be cleared
+					if tc.expectPreviousTransportCleared {
+						ctrl.previousPrometheusClientTransport = &http.Transport{}
+					}
+					if tc.expectPromClientCleared {
+						ctrl.promClient = &mockPrometheusClient{name: "old-client"}
+					}
 
-			// Mock createPrometheusClient
-			clientCreated := false
-			if tc.createPrometheusClientFunc != nil {
-				ctrl.createPrometheusClient = func(url, token string) (promapi.Client, *http.Transport, error) {
-					client, transport, err := tc.createPrometheusClientFunc(url, token)
-					if err == nil {
-						clientCreated = true
+					// Mock createPrometheusClient
+					clientCreated := false
+					if tc.createPrometheusClientFunc != nil {
+						ctrl.createPrometheusClient = func(url, token string) (promapi.Client, *http.Transport, error) {
+							client, transport, err := tc.createPrometheusClientFunc(url, token)
+							if err == nil {
+								clientCreated = true
+							}
+							return client, transport, err
+						}
 					}
-					return client, transport, err
-				}
-			}
 
-			// Call reconcileInClusterSAToken
-			err := ctrl.reconcileInClusterSAToken()
+					// Call reconcileInClusterSAToken
+					err := ctrl.reconcileInClusterSAToken()
 
-			// Verify error expectations
-			if tc.expectedErr != nil {
-				if err == nil {
-					t.Errorf("Expected error %q but got none", tc.expectedErr)
-				} else if err.Error() != tc.expectedErr.Error() {
-					t.Errorf("Expected error %q but got %q", tc.expectedErr, err.Error())
-				}
-			} else {
-				if err != nil {
-					t.Errorf("Expected no error but got: %v", err)
-				}
-			}
-
-			// Verify client creation expectations
-			if tc.expectClientCreated && !clientCreated {
-				t.Errorf("Expected prometheus client to be created but it wasn't")
-			}
-			if !tc.expectClientCreated && clientCreated {
-				t.Errorf("Expected prometheus client not to be created but it was")
-			}
+					// Verify error expectations
+					if tc.expectedErr != nil {
+						if err == nil {
+							t.Errorf("Expected error %q but got none", tc.expectedErr)
+						} else if err.Error() != tc.expectedErr.Error() {
+							t.Errorf("Expected error %q but got %q", tc.expectedErr, err.Error())
+						}
+					} else {
+						if err != nil {
+							t.Errorf("Expected no error but got: %v", err)
+						}
+					}
 
-			// Verify token expectations
-			if ctrl.currentPrometheusAuthToken != tc.expectCurrentToken {
-				t.Errorf("Expected current token to be %q but got %q", tc.expectCurrentToken, ctrl.currentPrometheusAuthToken)
-			}
+					// Verify client creation expectations
+					if tc.expectClientCreated && !clientCreated {
+						t.Errorf("Expected prometheus client to be created but it wasn't")
+					}
+					if !tc.expectClientCreated && clientCreated {
+						t.Errorf("Expected prometheus client not to be created but it was")
+					}
 
-			// Verify previous transport cleared when expected
-			if tc.expectPreviousTransportCleared {
-				if tc.expectClientCreated {
-					// Success case: new transport should be set
-					if ctrl.previousPrometheusClientTransport == nil {
-						t.Error("Expected previous transport to be set to new transport, but it was nil")
+					// Verify token expectations
+					if ctrl.currentPrometheusAuthToken != tc.expectCurrentToken {
+						t.Errorf("Expected current token to be %q but got %q", tc.expectCurrentToken, ctrl.currentPrometheusAuthToken)
 					}
-				} else if tc.expectedErr != nil {
-					// Failure case: transport should be nil
-					if ctrl.previousPrometheusClientTransport != nil {
-						t.Error("Expected previous transport to be cleared on error, but it was set")
+
+					// Verify previous transport cleared when expected
+					if tc.expectPreviousTransportCleared {
+						if tc.expectClientCreated {
+							// Success case: new transport should be set
+							if ctrl.previousPrometheusClientTransport == nil {
+								t.Error("Expected previous transport to be set to new transport, but it was nil")
+							}
+						} else if tc.expectedErr != nil {
+							// Failure case: transport should be nil
+							if ctrl.previousPrometheusClientTransport != nil {
+								t.Error("Expected previous transport to be cleared on error, but it was set")
+							}
+						}
 					}
-				}
-			}
 
-			// Verify promClient cleared when expected
-			if tc.expectPromClientCleared {
-				if ctrl.promClient != nil {
-					t.Error("Expected promClient to be cleared, but it was set")
-				}
-			}
+					// Verify promClient cleared when expected
+					if tc.expectPromClientCleared {
+						if ctrl.promClient != nil {
+							t.Error("Expected promClient to be cleared, but it was set")
+						}
+					}
 		})
 	}
 }