Implement DefaultChatClient.rooms

Part of #19.

Example/AblyChatExample/ContentView.swift

@@ -4,7 +4,7 @@ import SwiftUI
 struct ContentView: View {
     /// Just used to check that we can successfully import and use the AblyChat library. TODO remove this once we start building the library
     @State private var ablyChatClient = DefaultChatClient(
-        realtime: MockRealtime(key: ""),
+        realtime: MockRealtime.create(),
         clientOptions: ClientOptions()
     )
 

Example/AblyChatExample/Mocks/MockRealtime.swift

@@ -16,6 +16,15 @@ final class MockRealtime: NSObject, ARTRealtimeProtocol, Sendable {
 
     required init(token _: String) {}
 
+    /**
+     Creates an instance of MockRealtime.
+
+     This exists to give a convenient way to create an instance, because `init` is marked as unavailable in `ARTRealtimeProtocol`.
+     */
+    static func create() -> MockRealtime {
+        MockRealtime(key: "")
+    }
+
     func time(_: @escaping ARTDateTimeCallback) {
         fatalError("Not implemented")
     }

Sources/AblyChat/ChatClient.swift

@@ -10,28 +10,22 @@ public protocol ChatClient: AnyObject, Sendable {
 
 public typealias RealtimeClient = any(ARTRealtimeProtocol & Sendable)
 
-public final class DefaultChatClient: ChatClient {
-    public init(realtime _: RealtimeClient, clientOptions _: ClientOptions?) {
-        // This one doesn’t do `fatalError`, so that I can call it in the example app
+public actor DefaultChatClient: ChatClient {
+    public let realtime: RealtimeClient
+    public nonisolated let clientOptions: ClientOptions
+    public nonisolated let rooms: Rooms
+
+    public init(realtime: RealtimeClient, clientOptions: ClientOptions?) {
+        self.realtime = realtime
+        self.clientOptions = clientOptions ?? .init()
+        rooms = DefaultRooms(realtime: realtime, clientOptions: self.clientOptions)
     }
 
-    public var rooms: any Rooms {
+    public nonisolated var connection: any Connection {
         fatalError("Not yet implemented")
     }
 
-    public var connection: any Connection {
-        fatalError("Not yet implemented")
-    }
-
-    public var clientID: String {
-        fatalError("Not yet implemented")
-    }
-
-    public var realtime: RealtimeClient {
-        fatalError("Not yet implemented")
-    }
-
-    public var clientOptions: ClientOptions {
+    public nonisolated var clientID: String {
         fatalError("Not yet implemented")
     }
 }
@@ -44,4 +38,9 @@ public struct ClientOptions: Sendable {
         self.logHandler = logHandler
         self.logLevel = logLevel
     }
+
+    /// Used for comparing these instances in tests without having to make this Equatable, which I’m not yet sure makes sense (we’ll decide in https://github.com/ably-labs/ably-chat-swift/issues/10)
+    internal func isEqualForTestPurposes(_ other: ClientOptions) -> Bool {
+        logHandler === other.logHandler && logLevel == other.logLevel
+    }
 }

Sources/AblyChat/Rooms.swift

@@ -1,5 +1,26 @@
+import Ably
+
 public protocol Rooms: AnyObject, Sendable {
     func get(roomID: String, options: RoomOptions) throws -> any Room
     func release(roomID: String) async throws
     var clientOptions: ClientOptions { get }
 }
+
+internal actor DefaultRooms: Rooms {
+    /// Exposed so that we can test it.
+    internal nonisolated let realtime: RealtimeClient
+    internal nonisolated let clientOptions: ClientOptions
+
+    internal init(realtime: RealtimeClient, clientOptions: ClientOptions) {
+        self.realtime = realtime
+        self.clientOptions = clientOptions
+    }
+
+    internal nonisolated func get(roomID _: String, options _: RoomOptions) throws -> any Room {
+        fatalError("Not yet implemented")
+    }
+
+    internal func release(roomID _: String) async throws {
+        fatalError("Not yet implemented")
+    }
+}

Tests/AblyChatTests/DefaultChatClientTests.swift

@@ -0,0 +1,27 @@
+@testable import AblyChat
+import XCTest
+
+class DefaultChatClientTests: XCTestCase {
+    func test_init_withoutClientOptions() {
+        // Given: An instance of DefaultChatClient is created with nil clientOptions
+        let client = DefaultChatClient(realtime: MockRealtime.create(), clientOptions: nil)
+
+        // Then: It uses the default client options
+        let defaultOptions = ClientOptions()
+        XCTAssertTrue(client.clientOptions.isEqualForTestPurposes(defaultOptions))
+    }
+
+    func test_rooms() throws {
+        // Given: An instance of DefaultChatClient
+        let realtime = MockRealtime.create()
+        let options = ClientOptions()
+        let client = DefaultChatClient(realtime: realtime, clientOptions: options)
+
+        // Then: Its `rooms` property returns an instance of DefaultRooms with the same realtime client and client options
+        let rooms = client.rooms
+
+        let defaultRooms = try XCTUnwrap(rooms as? DefaultRooms)
+        XCTAssertIdentical(defaultRooms.realtime, realtime)
+        XCTAssertTrue(defaultRooms.clientOptions.isEqualForTestPurposes(options))
+    }
+}

Tests/AblyChatTests/Mocks/MockRealtime.swift

@@ -17,6 +17,15 @@ final class MockRealtime: NSObject, ARTRealtimeProtocol, Sendable {
 
     required init(token _: String) {}
 
+    /**
+     Creates an instance of MockRealtime.
+
+     This exists to give a convenient way to create an instance, because `init` is marked as unavailable in `ARTRealtimeProtocol`.
+     */
+    static func create() -> MockRealtime {
+        MockRealtime(key: "")
+    }
+
     func time(_: @escaping ARTDateTimeCallback) {
         fatalError("Not implemented")
     }