fix unit_test

Signed-off-by: Suleiman Dibirov <[email protected]>

Author: idsulik

pkg/skaffold/deploy/helm/util.go

@@ -60,7 +60,7 @@ func VerifyNoCycles(graph map[string][]string) error {
 // ensuring that releases are deployed after their dependencies.
 func calculateDeploymentOrder(graph map[string][]string) ([]string, error) {
 	visited := make(map[string]bool)
-	var order []string
+	order := make([]string, 0)
 
 	var visit func(node string) error
 	visit = func(node string) error {

pkg/skaffold/deploy/helm/util_test.go

@@ -6,7 +6,6 @@ import (
 
 	"github.com/GoogleContainerTools/skaffold/v2/pkg/skaffold/schema/latest"
 	"github.com/GoogleContainerTools/skaffold/v2/testutil"
-	"github.com/stretchr/testify/require"
 )
 
 func TestBuildDependencyGraph(t *testing.T) {
@@ -65,12 +64,11 @@ func TestBuildDependencyGraph(t *testing.T) {
 			graph, err := BuildDependencyGraph(test.releases)
 
 			if test.shouldErr {
-				require.Error(t, err)
+				t.CheckError(true, err)
 				return
 			}
 
-			require.NoError(t, err)
-			require.Equal(t, test.expected, graph)
+			t.CheckErrorAndDeepEqual(false, err, test.expected, graph)
 		})
 	}
 }
@@ -137,10 +135,9 @@ func TestVerifyNoCycles(t *testing.T) {
 			err := VerifyNoCycles(test.graph)
 
 			if test.shouldErr {
-				require.Error(t, err)
-				require.Contains(t, err.Error(), "cycle detected")
+				t.CheckErrorContains("cycle detected", err)
 			} else {
-				require.NoError(t, err)
+				t.RequireNoError(err)
 			}
 		})
 	}
@@ -199,24 +196,22 @@ func TestCalculateDeploymentOrder(t *testing.T) {
 	}
 
 	for _, test := range tests {
-		t.Run(test.description, func(t *testing.T) {
+		testutil.Run(t, test.description, func(t *testutil.T) {
 			order, err := calculateDeploymentOrder(test.graph)
 
 			if test.shouldErr {
-				require.Error(t, err)
+				t.CheckError(true, err)
 				return
 			}
 
-			require.NoError(t, err)
-			require.Equal(t, len(test.expected), len(order), "deployment order length mismatch")
+			t.CheckErrorAndDeepEqual(false, err, test.expected, order)
 
 			// Verify order satisfies dependencies
 			installed := make(map[string]bool)
 			for _, release := range order {
 				// Check all dependencies are installed
 				for _, dep := range test.graph[release] {
-					require.True(t, installed[dep],
-						"release %s deployed before dependency %s", release, dep)
+					t.CheckTrue(installed[dep])
 				}
 				installed[release] = true
 			}
@@ -297,14 +292,13 @@ func TestGroupReleasesByLevel(t *testing.T) {
 	}
 
 	for _, test := range tests {
-		t.Run(test.description, func(t *testing.T) {
+		testutil.Run(t, test.description, func(t *testutil.T) {
 			levels := groupReleasesByLevel(test.order, test.graph)
 
-			require.Equal(t, len(test.expected), len(levels), "number of levels mismatch")
+			t.CheckDeepEqual(len(test.expected), len(levels))
 
 			for level, releases := range test.expected {
-				require.ElementsMatch(t, releases, levels[level],
-					"releases at level %d don't match", level)
+				t.CheckDeepEqual(releases, levels[level])
 			}
 
 			// Verify level assignments are correct
@@ -314,9 +308,9 @@ func TestGroupReleasesByLevel(t *testing.T) {
 					for _, dep := range test.graph[release] {
 						for depLevel, depReleases := range levels {
 							if slices.Contains(depReleases, dep) {
-								require.Less(t, depLevel, level,
-									"dependency %s at level %d >= release %s at level %d",
-									dep, depLevel, release, level)
+								if depLevel >= level {
+									t.Errorf("dependency %s at level %d >= release %s at level %d", dep, depLevel, release, level)
+								}
 							}
 						}
 					}