Skip to content

Commit

Permalink
Removed comments and change var to val
Browse files Browse the repository at this point in the history
  • Loading branch information
Pgarrett committed Nov 15, 2024
1 parent ce48415 commit a3fc5fb
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ open class GraphQLActionTestCaseNamingStrategy(
return formatName(nameTokens)
}

// override fun resolveAmbiguities(duplicatedIndividuals: MutableSet<EvaluatedIndividual<*>>): Map<EvaluatedIndividual<*>, String> {
override fun resolveAmbiguities(duplicatedIndividuals: Set<EvaluatedIndividual<*>>): Map<EvaluatedIndividual<*>, String> {
// TODO do nothing at the moment. This will be completed with the experimental params disambiguation method
return emptyMap()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ open class NumberedTestCaseNamingStrategy(
return ""
}

// override fun resolveAmbiguities(duplicatedIndividuals: MutableSet<EvaluatedIndividual<*>>): Map<EvaluatedIndividual<*>, String> {
override fun resolveAmbiguities(duplicatedIndividuals: Set<EvaluatedIndividual<*>>): Map<EvaluatedIndividual<*>, String> {
// do nothing, plain numbered strategy will never have duplicate names
return emptyMap()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ open class RPCActionTestCaseNamingStrategy(
return formatName(nameTokens)
}

// override fun resolveAmbiguities(duplicatedIndividuals: MutableSet<EvaluatedIndividual<*>>): Map<EvaluatedIndividual<*>, String> {
override fun resolveAmbiguities(duplicatedIndividuals: Set<EvaluatedIndividual<*>>): Map<EvaluatedIndividual<*>, String> {
// TODO do nothing at the moment. This will be completed with the experimental params disambiguation method
return emptyMap()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ open class RestActionTestCaseNamingStrategy(
*/
override fun resolveAmbiguities(duplicatedIndividuals: Set<EvaluatedIndividual<*>>): Map<EvaluatedIndividual<*>, String> {
val solvedAmbiguities = mutableMapOf<EvaluatedIndividual<*>, String>()
var workingCopy = duplicatedIndividuals.toMutableSet()
val workingCopy = duplicatedIndividuals.toMutableSet()

val pathDisambiguatedIndividuals = solvePathAmbiguities(workingCopy)
solvedAmbiguities.putAll(pathDisambiguatedIndividuals)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ abstract class TestCaseNamingStrategy(
*
* @return a Map of EvaluatedIndividuals and the disambiguated test case name
*/
// protected abstract fun resolveAmbiguities(duplicatedIndividuals: MutableSet<EvaluatedIndividual<*>>): Map<EvaluatedIndividual<*>, String>
protected abstract fun resolveAmbiguities(duplicatedIndividuals: Set<EvaluatedIndividual<*>>): Map<EvaluatedIndividual<*>, String>

}

0 comments on commit a3fc5fb

Please sign in to comment.