From d51d78a7f4554a163b4254aa25cb1fed4fe30460 Mon Sep 17 00:00:00 2001
From: Basil Crow <me@basilcrow.com>
Date: Tue, 14 Dec 2021 19:06:03 -0800
Subject: [PATCH] Clean up build

---
 core/pom.xml                                  | 34 ++++++--------
 .../org/kohsuke/stapler/DispatcherTest.java   | 18 +++----
 .../org/kohsuke/stapler/Stapler2Test.java     | 13 ++---
 .../stapler/bind/JavaScriptProxyTest.java     |  6 +++
 .../interceptor/JsonOutputFilterTest.java     |  6 +++
 .../kohsuke/stapler/json/JsonBodyTest.java    |  4 +-
 .../stapler/json/SubmittedFormTest.java       |  2 +-
 groovy/pom.xml                                |  2 +
 jelly/pom.xml                                 | 15 +++---
 .../stapler/jelly/BindTagJQueryTest.java      |  2 +
 .../kohsuke/stapler/jelly/BindTagTest.java    |  2 +
 jrebel/pom.xml                                |  1 +
 jruby/pom.xml                                 |  7 ++-
 jsp/pom.xml                                   |  1 +
 pom.xml                                       | 47 ++++++++++++-------
 15 files changed, 96 insertions(+), 64 deletions(-)

diff --git a/core/pom.xml b/core/pom.xml
index 6e341723aa..6aab0b44a2 100644
--- a/core/pom.xml
+++ b/core/pom.xml
@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
   <modelVersion>4.0.0</modelVersion>
   <parent>
@@ -51,11 +52,6 @@
       <artifactId>commons-io</artifactId>
       <version>2.11.0</version>
     </dependency>
-    <dependency>
-      <groupId>commons-logging</groupId>
-      <artifactId>commons-logging</artifactId>
-      <version>1.2</version>
-    </dependency>
     <dependency>
       <groupId>commons-codec</groupId>
       <artifactId>commons-codec</artifactId>
@@ -111,6 +107,12 @@
       <version>20110809</version>
       <optional>true</optional>
     </dependency>
+    <dependency>
+      <groupId>commons-httpclient</groupId>
+      <artifactId>commons-httpclient</artifactId>
+      <version>3.1</version>
+      <scope>test</scope>
+    </dependency>
     <dependency>
       <groupId>junit</groupId>
       <artifactId>junit</artifactId>
@@ -119,6 +121,7 @@
     <dependency>
       <groupId>org.mockito</groupId>
       <artifactId>mockito-core</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.kohsuke.metainf-services</groupId>
@@ -128,22 +131,27 @@
     <dependency>
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-server</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-util</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-servlet</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-webapp</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
-      <groupId>org.jvnet.hudson</groupId>
-      <artifactId>htmlunit</artifactId>
+      <groupId>org.jenkins-ci.main</groupId>
+      <artifactId>jenkins-test-harness-htmlunit</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>com.jolira</groupId>
@@ -165,25 +173,13 @@
     <dependency>
       <groupId>org.jenkins-ci</groupId>
       <artifactId>test-annotations</artifactId>
-      <version>1.4</version>
       <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.hamcrest</groupId>
       <artifactId>hamcrest</artifactId>
-      <version>2.2</version>
       <scope>test</scope>
     </dependency>
-    <dependency>
-      <groupId>com.github.spotbugs</groupId>
-      <artifactId>spotbugs-annotations</artifactId>
-      <exclusions>
-        <exclusion>
-          <groupId>com.google.code.findbugs</groupId>
-          <artifactId>jsr305</artifactId>
-        </exclusion>
-      </exclusions>
-    </dependency>
   </dependencies>
 </project>
 
diff --git a/core/src/test/java/org/kohsuke/stapler/DispatcherTest.java b/core/src/test/java/org/kohsuke/stapler/DispatcherTest.java
index 17eee2e76e..e5740f09c6 100644
--- a/core/src/test/java/org/kohsuke/stapler/DispatcherTest.java
+++ b/core/src/test/java/org/kohsuke/stapler/DispatcherTest.java
@@ -5,7 +5,7 @@
 import com.gargoylesoftware.htmlunit.Page;
 import com.gargoylesoftware.htmlunit.TextPage;
 import com.gargoylesoftware.htmlunit.WebClient;
-import com.gargoylesoftware.htmlunit.WebRequestSettings;
+import com.gargoylesoftware.htmlunit.WebRequest;
 import org.apache.commons.io.IOUtils;
 import org.kohsuke.stapler.interceptor.RequirePOST;
 import org.kohsuke.stapler.json.JsonBody;
@@ -84,7 +84,7 @@ public void testVerbMatch() throws Exception {
     }
 
     private void check(WebClient wc, HttpMethod m) throws java.io.IOException {
-        TextPage p = wc.getPage(new WebRequestSettings(new URL(url, "verbMatch/"), m));
+        TextPage p = wc.getPage(new WebRequest(new URL(url, "verbMatch/"), m));
         assertEquals("Got " + m.name(), p.getContent());
     }
 
@@ -146,7 +146,7 @@ public void testInterceptorStage() throws Exception {
             assertEquals(404, e.getStatusCode());
         }
 
-        WebRequestSettings req = new WebRequestSettings(new URL(url, "interceptorStage/foo"), HttpMethod.POST);
+        WebRequest req = new WebRequest(new URL(url, "interceptorStage/foo"), HttpMethod.POST);
         req.setAdditionalHeader("Content-Type","application/json");
         req.setRequestBody("{x:3,y:5}");
         TextPage p = wc.getPage(req);
@@ -163,7 +163,7 @@ public void testInterceptorStageContentTypeWithCharset() throws Exception {
             assertEquals(404, e.getStatusCode());
         }
 
-        WebRequestSettings req = new WebRequestSettings(new URL(url, "interceptorStage/foo"), HttpMethod.POST);
+        WebRequest req = new WebRequest(new URL(url, "interceptorStage/foo"), HttpMethod.POST);
         req.setAdditionalHeader("Content-Type","application/json; charset=utf-8");
         req.setRequestBody("{x:3,y:5}");
         TextPage p = wc.getPage(req);
@@ -227,7 +227,7 @@ public void testPutInheritance() throws Exception {
         WebClient wc = new WebClient();
 
         // the request should get to the overriding method and it should still see all the annotations in the base type
-        WebRequestSettings wrs = new WebRequestSettings(new URL(url, "putInheritance/foo"), HttpMethod.PUT);
+        WebRequest wrs = new WebRequest(new URL(url, "putInheritance/foo"), HttpMethod.PUT);
         wrs.setRequestBody("Hello");
         TextPage p = wc.getPage(wrs);
         assertEquals("Hello World!", p.getContent());
@@ -241,7 +241,7 @@ public void testPutInheritance() throws Exception {
         }
 
         //Invoke Post as well
-        wrs = new WebRequestSettings(new URL(url, "putInheritance/acme"), HttpMethod.POST);
+        wrs = new WebRequest(new URL(url, "putInheritance/acme"), HttpMethod.POST);
         wrs.setRequestBody("Hello");
         p = wc.getPage(wrs);
         assertEquals("POST: Hello", p.getContent());
@@ -255,14 +255,14 @@ public void testInterfaceMethods() throws Exception {
         } catch (FailingHttpStatusCodeException x) {
             assertEquals(HttpServletResponse.SC_METHOD_NOT_ALLOWED, x.getStatusCode());
         }
-        assertEquals("default", wc.getPage(new WebRequestSettings(new URL(url, "usesInterfaceMethods/foo"), HttpMethod.POST)).getWebResponse().getContentAsString().trim());
+        assertEquals("default", wc.getPage(new WebRequest(new URL(url, "usesInterfaceMethods/foo"), HttpMethod.POST)).getWebResponse().getContentAsString().trim());
         try {
             wc.getPage(new URL(url, "overridesInterfaceMethods/foo"));
             fail();
         } catch (FailingHttpStatusCodeException x) {
             assertEquals(HttpServletResponse.SC_METHOD_NOT_ALLOWED, x.getStatusCode());
         }
-        assertEquals("due to UnionAnnotatedElement it is even inherited", "overridden", wc.getPage(new WebRequestSettings(new URL(url, "overridesInterfaceMethods/foo"), HttpMethod.POST)).getWebResponse().getContentAsString().trim());
+        assertEquals("due to UnionAnnotatedElement it is even inherited", "overridden", wc.getPage(new WebRequest(new URL(url, "overridesInterfaceMethods/foo"), HttpMethod.POST)).getWebResponse().getContentAsString().trim());
     }
     public interface InterfaceWithWebMethods {
         @RequirePOST
@@ -309,7 +309,7 @@ public void testRequirePostOnBase() throws Exception {
         }
 
         // POST should succeed
-        wc.getPage(new WebRequestSettings(url, HttpMethod.POST));
+        wc.getPage(new WebRequest(url, HttpMethod.POST));
         assertEquals(1, requirePostOnBase.hit);
     }
 
diff --git a/core/src/test/java/org/kohsuke/stapler/Stapler2Test.java b/core/src/test/java/org/kohsuke/stapler/Stapler2Test.java
index ec621775f4..472e2cb8ba 100644
--- a/core/src/test/java/org/kohsuke/stapler/Stapler2Test.java
+++ b/core/src/test/java/org/kohsuke/stapler/Stapler2Test.java
@@ -26,8 +26,10 @@
 
 package org.kohsuke.stapler;
 
+import static org.junit.Assert.assertThrows;
+
+import com.gargoylesoftware.htmlunit.FailingHttpStatusCodeException;
 import com.gargoylesoftware.htmlunit.WebClient;
-import com.gargoylesoftware.htmlunit.WebResponse;
 import java.net.URL;
 import javax.servlet.http.HttpServletResponse;
 import org.jvnet.hudson.test.For;
@@ -40,16 +42,15 @@ public class Stapler2Test extends JettyTestCase {
     @Issue("SECURITY-390")
     public void testTraceXSS() throws Exception {
         WebClient wc = new WebClient();
-        wc.setThrowExceptionOnFailingStatusCode(false);
-        WebResponse rsp;
+        FailingHttpStatusCodeException exc;
         Dispatcher.TRACE = true;
         try {
-            rsp = wc.getPage(new URL(this.url, "thing/<button>/x")).getWebResponse();
+            exc = assertThrows(FailingHttpStatusCodeException.class, () -> wc.getPage(new URL(this.url, "thing/<button>/x")).getWebResponse());
         } finally {
             Dispatcher.TRACE = false;
         }
-        assertEquals(HttpServletResponse.SC_NOT_FOUND, rsp.getStatusCode());
-        String html = rsp.getContentAsString();
+        assertEquals(HttpServletResponse.SC_NOT_FOUND, exc.getStatusCode());
+        String html = exc.getResponse().getContentAsString();
         assertTrue(html, html.contains("&lt;button&gt;"));
         assertFalse(html, html.contains("<button>"));
     }
diff --git a/core/src/test/java/org/kohsuke/stapler/bind/JavaScriptProxyTest.java b/core/src/test/java/org/kohsuke/stapler/bind/JavaScriptProxyTest.java
index 25b43ca9a3..42bf352ef5 100644
--- a/core/src/test/java/org/kohsuke/stapler/bind/JavaScriptProxyTest.java
+++ b/core/src/test/java/org/kohsuke/stapler/bind/JavaScriptProxyTest.java
@@ -42,11 +42,15 @@ public void handleAlert(Page page, String message) {
         HtmlPage page = wc.getPage(new URL(url, "/"));
 
         page.executeJavaScript("v.foo(3,'test',callback);");
+        wc.getJavaScriptEngine().processPostponedActions();
+        wc.waitForBackgroundJavaScript(10000);
         assertEquals("string:test3",msg[0]);
         msg[0] = null;
 
         // test null unmarshalling and marshalling
         page.executeJavaScript("v.foo(0,null,callback);");
+        wc.getJavaScriptEngine().processPostponedActions();
+        wc.waitForBackgroundJavaScript(10000);
         assertEquals("object:null",msg[0]);
     }
 
@@ -57,6 +61,8 @@ public void testAnonymousBind() throws Exception {
         WebClient wc = new WebClient();
         HtmlPage page = wc.getPage(new URL(url, "/bindAnonymous"));
         page.executeJavaScript("v.xyz('hello');");
+        wc.getJavaScriptEngine().processPostponedActions();
+        wc.waitForBackgroundJavaScript(10000);
         assertEquals("hello",anonymousValue);
     }
 
diff --git a/core/src/test/java/org/kohsuke/stapler/interceptor/JsonOutputFilterTest.java b/core/src/test/java/org/kohsuke/stapler/interceptor/JsonOutputFilterTest.java
index 58c5337ad2..b1ea922850 100644
--- a/core/src/test/java/org/kohsuke/stapler/interceptor/JsonOutputFilterTest.java
+++ b/core/src/test/java/org/kohsuke/stapler/interceptor/JsonOutputFilterTest.java
@@ -38,6 +38,8 @@ public void handleAlert(Page page, String message) {
         HtmlPage page = wc.getPage(new URL(url, "/"));
 
         page.executeJavaScript("v.getSomeExcludedData(callback);");
+        wc.getJavaScriptEngine().processPostponedActions();
+        wc.waitForBackgroundJavaScript(10000);
 
         Map json = (Map)JSON.parse(msg[0]);
         assertTrue(json.containsKey("name"));
@@ -57,6 +59,8 @@ public void handleAlert(Page page, String message) {
         HtmlPage page = wc.getPage(new URL(url, "/"));
 
         page.executeJavaScript("v.getSomeIncludedData(callback);");
+        wc.getJavaScriptEngine().processPostponedActions();
+        wc.waitForBackgroundJavaScript(10000);
 
         Map json = (Map)JSON.parse(msg[0]);
         assertTrue(json.containsKey("name"));
@@ -76,6 +80,8 @@ public void handleAlert(Page page, String message) {
         HtmlPage page = wc.getPage(new URL(url, "/"));
 
         page.executeJavaScript("v.getSomeExcludedList(callback);");
+        wc.getJavaScriptEngine().processPostponedActions();
+        wc.waitForBackgroundJavaScript(10000);
 
         Object[] json = (Object[])JSON.parse(msg[0]);
         assertEquals(3, json.length);
diff --git a/core/src/test/java/org/kohsuke/stapler/json/JsonBodyTest.java b/core/src/test/java/org/kohsuke/stapler/json/JsonBodyTest.java
index 4246d69ede..c13c00db4e 100644
--- a/core/src/test/java/org/kohsuke/stapler/json/JsonBodyTest.java
+++ b/core/src/test/java/org/kohsuke/stapler/json/JsonBodyTest.java
@@ -2,7 +2,7 @@
 
 import com.gargoylesoftware.htmlunit.HttpMethod;
 import com.gargoylesoftware.htmlunit.WebClient;
-import com.gargoylesoftware.htmlunit.WebRequestSettings;
+import com.gargoylesoftware.htmlunit.WebRequest;
 import com.gargoylesoftware.htmlunit.WebResponse;
 import java.net.URL;
 import org.kohsuke.stapler.test.JettyTestCase;
@@ -10,7 +10,7 @@
 public class JsonBodyTest extends JettyTestCase {
 
     public void testSmokes() throws Exception {
-        WebRequestSettings req = new WebRequestSettings(new URL(url, "double"), HttpMethod.POST);
+        WebRequest req = new WebRequest(new URL(url, "double"), HttpMethod.POST);
         req.setAdditionalHeader("Content-Type", "application/json");
         req.setRequestBody("{\"x\":10,\"y\":5}");
         WebResponse response = new WebClient().getPage(req).getWebResponse();
diff --git a/core/src/test/java/org/kohsuke/stapler/json/SubmittedFormTest.java b/core/src/test/java/org/kohsuke/stapler/json/SubmittedFormTest.java
index 9c908dfafb..adde9c9019 100644
--- a/core/src/test/java/org/kohsuke/stapler/json/SubmittedFormTest.java
+++ b/core/src/test/java/org/kohsuke/stapler/json/SubmittedFormTest.java
@@ -29,7 +29,7 @@ public void testMainFeature() throws Exception {
         HtmlPage page = wc.getPage(new URL(url, "/form.html"));
         HtmlForm f = page.getFormByName("main");
         f.getInputByName("json").setValueAttribute("{\"first\":\"Kohsuke\",\"last\":\"Kawaguchi\"}");
-        f.submit();
+        f.submit(null);
     }
 
     public HttpResponse doSubmit(@SubmittedForm JSONObject o) {
diff --git a/groovy/pom.xml b/groovy/pom.xml
index 1590c654d9..e4db4255c7 100644
--- a/groovy/pom.xml
+++ b/groovy/pom.xml
@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
   <modelVersion>4.0.0</modelVersion>
   <parent>
@@ -48,6 +49,7 @@
     <dependency>
       <groupId>org.mockito</groupId>
       <artifactId>mockito-core</artifactId>
+      <scope>test</scope>
     </dependency>
   </dependencies>
 </project>
diff --git a/jelly/pom.xml b/jelly/pom.xml
index 776dfa654a..01f7d04c40 100644
--- a/jelly/pom.xml
+++ b/jelly/pom.xml
@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
   <modelVersion>4.0.0</modelVersion>
   <parent>
@@ -36,10 +37,6 @@
         </exclusion>
       </exclusions>
     </dependency>
-    <dependency>
-      <groupId>commons-collections</groupId>
-      <artifactId>commons-collections</artifactId>
-    </dependency>
     <dependency>
       <groupId>org.dom4j</groupId>
       <artifactId>dom4j</artifactId>
@@ -73,8 +70,9 @@
       <scope>test</scope>
     </dependency>
     <dependency>
-      <groupId>org.jvnet.hudson</groupId>
-      <artifactId>htmlunit</artifactId>
+      <groupId>org.jenkins-ci.main</groupId>
+      <artifactId>jenkins-test-harness-htmlunit</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.kohsuke.metainf-services</groupId>
@@ -90,23 +88,26 @@
     <dependency>
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-server</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-util</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-servlet</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-webapp</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.jvnet.hudson</groupId>
       <artifactId>commons-jelly-tags-define</artifactId>
-      <version>1.0.1-hudson-20071021</version>
       <scope>test</scope>
       <exclusions>
         <exclusion>
diff --git a/jelly/src/test/java/org/kohsuke/stapler/jelly/BindTagJQueryTest.java b/jelly/src/test/java/org/kohsuke/stapler/jelly/BindTagJQueryTest.java
index 615e7dfcf5..01796dc8d3 100644
--- a/jelly/src/test/java/org/kohsuke/stapler/jelly/BindTagJQueryTest.java
+++ b/jelly/src/test/java/org/kohsuke/stapler/jelly/BindTagJQueryTest.java
@@ -33,6 +33,8 @@ public void test1() throws Exception {
         //Check that prototype is included in the page
         assertFalse(content.contains("/am/org/kohsuke/stapler/framework/prototype/prototype.js"));
         page.executeJavaScript("v.foo('hello world', 2);");
+        wc.getJavaScriptEngine().processPostponedActions();
+        wc.waitForBackgroundJavaScript(10000);
         assertEquals("hello world",value);
         assertEquals(2, number);
     }
diff --git a/jelly/src/test/java/org/kohsuke/stapler/jelly/BindTagTest.java b/jelly/src/test/java/org/kohsuke/stapler/jelly/BindTagTest.java
index b69521944b..23d4ad6f32 100644
--- a/jelly/src/test/java/org/kohsuke/stapler/jelly/BindTagTest.java
+++ b/jelly/src/test/java/org/kohsuke/stapler/jelly/BindTagTest.java
@@ -30,6 +30,8 @@ public void test1() throws Exception {
         //Check that prototype is included in the page
         assertTrue(content.contains("/am/org/kohsuke/stapler/framework/prototype/prototype.js"));
         page.executeJavaScript("v.foo('hello world');");
+        wc.getJavaScriptEngine().processPostponedActions();
+        wc.waitForBackgroundJavaScript(10000);
         assertEquals("hello world",value);
     }
 
diff --git a/jrebel/pom.xml b/jrebel/pom.xml
index 0d19822554..6391062454 100644
--- a/jrebel/pom.xml
+++ b/jrebel/pom.xml
@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
   <modelVersion>4.0.0</modelVersion>
   <parent>
diff --git a/jruby/pom.xml b/jruby/pom.xml
index be6e3d8006..fed3827e07 100644
--- a/jruby/pom.xml
+++ b/jruby/pom.xml
@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
   <modelVersion>4.0.0</modelVersion>
   <parent>
@@ -45,7 +46,6 @@
     <dependency>
       <groupId>org.jvnet.hudson</groupId>
       <artifactId>commons-jelly-tags-define</artifactId>
-      <version>1.0.1-hudson-20071021</version>
       <scope>test</scope>
       <exclusions>
         <exclusion>
@@ -82,17 +82,16 @@
     <dependency>
       <groupId>org.mockito</groupId>
       <artifactId>mockito-core</artifactId>
+      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.jenkins-ci</groupId>
       <artifactId>test-annotations</artifactId>
-      <version>1.4</version>
       <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.hamcrest</groupId>
-      <artifactId>hamcrest-all</artifactId>
-      <version>1.3</version>
+      <artifactId>hamcrest</artifactId>
       <scope>test</scope>
     </dependency>
   </dependencies>
diff --git a/jsp/pom.xml b/jsp/pom.xml
index aad90f99a7..eaaa313845 100644
--- a/jsp/pom.xml
+++ b/jsp/pom.xml
@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
   <modelVersion>4.0.0</modelVersion>
   <parent>
diff --git a/pom.xml b/pom.xml
index 85597e8fd0..d9c83fe3ca 100644
--- a/pom.xml
+++ b/pom.xml
@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
   <modelVersion>4.0.0</modelVersion>
   
@@ -56,7 +57,6 @@
   </pluginRepositories>
 
   <properties>
-    <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding> <!-- TODO move to parent POM -->
     <changelist>999999-SNAPSHOT</changelist>
     <scmTag>HEAD</scmTag>
     <spotbugs.excludeFilterFile>${project.basedir}/../src/spotbugs/spotbugs-excludes.xml</spotbugs.excludeFilterFile>
@@ -96,6 +96,11 @@
 
   <dependencyManagement>
   <dependencies>
+    <dependency>
+      <groupId>commons-logging</groupId>
+      <artifactId>commons-logging</artifactId>
+      <version>1.2</version>
+    </dependency>
     <dependency>
       <groupId>org.kohsuke.metainf-services</groupId>
       <artifactId>metainf-services</artifactId>
@@ -107,44 +112,41 @@
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-server</artifactId>
       <version>${jetty.version}</version>
-      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-util</artifactId>
       <version>${jetty.version}</version>
-      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-servlet</artifactId>
       <version>${jetty.version}</version>
-      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.eclipse.jetty</groupId>
       <artifactId>jetty-webapp</artifactId>
       <version>${jetty.version}</version>
-      <scope>test</scope>
     </dependency>
     <dependency>
       <groupId>org.mockito</groupId>
       <artifactId>mockito-core</artifactId>
       <version>4.1.0</version>
-      <scope>test</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.jenkins-ci.main</groupId>
+      <artifactId>jenkins-test-harness-htmlunit</artifactId>
+      <version>56.vf1eabfc274f8</version>
     </dependency>
     <dependency>
       <groupId>org.jvnet.hudson</groupId>
-      <artifactId>htmlunit</artifactId>
-      <version>2.6-hudson-2</version>
-      <scope>test</scope>
-      <exclusions>
-      	<exclusion>
-	      	<!--  hides JDK DOM classes in Eclipse -->
-      		<groupId>xml-apis</groupId>
-      		<artifactId>xml-apis</artifactId>
-      	</exclusion>
-      </exclusions>
+      <artifactId>commons-jelly-tags-define</artifactId>
+      <version>1.0.1-hudson-20071021</version>
+    </dependency>
+    <dependency>
+      <groupId>org.hamcrest</groupId>
+      <artifactId>hamcrest</artifactId>
+      <version>2.2</version>
     </dependency>
     <dependency>
       <groupId>commons-collections</groupId>
@@ -154,4 +156,17 @@
   </dependencies>
   </dependencyManagement>
 
+  <dependencies>
+    <dependency>
+      <groupId>com.github.spotbugs</groupId>
+      <artifactId>spotbugs-annotations</artifactId>
+      <optional>true</optional>
+      <exclusions>
+        <exclusion>
+          <groupId>com.google.code.findbugs</groupId>
+          <artifactId>jsr305</artifactId>
+        </exclusion>
+      </exclusions>
+    </dependency>
+  </dependencies>
 </project>