Channel presence now is exposed publicly as protocol. For internal ac…

…cess `internalPresence` (in internal class) exists, or `presence`, but manually casted to internal when needed. In other words, public channel exposes public `presence`, internal one exposes internal `presence` (but only public properties/methods are visible by default which of `ARTRealtimeChannelProtocol`). `internalPresence` is for explicit and more convenient way of using internal type.
ably · Nov 4, 2024 · 26d9bf7 · 26d9bf7
1 parent 4856ba6
commit 26d9bf7
Show file tree

Hide file tree

Showing 8 changed files with 138 additions and 139 deletions.
diff --git a/Source/ARTRealtimeChannel.m b/Source/ARTRealtimeChannel.m
@@ -77,8 +77,8 @@ - (ARTErrorInfo *)errorReason {
     return _internal.errorReason;
 }
 
-- (ARTRealtimePresence *)presence {
-    return [[ARTRealtimePresence alloc] initWithInternal:_internal.presence queuedDealloc:_dealloc];
+- (id<ARTRealtimePresenceProtocol>)presence {
+    return [[ARTRealtimePresence alloc] initWithInternal:_internal.internalPresence queuedDealloc:_dealloc];
 }
 
 #if TARGET_OS_IPHONE
@@ -224,7 +224,7 @@ - (void)setOptions:(ARTRealtimeChannelOptions *_Nullable)options callback:(nulla
 @end
 
 @interface ARTRealtimeChannelInternal () {
-    ARTRealtimePresenceInternal *_realtimePresence;
+    ARTRealtimePresenceInternal *_internalPresence;
     #if TARGET_OS_IPHONE
     ARTPushChannelInternal *_pushChannel;
     #endif
@@ -263,7 +263,7 @@ - (instancetype)initWithRealtime:(ARTRealtimeInternal *)realtime andName:(NSStri
         _restChannel = [_realtime.rest.channels _getChannel:self.name options:options addPrefix:true];
         _state = ARTRealtimeChannelInitialized;
         _attachSerial = nil;
-        _realtimePresence = [[ARTRealtimePresenceInternal alloc] initWithChannel:self logger:self.logger];
+        _internalPresence = [[ARTRealtimePresenceInternal alloc] initWithChannel:self logger:self.logger];
         _statesEventEmitter = [[ARTPublicEventEmitter alloc] initWithRest:_realtime.rest logger:logger];
         _messagesEventEmitter = [[ARTInternalEventEmitter alloc] initWithQueues:_queue userQueue:_userQueue];
         _attachedEventEmitter = [[ARTInternalEventEmitter alloc] initWithQueue:_queue];
@@ -324,8 +324,8 @@ - (ARTErrorInfo *)errorReason_nosync {
     return _errorReason;
 }
 
-- (ARTRealtimePresenceInternal *)presence {
-    return _realtimePresence;
+- (id<ARTRealtimePresenceProtocol>)presence {
+    return _internalPresence;
 }
 
 #if TARGET_OS_IPHONE
@@ -567,14 +567,14 @@ - (void)performTransitionToState:(ARTRealtimeChannelState)state withParams:(ARTC
             break;
         case ARTRealtimeChannelDetached:
             self.channelSerial = nil; // RTP5a1
-            [self.presence failsSync:params.errorInfo]; // RTP5a
+            [self.internalPresence failsSync:params.errorInfo]; // RTP5a
             break;
         case ARTRealtimeChannelFailed:
             self.channelSerial = nil; // RTP5a1
             self.attachResume = false;
             [_attachedEventEmitter emit:nil with:params.errorInfo];
             [_detachedEventEmitter emit:nil with:params.errorInfo];
-            [self.presence failsSync:params.errorInfo]; // RTP5a
+            [self.internalPresence failsSync:params.errorInfo]; // RTP5a
             break;
         default:
             break;
@@ -658,7 +658,7 @@ - (void)setAttached:(ARTProtocolMessage *)message {
             }
             ARTChannelStateChange *stateChange = [[ARTChannelStateChange alloc] initWithCurrent:state previous:state event:ARTChannelEventUpdate reason:message.error resumed:message.resumed];
             [self emit:stateChange.event with:stateChange];
-            [self.presence onAttached:message];
+            [self.internalPresence onAttached:message];
         }
         return;
     }
@@ -671,7 +671,7 @@ - (void)setAttached:(ARTProtocolMessage *)message {
     }
     params.resumed = message.resumed;
     [self performTransitionToState:ARTRealtimeChannelAttached withParams:params];
-    [self.presence onAttached:message];
+    [self.internalPresence onAttached:message];
     [_attachedEventEmitter emit:nil with:nil];
 }
 
@@ -709,7 +709,7 @@ - (void)setDetached:(ARTProtocolMessage *)message {
 
 - (void)failPendingPresenceWithState:(ARTState)state info:(nullable ARTErrorInfo *)info {
     ARTStatus *const status = [ARTStatus state:state info:info];
-    [self.presence failPendingPresence:status];
+    [self.internalPresence failPendingPresence:status];
 }
 
 - (void)detachChannel:(ARTChannelStateChangeParams *)params {
@@ -804,11 +804,11 @@ - (void)onPresence:(ARTProtocolMessage *)message {
     if (message.channelSerial) {
         self.channelSerial = message.channelSerial;
     }
-    [self.presence onMessage:message];
+    [self.internalPresence onMessage:message];
 }
 
 - (void)onSync:(ARTProtocolMessage *)message {
-    [self.presence onSync:message];
+    [self.internalPresence onSync:message];
 }
 
 - (void)onError:(ARTProtocolMessage *)msg {
@@ -1026,8 +1026,8 @@ - (void)detachAfterChecks {
         [self->_detachedEventEmitter emit:nil with:errorInfo];
     }] startTimer];
 
-    if (self.presence.syncInProgress_nosync) {
-        [self.presence failsSync:[ARTErrorInfo createWithCode:ARTErrorChannelOperationFailed message:@"channel is being DETACHED"]];
+    if (self.internalPresence.syncInProgress_nosync) {
+        [self.internalPresence failsSync:[ARTErrorInfo createWithCode:ARTErrorChannelOperationFailed message:@"channel is being DETACHED"]];
     }
 }
 

diff --git a/Source/ARTRealtimeChannels.m b/Source/ARTRealtimeChannels.m
@@ -116,7 +116,7 @@ - (void)release:(NSString *)name callback:(ARTCallback)cb {
     [channel _detach:^(ARTErrorInfo *errorInfo) {
         [channel off_nosync];
         [channel _unsubscribe];
-        [channel.presence _unsubscribe];
+        [channel.internalPresence _unsubscribe];
 
         // Only release if the stored channel now is the same as whne.
         // Otherwise, subsequent calls to this release method race, and

diff --git a/Source/PrivateHeaders/Ably/ARTRealtimeChannel+Private.h b/Source/PrivateHeaders/Ably/ARTRealtimeChannel+Private.h
@@ -19,7 +19,7 @@ NS_ASSUME_NONNULL_BEGIN
 
 @interface ARTRealtimeChannelInternal : ARTChannel <ARTRealtimeChannelProtocol>
 
-@property (readonly) ARTRealtimePresenceInternal *presence;
+@property (readonly) ARTRealtimePresenceInternal *internalPresence;
 #if TARGET_OS_IPHONE
 @property (readonly) ARTPushChannelInternal *push;
 #endif

diff --git a/Source/include/Ably/ARTRealtimeChannel.h b/Source/include/Ably/ARTRealtimeChannel.h
@@ -9,7 +9,6 @@
 
 NS_ASSUME_NONNULL_BEGIN
 
-@class ARTRealtimePresence;
 @class ARTRealtimeChannelOptions;
 @class ARTChannelProperties;
 #if TARGET_OS_IPHONE
@@ -39,6 +38,11 @@ NS_ASSUME_NONNULL_BEGIN
 /// :nodoc: TODO: docstring
 @property (readonly, nullable, getter=getOptions) ARTRealtimeChannelOptions *options;
 
+/**
+ * An `ARTRealtimePresence` object.
+ */
+@property (readonly) id<ARTRealtimePresenceProtocol> presence;
+
 /**
  * A shortcut for the `-[ARTRealtimeChannelProtocol attach:]` method.
  */
@@ -180,10 +184,6 @@ ART_EMBED_INTERFACE_EVENT_EMITTER(ARTChannelEvent, ARTChannelStateChange *)
 NS_SWIFT_SENDABLE
 @interface ARTRealtimeChannel : NSObject <ARTRealtimeChannelProtocol>
 
-/**
- * An `ARTRealtimePresence` object.
- */
-@property (readonly) ARTRealtimePresence *presence;
 #if TARGET_OS_IPHONE
 /**
  * An `ARTPushChannel` object.

diff --git a/Source/include/Ably/ARTRealtimePresence.h b/Source/include/Ably/ARTRealtimePresence.h
@@ -3,7 +3,6 @@
 #import <Ably/ARTRestPresence.h>
 #import <Ably/ARTDataQuery.h>
 #import <Ably/ARTEventEmitter.h>
-#import <Ably/ARTRealtimeChannel.h>
 
 NS_ASSUME_NONNULL_BEGIN
 

diff --git a/Test/Tests/RealtimeClientChannelTests.swift b/Test/Tests/RealtimeClientChannelTests.swift
@@ -189,14 +189,14 @@ class RealtimeClientChannelTests: XCTestCase {
             channel2.attach()
 
             XCTAssertFalse(channel2.presence.syncComplete)
-            XCTAssertEqual(channel1.internal.presence.members.count, 1)
-            XCTAssertEqual(channel2.internal.presence.members.count, 0)
+            XCTAssertEqual(channel1.internal.internalPresence.members.count, 1)
+            XCTAssertEqual(channel2.internal.internalPresence.members.count, 0)
         }
 
         expect(channel2.presence.syncComplete).toEventually(beTrue(), timeout: testTimeout)
 
-        XCTAssertEqual(channel1.internal.presence.members.count, 1)
-        expect(channel2.internal.presence.members).toEventually(haveCount(1), timeout: testTimeout)
+        XCTAssertEqual(channel1.internal.internalPresence.members.count, 1)
+        expect(channel2.internal.internalPresence.members).toEventually(haveCount(1), timeout: testTimeout)
 
         waitUntil(timeout: testTimeout) { done in
             channel1.publish("message", data: nil) { errorInfo in
@@ -212,14 +212,14 @@ class RealtimeClientChannelTests: XCTestCase {
             }
         }
 
-        expect(channel1.internal.presence.members).toEventually(haveCount(2), timeout: testTimeout)
-        expect(channel1.internal.presence.members.keys).to(allPass { $0.hasPrefix("\(channel1.internal.connectionId):Client") || $0.hasPrefix("\(channel2.internal.connectionId):Client") })
-        expect(channel1.internal.presence.members.values).to(allPass { $0.action == .present })
+        expect(channel1.internal.internalPresence.members).toEventually(haveCount(2), timeout: testTimeout)
+        expect(channel1.internal.internalPresence.members.keys).to(allPass { $0.hasPrefix("\(channel1.internal.connectionId):Client") || $0.hasPrefix("\(channel2.internal.connectionId):Client") })
+        expect(channel1.internal.internalPresence.members.values).to(allPass { $0.action == .present })
 
-        expect(channel2.internal.presence.members).toEventually(haveCount(2), timeout: testTimeout)
-        expect(channel2.internal.presence.members.keys).to(allPass { $0.hasPrefix("\(channel1.internal.connectionId):Client") || $0.hasPrefix("\(channel2.internal.connectionId):Client") })
-        XCTAssertEqual(channel2.internal.presence.members["\(channel1.internal.connectionId):Client 1"]!.action, ARTPresenceAction.present)
-        XCTAssertEqual(channel2.internal.presence.members["\(channel2.internal.connectionId):Client 2"]!.action, ARTPresenceAction.present)
+        expect(channel2.internal.internalPresence.members).toEventually(haveCount(2), timeout: testTimeout)
+        expect(channel2.internal.internalPresence.members.keys).to(allPass { $0.hasPrefix("\(channel1.internal.connectionId):Client") || $0.hasPrefix("\(channel2.internal.connectionId):Client") })
+        XCTAssertEqual(channel2.internal.internalPresence.members["\(channel1.internal.connectionId):Client 1"]!.action, ARTPresenceAction.present)
+        XCTAssertEqual(channel2.internal.internalPresence.members["\(channel2.internal.connectionId):Client 2"]!.action, ARTPresenceAction.present)
     }
 
     // RTL2