Cleanup ee9 DefaultServletTest

+ Reenable disabled test testListingContextBreakout
jetty · Jun 17, 2024 · 7cf5479 · 7cf5479
1 parent bbc3430
commit 7cf5479
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/...ee9/jetty-ee9-servlet/src/test/java/org/eclipse/jetty/ee9/servlet/DefaultServletTest.java b/...ee9/jetty-ee9-servlet/src/test/java/org/eclipse/jetty/ee9/servlet/DefaultServletTest.java
@@ -66,7 +66,6 @@
 import org.hamcrest.Matchers;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.condition.OS;
 import org.junit.jupiter.api.extension.ExtendWith;
@@ -530,7 +529,7 @@ public static Stream<Arguments> contextBreakoutScenarios()
             );
 
             // A Raw Question mark in the prefix can be interpreted as a query section
-            if (prefix.contains("?"))
+            if (prefix.contains("?") || prefix.contains(";"))
             {
                 scenarios.addScenario(
                     "GET " + prefix + "/../index.html",
@@ -635,14 +634,15 @@ public static Stream<Arguments> contextBreakoutScenarios()
 
     @ParameterizedTest
     @MethodSource("contextBreakoutScenarios")
-    @Disabled // TODO
     public void testListingContextBreakout(Scenario scenario) throws Exception
     {
         Path docRoot = workDir.getEmptyPathDir().resolve("docroot");
         FS.ensureDirExists(docRoot);
 
         startServer((context) ->
         {
+            context.setBaseResource(ResourceFactory.of(context).newResource(docRoot));
+
             ServletHolder defholder = context.addServlet(DefaultServlet.class, "/");
             defholder.setInitParameter("dirAllowed", "true");
             defholder.setInitParameter("redirectWelcome", "false");