style

Removing commented out code that is no longer needed/used.

Author: NoelStephensUnity

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

@@ -356,11 +356,6 @@ public bool IsTestComplete()
             return m_FinishedTests;
         }
 
-        public void Awake()
-        {
-            //InitializeTest();
-        }
-
         public void AssertAllValuesAreCorrect()
         {
             Assert.AreEqual(false, m_NetworkVariableBool.Value);

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

@@ -25,25 +25,6 @@ internal override bool ShouldCreatePlayerPrefab()
             return false;
         }
 
-        ///// <summary>
-        ///// Assure no player prefab is assigned.
-        ///// </summary>
-        //protected override void OnServerAndClientsCreated()
-        //{
-        //    foreach (var networkManager in m_NetworkManagers)
-        //    {
-        //        networkManager.NetworkConfig.Prefabs.Remove(m_PlayerPrefab);
-        //        networkManager.NetworkConfig.PlayerPrefab = null;
-        //    }
-        //    base.OnServerAndClientsCreated();
-        //}
-
-        //protected override void OnNewClientCreated(NetworkManager networkManager)
-        //{
-        //    networkManager.NetworkConfig.PlayerPrefab = null;
-        //    base.OnNewClientCreated(networkManager);
-        //}
-
         /// <summary>
         /// Do not wait for spawned players as there are none.
         /// </summary>

com.unity.netcode.gameobjects/Tests/Runtime/NetworkVariable/NetworkVariableTests.cs

@@ -368,20 +368,6 @@ public void AllNetworkVariableTypes([Values] HostOrServer useHost)
                 StopOneClientWithTimeTravel(networkManager);
             }
 
-            // Create, instantiate, and host
-            // This would normally go in Setup, but since every other test but this one
-            //  uses NetworkManagerHelper, and it does its own NetworkManager setup / teardown,
-            //  for now we put this within this one test until we migrate it to MIH
-            //Assert.IsTrue(NetworkManagerHelper.StartNetworkManager(out NetworkManager server, useHost == HostOrServer.Host ? NetworkManagerHelper.NetworkManagerOperatingMode.Host : NetworkManagerHelper.NetworkManagerOperatingMode.Server));
-
-            //Assert.IsTrue(server.IsHost == (useHost == HostOrServer.Host), $"{nameof(useHost)} does not match the server.IsHost value!");
-
-            //Guid gameObjectId = NetworkManagerHelper.AddGameNetworkObject("NetworkVariableTestComponent");
-
-            //var networkVariableTestComponent = NetworkManagerHelper.AddComponentToObject<NetworkVariableTestComponent>(gameObjectId);
-
-            //NetworkManagerHelper.SpawnNetworkObject(gameObjectId);
-
             var instance = SpawnObject(prefabToSpawn, authority);
             var networkVariableTestComponent = instance.GetComponent<NetworkVariableTestComponent>();
 
@@ -400,16 +386,8 @@ public void AllNetworkVariableTypes([Values] HostOrServer useHost)
             Assert.IsTrue(networkVariableTestComponent.DidAllValuesChange());
             networkVariableTestComponent.AssertAllValuesAreCorrect();
 
-            // Disable this once we are done.
-
-
-
-
-            // This would normally go in Teardown, but since every other test but this one
-            //  uses NetworkManagerHelper, and it does its own NetworkManager setup / teardown,
-            //  for now we put this within this one test until we migrate it to MIH
+            // Stop the authority NetworkManager instance
             StopOneClientWithTimeTravel(authority);
-            //NetworkManagerHelper.ShutdownNetworkManager();
         }
 
         [Test]