Merge pull request #426 from Ecwid/ECWID_144513

ECWID-144513 startersite error Caused by IllegalStateException: Expected but was STRING at line column path at JsonReader.beginObject(JsonReader.java:) - removed unused imports

Author: Alexis2004

src/main/kotlin/com/ecwid/apiclient/v3/ApiClientHelper.kt

@@ -196,7 +196,7 @@ class ApiClientHelper private constructor(
 					val responseBody = responseBytes.asString()
 					logErrorResponseIfNeeded(requestId, requestTime, httpResponse.statusCode, responseBody)
 					val ecwidError = if (responseBody.isNotBlank()) {
-						jsonTransformer.deserialize(responseBody, EcwidApiError::class.java)
+						jsonTransformer.deserializeOrNull(responseBody, EcwidApiError::class.java)
 					} else {
 						null
 					}

src/main/kotlin/com/ecwid/apiclient/v3/jsontransformer/JsonTransformer.kt

@@ -2,5 +2,6 @@ package com.ecwid.apiclient.v3.jsontransformer
 
 interface JsonTransformer {
 	fun serialize(src: Any?, srcExt: Any? = null): String
-	fun <V> deserialize(json: String, clazz: Class<V>): V?
+	fun <V> deserialize(json: String, clazz: Class<V>): V
+	fun <V> deserializeOrNull(json: String, clazz: Class<V>): V?
 }

src/main/kotlin/com/ecwid/apiclient/v3/jsontransformer/gson/GsonTransformer.kt

@@ -36,13 +36,21 @@ class GsonTransformer(polymorphicTypes: List<PolymorphicType<*>>) : JsonTransfor
 		}
 	}
 
-	override fun <V> deserialize(json: String, clazz: Class<V>): V? {
+	override fun <V> deserialize(json: String, clazz: Class<V>): V {
 		try {
 			return gson.fromJson(json, clazz)
 		} catch (e: JsonParseException) {
 			throw JsonDeserializationException(e.message, e)
 		}
 	}
+
+	override fun <V> deserializeOrNull(json: String, clazz: Class<V>): V? {
+		return try {
+			return gson.fromJson(json, clazz)
+		} catch (e: JsonParseException) {
+			null
+		}
+	}
 }
 
 private fun JsonObject.mergeJsonObject(from: JsonObject) {

src/test/kotlin/com/ecwid/apiclient/v3/jsontransformer/gson/GsonTransformerTest.kt

@@ -230,6 +230,13 @@ internal class GsonTransformerTest {
 		)
 	}
 
+	@Test
+	fun `deserializeOrNull of broken ParsedResponseWithExt`() {
+		val json = "'testField': {'baseField': 'base', 'extField': 'ext'}}"
+		val deserializedValue = transformer.deserializeOrNull(json, TestParsedResponseWithExt::class.java)
+		assertEquals(null, deserializedValue)
+	}
+
 }
 
 private fun assertJsonEquals(expected: String, actual: String) {