possible fix for #698: making sure all calls to Gson().fromJson are in a try/catch

Author: arcuri82

core/src/main/kotlin/org/evomaster/core/EMConfig.kt

@@ -260,6 +260,10 @@ class EMConfig {
             They can be check only once all fields have been updated
          */
 
+        if(!blackBox && bbSwaggerUrl.isNotBlank()){
+            throw IllegalArgumentException("'bbSwaggerUrl' should be set only in black-box mode")
+        }
+
         if (blackBox && !bbExperiments) {
 
             if(problemType == ProblemType.DEFAULT){

core/src/main/kotlin/org/evomaster/core/output/TestSuiteSplitter.kt

@@ -278,7 +278,11 @@ object TestSuiteSplitter {
      * if it contains the field "errors" in its body.
      */
     fun assessFailed(result: GraphQlCallResult): Boolean{
-        val resultBody = Gson().fromJson(result.getBody(), HashMap::class.java)
+        val resultBody = try {
+            Gson().fromJson(result.getBody(), HashMap::class.java)
+        } catch (e : JsonSyntaxException){
+            return true //TODO should this be treated specially???
+        }
         val errMsg = resultBody?.get("errors")
         return (resultBody!=null && errMsg != null)
     }

core/src/main/kotlin/org/evomaster/core/output/oracles/SchemaOracle.kt

@@ -1,6 +1,7 @@
 package org.evomaster.core.output.oracles
 
 import com.google.gson.Gson
+import com.google.gson.JsonSyntaxException
 import io.swagger.v3.oas.models.media.*
 import org.evomaster.core.logging.LoggingUtil
 import org.evomaster.core.output.Lines
@@ -342,7 +343,12 @@ class SchemaOracle : ImplementedOracle() {
         var supported = true
 
         if (res.getBodyType()?.isCompatible(MediaType.APPLICATION_JSON_TYPE) == true){
-            val actualObject = Gson().fromJson(res.getBody(), Object::class.java)
+            val actualObject = try {
+                Gson().fromJson(res.getBody(), Object::class.java)
+            } catch (e: JsonSyntaxException) {
+                return false
+            }
+
             if  (actualObject is Map<*,*>)
                 supported = supportedObject(actualObject, call)
             else if (actualObject is List<*>

core/src/main/kotlin/org/evomaster/core/output/service/ApiTestCaseWriter.kt

@@ -1,6 +1,7 @@
 package org.evomaster.core.output.service
 
 import com.google.gson.Gson
+import com.google.gson.JsonSyntaxException
 import org.evomaster.core.database.DbAction
 import org.evomaster.core.database.DbActionResult
 import org.evomaster.core.output.CookieWriter
@@ -68,14 +69,22 @@ abstract class ApiTestCaseWriter : TestCaseWriter() {
         when (bodyString?.trim()?.first()) {
             //TODO this should be handled recursively, and not ad-hoc here...
             '[' -> {
+                try{
                 // This would be run if the JSON contains an array of objects.
                 val list = Gson().fromJson(bodyString, List::class.java)
                 handleAssertionsOnList(list, lines, "", bodyVarName)
+                } catch (e: JsonSyntaxException) {
+                    lines.add("/* Failed to parse JSON response */")
+                }
             }
             '{' -> {
                 // JSON contains an object
-                val resContents = Gson().fromJson(bodyString, Map::class.java)
-                handleAssertionsOnObject(resContents as Map<String, *>, lines, "", bodyVarName)
+                try {
+                    val resContents = Gson().fromJson(bodyString, Map::class.java)
+                    handleAssertionsOnObject(resContents as Map<String, *>, lines, "", bodyVarName)
+                } catch (e: JsonSyntaxException) {
+                    lines.add("/* Failed to parse JSON response */")
+                }
             }
             '"' -> {
                 lines.add(bodyIsString(bodyString, GeneUtils.EscapeMode.BODY, bodyVarName))