remove unnecessary wrapper

Signed-off-by: Geoff Flarity <[email protected]>

Author: gflarity

operator/e2e/tests/gang_scheduling_test.go

@@ -21,6 +21,8 @@ package tests
 import (
 	"context"
 	"testing"
+
+	"github.com/ai-dynamo/grove/operator/e2e/utils"
 )
 
 // Test_GS1_GangSchedulingWithFullReplicas tests gang-scheduling behavior with insufficient resources
@@ -149,7 +151,7 @@ func Test_GS2_GangSchedulingWithScalingFullReplicas(t *testing.T) {
 		t.Fatalf("Failed to wait for scaled pods to be created: %v", err)
 	}
 
-	if err := verifyPodPhases(tc, pods, expectedPods, 4); err != nil {
+	if err := utils.VerifyPodPhases(pods, expectedPods, 4); err != nil {
 		t.Fatalf("Pod phase verification failed: %v", err)
 	}
 

operator/e2e/tests/setup.go

@@ -127,11 +127,6 @@ func waitForPodCountAndPhases(tc TestContext, expectedTotal, expectedRunning, ex
 	return utils.WaitForPodCountAndPhases(tc.Ctx, tc.Clientset, tc.Namespace, tc.getLabelSelector(), expectedTotal, expectedRunning, expectedPending, tc.Timeout, tc.Interval)
 }
 
-// verifyPodPhases is a wrapper around utils.VerifyPodPhases that accepts TestContext
-func verifyPodPhases(tc TestContext, pods *v1.PodList, expectedRunning, expectedPending int) error {
-	return utils.VerifyPodPhases(pods, expectedRunning, expectedPending)
-}
-
 // prepareTestCluster is a helper function that prepares the shared cluster for a test
 // with the specified number of worker nodes and returns the necessary clients and a cleanup function.
 // The cleanup function will fatally fail the test if workload cleanup fails.