fix: add comments and backing field for time (#618)

* fix: add comments and backing field for time

* readonly to fix code smell

* add tests

Assets/Mirage/Runtime/NetworkClient.cs

@@ -82,7 +82,14 @@ public class NetworkClient : MonoBehaviour, INetworkClient
         /// </summary>
         public bool IsConnected => connectState == ConnectState.Connected;
 
-        public readonly NetworkTime Time = new NetworkTime();
+        readonly NetworkTime _time = new NetworkTime();
+        /// <summary>
+        /// Time kept in this client
+        /// </summary>
+        public NetworkTime Time
+        {
+            get { return _time; }
+        }
 
         /// <summary>
         /// NetworkClient can connect to local server in host mode too

Assets/Mirage/Runtime/NetworkServer.cs

@@ -122,8 +122,14 @@ public class NetworkServer : MonoBehaviour, INetworkServer
         /// </summary>
         public bool Active { get; private set; }
 
-        // Time kept in this server
-        public readonly NetworkTime Time = new NetworkTime();
+        readonly NetworkTime _time = new NetworkTime();
+        /// <summary>
+        /// Time kept in this server
+        /// </summary>
+        public NetworkTime Time
+        {
+            get { return _time; }
+        }
 
         /// <summary>
         /// This shuts down the server and disconnects all clients.

Assets/Tests/Runtime/Host/NetworkClientTest.cs

@@ -79,5 +79,11 @@ public void DisconnectedNotNullTest()
         {
             Assert.That(client.Disconnected, Is.Not.Null);
         }
+
+        [Test]
+        public void TimeNotNullTest()
+        {
+            Assert.That(client.Time, Is.Not.Null);
+        }
     }
 }

Assets/Tests/Runtime/Host/NetworkServerTest.cs

@@ -82,5 +82,11 @@ public void OnStopHostNotNullTest()
         {
             Assert.That(server.OnStopHost, Is.Not.Null);
         }
+
+        [Test]
+        public void TimeNotNullTest()
+        {
+            Assert.That(server.Time, Is.Not.Null);
+        }
     }
 }