[test] Jetty test follow-ups: Windows IT, logging, ClassRule

- WindowsPathResourceIT in exist-webdav (Windows CI integration job)
- jetty-util test dependency for dependency analyze
- org.exist.jetty WARN logging in integration test log4j2 configs
- ControllerTest @ClassRule (one server per class)

Author: duncdrum

exist-core/src/test/java/org/exist/http/urlrewrite/ControllerTest.java

@@ -30,7 +30,7 @@
 import org.apache.http.entity.ContentType;
 import org.exist.http.AbstractHttpTest;
 import org.exist.test.ExistWebServer;
-import org.junit.Rule;
+import org.junit.ClassRule;
 import org.junit.Test;
 
 import java.io.IOException;
@@ -50,8 +50,8 @@ public class ControllerTest extends AbstractHttpTest {
     private static final String LEGACY_CONTROLLER_XQUERY = "<controller>xql</controller>";
     private static final String TEST_DOCUMENT_NAME = "test.xml";
 
-    @Rule
-    public final ExistWebServer existWebServer = new ExistWebServer(true, false, true, true, false);
+    @ClassRule
+    public static final ExistWebServer existWebServer = new ExistWebServer(true, false, true, true, false);
 
     @Test
     public void findsLegacyController() throws IOException {

exist-core/src/test/resources/log4j2.xml

@@ -29,6 +29,7 @@
         </Console>
     </Appenders>
     <Loggers>
+        <Logger name="org.exist.jetty" level="WARN"/>
         <Root level="OFF">
             <AppenderRef ref="Console"/>
         </Root>

extensions/exquery/restxq/src/test/resources/log4j2.xml

@@ -6,6 +6,7 @@
         </Console>
     </Appenders>
     <Loggers>
+        <Logger name="org.exist.jetty" level="WARN"/>
         <Root level="OFF">
             <AppenderRef ref="Console"/>
         </Root>

extensions/modules/file/src/test/resources/log4j2.xml

@@ -29,6 +29,7 @@
         </Console>
     </Appenders>
     <Loggers>
+        <Logger name="org.exist.jetty" level="WARN"/>
         <Root level="OFF">
             <AppenderRef ref="Console"/>
         </Root>

extensions/webdav/pom.xml

@@ -153,6 +153,11 @@
         </dependency>
 
         <!-- needed for starting up a jetty server -->
+        <dependency>
+            <groupId>org.eclipse.jetty</groupId>
+            <artifactId>jetty-util</artifactId>
+            <scope>test</scope>
+        </dependency>
         <dependency>
             <groupId>org.eclipse.jetty</groupId>
             <artifactId>jetty-deploy</artifactId>

extensions/webdav/src/test/java/org/exist/jetty/WindowsPathResourceIT.java

@@ -0,0 +1,77 @@
+/*
+ * eXist-db Open Source Native XML Database
+ * Copyright (C) 2001 The eXist-db Authors
+ *
+ * info@exist-db.org
+ * http://www.exist-db.org
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+ */
+package org.exist.jetty;
+
+import org.eclipse.jetty.util.resource.PathResource;
+import org.eclipse.jetty.util.resource.Resource;
+import org.eclipse.jetty.util.resource.ResourceFactory;
+import org.junit.Test;
+
+import java.io.File;
+import java.nio.file.Files;
+import java.nio.file.InvalidPathException;
+import java.nio.file.Path;
+
+import static org.junit.Assert.assertNotSame;
+import static org.junit.Assert.assertTrue;
+import static org.junit.Assume.assumeTrue;
+
+/**
+ * Windows integration regression test for Jetty 12.1 {@link PathResource#resolve(String)} on
+ * drive URIs ({@code /D:/...}). Lives in {@code exist-webdav} so it runs on Windows CI
+ * ({@code verify -DskipUnitTests=true}).
+ */
+public class WindowsPathResourceIT {
+
+    @Test
+    public void resolveWebInfOnWindowsDriveUri() throws Exception {
+        assumeTrue("Windows-only PathResource URI regression", File.separatorChar == '\\');
+
+        final ResourceFactory resourceFactory = ResourceFactory.root();
+        final Path webapp = Files.createTempDirectory("webapp");
+        Files.createDirectory(webapp.resolve("WEB-INF"));
+        try {
+            final Resource resource = resourceFactory.newResource(webapp);
+            assumeTrue("Expected PathResource for local webapp directory", resource instanceof PathResource);
+            final String uriPath = resource.getURI().getPath();
+            assumeTrue("Expected absolute Windows drive URI path, got: " + uriPath,
+                    uriPath != null && uriPath.matches("/[A-Za-z]:/.*"));
+
+            final PathResource pathResource = (PathResource) resource;
+            try {
+                pathResource.resolve("WEB-INF/");
+                // Jetty version may already fix resolve; wrapped path must still work.
+            } catch (final InvalidPathException e) {
+                // Expected on Jetty 12.1.x with /X:/... URI paths.
+            }
+
+            final Resource wrapped = WindowsPathResource.wrapIfNeeded(pathResource, resourceFactory);
+            assertNotSame(pathResource, wrapped);
+
+            final Resource webInf = wrapped.resolve("WEB-INF/");
+            assertTrue("WEB-INF should resolve to a directory", webInf.isDirectory());
+        } finally {
+            Files.deleteIfExists(webapp.resolve("WEB-INF"));
+            Files.deleteIfExists(webapp);
+        }
+    }
+}

extensions/webdav/src/test/resources/log4j2.xml

@@ -29,6 +29,7 @@
         </Console>
     </Appenders>
     <Loggers>
+        <Logger name="org.exist.jetty" level="WARN"/>
         <Root level="OFF">
             <AppenderRef ref="Console"/>
         </Root>