Skip to content

Commit 1f5e440

Browse files
committed
test: adding log assert for disconnect test
1 parent 1da26e5 commit 1f5e440

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

Assets/Tests/Runtime/ClientServer/NetworkClientDisconnectTest.cs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,9 @@ public IEnumerator DisconnectEventWhenSentInvalidPacket()
8585
(var clientSocket, var serverEndPoint) = GetSocketAndEndPoint();
8686
var badMessage = CreateInvalidPacket();
8787

88+
// Expect error log because invalid packet triggers server disconnect
89+
LogAssert.Expect(LogType.Error, new System.Text.RegularExpressions.Regex(".*Received invalid fragment. Disconnecting.*"));
90+
8891
clientSocket.Send(serverEndPoint, badMessage.AsSpan(0, 20));
8992

9093
var called = 0;

Assets/Tests/SocketLayer/StatefulPeerTest.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,8 @@ public void SafeDisconnectFromErrorShouldDisconnectStatefulConnection()
116116
const int aboveMTU = MAX_PACKET_SIZE + 10;
117117
socket.AsMock().QueueReceiveCall(new byte[1000], clientStatefulHandle, length: aboveMTU);
118118

119-
LogAssert.Expect(LogType.Error, $"Socket returned length above MTU. MaxPacketSize:{MAX_PACKET_SIZE} length:{aboveMTU}");
119+
// Expect error log because packet length above MTU is invalid and triggers safe disconnect
120+
LogAssert.Expect(LogType.Error, new System.Text.RegularExpressions.Regex($".*Socket returned length above MTU. MaxPacketSize:{MAX_PACKET_SIZE} length:{aboveMTU}"));
120121
peer.UpdateTest();
121122

122123
disconnectAction.Received(1).Invoke(conn, DisconnectReason.InvalidPacket);

0 commit comments

Comments
 (0)