Fix failing tests in CI

Author: manuel-alvarez-alvarez

dd-java-agent/instrumentation/armeria-jetty/src/test/jetty11/groovy/ArmeriaJetty11ServerTest.groovy

@@ -114,4 +114,9 @@ class ArmeriaJetty11ServerTest extends HttpServerTest<ArmeriaServer> {
   boolean testEncodedPath() {
     return false
   }
+
+  @Override
+  boolean testSessionId() {
+    true
+  }
 }

dd-java-agent/instrumentation/jetty-11/src/test/groovy/Jetty11Test.groovy

@@ -91,7 +91,7 @@ abstract class Jetty11Test extends HttpServerTest<Server> {
   }
 }
 
-class Jetty11V0ForkedTest extends Jetty11Test implements TestingGenericHttpNamingConventions.ServerV0 {
+class  Jetty11V0ForkedTest extends Jetty11Test implements TestingGenericHttpNamingConventions.ServerV0 {
 }
 
 class Jetty11V1ForkedTest extends Jetty11Test implements TestingGenericHttpNamingConventions.ServerV1 {

dd-java-agent/instrumentation/jetty-11/src/test/groovy/JettyAsyncHandlerTest.groovy

@@ -15,6 +15,11 @@ class JettyAsyncHandlerTest extends Jetty11Test implements TestingGenericHttpNam
     new ContinuationTestHandler(super.handler())
   }
 
+  @Override
+  boolean testSessionId() {
+    false // continuation test handler not working with sessions
+  }
+
   static class ContinuationTestHandler implements Handler {
     @Delegate
     private final Handler delegate

dd-java-agent/instrumentation/jetty-11/src/testFixtures/groovy/JettyServer.groovy

@@ -20,8 +20,7 @@ class JettyServer implements HttpServer {
   final server = new Server(0) // select random open port
 
   JettyServer(Handler handler) {
-    sessionHandler.handler = handler
-    server.handler = sessionHandler
+    server.handler = handler
     server.addBean(errorHandler)
   }
 
@@ -49,6 +48,7 @@ class JettyServer implements HttpServer {
 
   static AbstractHandler servletHandler(Class<? extends Servlet> servlet) {
     ServletContextHandler handler = new ServletContextHandler(null, "/context-path")
+    final sessionHandler = new SessionHandler()
     handler.sessionHandler = sessionHandler
     handler.errorHandler = errorHandler
     handler.servletHandler.addFilterWithMapping(EnableMultipartFilter, '/*', FilterMapping.ALL)
@@ -75,6 +75,4 @@ class JettyServer implements HttpServer {
       }
     }
   }
-
-  static sessionHandler = new SessionHandler()
 }

dd-java-agent/instrumentation/jetty-12/build.gradle

@@ -32,9 +32,14 @@ addTestSuiteExtendingForDir('ee10LatestDepTest', 'latestDepTest', 'test/ee10')
   }
 }
 
+forbiddenApisMain_java17 {
+  failOnMissingClasses = false
+}
+
 compileTestGroovy {
   javaLauncher = getJavaLauncherFor(17)
 }
+
 dependencies {
   main_java17CompileOnly ("org.eclipse.jetty:jetty-server:12.0.0") {
     exclude group: 'org.slf4j', module: 'slf4j-api'

dd-java-agent/instrumentation/jetty-12/src/test/ee8/groovy/JettyServer.groovy

@@ -1,5 +1,3 @@
-import org.eclipse.jetty.session.DefaultSessionIdManager
-
 import static datadog.trace.agent.test.base.HttpServerTest.ServerEndpoint.NOT_FOUND
 import static datadog.trace.agent.test.base.HttpServerTest.ServerEndpoint.UNKNOWN