Add fallback when variation SKU is a number

Author: rachelmcr

Networking/Networking/Model/Product/ProductVariation.swift

@@ -206,7 +206,12 @@ public struct ProductVariation: Codable, GeneratedCopiable, Equatable, Generated
         let statusKey = try container.decode(String.self, forKey: .statusKey)
         let status = ProductStatus(rawValue: statusKey)
         let description = try container.decodeIfPresent(String.self, forKey: .description)
-        let sku = try container.decodeIfPresent(String.self, forKey: .sku)
+
+        // Even though a plain install of WooCommerce Core provides String values,
+        // some plugins alter the field value from String to Int or Decimal.
+        let sku = container.failsafeDecodeIfPresent(targetType: String.self,
+                                                    forKey: .sku,
+                                                    alternativeTypes: [.decimal(transform: { NSDecimalNumber(decimal: $0).stringValue })])
 
         // Even though a plain install of WooCommerce Core provides string values,
         // some plugins alter the field value from String to Int or Decimal.

Networking/NetworkingTests/Mapper/ProductVariationMapperTests.swift

@@ -40,6 +40,7 @@ final class ProductVariationMapperTests: XCTestCase {
         XCTAssertFalse(productVariation.manageStock)
         XCTAssertTrue(productVariation.purchasable)
         XCTAssertEqual(productVariation.permalink, "")
+        XCTAssertEqual(productVariation.sku, "12345")
     }
 
     /// Test that the fields for variations of a subscription product are properly parsed.

Networking/NetworkingTests/Responses/product-variation-alternative-types.json

@@ -8,7 +8,7 @@
             "date_modified_gmt": "2019-11-14T13:06:42",
             "description": "<p>Nutty chocolate marble, 99% and organic.</p>\n",
             "permalink": 0,
-            "sku": "99%-nuts-marble",
+            "sku": 12345,
             "price": 13.99,
             "regular_price": 16,
             "sale_price": 9.99,