Fix some mistakes from cb427d8

Author: lawrence-forooghian

Sources/AblyLiveObjects/Internal/ObjectsPool.swift

@@ -143,7 +143,7 @@ internal struct ObjectsPool {
         return entry
     }
 
-    /// Applies the objects gathered during an `OBJECT_SYNC` to this `ObjectsPool`, per RTO5c1.
+    /// Applies the objects gathered during an `OBJECT_SYNC` to this `ObjectsPool`, per RTO5c1 and RTO5c2.
     internal mutating func applySyncObjectsPool(
         _ syncObjectsPool: [ObjectState],
         logger: AblyPlugin.Logger,

Tests/AblyLiveObjectsTests/DefaultRealtimeObjectsTests.swift

@@ -155,9 +155,9 @@ struct DefaultRealtimeObjectsTests {
 
         // MARK: - RTO5c: Post-Sync Behavior Tests
 
-        // @spec(RTO5c2, RTO5c2a) Objects not in sync are removed, except root
+        // A smoke test that the RTO5c post-sync behaviours get performed. They are tested in more detail in the ObjectsPool.applySyncObjectsPool tests.
         @Test
-        func removesObjectsNotInSyncButPreservesRoot() async throws {
+        func performsPostSyncSteps() async throws {
             let realtimeObjects = DefaultRealtimeObjectsTests.createDefaultRealtimeObjects()
 
             // Perform sync with only one object (RTO5a5 case)
@@ -172,9 +172,6 @@ struct DefaultRealtimeObjectsTests {
             let finalPool = realtimeObjects.testsOnly_objectsPool
             #expect(finalPool.entries["root"] != nil) // Root preserved
             #expect(finalPool.entries["map:synced@1"] != nil) // Synced object added
-
-            // Note: We rely on applySyncObjectsPool being tested separately for RTO5c2 removal behavior
-            // as the side effect of removing pre-existing objects is tested in ObjectsPoolTests
         }
 
         // MARK: - Error Handling Tests

Tests/AblyLiveObjectsTests/ObjectsPoolTests.swift

@@ -204,7 +204,7 @@ struct ObjectsPoolTests {
 
         // MARK: - RTO5c2 Tests
 
-        // @spec(RTO5c2) Remove objects not received during sync
+        // @spec RTO5c2
         @Test
         func removesObjectsNotInSync() throws {
             let logger = TestLogger()
@@ -231,7 +231,7 @@ struct ObjectsPoolTests {
             #expect(pool.entries["counter:hash@1"] == nil) // Should be removed
         }
 
-        // @spec(RTO5c2a) Root object must not be removed
+        // @spec RTO5c2a
         @Test
         func doesNotRemoveRootObject() throws {
             let logger = TestLogger()
@@ -247,7 +247,7 @@ struct ObjectsPoolTests {
             #expect(pool.entries["map:hash@1"] == nil) // Should be removed
         }
 
-        // @spec(RTO5c1, RTO5c2) Complete sync scenario with mixed operations
+        // A more complete example of the behaviours described in RTO5c1 and RTO5c2.
         @Test
         func handlesComplexSyncScenario() throws {
             let logger = TestLogger()