Test-update

Some general clean up and separating areas of the NetworkTransformOrderOfOperations by region to help with navigating through the different parts.

Author: NoelStephensUnity

com.unity.netcode.gameobjects/Tests/Runtime/NetworkTransform/NetworkTransformOrderOfOperations.cs

@@ -16,6 +16,7 @@ namespace Unity.Netcode.RuntimeTests
     [TestFixture(HostOrServer.Server)]
     internal class NetworkTransformOrderOfOperations : IntegrationTestWithApproximation
     {
+        #region Configuration, Properties, and Overrides
         protected override int NumberOfClients => 4;
 
         private NetworkObject m_GenericObject;
@@ -30,6 +31,12 @@ public NetworkTransformOrderOfOperations(HostOrServer host) : base(host)
         {
         }
 
+        protected override IEnumerator OnSetup()
+        {
+            SpawnSequenceController.Clear();
+            return base.OnSetup();
+        }
+
         protected override void OnServerAndClientsCreated()
         {
             m_ObjectToTest = CreateNetworkObjectPrefab("TestObject").GetComponent<NetworkObject>();
@@ -40,6 +47,9 @@ protected override void OnServerAndClientsCreated()
             base.OnServerAndClientsCreated();
         }
 
+        #endregion
+
+        #region (Wait for) Conditional Methods
         private bool VerifyGenericsSpawned(StringBuilder errorLog)
         {
             var conditionMet = true;
@@ -81,12 +91,6 @@ private IEnumerator SpawnGenericParents()
             AssertOnTimeout("Failure to spawn generics on one or more clients!");
         }
 
-        protected override IEnumerator OnSetup()
-        {
-            SpawnSequenceController.Clear();
-            return base.OnSetup();
-        }
-
         private bool TransformsMatch(StringBuilder errorLog)
         {
             var hasErrors = false;
@@ -146,7 +150,9 @@ private bool TransformsMatch(StringBuilder errorLog)
             }
             return !hasErrors;
         }
+        #endregion
 
+        #region OrderOfOperations Core Test Methods
         [UnityTest]
 
         public IEnumerator OrderOfOperations()
@@ -260,7 +266,9 @@ private IEnumerator RunTestSequences(bool spawnWithObservers = true, bool spawnW
             // Reset the controller's global settings
             SpawnSequenceController.Clear();
         }
+        #endregion
 
+        #region Test Sequence Configurations
         /// <summary>
         /// Test-1:
         /// Authority-> Spawn, change ownership, (wait), parent
@@ -623,7 +631,9 @@ private void ConfigureSequencesTest10(NetworkObject parent1, NetworkObject paren
             SpawnSequenceController.AddAction(parentSequence1);
             SpawnSequenceController.AddAction(parentRpc);
         }
+        #endregion
 
+        #region Sequence Class Definitions
         internal class NetworkShowSequence : SpawnSequence
         {
             public List<ulong> Clients = new List<ulong>();
@@ -954,7 +964,6 @@ public void TeleportRpc(Vector3 position, Quaternion rotation, RpcParams rpcPara
             }
         }
 
-
         public class ReferenceRpcHelper : NetworkBehaviour
         {
             [Rpc(SendTo.NotMe)]
@@ -993,5 +1002,6 @@ public void ReferenceParentRpc(NetworkObjectReference targetChildReference, Netw
                 networkObjectChild.TrySetParent(networkObjectParent);
             }
         }
+        #endregion
     }
 }