test(ddc/goosefs): migrate transform_api_gateway tests to ginkgo (#5554)

* test(ddc/goosefs): migrate transform_api_gateway tests to ginkgo

Signed-off-by: Harsh <harshmastic@gmail.com>

* fix: set shouldMatch=true for success cases

Signed-off-by: Harsh <harshmastic@gmail.com>

---------

Signed-off-by: Harsh <harshmastic@gmail.com>

Author: hxrshxz

pkg/ddc/goosefs/transform_api_gateway_test.go

@@ -17,75 +17,80 @@ limitations under the License.
 package goosefs
 
 import (
-	"testing"
+	. "github.com/onsi/ginkgo/v2"
+	. "github.com/onsi/gomega"
 
 	datav1alpha1 "github.com/fluid-cloudnative/fluid/api/v1alpha1"
 )
 
-func TestTransformAPIGateway(t *testing.T) {
-	var engine = &GooseFSEngine{}
-	var tests = []struct {
-		runtime *datav1alpha1.GooseFSRuntime
-		value   *GooseFS
-	}{
-		{
-			runtime: &datav1alpha1.GooseFSRuntime{
+var _ = Describe("GooseFS", func() {
+	DescribeTable("transformAPIGateway",
+		func(runtime *datav1alpha1.GooseFSRuntime, value *GooseFS, expectError bool, shouldMatch bool) {
+			engine := &GooseFSEngine{}
+			err := engine.transformAPIGateway(runtime, value)
+
+			if expectError {
+				Expect(err).To(HaveOccurred())
+			} else {
+				Expect(err).NotTo(HaveOccurred())
+				if shouldMatch {
+					Expect(runtime.Spec.APIGateway.Enabled).To(Equal(value.APIGateway.Enabled))
+				}
+			}
+		},
+		Entry("should sync when runtime enabled and value disabled",
+			&datav1alpha1.GooseFSRuntime{
 				Spec: datav1alpha1.GooseFSRuntimeSpec{
 					APIGateway: datav1alpha1.GooseFSCompTemplateSpec{
 						Enabled: true,
 					},
 				},
 			},
-			value: &GooseFS{
+			&GooseFS{
 				APIGateway: APIGateway{
 					Enabled: false,
 				},
 			},
-		},
-		{
-			runtime: &datav1alpha1.GooseFSRuntime{
+			false,
+			true,
+		),
+		Entry("should sync when runtime disabled and value enabled",
+			&datav1alpha1.GooseFSRuntime{
 				Spec: datav1alpha1.GooseFSRuntimeSpec{
 					APIGateway: datav1alpha1.GooseFSCompTemplateSpec{
 						Enabled: false,
 					},
 				},
 			},
-			value: &GooseFS{
+			&GooseFS{
 				APIGateway: APIGateway{
 					Enabled: true,
 				},
 			},
-		},
-		{
-			runtime: nil,
-			value: &GooseFS{
+			false,
+			true,
+		),
+		Entry("should return error when runtime is nil",
+			nil,
+			&GooseFS{
 				APIGateway: APIGateway{
 					Enabled: false,
 				},
 			},
-		},
-		{
-			runtime: &datav1alpha1.GooseFSRuntime{
+			true,
+			false,
+		),
+		Entry("should return error when value is nil",
+			&datav1alpha1.GooseFSRuntime{
 				Spec: datav1alpha1.GooseFSRuntimeSpec{
 					APIGateway: datav1alpha1.GooseFSCompTemplateSpec{
 						Enabled: true,
 					},
 				},
 			},
-			value: nil,
-		},
-	}
-	for _, test := range tests {
-		err := engine.transformAPIGateway(test.runtime, test.value)
-		if test.runtime == nil || test.value == nil {
-			if err == nil {
-				t.Errorf("should return err if it's possible to lead to nil pointer")
-			}
-		} else if test.runtime.Spec.APIGateway.Enabled != test.value.APIGateway.Enabled {
-			t.Errorf("testcase cannot paas because of wrong result,%t != %t",
-				test.runtime.Spec.APIGateway.Enabled,
-				test.value.APIGateway.Enabled)
-		}
-	}
-
-}
+			nil,
+			true,
+			false,
+		),
+	)
+})