Fix variation attributes deserializing

Author: samiuelson

libs/fluxc-plugin/src/main/kotlin/org/wordpress/android/fluxc/model/pos/WooPosVariationApiResponse.kt

@@ -49,7 +49,7 @@ data class WooPosVariationApiResponse(
     val backordered: Boolean = false,
 
     @SerializedName("attributes")
-    val attributes: List<VariationAttribute> = emptyList(),
+    val attributesJson: String = "",
 
     @SerializedName("image")
     val image: VariationImage? = null,
@@ -60,17 +60,6 @@ data class WooPosVariationApiResponse(
     @SerializedName("name")
     val name: String = ""
 ) {
-    data class VariationAttribute(
-        @SerializedName("id")
-        val id: Long = 0L,
-
-        @SerializedName("name")
-        val name: String = "",
-
-        @SerializedName("option")
-        val option: String = ""
-    )
-
     data class VariationImage(
         @SerializedName("id")
         val id: Long = 0L,

libs/fluxc-plugin/src/main/kotlin/org/wordpress/android/fluxc/network/rest/wpcom/wc/product/pos/WooPosProductResponseToEntityMapper.kt

@@ -57,7 +57,7 @@ fun WooPosVariationApiResponse.mapToPosVariationModel(
         stockStatus = this.stockStatus,
         manageStock = this.manageStock,
         backordered = this.backordered,
-        attributesJson = this.attributes.toString(),
+        attributesJson = this.attributesJson,
         imageUrl = this.image?.src ?: "",
         status = this.status,
         downloadable = this.downloadable

libs/fluxc-plugin/src/test/java/org/wordpress/android/fluxc/network/rest/wpcom/wc/product/pos/WooPosVariationMapperTest.kt

@@ -26,10 +26,7 @@ class WooPosVariationMapperTest {
             stockStatus = "instock",
             manageStock = true,
             backordered = false,
-            attributes = listOf(
-                WooPosVariationApiResponse.VariationAttribute(1, "Color", "Blue"),
-                WooPosVariationApiResponse.VariationAttribute(2, "Size", "Large")
-            ),
+            attributesJson = """[{"id":1,"name":"Color","option":"Blue"},{"id":2,"name":"Size","option":"Large"}]""",
             image = WooPosVariationApiResponse.VariationImage(
                 id = 789,
                 src = "https://example.com/image.jpg",
@@ -105,7 +102,7 @@ class WooPosVariationMapperTest {
         val response = WooPosVariationApiResponse(
             id = 123L,
             productId = 456L,
-            attributes = emptyList()
+            attributesJson = "[]"
         )
 
         // When
@@ -121,11 +118,7 @@ class WooPosVariationMapperTest {
         val response = WooPosVariationApiResponse(
             id = 123L,
             productId = 456L,
-            attributes = listOf(
-                WooPosVariationApiResponse.VariationAttribute(1, "Color", "Red"),
-                WooPosVariationApiResponse.VariationAttribute(2, "Size", "Medium"),
-                WooPosVariationApiResponse.VariationAttribute(3, "Material", "Cotton")
-            )
+            attributesJson = """[{"id":1,"name":"Color","option":"Red"},{"id":2,"name":"Size","option":"Medium"},{"id":3,"name":"Material","option":"Cotton"}]"""
         )
 
         // When