Marked IsSyncInProgress as a public deprecated method

ably · Apr 19, 2024 · 1ded60a · 1ded60a
1 parent 8be657f
commit 1ded60a
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 7 deletions.
diff --git a/src/IO.Ably.Shared/Realtime/Presence.cs b/src/IO.Ably.Shared/Realtime/Presence.cs
@@ -43,18 +43,24 @@ internal Presence(IConnectionManager connection, RealtimeChannel channel, string
         /// </summary>
         ///
         [Obsolete("This property is deprecated, use SyncComplete instead")]
-        public bool IsSyncComplete => SyncComplete; // RTP13
+        public bool IsSyncComplete => SyncComplete; // RTP13.
 
         /// <summary>
         /// Checks if presence sync has ended.
         /// </summary>
         ///
-        public bool SyncComplete => MembersMap.SyncCompleted && !IsSyncInProgress; // RTP13
+        public bool SyncComplete => MembersMap.SyncCompleted && !SyncInProgress; // RTP13
 
         /// <summary>
         /// Indicates whether there is currently a sync in progress.
         /// </summary>
-        internal bool IsSyncInProgress => MembersMap.SyncInProgress;
+        [Obsolete("This property is internal, will be removed in the future")]
+        public bool IsSyncInProgress => SyncInProgress;
+
+        /// <summary>
+        /// Indicates whether there is currently a sync in progress.
+        /// </summary>
+        internal bool SyncInProgress => MembersMap.SyncInProgress;
 
         /// <summary>
         /// Indicates all members present on the channel.
@@ -540,7 +546,7 @@ internal void OnSyncMessage(ProtocolMessage protocolMessage)
                 /* If a new sequence identifier is sent from Ably, then the client library
                  * must consider that to be the start of a new sync sequence
                  * and any previous in-flight sync should be discarded. (part of RTP18)*/
-                if (IsSyncInProgress && _currentSyncChannelSerial.IsNotEmpty() && _currentSyncChannelSerial != syncSequenceId)
+                if (SyncInProgress && _currentSyncChannelSerial.IsNotEmpty() && _currentSyncChannelSerial != syncSequenceId)
                 {
                     EndSync();
                 }
@@ -621,15 +627,15 @@ internal void OnPresence(PresenceMessage[] messages)
 
         internal void StartSync()
         {
-            if (!IsSyncInProgress)
+            if (!SyncInProgress)
             {
                 MembersMap.StartSync();
             }
         }
 
         private void EndSync()
         {
-            if (!IsSyncInProgress)
+            if (!SyncInProgress)
             {
                 return;
             }

diff --git a/src/IO.Ably.Tests.Shared/Rest/SandboxSpecExtension.cs b/src/IO.Ably.Tests.Shared/Rest/SandboxSpecExtension.cs
@@ -47,7 +47,7 @@ internal static async Task WaitForAttachedState(this IRealtimeChannel channel, T
         internal static async Task<bool> WaitSync(this Presence presence, TimeSpan? waitSpan = null)
         {
             TaskCompletionSource<bool> taskCompletionSource = new TaskCompletionSource<bool>();
-            var inProgress = presence.IsSyncInProgress;
+            var inProgress = presence.SyncInProgress;
             if (inProgress == false)
             {
                 return true;