Skip to content

Commit b800213

Browse files
authored
Fix cxf latest dep tests (#8449)
1 parent b43be08 commit b800213

File tree

3 files changed

+15
-3
lines changed

3 files changed

+15
-3
lines changed

instrumentation/jaxrs-client/jaxrs-client-2.0-testing/build.gradle.kts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@ plugins {
33
}
44

55
dependencies {
6-
testInstrumentation(project(":instrumentation:http-url-connection:javaagent"))
7-
86
testImplementation("javax.ws.rs:javax.ws.rs-api:2.0.1")
97

108
testLibrary("org.glassfish.jersey.core:jersey-client:2.0")
@@ -17,6 +15,8 @@ dependencies {
1715
testImplementation("javax.xml.bind:jaxb-api:2.2.3")
1816

1917
testInstrumentation(project(":instrumentation:apache-httpclient:apache-httpclient-4.0:javaagent"))
18+
testInstrumentation(project(":instrumentation:http-url-connection:javaagent"))
19+
testInstrumentation(project(":instrumentation:java-http-client:javaagent"))
2020

2121
latestDepTestLibrary("org.glassfish.jersey.inject:jersey-hk2:2.+")
2222
latestDepTestLibrary("org.glassfish.jersey.core:jersey-client:2.+")
@@ -31,4 +31,6 @@ tasks.withType<Test>().configureEach {
3131
// required on jdk17
3232
jvmArgs("--add-opens=java.base/java.net=ALL-UNNAMED")
3333
jvmArgs("-XX:+IgnoreUnrecognizedVMOptions")
34+
35+
systemProperty("testLatestDeps", findProperty("testLatestDeps") as Boolean)
3436
}

instrumentation/jaxrs-client/jaxrs-client-2.0-testing/src/test/groovy/JaxRsClientTest.groovy

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,10 @@ class CxfClientTest extends JaxRsClientTest {
174174
Throwable clientSpanError(URI uri, Throwable exception) {
175175
switch (uri.toString()) {
176176
case "http://localhost:61/": // unopened port
177+
if (exception.getCause() instanceof ConnectException) {
178+
exception = exception.getCause()
179+
}
180+
break
177181
case "https://192.0.2.1/": // non routable address
178182
if (exception.getCause() != null) {
179183
exception = exception.getCause()
@@ -182,6 +186,11 @@ class CxfClientTest extends JaxRsClientTest {
182186
return exception
183187
}
184188

189+
@Override
190+
boolean testWithClientParent() {
191+
!Boolean.getBoolean("testLatestDeps")
192+
}
193+
185194
@Override
186195
String userAgent() {
187196
"Apache"
@@ -191,6 +200,7 @@ class CxfClientTest extends JaxRsClientTest {
191200
ClientBuilder builder() {
192201
return new ClientBuilderImpl()
193202
.property("http.connection.timeout", (long) CONNECT_TIMEOUT_MS)
203+
.property("org.apache.cxf.transport.http.forceVersion", "1.1")
194204
}
195205

196206
@Override

instrumentation/jaxrs/jaxrs-2.0/jaxrs-2.0-cxf-3.2/javaagent/build.gradle.kts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ dependencies {
4242
testLibrary("org.apache.cxf:cxf-rt-transports-http-jetty:3.2.0")
4343
testLibrary("org.apache.cxf:cxf-rt-ws-policy:3.2.0")
4444

45-
latestDepTestLibrary("org.eclipse.jetty:jetty-webapp:9.+")
45+
latestDepTestLibrary("org.eclipse.jetty:jetty-webapp:10.+")
4646
latestDepTestLibrary("org.apache.cxf:cxf-rt-frontend-jaxrs:3.+")
4747
latestDepTestLibrary("org.apache.cxf:cxf-rt-transports-http-jetty:3.+")
4848
latestDepTestLibrary("org.apache.cxf:cxf-rt-ws-policy:3.+")

0 commit comments

Comments
 (0)