update: fips unit test move to use gomega

Signed-off-by: Wen Zhou <[email protected]>

Author: zdtsw

pkg/cluster/cluster_config_test.go

@@ -35,6 +35,7 @@ func (c *erroringClient) Get(ctx context.Context, key types.NamespacedName, obj
 }
 
 func TestIsFipsEnabled(t *testing.T) {
+	t.Parallel()
 	var genericError = errors.New("generic client error")
 
 	// Define test cases
@@ -152,6 +153,9 @@ invalid: yaml`,
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
+			t.Parallel()
+			g := NewWithT(t)
+
 			// Create a fake client
 			var fakeClient client.Client
 			if tc.configMap != nil || tc.clientErr != nil {
@@ -172,30 +176,22 @@ invalid: yaml`,
 			}
 
 			// Call the function under test
-			ctx := t.Context()
+			ctx := context.Background()
 			result, err := cluster.IsFipsEnabled(ctx, fakeClient)
 
 			// Check the result
-			if result != tc.expectedResult {
-				t.Errorf("IsFIPSEnabled() = %v, want %v", result, tc.expectedResult)
-			}
+			g.Expect(result).To(Equal(tc.expectedResult))
 
-			// Check the error.  We need to handle nil vs. non-nil errors carefully.
+			// Check the error
 			if tc.expectedError != nil {
-				switch {
-				case err == nil:
-					t.Errorf("IsFIPSEnabled() error = nil, want %v", tc.expectedError)
-				case k8serr.IsNotFound(tc.expectedError):
-					if !k8serr.IsNotFound(err) {
-						t.Errorf("IsFipsEnabled() error = %v, want NotFound error", err)
-					}
-				default:
-					if err.Error() != tc.expectedError.Error() {
-						t.Errorf("IsFIPSEnabled() error = %v, want %v", err, tc.expectedError)
-					}
+				g.Expect(err).To(HaveOccurred())
+				if k8serr.IsNotFound(tc.expectedError) {
+					g.Expect(k8serr.IsNotFound(err)).To(BeTrue())
+				} else {
+					g.Expect(err.Error()).To(Equal(tc.expectedError.Error()))
 				}
-			} else if err != nil {
-				t.Errorf("IsFIPSEnabled() error = %v, want nil", err)
+			} else {
+				g.Expect(err).ShouldNot(HaveOccurred())
 			}
 		})
 	}