Remove usage of NSLog in ARTPublicEventEmitter

Replace it with a call to an ARTLog instance. The test change is because the first recorded log message will now be a "PublicEventEmitter Notification emitted" message instead of the previous "channel state transitions" one.
ably · Dec 7, 2022 · b2e024c · b2e024c
1 parent ce62489
commit b2e024c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Source/ARTEventEmitter.m b/Source/ARTEventEmitter.m
@@ -332,7 +332,7 @@ - (instancetype)initWithRest:(ARTRestInternal *)rest {
                                                  object:nil
                                                   queue:nil
                                              usingBlock:^(NSNotification *notification) {
-                                                 NSLog(@"VERBOSE: PublicEventEmitter Notification emitted %@", notification.name);
+                                                 [rest.logger verbose:@"PublicEventEmitter Notification emitted %@", notification.name];
                                              }];
         }
     }

diff --git a/Spec/Tests/UtilitiesTests.swift b/Spec/Tests/UtilitiesTests.swift
@@ -453,7 +453,7 @@ class UtilitiesTests: XCTestCase {
         }
 
         expect(realtime.internal.logger.history.count).toNot(beGreaterThan(100))
-        expect(realtime.internal.logger.history.map { $0.message }.first).to(contain("channel state transitions from 1 - Attaching to 2 - Attached"))
+        expect(realtime.internal.logger.history.filter { $0.message.contains("channel state transitions from 1 - Attaching to 2 - Attached") }).to(haveCount(1))
         expect(realtime.internal.logger.history.filter { $0.message.contains("realtime state transitions to 2 - Connected") }).to(haveCount(1))
     }