add comment (#4650)

yuio0801 · web-flow · commit 404885a67283 · 2025-03-20T04:04:58.000Z
Signed-off-by: yuio0801 &lt;2632656591@qq.com&gt;
diff --git a/pkg/ddc/alluxio/metadata_test.go b/pkg/ddc/alluxio/metadata_test.go
@@ -349,6 +349,14 @@ func TestSyncMetadataWithoutMaster(t *testing.T) {
 	}
 }
 
+// TestShouldSyncMetadata tests the behavior of the shouldSyncMetadata method.
+// This test verifies whether metadata synchronization is correctly determined
+// under different Dataset and AlluxioRuntime configurations.
+// Test cases include:
+// 1. A Dataset with metadata already synced should not trigger synchronization.
+// 2. A Dataset with an empty UfsTotal should trigger synchronization by default.
+// 3. A Dataset with AutoSync enabled should trigger synchronization.
+// 4. A Dataset with AutoSync disabled should not trigger synchronization.
 func TestShouldSyncMetadata(t *testing.T) {
 	datasetInputs := []datav1alpha1.Dataset{
 		{

Original file line number	Diff line number	Diff line change
`@@ -349,6 +349,14 @@ func TestSyncMetadataWithoutMaster(t *testing.T) {`
`349`	`349`	`}`
`350`	`350`	`}`
`351`	`351`
	`352`	`+// TestShouldSyncMetadata tests the behavior of the shouldSyncMetadata method.`
	`353`	`+// This test verifies whether metadata synchronization is correctly determined`
	`354`	`+// under different Dataset and AlluxioRuntime configurations.`
	`355`	`+// Test cases include:`
	`356`	`+// 1. A Dataset with metadata already synced should not trigger synchronization.`
	`357`	`+// 2. A Dataset with an empty UfsTotal should trigger synchronization by default.`
	`358`	`+// 3. A Dataset with AutoSync enabled should trigger synchronization.`
	`359`	`+// 4. A Dataset with AutoSync disabled should not trigger synchronization.`
`352`	`360`	`func TestShouldSyncMetadata(t *testing.T) {`
`353`	`361`	`datasetInputs := []datav1alpha1.Dataset{`
`354`	`362`	`{`