150-1 PVP fixes

Author: michalChrobot

com.unity.netcode.gameobjects/TestHelpers/Runtime/NetcodeIntegrationTestHelpers.cs

@@ -425,7 +425,6 @@ private static void SceneManagerValidationAndTestRunnerInitialization(NetworkMan
         /// <param name="server">The Server NetworkManager</param>
         /// <param name="clients">The Clients NetworkManager</param>
         /// <param name="callback">called immediately after server is started and before client(s) are started</param>
-        /// <returns></returns>
         public static bool Start(bool host, NetworkManager server, NetworkManager[] clients, BeforeClientStartCallback callback = null)
         {
             if (s_IsStarted)

com.unity.netcode.gameobjects/Tests/Runtime/Components/BufferDataValidationComponent.cs

@@ -54,7 +54,6 @@ private void Start()
         /// <summary>
         /// Returns back whether the test has completed the total number of iterations
         /// </summary>
-        /// <returns></returns>
         public bool IsTestComplete()
         {
             if (m_CurrentBufferSize > MaximumBufferSize || TestFailed)

com.unity.netcode.gameobjects/Tests/Runtime/NetworkVariableBaseInitializesWhenPersisted.cs

@@ -334,6 +334,7 @@ public void Initialize(NetworkManager networkManager)
             /// <summary>
             /// This validates that the instances persisted to the next test set and persisted
             /// between network sessions
+            /// </summary>
             public bool ValidateInstanceSpawnCount(int minCount)
             {
                 // First pass we should have no instances

pvpExceptions.json

@@ -10,14 +10,6 @@
           "Failed to find a suitable OpenCL device, baking cannot use GPU lightmapper."
         ]
       },
-      "PVP-150-1": {
-        "errors": [
-          "Unity.Netcode.TestHelpers.Runtime.NetcodeIntegrationTestHelpers: bool Start(bool, NetworkManager, NetworkManager[], BeforeClientStartCallback): empty <returns> tag",
-          "Unity.Netcode.TestHelpers.Runtime.NetcodeIntegrationTestHelpers: IEnumerator WaitForClientsConnected(NetworkManager[], ResultWrapper<bool>, float): XML is not well-formed: An identifier was expected",
-          "Unity.Netcode.RuntimeTests.BufferDataValidationComponent: bool IsTestComplete(): empty <returns> tag",
-          "Unity.Netcode.RuntimeTests.NetworkVariableBaseInitializesWhenPersisted.PrefabInstanceHandler: bool ValidateInstanceSpawnCount(int): XML is not well-formed: Expected an end tag for element 'summary'"
-        ]
-      },
       "PVP-151-1": {
         "errors": [
           "Unity.Netcode.NetworkConfig: Prefabs: undocumented",