This PR is to add comments to TestInvokeCleanCache in pkg\ddc\alluxio\cache_test.go. (#4905)

nellish123 · Srj · cheyang · web-flow · commit 5d4b73487545 · 2025-05-25T10:15:43.000Z
* Add comments/notation to TestInvokeCleanCache  in pkg\ddc\alluxio\cache_test.go.

Signed-off-by: Srj &lt;sunrunjia@smail.nju.edu.cn&gt;

* add comments to TestInvokeCleanCache in pkg\ddc\alluxio\cache_test.go.

Signed-off-by: Srj &lt;sunrunjia@smail.nju.edu.cn&gt;

* fix go fmt issue of pkg/ddc/alluxio/cache_test.go

Signed-off-by: cheyang &lt;cheyang@163.com&gt;

---------

Signed-off-by: Srj &lt;sunrunjia@smail.nju.edu.cn&gt;
Signed-off-by: cheyang &lt;cheyang@163.com&gt;
Co-authored-by: Srj &lt;sunrunjia@smail.nju.edu.cn&gt;
Co-authored-by: cheyang &lt;cheyang@163.com&gt;
diff --git a/pkg/ddc/alluxio/cache_test.go b/pkg/ddc/alluxio/cache_test.go
@@ -231,6 +231,15 @@ func TestPatchDatasetStatus(t *testing.T) {
 	}
 }
 
+// TestInvokeCleanCache tests the behavior of the invokeCleanCache function in the AlluxioEngine.
+// It simulates different StatefulSet statuses using a fake client to verify whether the function
+// correctly determines if an error should be returned based on the readiness state of the replicas.
+//
+// Parameters:
+// - t (*testing.T): The testing context used to run assertions.
+//
+// Returns:
+// - None. The function uses t.Errorf to report test failures.
 func TestInvokeCleanCache(t *testing.T) {
 	masterInputs := []*appsv1.StatefulSet{
 		{

Original file line number	Diff line number	Diff line change
`@@ -231,6 +231,15 @@ func TestPatchDatasetStatus(t *testing.T) {`
`231`	`231`	`}`
`232`	`232`	`}`
`233`	`233`
	`234`	`+// TestInvokeCleanCache tests the behavior of the invokeCleanCache function in the AlluxioEngine.`
	`235`	`+// It simulates different StatefulSet statuses using a fake client to verify whether the function`
	`236`	`+// correctly determines if an error should be returned based on the readiness state of the replicas.`
	`237`	`+//`
	`238`	`+// Parameters:`
	`239`	`+// - t (*testing.T): The testing context used to run assertions.`
	`240`	`+//`
	`241`	`+// Returns:`
	`242`	`+// - None. The function uses t.Errorf to report test failures.`
`234`	`243`	`func TestInvokeCleanCache(t *testing.T) {`
`235`	`244`	`masterInputs := []*appsv1.StatefulSet{`
`236`	`245`	`{`