fixing setup

MirageNet · Mar 25, 2021 · d94df03 · d94df03
1 parent 133fe5c
commit d94df03
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/Assets/Tests/Common/LocalConnections.cs b/Assets/Tests/Common/LocalConnections.cs
@@ -2,7 +2,6 @@
 
 namespace Mirage.Tests
 {
-
     public static class LocalConnections
     {
         public static (NetworkPlayer, NetworkPlayer) PipedConnections(IMessageHandler serverHandler, IMessageHandler clientHandler)
@@ -13,6 +12,7 @@ public static (NetworkPlayer, NetworkPlayer) PipedConnections(IMessageHandler se
 
             return (toServer, toClient);
         }
+
         public static (NetworkPlayer, NetworkPlayer) PipedConnections()
         {
             return PipedConnections(Substitute.For<IMessageHandler>(), Substitute.For<IMessageHandler>());

diff --git a/Assets/Tests/Runtime/Serializers/MessageBrokerTest.cs b/Assets/Tests/Runtime/Serializers/MessageBrokerTest.cs
@@ -81,21 +81,21 @@ void ParsePacket(ArraySegment<byte> data)
                 Array.Copy(data.Array, data.Offset, lastSerializedPacket, 0, data.Count);
             }
 
-            connection.Send(
-                Arg.Do<ArraySegment<byte>>(ParsePacket), Channel.Unreliable);
+            // add ParsePacket function to Substitute to validate
+            connection.Send(Arg.Do<ArraySegment<byte>>(ParsePacket), Channel.Unreliable);
 
-            player = new NetworkPlayer(connection, Substitute.For<IMessageHandler>());
+            player = Substitute.For<INetworkPlayer>();
+            player.Connection.Returns(connection);
 
             serializedMessage = MessagePacker.Pack(new ReadyMessage());
-            //messageBroker.RegisterHandler<ReadyMessage>(message => { });
+            messageBroker.RegisterHandler<ReadyMessage>(message => { });
 
             delivered = Substitute.For<Action<INetworkPlayer, object>>();
             lost = Substitute.For<Action<INetworkPlayer, object>>();
 
             messageBroker.NotifyDelivered += delivered;
             messageBroker.NotifyLost += lost;
         }
-        #region Notify
 
 
         [Test]
@@ -282,7 +282,5 @@ public void NotAcknowledgedYet()
 
             delivered.Received().Invoke(player, 5);
         }
-
-        #endregion
     }
 }