Skip to content

Commit 1d17d89

Browse files
committed
test: retry for tomcat test cases
1 parent 3d36490 commit 1d17d89

File tree

4 files changed

+8
-2
lines changed

4 files changed

+8
-2
lines changed

gradle/libs.versions.toml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ java-websocket = "1.6.0"
2222
mockito = "5.18.0"
2323
hamcrest = "3.0"
2424
junit-jupiter = "5.13.4"
25+
junit-pioneer = "2.3.0"
2526
junit-platform = "1.13.4"
2627
testcontainers = "1.21.3"
2728

@@ -49,6 +50,7 @@ mockito-core = { module = "org.mockito:mockito-core", version.ref = "mockito" }
4950
mockito-junit-jupiter = { module = "org.mockito:mockito-junit-jupiter", version.ref = "mockito" }
5051
hamcrest = { module = "org.hamcrest:hamcrest", version.ref = "hamcrest" }
5152
junit-jupiter = { module = "org.junit.jupiter:junit-jupiter", version.ref = "junit-jupiter" }
53+
junit-pioneer = { module = "org.junit-pioneer:junit-pioneer", version.ref = "junit-pioneer" }
5254
junit-platform-launcher = { module = "org.junit.platform:junit-platform-launcher", version.ref = "junit-platform" }
5355
junit-platform-reporting = { module = "org.junit.platform:junit-platform-reporting", version.ref = "junit-platform" }
5456
testcontainers = { module = "org.testcontainers:testcontainers", version.ref = "testcontainers" }

integration-test/build.gradle.kts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ dependencies {
2727
testImplementation(libs.okhttp3)
2828
testImplementation(libs.junit.platform.reporting)
2929
testImplementation(libs.junit.jupiter)
30+
testImplementation(libs.junit.pioneer)
3031
testRuntimeOnly(libs.junit.platform.launcher)
3132
testImplementation(libs.hamcrest)
3233
testImplementation(libs.bundles.testcontainers)

integration-test/src/test/java/com/reajason/javaweb/integration/probe/tomcat/Tomcat5ContainerTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
import lombok.extern.slf4j.Slf4j;
99
import org.junit.jupiter.api.AfterAll;
1010
import org.junit.jupiter.api.Test;
11+
import org.junitpioneer.jupiter.RetryingTest;
1112
import org.objectweb.asm.Opcodes;
1213
import org.testcontainers.containers.GenericContainer;
1314
import org.testcontainers.containers.wait.strategy.Wait;
@@ -40,7 +41,8 @@ public static void stopContainer() {
4041
System.out.println(container.getLogs());
4142
}
4243

43-
@Test
44+
// 存在首次请求,Tomcat 无法通过 req.getParameter 拿到参数的情况,因此需要重试
45+
@RetryingTest(3)
4446
void testJDK() {
4547
String url = getUrl(container);
4648
String data = VulTool.post(url + "/b64", DetectionTool.getJdkDetection());

integration-test/src/test/java/com/reajason/javaweb/integration/probe/tomcat/Tomcat9ContainerTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
import lombok.extern.slf4j.Slf4j;
99
import net.bytebuddy.jar.asm.Opcodes;
1010
import org.junit.jupiter.api.Test;
11+
import org.junitpioneer.jupiter.RetryingTest;
1112
import org.testcontainers.containers.GenericContainer;
1213
import org.testcontainers.containers.wait.strategy.Wait;
1314
import org.testcontainers.junit.jupiter.Container;
@@ -34,7 +35,7 @@ public class Tomcat9ContainerTest {
3435
.waitingFor(Wait.forHttp("/app"))
3536
.withExposedPorts(8080);
3637

37-
@Test
38+
@RetryingTest(3)
3839
void testJDK() {
3940
String url = getUrl(container);
4041
String data = VulTool.post(url + "/b64", DetectionTool.getJdkDetection());

0 commit comments

Comments
 (0)