Browse Source

Fix latest dep tests (#9892)

Lauri Tulmin 1 year ago
parent
commit
dedfc204c1

+ 1 - 1
instrumentation/spring/spring-security-config-6.0/library/build.gradle.kts

@@ -12,7 +12,7 @@ dependencies {
   implementation(project(":instrumentation:reactor:reactor-3.1:library"))
 
   testImplementation(project(":testing-common"))
-  testImplementation("org.springframework:spring-test:6.0.0")
+  testLibrary("org.springframework:spring-test:6.0.0")
 }
 
 otelJava {

+ 1 - 0
instrumentation/spring/spring-web/spring-web-3.1/library/src/test/java/io/opentelemetry/instrumentation/spring/web/v3_1/SpringWebInstrumentationTest.java

@@ -37,6 +37,7 @@ public class SpringWebInstrumentationTest extends AbstractHttpClientTest<HttpEnt
   static RestTemplate restTemplate;
 
   @BeforeAll
+  @SuppressWarnings("PreferJavaTimeOverload")
   static void setUp() {
     SimpleClientHttpRequestFactory requestFactory = new SimpleClientHttpRequestFactory();
     requestFactory.setConnectTimeout((int) CONNECTION_TIMEOUT.toMillis());

+ 2 - 0
instrumentation/spring/spring-webflux/spring-webflux-5.3/library/build.gradle.kts

@@ -14,6 +14,8 @@ dependencies {
   testLibrary("org.springframework.boot:spring-boot-starter-webflux:2.4.0")
   testLibrary("org.springframework.boot:spring-boot-starter-test:2.4.0")
   testLibrary("org.springframework.boot:spring-boot-starter-reactor-netty:2.4.0")
+  // can be remove after starter is update to depend on spring 6.1
+  latestDepTestLibrary("org.springframework:spring-context:+")
 }
 
 val latestDepTest = findProperty("testLatestDeps") as Boolean

+ 1 - 1
instrumentation/spring/spring-webflux/spring-webflux-5.3/library/src/main/java/io/opentelemetry/instrumentation/spring/webflux/v5_3/WebfluxServerHttpAttributesGetter.java

@@ -20,7 +20,7 @@ enum WebfluxServerHttpAttributesGetter
 
   @Override
   public String getHttpRequestMethod(ServerWebExchange request) {
-    return request.getRequest().getMethodValue();
+    return request.getRequest().getMethod().name();
   }
 
   @Override