sort set policy values before validation

Azure · Jan 21, 2025 · 55c1158 · 55c1158
1 parent c670d53
commit 55c1158
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/npm/pkg/dataplane/ipsets/ipsetmanager_windows_test.go b/npm/pkg/dataplane/ipsets/ipsetmanager_windows_test.go
@@ -2,6 +2,8 @@ package ipsets
 
 import (
 	"fmt"
+	"sort"
+	"strings"
 	"testing"
 
 	"github.com/Azure/azure-container-networking/common"
@@ -379,7 +381,14 @@ func verifyHNSCache(t *testing.T, expected map[string]hcn.SetPolicySetting, hns
 	for setName, setObj := range expected {
 		cacheObj := hns.Cache.SetPolicy(setObj.Id)
 		require.NotNil(t, cacheObj)
-		require.Equal(t, setObj, *cacheObj, fmt.Sprintf("%s mismatch in cache", setName))
+
+		// make values always sorted for testing consistency
+		members := strings.Split(cacheObj.Values, ",")
+		sort.Strings(members)
+		copyOfCachedObj := *cacheObj
+		copyOfCachedObj.Values = strings.Join(members, ",")
+
+		require.Equal(t, setObj, copyOfCachedObj, fmt.Sprintf("%s mismatch in cache", setName))
 	}
 }