Fix failing test

elastic · Jan 8, 2025 · e30d4ae · e30d4ae
1 parent fac3455
commit e30d4ae
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/...5/src/test/java/co/elastic/apm/agent/springwebflux/AbstractServerInstrumentationTest.java b/...5/src/test/java/co/elastic/apm/agent/springwebflux/AbstractServerInstrumentationTest.java
@@ -452,6 +452,10 @@ protected ErrorCaptureImpl getFirstError() {
     }
 
     static TransactionImpl checkTransaction(TransactionImpl transaction, String expectedName, String expectedMethod, int expectedStatus) {
+        return checkTransaction(transaction, expectedName, expectedMethod, expectedStatus, "5.3.30");
+    }
+
+    static TransactionImpl checkTransaction(TransactionImpl transaction, String expectedName, String expectedMethod, int expectedStatus, String expectedFrameworkVersion) {
         assertThat(transaction.getType()).isEqualTo("request");
         assertThat(transaction.getNameAsString()).isEqualTo(expectedName);
 
@@ -468,7 +472,7 @@ static TransactionImpl checkTransaction(TransactionImpl transaction, String expe
             .isEqualTo("Spring Webflux");
 
         assertThat(transaction.getFrameworkVersion())
-            .isEqualTo("5.3.30");
+            .isEqualTo(expectedFrameworkVersion);
 
         return transaction;
     }

diff --git a/...ebflux-spring5/src/test/java/co/elastic/apm/agent/springwebflux/ServletContainerTest.java b/...ebflux-spring5/src/test/java/co/elastic/apm/agent/springwebflux/ServletContainerTest.java
@@ -74,7 +74,7 @@ void shouldOnlyCreateOneTransaction() throws InterruptedException {
         TransactionImpl transaction = reporter.getFirstTransaction(200);
 
         // transaction naming should be set by webflux instrumentation
-        AbstractServerInstrumentationTest.checkTransaction(transaction, "GET /functional/with-parameters/{id}", "GET", 200);
+        AbstractServerInstrumentationTest.checkTransaction(transaction, "GET /functional/with-parameters/{id}", "GET", 200, "6.2.0");
 
         // transaction HTTP part should be provided by servlet instrumentation
         AbstractServerInstrumentationTest.checkUrl(client, transaction, "/with-parameters/42");