Re-annotate according to the feedback request.

Signed-off-by: konjacq <konjacq@outlook.com>

Author: konjacq

pkg/ddc/juicefs/master_internal_test.go

@@ -53,7 +53,6 @@ func init() {
 // It covers cases where the Helm release exists, doesn't exist, encounters errors during checking/installation,
 // and verifies the correct handling of each situation through mock implementations and hook injections.
 func TestSetupMasterInternal(t *testing.T) {
-	// Mock implementations for helm.CheckRelease under different scenarios
 	mockExecCheckReleaseCommonFound := func(name string, namespace string) (exist bool, err error) {
 		return true, nil
 	}
@@ -63,16 +62,13 @@ func TestSetupMasterInternal(t *testing.T) {
 	mockExecCheckReleaseErr := func(name string, namespace string) (exist bool, err error) {
 		return false, errors.New("fail to check release")
 	}
-
-	// Mock implementations for helm.InstallRelease under different scenarios
 	mockExecInstallReleaseCommon := func(name string, namespace string, valueFile string, chartName string) error {
 		return nil
 	}
 	mockExecInstallReleaseErr := func(name string, namespace string, valueFile string, chartName string) error {
 		return errors.New("fail to install dataload chart")
 	}
 
-	// Helper functions for unhooking mocked implementations
 	wrappedUnhookCheckRelease := func() {
 		err := gohook.UnHook(helm.CheckRelease)
 		if err != nil {
@@ -86,7 +82,6 @@ func TestSetupMasterInternal(t *testing.T) {
 		}
 	}
 
-	// Setup test Kubernetes objects
 	juicefsSecret := &v1.Secret{
 		ObjectMeta: metav1.ObjectMeta{
 			Name:      "test",
@@ -105,7 +100,6 @@ func TestSetupMasterInternal(t *testing.T) {
 	testObjs := []runtime.Object{}
 	testObjs = append(testObjs, (*juicefsruntime).DeepCopy(), (*juicefsSecret).DeepCopy())
 
-	// Create test dataset objects
 	var datasetInputs = []datav1alpha1.Dataset{
 		{
 			ObjectMeta: metav1.ObjectMeta{
@@ -132,13 +126,11 @@ func TestSetupMasterInternal(t *testing.T) {
 	}
 	client := fake.NewFakeClientWithScheme(testScheme, testObjs...)
 
-	// Build runtime information
 	runtimeInfo, err := base.BuildRuntimeInfo("hbase", "fluid", "juicefs")
 	if err != nil {
 		t.Errorf("fail to create the runtimeInfo with error %v", err)
 	}
 
-	// Initialize JuiceFS engine instance
 	engine := JuiceFSEngine{
 		name:      "test",
 		namespace: "fluid",
@@ -151,14 +143,12 @@ func TestSetupMasterInternal(t *testing.T) {
 		},
 		runtimeInfo: runtimeInfo,
 	}
-
-	// Setup port allocator for testing
 	err = portallocator.SetupRuntimePortAllocator(client, &net.PortRange{Base: 10, Size: 100}, "bitmap", GetReservedPorts)
 	if err != nil {
 		t.Fatal(err.Error())
 	}
 
-	// Test case 1: Existing Helm release scenario
+	// check release found
 	err = gohook.Hook(helm.CheckRelease, mockExecCheckReleaseCommonFound, nil)
 	if err != nil {
 		t.Fatal(err.Error())
@@ -169,7 +159,7 @@ func TestSetupMasterInternal(t *testing.T) {
 	}
 	wrappedUnhookCheckRelease()
 
-	// Test case 2: Error during release check
+	// check release error
 	err = gohook.Hook(helm.CheckRelease, mockExecCheckReleaseErr, nil)
 	if err != nil {
 		t.Fatal(err.Error())
@@ -180,13 +170,13 @@ func TestSetupMasterInternal(t *testing.T) {
 	}
 	wrappedUnhookCheckRelease()
 
-	// Test case 3: Missing Helm release scenario
+	// check release not found
 	err = gohook.Hook(helm.CheckRelease, mockExecCheckReleaseCommonNotFound, nil)
 	if err != nil {
 		t.Fatal(err.Error())
 	}
 
-	// Subcase 3a: Error during release installation
+	// install release with error
 	err = gohook.Hook(helm.InstallRelease, mockExecInstallReleaseErr, nil)
 	if err != nil {
 		t.Fatal(err.Error())
@@ -197,7 +187,7 @@ func TestSetupMasterInternal(t *testing.T) {
 	}
 	wrappedUnhookInstallRelease()
 
-	// Subcase 3b: Successful release installation
+	// install release successfully
 	err = gohook.Hook(helm.InstallRelease, mockExecInstallReleaseCommon, nil)
 	if err != nil {
 		t.Fatal(err.Error())