Fixes alterative valid orderings for tests (#5904)

The unit tests can have multiple valid expectations that are not 100% deterministic depending on ordering used when looping through maps in go.

This updates the test cases to allow multiple expected valid values.

Author: wbreza

cli/azd/pkg/project/importer_test.go

@@ -505,12 +505,12 @@ func TestImportManagerServiceStableWithDependencyOrdering(t *testing.T) {
 	})
 
 	tests := []struct {
-		name        string
-		services    map[string]*ServiceConfig
-		resources   map[string]*ResourceConfig
-		expected    []string
-		shouldError bool
-		errorMsg    string
+		name               string
+		services           map[string]*ServiceConfig
+		resources          map[string]*ResourceConfig
+		expectedVariations [][]string // All valid orderings (single slice for deterministic cases)
+		shouldError        bool
+		errorMsg           string
 	}{
 		{
 			name: "no dependencies - alphabetical order maintained",
@@ -519,7 +519,9 @@ func TestImportManagerServiceStableWithDependencyOrdering(t *testing.T) {
 				"alpha": {Name: "alpha", Uses: []string{}},
 				"beta":  {Name: "beta", Uses: []string{}},
 			},
-			expected: []string{"alpha", "beta", "zebra"}, // Alphabetical order when no dependencies
+			expectedVariations: [][]string{
+				{"alpha", "beta", "zebra"}, // Alphabetical order when no dependencies
+			},
 		},
 		{
 			name: "simple dependency chain",
@@ -528,7 +530,9 @@ func TestImportManagerServiceStableWithDependencyOrdering(t *testing.T) {
 				"backend":  {Name: "backend", Uses: []string{"database"}},
 				"database": {Name: "database", Uses: []string{}},
 			},
-			expected: []string{"database", "backend", "frontend"},
+			expectedVariations: [][]string{
+				{"database", "backend", "frontend"},
+			},
 		},
 		{
 			name: "complex dependencies",
@@ -539,7 +543,10 @@ func TestImportManagerServiceStableWithDependencyOrdering(t *testing.T) {
 				"storage":  {Name: "storage", Uses: []string{"database"}},
 				"database": {Name: "database", Uses: []string{}},
 			},
-			expected: []string{"database", "auth", "storage", "api", "web"},
+			expectedVariations: [][]string{
+				{"database", "auth", "storage", "api", "web"}, // Original expected order
+				{"database", "storage", "auth", "api", "web"}, // Alternative valid order
+			},
 		},
 		{
 			name: "service depending on resource",
@@ -550,7 +557,9 @@ func TestImportManagerServiceStableWithDependencyOrdering(t *testing.T) {
 			resources: map[string]*ResourceConfig{
 				"database": {Name: "database", Type: "db.postgres"},
 			},
-			expected: []string{"api", "web"}, // Resource dependencies don't affect service ordering
+			expectedVariations: [][]string{
+				{"api", "web"}, // Resource dependencies don't affect service ordering
+			},
 		},
 		{
 			name: "circular dependency",
@@ -593,12 +602,26 @@ func TestImportManagerServiceStableWithDependencyOrdering(t *testing.T) {
 				require.Contains(t, err.Error(), tt.errorMsg)
 			} else {
 				require.NoError(t, err)
-				require.Len(t, result, len(tt.expected))
+				require.Len(t, result, len(tt.expectedVariations[0]))
+
+				// Get the actual service names for comparison
+				actualOrder := make([]string, len(result))
+				for i, svc := range result {
+					actualOrder[i] = svc.Name
+				}
+
+				// Check if the actual order matches any of the expected variations
+				matchesAnyVariation := false
+				for _, expectedVariation := range tt.expectedVariations {
+					if slices.Equal(actualOrder, expectedVariation) {
+						matchesAnyVariation = true
+						break
+					}
+				}
 
-				// Check exact order for all test cases
-				for i, expected := range tt.expected {
-					assert.Equal(t, expected, result[i].Name,
-						"Service at position %d should be %s, got %s", i, expected, result[i].Name)
+				if !matchesAnyVariation {
+					t.Errorf("Actual order %v does not match any expected variations: %v",
+						actualOrder, tt.expectedVariations)
 				}
 			}
 		})