This PR is to add comments toTestJuiceFSEngine_setParallelMigrateOptions in pkg/ddc/juicefs/data_migrate_test.go. (#4574)

1wyllyw1 · web-flow · commit 179e1408d0e6 · 2025-03-09T12:42:45.000Z
* Add Notation to TestJuiceFSEngine_setParallelMigrateOptions in pkg/ddc/juicefs/data_migrate_test.go.

Signed-off-by: 1wyllyw1 &lt;1045851613@qq.com&gt;

* Add Notation to TestJuiceFSEngine_setParallelMigrateOptions in pkg/ddc/juicefs/data_migrate_test.go.

Signed-off-by: 1wyllyw1 &lt;1045851613@qq.com&gt;

---------

Signed-off-by: 1wyllyw1 &lt;1045851613@qq.com&gt;
diff --git a/pkg/ddc/juicefs/data_migrate_test.go b/pkg/ddc/juicefs/data_migrate_test.go
@@ -597,6 +597,12 @@ func TestJuiceFSEngine_genDataUrl(t *testing.T) {
 	}
 }
 
+// TestJuiceFSEngine_setParallelMigrateOptions tests the setParallelMigrateOptions method of JuiceFSEngine.
+// This method is responsible for setting parallel migration options, including SSH port and SSH ready timeout.
+// The test cases cover:
+// 1. Normal scenario where parallel migration options are set correctly.
+// 2. Default values are used when no parallel options are provided.
+// 3. Error scenario where invalid parallel options are provided.
 func TestJuiceFSEngine_setParallelMigrateOptions(t *testing.T) {
 	type args struct {
 		dataMigrateInfo *cdatamigrate.DataMigrateInfo

Original file line number	Diff line number	Diff line change
`@@ -597,6 +597,12 @@ func TestJuiceFSEngine_genDataUrl(t *testing.T) {`
`597`	`597`	`}`
`598`	`598`	`}`
`599`	`599`
	`600`	`+// TestJuiceFSEngine_setParallelMigrateOptions tests the setParallelMigrateOptions method of JuiceFSEngine.`
	`601`	`+// This method is responsible for setting parallel migration options, including SSH port and SSH ready timeout.`
	`602`	`+// The test cases cover:`
	`603`	`+// 1. Normal scenario where parallel migration options are set correctly.`
	`604`	`+// 2. Default values are used when no parallel options are provided.`
	`605`	`+// 3. Error scenario where invalid parallel options are provided.`
`600`	`606`	`func TestJuiceFSEngine_setParallelMigrateOptions(t *testing.T) {`
`601`	`607`	`type args struct {`
`602`	`608`	`dataMigrateInfo *cdatamigrate.DataMigrateInfo`