Skip to content

Commit ba2ae2f

Browse files
test: make changed2 always differ from original2 in collection generators
With RandGenBool an overwrite keeps the existing value half the time, so when no elements were added or removed, changed2 could equal original2 and the "do not match" assertions failed. Each overwrite now redraws until it differs from the original element.
1 parent 813d44a commit ba2ae2f

1 file changed

Lines changed: 15 additions & 0 deletions

File tree

‎com.unity.netcode.gameobjects/Tests/Runtime/NetworkVariable/NetworkVariableTests.cs‎

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1870,6 +1870,11 @@ public string ArrayStr<T>(NativeArray<T> arr) where T : unmanaged
18701870
{
18711871
var idx = rand.Next(changed2.Length - 1);
18721872
var item = generator(rand);
1873+
// Ensures changed2 differs from original2 when its length matches original2.
1874+
while (idx < original2.Length && EqualityComparer<T>.Default.Equals(item, original2[idx]))
1875+
{
1876+
item = generator(rand);
1877+
}
18731878
changed2[idx] = item;
18741879
}
18751880

@@ -2165,6 +2170,11 @@ public string DictionaryStr<TKey, TVal>(Dictionary<TKey, TVal> list)
21652170
{
21662171
var idx = rand.Next(changed2.Count - 1);
21672172
var item = generator(rand);
2173+
// Ensures changed2 differs from original2 when no elements are added or removed.
2174+
while (EqualityComparer<T>.Default.Equals(item, original2[idx]))
2175+
{
2176+
item = generator(rand);
2177+
}
21682178
changed2[idx] = item;
21692179
}
21702180

@@ -3663,6 +3673,11 @@ public string NativeHashMapStr<TKey, TVal>(NativeHashMap<TKey, TVal> list)
36633673
{
36643674
var idx = rand.Next(changed2.Length - 1);
36653675
var item = generator(rand);
3676+
// Ensures changed2 differs from original2 when no elements are added or removed.
3677+
while (EqualityComparer<T>.Default.Equals(item, original2[idx]))
3678+
{
3679+
item = generator(rand);
3680+
}
36663681
changed2[idx] = item;
36673682
}
36683683

0 commit comments

Comments
 (0)