Clean up to remove redundant test case

Signed-off-by: Raunak Madan <[email protected]>

Author: CodexRaunak

integration-tests/meshsync_as_library_with_k8s_cluster_custom_broker_test_cases_test.go

@@ -21,6 +21,9 @@ var meshsyncLibraryWithK8SClusterCustomBrokerTestCaseData []meshsyncLibraryWithK
 	{
 		name: "output mode channel: number of messages received from meshsync is greater than zero",
 		meshsyncRunOptions: []libmeshsync.OptionsSetter{
+			// adding nil to options to test that meshsync can handle nil options setter
+			// must not fail when has nil in options setter
+			nil,
 			libmeshsync.WithOutputMode(config.OutputModeBroker),
 			libmeshsync.WithBrokerHandler(channel.NewChannelBrokerHandler()),
 			libmeshsync.WithStopAfterDuration(8 * time.Second),
@@ -50,18 +53,6 @@ var meshsyncLibraryWithK8SClusterCustomBrokerTestCaseData []meshsyncLibraryWithK
 		},
 	},
 	// TODO
-	// remove this as a separate test case,
-	// add nil to []libmeshsync.OptionsSetter in the previous test case with a comment
-	{
-		name: "output mode channel: must not fail when has nil in options setter",
-		meshsyncRunOptions: []libmeshsync.OptionsSetter{
-			nil,
-			libmeshsync.WithOutputMode(config.OutputModeBroker),
-			libmeshsync.WithBrokerHandler(channel.NewChannelBrokerHandler()),
-			libmeshsync.WithStopAfterDuration(0 * time.Second),
-		},
-	},
-	// TODO
 	// this is not an output mode test
 	// we do not need to run libmeshsync.Run, as we only test call to GetClusterID
 	// we still need k8s cluster in place;