Implement spec points that say not to access certain fields

Based on [1] at 2e975cb.

[1] ably/specification#335

Author: lawrence-forooghian

Sources/AblyLiveObjects/Protocol/ObjectMessage.swift

@@ -117,9 +117,13 @@ internal extension ObjectOperation {
         counterOp = wireObjectOperation.counterOp
         map = wireObjectOperation.map.map { .init(wireMap: $0) }
         counter = wireObjectOperation.counter
-        nonce = wireObjectOperation.nonce
-        initialValue = wireObjectOperation.initialValue
-        initialValueEncoding = wireObjectOperation.initialValueEncoding
+
+        // Do not access on inbound data, per OOP3g
+        nonce = nil
+        // Do not access on inbound data, per OOP3h
+        initialValue = nil
+        // Do not access on inbound data, per OOP3i
+        initialValueEncoding = nil
     }
 
     /// Converts this `ObjectOperation` to a `WireObjectOperation`.

Sources/AblyLiveObjects/Protocol/WireObjectMessage.swift

@@ -158,7 +158,7 @@ internal struct WireObjectOperation {
     internal var map: WireMap? // OOP3e
     internal var counter: WireCounter? // OOP3f
     internal var nonce: String? // OOP3g
-    // TODO: Not yet clear how to encode / decode this property; I assume it will be properly specified later. Do in https://github.com/ably/ably-cocoa-liveobjects-plugin/issues/12
+    // TODO: Not yet clear how to encode this property; I assume it will be properly specified later. Do in https://github.com/ably/ably-cocoa-liveobjects-plugin/issues/12
     internal var initialValue: Data? // OOP3h
     internal var initialValueEncoding: String? // OOP3i
 }
@@ -183,8 +183,13 @@ extension WireObjectOperation: WireObjectCodable {
         counterOp = try wireObject.optionalDecodableValueForKey(WireKey.counterOp.rawValue)
         map = try wireObject.optionalDecodableValueForKey(WireKey.map.rawValue)
         counter = try wireObject.optionalDecodableValueForKey(WireKey.counter.rawValue)
-        nonce = try wireObject.optionalStringValueForKey(WireKey.nonce.rawValue)
-        initialValueEncoding = try wireObject.optionalStringValueForKey(WireKey.initialValueEncoding.rawValue)
+
+        // Do not access on inbound data, per OOP3g
+        nonce = nil
+        // Do not access on inbound data, per OOP3h
+        initialValue = nil
+        // Do not access on inbound data, per OOP3i
+        initialValueEncoding = nil
     }
 
     internal var toWireObject: [String: WireValue] {

Tests/AblyLiveObjectsTests/WireObjectMessageTests.swift

@@ -160,6 +160,9 @@ enum WireObjectMessageTests {
     }
 
     struct WireObjectOperationTests {
+        // @spec OOP3g
+        // @spec OOP3h
+        // @spec OOP3i
         @Test
         func decodesAllFields() throws {
             let wire: [String: WireValue] = [
@@ -182,8 +185,13 @@ enum WireObjectMessageTests {
             #expect(op.map?.entries?["key1"]?.data.string == "value1")
             #expect(op.map?.entries?["key1"]?.tombstone == false)
             #expect(op.counter?.count == 42)
-            #expect(op.nonce == "nonce1")
-            #expect(op.initialValueEncoding == "utf8")
+
+            // Per OOP3g we should not try and extract this
+            #expect(op.nonce == nil)
+            // Per OOP3h we should not try and extract this
+            #expect(op.initialValueEncoding == nil)
+            // Per OOP3i we should not try and extract this
+            #expect(op.initialValue == nil)
         }
 
         @Test