Fix the tests

Author: EmandM

com.unity.netcode.gameobjects/Runtime/Logging/LogBuilder.cs

@@ -7,7 +7,7 @@ internal class LogBuilder
         private readonly StringBuilder m_Builder = new();
         private const string k_OpenBracket = "[";
         private const string k_CloseBracket = "]";
-        private const string k_Separator = ": ";
+        private const string k_Separator = ":";
 
         public void Reset()
         {

com.unity.netcode.gameobjects/Runtime/Messaging/Messages/RpcMessages.cs

@@ -36,7 +36,6 @@ public static unsafe bool Deserialize(ref FastBufferReader reader, ref NetworkCo
         public static void Handle(ref NetworkContext context, ref RpcMetadata metadata, ref FastBufferReader payload, ref __RpcParams rpcParams)
         {
             var networkManager = (NetworkManager)context.SystemOwner;
-            using var logContext = networkManager.Log.AddDisposableInfo("SenderClientId", context.SenderId);
 
             if (!networkManager.SpawnManager.SpawnedObjects.TryGetValue(metadata.NetworkObjectId, out var networkObject))
             {
@@ -68,7 +67,7 @@ public static void Handle(ref NetworkContext context, ref RpcMetadata metadata,
             if ((permission == RpcInvokePermission.Server && rpcParams.SenderId != NetworkManager.ServerClientId) ||
                 (permission == RpcInvokePermission.Owner && rpcParams.SenderId != networkObject.OwnerClientId))
             {
-                networkManager.Log.ErrorServer(new Context(LogLevel.Normal, "Rpc message received from a client permission to perform this operation!. Dropping RPC message").AddNetworkBehaviour(networkBehaviour));
+                networkManager.Log.ErrorServer(new Context(LogLevel.Normal, "Rpc message received from a client without permission to perform this operation!. Dropping RPC message").AddNetworkBehaviour(networkBehaviour));
                 return;
             }
 

com.unity.netcode.gameobjects/Tests/Runtime/Rpc/RpcInvocationTests.cs

@@ -139,7 +139,16 @@ public IEnumerator RpcInvokePermissionSendingTests()
         [UnityTest]
         public IEnumerator RpcInvokePermissionReceivingTests()
         {
-            var firstClient = GetNonAuthorityNetworkManager(0);
+            NetworkManager firstClient = null;
+            foreach (var networkManager in m_NetworkManagers)
+            {
+                if (firstClient == null && !networkManager.IsServer && !networkManager.LocalClient.IsSessionOwner)
+                {
+                    firstClient = networkManager;
+                }
+
+                networkManager.LogLevel = LogLevel.Error;
+            }
 
             var spawnedObject = SpawnObject(m_Prefab, firstClient).GetComponent<NetworkObject>();