Change builder place, and add some comments

Author: suarezrominajulieta

…/builder/JsonPatchDocumentGeneBuilder.kt …/builder/JsonPatchDocumentGeneBuilder.ktcore/src/main/kotlin/org/evomaster/core/search/builder/JsonPatchDocumentGeneBuilder.kt renamed to core/src/main/kotlin/org/evomaster/core/search/gene/builder/JsonPatchDocumentGeneBuilder.kt core/src/main/kotlin/org/evomaster/core/search/builder/JsonPatchDocumentGeneBuilder.kt renamed to core/src/main/kotlin/org/evomaster/core/search/gene/builder/JsonPatchDocumentGeneBuilder.kt

@@ -82,7 +82,7 @@ object JsonPatchDocumentGeneBuilder {
 
     private fun generateRandomPaths(randomness: Randomness): List<String> =
         generateSequence { "/${randomness.nextWordString(2, 6)}" }
-            .take(RANDOM_PATH_COUNT * 2)
+            .take(RANDOM_PATH_COUNT * 2) // generate extra candidates so that after deduplication we still have RANDOM_PATH_COUNT distinct paths
             .distinct()
             .take(RANDOM_PATH_COUNT)
             .toList()

core/src/main/kotlin/org/evomaster/core/search/gene/jsonpatch/JsonPatchOperationGene.kt

@@ -76,6 +76,8 @@ abstract class JsonPatchOperationGene(
         children.filter { it.isMutable() }.forEach { it.randomize(randomness, tryToForceNewValue) }
     }
 
+    // Set to false due that for patch operation genes, all variation is already delegated to children (the path/value
+    // sub-genes and, at the document level, ArrayGene and ChoiceGene, that selects which operation is active).
     override fun customShouldApplyShallowMutation(
         randomness: Randomness,
         selectionStrategy: SubsetGeneMutationSelectionStrategy,