fix: NetworkRigidbodyBase 2D rigid body issues (#4012)

* fix

Fixing issue where Rigidbody2D was not applying rotation correctly. (4009)
Fixing issue where `NetworkRigidbodyBase` was always checking the 3D rigid body's interpolation mode when determining if it is kinematic and needs to put the rigid body to sleep and then switch to interpolation. (3924)

* style

Removing static using directive (not used).

* update

Missed the return...

* test

Did a bit of an overhaul on the original NetworkRigidbodyTest. It should now run with distributed authority tests and includes a 2D rigidbody in the tests as well.

* stye

Removing trailing space on comment

* test

Updating the NetworkRigidbodyTest to do a 2nd spawn pass that swaps the session owner and non-session owner clients to validate it works when a non-session authority client is the spawn authority.

* update

Removing Rigidbody tests no longer used and already covered by `NetworkRigidbodyTests`.
Moving `RigidbodyContactEventManagerTests` into its own file.

Author: NoelStephensUnity

com.unity.netcode.gameobjects/Runtime/Components/NetworkRigidBodyBase.cs

@@ -2,6 +2,7 @@
 using System.Runtime.CompilerServices;
 using UnityEngine;
 
+
 namespace Unity.Netcode.Components
 {
     /// <summary>
@@ -571,7 +572,7 @@ private void InternalMoveRotation2D(Quaternion rotation)
         {
             var quaternion = Quaternion.identity;
             var angles = quaternion.eulerAngles;
-            angles.z = m_InternalRigidbody2D.rotation;
+            angles.z = rotation.z;
             quaternion.eulerAngles = angles;
             m_InternalRigidbody2D.MoveRotation(quaternion);
         }
@@ -845,6 +846,28 @@ public void SetIsKinematic(bool isKinematic)
             PostSetIsKinematic();
         }
 
+
+        [MethodImpl(MethodImplOptions.AggressiveInlining)]
+        private bool HasInterpolationTypeSet(InterpolationTypes interpolationType)
+        {
+#if COM_UNITY_MODULES_PHYSICS && COM_UNITY_MODULES_PHYSICS2D
+            if (m_IsRigidbody2D)
+            {
+                return interpolationType == InterpolationTypes.Extrapolate ? m_InternalRigidbody2D.interpolation == RigidbodyInterpolation2D.Extrapolate : m_InternalRigidbody2D.interpolation == RigidbodyInterpolation2D.Interpolate;
+            }
+            else
+            {
+                return interpolationType == InterpolationTypes.Extrapolate ? m_InternalRigidbody.interpolation == RigidbodyInterpolation.Extrapolate : m_InternalRigidbody.interpolation == RigidbodyInterpolation.Interpolate;
+            }
+#endif
+#if COM_UNITY_MODULES_PHYSICS && !COM_UNITY_MODULES_PHYSICS2D
+            return interpolationType == InterpolationTypes.Extrapolate ? m_InternalRigidbody2D.interpolation == RigidbodyInterpolation2D.Extrapolate : m_InternalRigidbody2D.interpolation == RigidbodyInterpolation2D.Interpolate;
+#endif
+#if !COM_UNITY_MODULES_PHYSICS && COM_UNITY_MODULES_PHYSICS2D
+            return interpolationType == InterpolationTypes.Extrapolate ? m_InternalRigidbody.interpolation == RigidbodyInterpolation.Extrapolate : m_InternalRigidbody.interpolation == RigidbodyInterpolation.Interpolate;
+#endif
+        }
+
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         private void PostSetIsKinematic()
         {
@@ -853,26 +876,29 @@ private void PostSetIsKinematic()
             {
                 return;
             }
+
             if (UseRigidBodyForMotion)
             {
-                // Only if the NetworkTransform is set to interpolate do we need to check for extrapolation
-                if (NetworkTransform.Interpolate && m_OriginalInterpolation == InterpolationTypes.Extrapolate)
+                // Exit early if the original interpolation type is not set to extrapolate or NetworkTransform interpolate is disabled
+                if (m_OriginalInterpolation != InterpolationTypes.Extrapolate || !NetworkTransform.Interpolate)
                 {
-                    if (IsKinematic())
-                    {
-                        // If not already set to interpolate then set the Rigidbody to interpolate
-                        if (m_InternalRigidbody.interpolation == RigidbodyInterpolation.Extrapolate)
-                        {
-                            // Sleep until the next fixed update when switching from extrapolation to interpolation
-                            SleepRigidbody();
-                            SetInterpolation(InterpolationTypes.Interpolate);
-                        }
-                    }
-                    else
-                    {
-                        // Switch it back to the original interpolation if non-kinematic (doesn't require sleep).
-                        SetInterpolation(m_OriginalInterpolation);
-                    }
+                    return;
+                }
+
+                // Otherwise, if this is the active physics object
+                if (!IsKinematic())
+                {
+                    // switch it back to the original interpolation and exit early
+                    SetInterpolation(m_OriginalInterpolation);
+                    return;
+                }
+
+                // If the Rigidbody or Rigidbody2D is currently configured to extrapolate
+                if (HasInterpolationTypeSet(InterpolationTypes.Extrapolate))
+                {
+                    // sleep until the next fixed update when switching from extrapolation to interpolation
+                    SleepRigidbody();
+                    SetInterpolation(InterpolationTypes.Interpolate);
                 }
             }
             else