refactor: move collections to Mirage.Collections (MirageNet#698)

Following .net example, the collections have been moved to a folder
and a namespace.

BREAKING CHANGE: collections moved to Mirage.Collections

Assets/Mirage/Editor/InspectorHelper.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.Reflection;
+using Mirage.Collections;
 using UnityEngine;
 
 namespace Mirage

Assets/Mirage/Editor/NetworkBehaviourInspector.cs

@@ -2,6 +2,7 @@
 using System.Collections;
 using System.Collections.Generic;
 using System.Reflection;
+using Mirage.Collections;
 using UnityEditor;
 using UnityEngine;
 using Mirage.Logging;

Assets/Mirage/Runtime/ISyncObject.cs → Assets/Mirage/Runtime/Collections/ISyncObject.cs

@@ -1,6 +1,6 @@
 using System;
 
-namespace Mirage
+namespace Mirage.Collections
 {
     /// <summary>
     /// A sync object is an object that can synchronize it's state

Assets/Mirage/Runtime/SyncDictionary.cs → Assets/Mirage/Runtime/Collections/SyncDictionary.cs

@@ -3,7 +3,7 @@
 using System.Collections.Generic;
 using JetBrains.Annotations;
 
-namespace Mirage
+namespace Mirage.Collections
 {
     public class SyncIDictionary<TKey, TValue> : IDictionary<TKey, TValue>, ISyncObject, IReadOnlyDictionary<TKey, TValue>
     {

Assets/Mirage/Runtime/SyncList.cs → Assets/Mirage/Runtime/Collections/SyncList.cs

@@ -2,7 +2,7 @@
 using System.Collections;
 using System.Collections.Generic;
 
-namespace Mirage
+namespace Mirage.Collections
 {
     public class SyncList<T> : IList<T>, IReadOnlyList<T>, ISyncObject
     {

Assets/Mirage/Runtime/SyncSet.cs → Assets/Mirage/Runtime/Collections/SyncSet.cs

@@ -2,7 +2,7 @@
 using System.Collections;
 using System.Collections.Generic;
 
-namespace Mirage
+namespace Mirage.Collections
 {
     public class SyncSet<T> : ISet<T>, ISyncObject
     {

Assets/Mirage/Runtime/NetworkBehaviour.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using Cysharp.Threading.Tasks;
+using Mirage.Collections;
 using Mirage.RemoteCalls;
 using UnityEngine;
 using Mirage.Logging;

Assets/Mirage/Weaver/Processors/SyncObjectProcessor.cs

@@ -1,5 +1,6 @@
 using System.Collections.Generic;
 using System.Linq;
+using Mirage.Collections;
 using Mono.Cecil;
 using Mono.Cecil.Cil;
 

Assets/Tests/Runtime/Serializers/SyncDictionaryTest.cs → Assets/Tests/Runtime/Collections/SyncDictionaryTest.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using Mirage.Collections;
 using NSubstitute;
 using NUnit.Framework;
 

Assets/Tests/Runtime/Serializers/SyncListClassTest.cs → Assets/Tests/Runtime/Collections/SyncListClassTest.cs

@@ -1,4 +1,5 @@
 using System.Linq;
+using Mirage.Collections;
 using NUnit.Framework;
 
 namespace Mirage.Tests

Assets/Tests/Runtime/Serializers/SyncListStructTest.cs → Assets/Tests/Runtime/Collections/SyncListStructTest.cs

@@ -1,3 +1,4 @@
+using Mirage.Collections;
 using NUnit.Framework;
 
 namespace Mirage.Tests

Assets/Tests/Runtime/Serializers/SyncListTest.cs → Assets/Tests/Runtime/Collections/SyncListTest.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using Mirage.Collections;
 using NSubstitute;
 using NUnit.Framework;
 

Assets/Tests/Runtime/Serializers/SyncSetTest.cs → Assets/Tests/Runtime/Collections/SyncSetTest.cs

@@ -1,5 +1,6 @@
 using System;
 using System.Collections.Generic;
+using Mirage.Collections;
 using NSubstitute;
 using NUnit.Framework;
 

Assets/Tests/Runtime/Host/NetworkBehaviourTests.cs

@@ -1,3 +1,4 @@
+using Mirage.Collections;
 using NUnit.Framework;
 using UnityEngine;
 

Assets/Tests/Runtime/Serializers/NetworkBehaviourSerializeTest.cs

@@ -1,4 +1,5 @@
 using System.Collections.Generic;
+using Mirage.Collections;
 using NUnit.Framework;
 using UnityEngine;
 

Assets/Tests/Weaver/MonoBehaviourTests.cs

@@ -21,7 +21,7 @@ public void MonoBehaviourSyncVar()
         public void MonoBehaviourSyncList()
         {
             HasError("potato is a SyncObject and must be inside a NetworkBehaviour.  MonoBehaviourSyncList is not a NetworkBehaviour",
-                "Mirage.SyncList`1<System.Int32> MonoBehaviourTests.MonoBehaviourSyncList.MonoBehaviourSyncList::potato");
+                "Mirage.Collections.SyncList`1<System.Int32> MonoBehaviourTests.MonoBehaviourSyncList.MonoBehaviourSyncList::potato");
         }
 
         [Test]

Assets/Tests/Weaver/MonoBehaviourTests~/MonoBehaviourSyncList.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 using UnityEngine;
 
 namespace MonoBehaviourTests.MonoBehaviourSyncList

Assets/Tests/Weaver/SyncDictionaryTests~/SyncDictionary.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncDictionaryTests.SyncDictionary
 {

Assets/Tests/Weaver/SyncDictionaryTests~/SyncDictionaryErrorForGenericStructItem.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncDictionaryTests.SyncDictionaryErrorForGenericStructItem
 {

Assets/Tests/Weaver/SyncDictionaryTests~/SyncDictionaryErrorForGenericStructKey.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncDictionaryTests.SyncDictionaryErrorForGenericStructKey
 {

Assets/Tests/Weaver/SyncDictionaryTests~/SyncDictionaryErrorWhenUsingGenericInNetworkBehaviour.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncDictionaryTests.SyncDictionaryErrorWhenUsingGenericInNetworkBehaviour
 {

Assets/Tests/Weaver/SyncDictionaryTests~/SyncDictionaryGenericAbstractInheritance.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncDictionaryTests.SyncDictionaryGenericAbstractInheritance
 {

Assets/Tests/Weaver/SyncDictionaryTests~/SyncDictionaryGenericInheritance.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncDictionaryTests.SyncDictionaryGenericInheritance
 {

Assets/Tests/Weaver/SyncDictionaryTests~/SyncDictionaryInheritance.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncDictionaryTests.SyncDictionaryInheritance
 {

Assets/Tests/Weaver/SyncDictionaryTests~/SyncDictionaryStructItem.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncDictionaryTests.SyncDictionaryStructItem
 {

Assets/Tests/Weaver/SyncDictionaryTests~/SyncDictionaryStructKey.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncDictionaryTests.SyncDictionaryStructKey
 {

Assets/Tests/Weaver/SyncListTests~/SyncList.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncList
 {

Assets/Tests/Weaver/SyncListTests~/SyncListByteValid.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListByteValid
 {

Assets/Tests/Weaver/SyncListTests~/SyncListErrorForGenericStruct.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListErrorForGenericStruct
 {

Assets/Tests/Weaver/SyncListTests~/SyncListErrorForInterface.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListErrorForInterface
 {

Assets/Tests/Weaver/SyncListTests~/SyncListErrorWhenUsingGenericListInNetworkBehaviour.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListErrorWhenUsingGenericListInNetworkBehaviour
 {

Assets/Tests/Weaver/SyncListTests~/SyncListGenericAbstractInheritance.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListGenericAbstractInheritance
 {

Assets/Tests/Weaver/SyncListTests~/SyncListGenericInheritance.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListGenericInheritance
 {

Assets/Tests/Weaver/SyncListTests~/SyncListGenericInheritanceWithMultipleGeneric.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListGenericInheritanceWithMultipleGeneric
 {

Assets/Tests/Weaver/SyncListTests~/SyncListGenericStructWithCustomMethods.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListGenericStructWithCustomMethods
 {

Assets/Tests/Weaver/SyncListTests~/SyncListInheritance.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListInheritance
 {

Assets/Tests/Weaver/SyncListTests~/SyncListNestedInAbstractClass.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListNestedInAbstractClass
 {

Assets/Tests/Weaver/SyncListTests~/SyncListNestedInAbstractClassWithInvalid.cs

@@ -1,5 +1,6 @@
 using Mirage;
 using UnityEngine;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListNestedInAbstractClassWithInvalid
 {

Assets/Tests/Weaver/SyncListTests~/SyncListNestedInStruct.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListNestedInStruct
 {
@@ -11,7 +12,7 @@ public struct SomeData
         {
             public int usefulNumber;
 
-            public class SyncList : Mirage.SyncList<SomeData> { }
+            public class SyncList : Mirage.Collections.SyncList<SomeData> { }
         }
     }
 }

Assets/Tests/Weaver/SyncListTests~/SyncListNestedInStructWithInvalid.cs

@@ -1,5 +1,6 @@
 using Mirage;
 using UnityEngine;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListNestedInStructWithInvalid
 {
@@ -13,7 +14,7 @@ public struct SomeData
             public int usefulNumber;
             public Object target;
 
-            public class SyncList : Mirage.SyncList<SomeData> { }
+            public class SyncList : Mirage.Collections.SyncList<SomeData> { }
         }
     }
 }

Assets/Tests/Weaver/SyncListTests~/SyncListNestedStruct.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListNestedStruct
 {

Assets/Tests/Weaver/SyncListTests~/SyncListStruct.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncListTests.SyncListStruct
 {

Assets/Tests/Weaver/SyncSetTests~/SyncSetByteValid.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncSetTests.SyncSetByteValid
 {

Assets/Tests/Weaver/SyncSetTests~/SyncSetGenericAbstractInheritance.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncSetTests.SyncSetGenericAbstractInheritance
 {

Assets/Tests/Weaver/SyncSetTests~/SyncSetGenericInheritance.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncSetTests.SyncSetGenericInheritance
 {

Assets/Tests/Weaver/SyncSetTests~/SyncSetInheritance.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncSetTests.SyncSetInheritance
 {

Assets/Tests/Weaver/SyncSetTests~/SyncSetStruct.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncSetTests.SyncSetStruct
 {

Assets/Tests/Weaver/SyncVarTests.cs

@@ -73,7 +73,7 @@ public void SyncVarsCantBeArray()
         [Test]
         public void SyncVarsSyncList()
         {
-            HasWarning("syncints has [SyncVar] attribute. SyncLists should not be marked with SyncVar", "Mirage.SyncList`1<System.Int32> SyncVarTests.SyncVarsSyncList.SyncVarsSyncList::syncints");
+            HasWarning("syncints has [SyncVar] attribute. SyncLists should not be marked with SyncVar", "Mirage.Collections.SyncList`1<System.Int32> SyncVarTests.SyncVarsSyncList.SyncVarsSyncList::syncints");
         }
 
         [Test]

Assets/Tests/Weaver/SyncVarTests~/SyncVarsSyncList.cs

@@ -1,4 +1,5 @@
 using Mirage;
+using Mirage.Collections;
 
 namespace SyncVarTests.SyncVarsSyncList
 {

doc/Articles/Guides/Sync/SyncDictionary.md

@@ -1,10 +1,10 @@
 # SyncDictionary
-A <xref:Mirage.SyncDictionary`2> is an associative array containing an unordered list of key, value pairs. Keys and values can be any of [Mirage supported types](../DataTypes.md).
+A <xref:Mirage.Collections.SyncDictionary`2> is an associative array containing an unordered list of key, value pairs. Keys and values can be any of [Mirage supported types](../DataTypes.md).
 
 SyncDictionary works much like [SyncLists](SyncLists.md): when you make a change on the server, the change is propagated to all clients and the appropriate callback is called.
 
 ## Usage
-Add a field of type `SyncDictionary<TKey, TValue>` on any <xref:Mirage.NetworkBehaviour> where `TKey` and `TValue` can be any supported Mirage type and initialize it.
+Add a field of type <xref:Mirage.Collections.SyncDictionary`2> on any <xref:Mirage.NetworkBehaviour> where `TKey` and `TValue` can be any supported Mirage type and initialize it.
 
 > [!IMPORTANT]
 > You need to initialize the SyncDictionary immediately after definition in order for them to work. You can mark them as `readonly` to enforce proper usage.
@@ -13,6 +13,7 @@ Add a field of type `SyncDictionary<TKey, TValue>` on any <xref:Mirage.NetworkBe
 ```cs
 using UnityEngine;
 using Mirage;
+using Mirage.Collections;
 
 [System.Serializable]
 public struct Item
@@ -43,7 +44,7 @@ public class Player : NetworkBehaviour
 ## Callbacks
 You can detect when a SyncDictionary changes on the client and/or server. This is especially useful for refreshing your UI, character appearance etc.
 
-There are different callbacks for different operations, such as `OnChange` (any change to the dictionary), `OnInsert` (adding new element) etc. Please check the [SyncDictionary API reference](xref:Mirage.SyncIDictionary`2) for the complete list of callbacks.
+There are different callbacks for different operations, such as `OnChange` (any change to the dictionary), `OnInsert` (adding new element) etc. Please check the [SyncDictionary API reference](xref:Mirage.Collections.SyncIDictionary`2) for the complete list of callbacks.
 
 Depending on where you want to invoke the callbacks, you can use these methods to register them:
 - `Awake` for both client and server
@@ -56,6 +57,7 @@ Depending on where you want to invoke the callbacks, you can use these methods t
 ### Example
 ```cs
 using Mirage;
+using Mirage.Collections;
 
 public class Player : NetworkBehaviour {
     public readonly SyncDictionary<stirng, Item> Equipment = new SyncDictionary<string, Item>();