Tighten check on get reusing room

The existing check was not great, since the factory only ever returns a
single instance.

Tests/AblyChatTests/DefaultRoomsTests.swift

@@ -40,15 +40,17 @@ struct DefaultRoomsTests {
         ]))
         let options = RoomOptions()
         let roomToReturn = MockRoom(options: options)
-        let rooms = DefaultRooms(realtime: realtime, clientOptions: .init(), logger: TestLogger(), roomFactory: MockRoomFactory(room: roomToReturn))
+        let roomFactory = MockRoomFactory(room: roomToReturn)
+        let rooms = DefaultRooms(realtime: realtime, clientOptions: .init(), logger: TestLogger(), roomFactory: roomFactory)
 
         let roomID = "basketball"
         let firstRoom = try await rooms.get(roomID: roomID, options: options)
 
         // When: get(roomID:options:) is called with the same room ID
         let secondRoom = try await rooms.get(roomID: roomID, options: options)
 
-        // Then: It returns the same room object
+        // Then: It does not create another room, and returns the same room object
+        #expect(await roomFactory.createRoomCallCount == 1)
         #expect(secondRoom === firstRoom)
     }
 

Tests/AblyChatTests/Mocks/MockRoomFactory.swift

@@ -2,6 +2,7 @@
 
 actor MockRoomFactory: RoomFactory {
     private var room: MockRoom?
+    private(set) var createRoomCallCount = 0
     private(set) var createRoomArguments: (realtime: RealtimeClient, chatAPI: ChatAPI, roomID: String, options: RoomOptions, logger: any InternalLogger)?
 
     init(room: MockRoom? = nil) {
@@ -13,6 +14,7 @@ actor MockRoomFactory: RoomFactory {
     }
 
     func createRoom(realtime: RealtimeClient, chatAPI: ChatAPI, roomID: String, options: RoomOptions, logger: any InternalLogger) async throws -> MockRoom {
+        createRoomCallCount += 1
         createRoomArguments = (realtime: realtime, chatAPI: chatAPI, roomID: roomID, options: options, logger: logger)
 
         guard let room else {