fix: InvalidDataException not found problem

Revert "Use standard exception instead of our own"

This reverts commit 10d2a1e.

Assets/Mirror/Runtime/InvalidMessageException.cs

@@ -0,0 +1,25 @@
+using System;
+using System.Runtime.Serialization;
+
+namespace Mirror
+{
+
+    [Serializable]
+    public class InvalidMessageException : Exception
+    {
+        public InvalidMessageException()
+        {
+        }
+
+        public InvalidMessageException(string message) : base(message)
+        {
+        }
+
+        public InvalidMessageException(string message, Exception innerException) : base(message, innerException)
+        {
+        }
+        protected InvalidMessageException(SerializationInfo info, StreamingContext context) : base(info, context)
+        {
+        }
+    }
+}

Assets/Mirror/Runtime/NetworkIdentity.cs

@@ -8,7 +8,6 @@
 using UnityEngine.Events;
 #if UNITY_EDITOR
 using UnityEditor;
-using System.IO;
 #if UNITY_2018_3_OR_NEWER
 using UnityEditor.Experimental.SceneManagement;
 #endif
@@ -955,7 +954,7 @@ void OnDeserializeSafely(NetworkBehaviour comp, NetworkReader reader, bool initi
             byte barrierData = reader.ReadByte();
             if (barrierData != Barrier)
             {
-                throw new InvalidDataException("OnDeserialize failed for: object=" + name + " component=" + comp.GetType() + " sceneId=" + sceneId + ". Possible Reasons:\n  * Do " + comp.GetType() + "'s OnSerialize and OnDeserialize calls write the same amount of data? \n  * Was there an exception in " + comp.GetType() + "'s OnSerialize/OnDeserialize code?\n  * Are the server and client the exact same project?\n  * Maybe this OnDeserialize call was meant for another GameObject? The sceneIds can easily get out of sync if the Hierarchy was modified only in the client OR the server. Try rebuilding both.\n\n");
+                throw new InvalidMessageException("OnDeserialize failed for: object=" + name + " component=" + comp.GetType() + " sceneId=" + sceneId + ". Possible Reasons:\n  * Do " + comp.GetType() + "'s OnSerialize and OnDeserialize calls write the same amount of data? \n  * Was there an exception in " + comp.GetType() + "'s OnSerialize/OnDeserialize code?\n  * Are the server and client the exact same project?\n  * Maybe this OnDeserialize call was meant for another GameObject? The sceneIds can easily get out of sync if the Hierarchy was modified only in the client OR the server. Try rebuilding both.\n\n");
             }
         }
 

Assets/Tests/Editor/InvalidMessageExceptionTest.cs

@@ -0,0 +1,40 @@
+using NUnit.Framework;
+
+namespace Mirror.Tests
+{
+    [TestFixture]
+    public class InvalidMessageExceptionTest
+    {
+        [Test]
+        public void InvalidMessageTest()
+        {
+            Assert.Throws<InvalidMessageException>(() =>
+            {
+                throw new InvalidMessageException();
+            });
+        }
+
+        [Test]
+        public void InvalidMessageWithTextTest()
+        {
+            InvalidMessageException ex = Assert.Throws<InvalidMessageException>(() =>
+            {
+                throw new InvalidMessageException("Test Message");
+            });
+
+            Assert.That(ex.Message, Is.EqualTo("Test Message"));
+        }
+
+        [Test]
+        public void InvalidMessageWithTextAndInnerTest()
+        {
+            InvalidMessageException ex = Assert.Throws<InvalidMessageException>(() =>
+            {
+                throw new InvalidMessageException("Test Message Too", new System.Exception());
+            });
+
+            Assert.That(ex.Message, Is.EqualTo("Test Message Too"));
+            Assert.That(ex.InnerException, Is.TypeOf(typeof(System.Exception)));
+        }
+    }
+}

Assets/Tests/Editor/NetworkIdentityTests.cs

@@ -10,7 +10,6 @@
 using UnityEngine.Events;
 
 using static Mirror.Tests.LocalConnections;
-using System.IO;
 
 namespace Mirror.Tests
 {
@@ -653,7 +652,7 @@ public void OnDeserializeSafelyShouldDetectAndHandleDeSerializationMismatch()
 
             // deserialize all
             var reader = new NetworkReader(ownerWriter.ToArray());
-            Assert.Throws<InvalidDataException>(() =>
+            Assert.Throws<InvalidMessageException>(() =>
             {
                 identity.OnDeserializeAllSafely(reader, true);
             });