candidate fix

Author: arcuri82

core/src/main/kotlin/org/evomaster/core/problem/rest/service/CallGraphService.kt

@@ -65,7 +65,7 @@ class CallGraphService {
     /**
      * Check in the schema if there is any action which is a direct child of [a] and last path element is a parameter
      */
-    fun hasParameterChild(a: RestCallAction): Boolean {
+    fun isThereChildActionWithParameter(a: RestCallAction): Boolean {
         return sampler.seeAvailableActions()
             .filterIsInstance<RestCallAction>()
             .map { it.path }
@@ -81,9 +81,9 @@ class CallGraphService {
 
     fun resolveLocationForParentOfChildOperationUsingCreatedResource(create: RestCallAction): String? {
 
-        if(hasParameterChild(create)) {
+        if(isThereChildActionWithParameter(create)) {
             //simple case
-            return create.resolvedPath()
+            return create.resolvedOnlyPath()
         }
 
         /*

core/src/main/kotlin/org/evomaster/core/problem/rest/service/fitness/AbstractRestFitness.kt

@@ -877,8 +877,10 @@ abstract class AbstractRestFitness : HttpWsFitness<RestIndividual>() {
                     it is or not a valid char.
                     Furthermore, likely needed to be done in resolveLocation,
                     or at least check how RestAssured would behave
+                    TODO update RestPathTest, check TODO there, once fixed
                  */
                 //it.replace("\"", "")
+                //FIXME outputFormat shouldn't really be used here
                 GeneUtils.applyEscapes(it, GeneUtils.EscapeMode.URI, configuration.outputFormat)
             }
 

core/src/test/kotlin/org/evomaster/core/problem/rest/RestPathTest.kt

@@ -1,6 +1,7 @@
 package org.evomaster.core.problem.rest
 
 import io.swagger.v3.oas.models.parameters.Parameter
+import org.evomaster.core.output.OutputFormat
 import org.evomaster.core.problem.rest.builder.RestActionBuilderV3
 import org.evomaster.core.problem.rest.data.HttpVerb
 import org.evomaster.core.problem.rest.data.RestPath
@@ -10,13 +11,15 @@ import org.evomaster.core.search.gene.collection.ArrayGene
 import org.evomaster.core.search.gene.wrapper.CustomMutationRateGene
 import org.evomaster.core.search.gene.numeric.IntegerGene
 import org.evomaster.core.search.gene.string.StringGene
+import org.evomaster.core.search.gene.utils.GeneUtils
 import org.glassfish.jersey.uri.internal.JerseyUriBuilder
 import org.junit.jupiter.api.Assertions.*
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.params.ParameterizedTest
 import org.junit.jupiter.params.provider.ValueSource
 import java.net.URISyntaxException
 import org.junit.jupiter.api.assertThrows
+import org.mockserver.configuration.Configuration.configuration
 
 internal class RestPathTest{
 
@@ -35,6 +38,15 @@ internal class RestPathTest{
         assertNotEquals(x, uri)
 
         JerseyUriBuilder.fromUri(uri).build()
+
+        // check escape
+        //TODO update once fixing AbstractRestFitness
+        val y = "/api/satsendring/kjorsatsendring/Trigget satsendring for fagsakene []"
+        JerseyUriBuilder.fromUri(y).build()
+
+        val e = GeneUtils.applyEscapes(y, GeneUtils.EscapeMode.URI, OutputFormat.JAVA_JUNIT_4)
+        //FIXME spaces are not escaped
+        //assertNotEquals(y,e)
     }