Add back attributes encoding to Product entity (#14046)

Author: pmusolino

Networking/Networking/Model/Product/Product.swift

@@ -746,6 +746,9 @@ public struct Product: Codable, GeneratedCopiable, Equatable, GeneratedFakeable
         try container.encode(minAllowedQuantity, forKey: .minAllowedQuantity)
         try container.encode(groupOfQuantity, forKey: .groupOfQuantity)
 
+        // Attributes
+        try container.encode(attributes, forKey: .attributes)
+
         // Password
         try container.encode(password, forKey: .password)
 

Networking/NetworkingTests/Mapper/ProductMapperTests.swift

@@ -515,7 +515,7 @@ final class ProductMapperTests: XCTestCase {
 
         // Given
         let product = try XCTUnwrap(mapLoadSubscriptionProductResponse())
-        let subscriptionSettings = try XCTUnwrap(product.subscription)
+        _ = try XCTUnwrap(product.subscription)
 
         // When
         let encoder = JSONEncoder()
@@ -549,6 +549,51 @@ final class ProductMapperTests: XCTestCase {
         XCTAssertEqual(encodedMetadata?[9]["key"] as? String, customFields[1].key)
         XCTAssertEqual(encodedMetadata?[9]["value"] as? String, customFields[1].value)
     }
+
+    /// Test that attributes are properly encoded.
+    ///
+    func test_attributes_are_properly_encoded() throws {
+        // Given
+        let product = try XCTUnwrap(mapLoadProductResponse().first)
+
+
+        // When
+        let encoder = JSONEncoder()
+        let encodedData = try encoder.encode(product)
+        let encodedJSON = try JSONSerialization.jsonObject(with: encodedData, options: []) as? [String: Any]
+
+        // Then
+
+        let attribute1 = ProductAttribute(siteID: dummySiteID,
+                                          attributeID: 0,
+                                          name: "Color",
+                                          position: 1,
+                                          visible: true,
+                                          variation: true,
+                                          options: ["Purple", "Yellow", "Hot Pink", "Lime Green", "Teal"])
+
+        let attribute2 = ProductAttribute(siteID: dummySiteID,
+                                          attributeID: 0,
+                                          name: "Size",
+                                          position: 0,
+                                          visible: true,
+                                          variation: true,
+                                          options: ["Small", "Medium", "Large"])
+        let attributes = [attribute1, attribute2]
+
+        XCTAssertNotNil(encodedJSON)
+        let encodedAttributes = encodedJSON?["attributes"] as? [[String: Any]]
+        XCTAssertEqual(encodedAttributes?.count, attributes.count)
+
+        // Verify all attributes are properly encoded
+        for (index, attribute) in attributes.enumerated() {
+            XCTAssertEqual(encodedAttributes?[index]["name"] as? String, attribute.name)
+            XCTAssertEqual(encodedAttributes?[index]["position"] as? Int, attribute.position)
+            XCTAssertEqual(encodedAttributes?[index]["visible"] as? Bool, attribute.visible)
+            XCTAssertEqual(encodedAttributes?[index]["variation"] as? Bool, attribute.variation)
+            XCTAssertEqual(encodedAttributes?[index]["options"] as? [String], attribute.options)
+        }
+    }
 }
 
 

RELEASE-NOTES.txt

@@ -10,6 +10,7 @@
 - [*] Blaze: Schedule a reminder local notification asking to continue abandoned campaign creation flow. [https://github.com/woocommerce/woocommerce-ios/pull/13950]
 - [**] Fixed formatting issues for discount and cash payment input values when the device and store currency settings are different. [https://github.com/woocommerce/woocommerce-ios/pull/13979]
 - [*] Blaze: Handle tap on local notifications to open campaign creation. [https://github.com/woocommerce/woocommerce-ios/pull/13968]
+- [**] Fixed an issue that prevented users from adding prices to product variations. [https://github.com/woocommerce/woocommerce-ios/pull/14046]
 - [internal] Mobile Payments: Log errors from Stripe Terminal [https://github.com/woocommerce/woocommerce-ios/pull/13976]