Log history

Source/ARTLog+Private.h

@@ -13,7 +13,7 @@
 
 ART_ASSUME_NONNULL_BEGIN
 
-@interface ARTLogLine : NSObject
+@interface ARTLogLine : NSObject <NSCoding>
 
 @property(nonatomic, readonly, strong) NSDate *date;
 @property(nonatomic, readonly) ARTLogLevel level;
@@ -27,9 +27,11 @@ ART_ASSUME_NONNULL_BEGIN
 
 @interface ARTLog ()
 
-@property(readonly, getter=getCaptured) __GENERIC(NSArray, ARTLogLine *) *captured;
+@property (readonly) NSArray<ARTLogLine *> *captured;
+@property (readonly) NSArray<ARTLogLine *> *history;
 
 - (instancetype)initCapturingOutput:(BOOL)capturing;
+- (instancetype)initCapturingOutput:(BOOL)capturing historyLines:(NSUInteger)historyLines;
 
 @end
 

Source/ARTLog.m

@@ -43,23 +43,54 @@ - (NSString *)toString {
     return [NSString stringWithFormat:@"%s: %@", logLevelName(self.level), self.message];
 }
 
+- (NSString *)description {
+    return [self toString];
+}
+
+#pragma mark - NSCoding
+
+- (id)initWithCoder:(NSCoder *)decoder {
+    self = [super init];
+    if (!self) {
+        return nil;
+    }
+    _date = [decoder decodeObjectForKey:@"date"];
+    _level = [[decoder decodeObjectForKey:@"level"] unsignedIntValue];
+    _message = [decoder decodeObjectForKey:@"message"];
+    return self;
+}
+
+- (void)encodeWithCoder:(NSCoder *)encoder {
+    [encoder encodeObject:self.date forKey:@"date"];
+    [encoder encodeObject:[NSNumber numberWithUnsignedInteger:self.level] forKey:@"level"];
+    [encoder encodeObject:self.message forKey:@"message"];
+}
+
 @end
 
 @implementation ARTLog {
-    NSMutableArray *_captured;
+    NSMutableArray<ARTLogLine *> *_captured;
+    NSMutableArray<ARTLogLine *> *_history;
+    NSUInteger _historyLines;
 }
 
 - (instancetype)init {
-    return [self initCapturingOutput:false];
+    return [self initCapturingOutput:true];
 }
 
 - (instancetype)initCapturingOutput:(BOOL)capturing {
+    return [self initCapturingOutput:true historyLines:10];
+}
+
+- (instancetype)initCapturingOutput:(BOOL)capturing historyLines:(NSUInteger)historyLines {
     if (self = [super init]) {
         // Default
         self->_logLevel = ARTLogLevelWarn;
         if (capturing) {
             self->_captured = [[NSMutableArray alloc] init];
         }
+        _history = [[NSMutableArray alloc] init];
+        _historyLines = historyLines;
     }
     return self;
 }
@@ -72,9 +103,17 @@ - (void)log:(NSString *)message level:(ARTLogLevel)level {
             [_captured addObject:logLine];
         }
     }
+    [_history insertObject:logLine atIndex:0];
+    if (_history.count > _historyLines) {
+        [_history removeLastObject];
+    }
+}
+
+- (NSArray<ARTLogLine *> *)history {
+    return _history;
 }
 
-- (NSArray *)getCaptured {
+- (NSArray *)captured {
     if (!_captured) {
         [NSException raise:NSInternalInconsistencyException format:@"tried to get captured output in non-capturing instance; use initCapturingOutput:true if you want captured output."];
     }

Spec/Utilities.swift

@@ -386,6 +386,27 @@ class Utilities: QuickSpec {
                     }
                 }
             }
+
+            context("Logger") {
+
+                it("should have a history of logs") {
+                    let options = AblyTests.commonAppSetup()
+                    let realtime = ARTRealtime(options: options)
+                    let channel = realtime.channels.get("foo")
+
+                    waitUntil(timeout: testTimeout) { done in
+                        channel.attach { error in
+                            expect(error).to(beNil())
+                            done()
+                        }
+                    }
+
+                    expect(realtime.logger.history).to(haveCount(10))
+                    expect(realtime.logger.history.map{ $0.message }.first).to(contain("channel state transitions to 2 - Attached"))
+                    expect(realtime.logger.history.filter{ $0.message.containsString("realtime state transitions to 2 - Connected") }).to(haveCount(1))
+                }
+
+            }
         }
     }
 }