Skip to content

Commit 6451428

Browse files
committed
clean-up
1 parent 6182b62 commit 6451428

2 files changed

Lines changed: 8 additions & 17 deletions

File tree

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

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -827,7 +827,7 @@ abstract class HttpWsTestCaseWriter : ApiTestCaseWriter() {
827827
lines.indented {
828828
lines.add("get(\"${verifier.stub}\")")
829829
lines.indented {
830-
lines.add(".withMetadata(Metadata.metadata().attr(\"ssrf\", \"${action.getName()}\"))")
830+
lines.add(".withMetadata(Metadata.metadata().attr(SSRF_METADATA_TAG, \"${action.getName()}\"))")
831831
lines.add(".atPriority(1)")
832832
lines.add(".willReturn(")
833833
lines.indented {
@@ -859,21 +859,11 @@ abstract class HttpWsTestCaseWriter : ApiTestCaseWriter() {
859859
private fun handleCallbackVerifierRequests(lines: Lines, action: Action, verifier: ActionStubMapping, assertTrue: Boolean) {
860860
if (assertTrue) {
861861
lines.addSingleCommentLine("Verifying that the request is successfully made to HttpCallbackVerifier after test execution.")
862-
lines.add("assertTrue(verifierHasServedRequests(${verifier.getVerifierName()}, \"${action.getName()}\"))")
863-
// lines.add("assertTrue(${verifier.getVerifierName()}")
862+
lines.add("assertTrue(verifierHasReceivedRequests(${verifier.getVerifierName()}, \"${action.getName()}\"))")
864863
} else {
865864
lines.addSingleCommentLine("Verifying that there are no requests made to HttpCallbackVerifier before test execution.")
866-
// lines.add("assertFalse(${verifier.getVerifierName()}")
867-
lines.add("assertFalse(verifierHasServedRequests(${verifier.getVerifierName()}, \"${action.getName()}\"))")
868-
}
869-
// lines.indented {
870-
// if (format.isKotlin()) {
871-
// lines.add(".allServeEvents")
872-
// lines.add(".filter { it.wasMatched && it.stubMapping.metadata != null }")
873-
// lines.add(".any { it.stubMapping.metadata.getString(\"ssrf\") == \"${action.getName()}\" }")
874-
// }
875-
// }
876-
// lines.add(")")
865+
lines.add("assertFalse(verifierHasReceivedRequests(${verifier.getVerifierName()}, \"${action.getName()}\"))")
866+
}
877867
}
878868

879869
}

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

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -674,6 +674,7 @@ class TestSuiteWriter {
674674
if (config.ssrf && solution.hasSsrfFaults()) {
675675
httpCallbackVerifier.getActionVerifierMappings().forEach { v ->
676676
addStatement("private lateinit var ${v.getVerifierName()}: WireMockServer", lines)
677+
addStatement("private const val SSRF_METADATA_TAG: String = \"SSRF\" ", lines)
677678
}
678679
}
679680

@@ -1154,10 +1155,10 @@ class TestSuiteWriter {
11541155
lines.endCommentBlock()
11551156
when {
11561157
format.isKotlin() -> {
1157-
lines.add("fun verifierHasServedRequests(verifier: WireMockServer, actionName: String) : Boolean")
1158+
lines.add("fun verifierHasReceivedRequests(verifier: WireMockServer, actionName: String) : Boolean")
11581159
}
11591160
format.isJava() -> {
1160-
lines.add("public boolean verifierHasServedRequests(WireMockServer verifier, String actionName)")
1161+
lines.add("public boolean verifierHasReceivedRequests(WireMockServer verifier, String actionName)")
11611162
}
11621163
}
11631164
lines.block {
@@ -1166,7 +1167,7 @@ class TestSuiteWriter {
11661167
if (format.isKotlin()) {
11671168
lines.add(".allServeEvents")
11681169
lines.add(".filter { it.wasMatched && it.stubMapping.metadata != null }")
1169-
lines.add(".any { it.stubMapping.metadata.getString(\"ssrf\") == actionName }")
1170+
lines.add(".any { it.stubMapping.metadata.getString(SSRF_METADATA_TAG) == actionName }")
11701171
}
11711172
}
11721173
}

0 commit comments

Comments
 (0)