Skip to content

Commit 7e6c91b

Browse files
test - fix
The server no longer sends the OwnershipChangeMessage to itself. The TrackOwnershipChangeSentMetric test was processing that extra message to just get to the actual ChangeOwnershipMessage metric. I removed that portion of this test.
1 parent 1c8d423 commit 7e6c91b

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

com.unity.netcode.gameobjects/Tests/Runtime/Metrics/OwnershipChangeMetricsTests.cs

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -73,13 +73,6 @@ public IEnumerator TrackOwnershipChangeSentMetric()
7373

7474
var ownershipChangeSent = metricValues.First();
7575
Assert.AreEqual(networkObject.NetworkObjectId, ownershipChangeSent.NetworkId.NetworkId);
76-
Assert.AreEqual(Server.LocalClientId, ownershipChangeSent.Connection.Id);
77-
Assert.AreEqual(0, ownershipChangeSent.BytesCount);
78-
79-
// The first metric is to the server(self), so its size is now correctly reported as 0.
80-
// Let's check the last one instead, to have a valid value
81-
ownershipChangeSent = metricValues.Last();
82-
Assert.AreEqual(networkObject.NetworkObjectId, ownershipChangeSent.NetworkId.NetworkId);
8376
Assert.AreEqual(Client.LocalClientId, ownershipChangeSent.Connection.Id);
8477

8578
var serializedLength = GetWriteSizeForOwnerChange(networkObject, 1);

0 commit comments

Comments
 (0)