Fix test that passes a nil mutator to 'move'

dlipovetsky · dlipovetsky · commit e143580d4fb6 · 2024-07-26T14:20:43.000-07:00
The call is incorrect, but the test does not fail, because it enables
dry-run, and that does not exercise the mutators.
diff --git a/cmd/clusterctl/client/cluster/mover_test.go b/cmd/clusterctl/client/cluster/mover_test.go
@@ -1203,7 +1203,7 @@ func Test_objectMover_move_dryRun(t *testing.T) {
 				dryRun:    true,
 			}
 
-			err := mover.move(ctx, graph, toProxy, nil)
+			err := mover.move(ctx, graph, toProxy)
 			if tt.wantErr {
 				g.Expect(err).To(HaveOccurred())
 				return

Original file line number	Diff line number	Diff line change
`@@ -1203,7 +1203,7 @@ func Test_objectMover_move_dryRun(t *testing.T) {`
`1203`	`1203`	`dryRun: true,`
`1204`	`1204`	`}`
`1205`	`1205`
`1206`		`- err := mover.move(ctx, graph, toProxy, nil)`
	`1206`	`+ err := mover.move(ctx, graph, toProxy)`
`1207`	`1207`	`if tt.wantErr {`
`1208`	`1208`	`g.Expect(err).To(HaveOccurred())`
`1209`	`1209`	`return`