Changeset 19155 in josm

Timestamp:

2024-07-29T17:57:38+02:00 (4 months ago)

Author:

taylor.smock

Message:

Fix tests broken by r19152

Location:

trunk/test/unit/org/openstreetmap/josm

Files:

• actions/AddImageryLayerActionTest.java (modified) (5 diffs)
• actions/downloadtasks/AbstractDownloadTaskTestParent.java (modified) (4 diffs)
• data/cache/JCSCachedTileLoaderJobTest.java (modified) (27 diffs)
• data/imagery/TMSCachedTileLoaderJobTest.java (modified) (12 diffs)
• data/imagery/WMSEndpointTileSourceTest.java (modified) (13 diffs)
• data/imagery/WMTSTileSourceTest.java (modified) (7 diffs)
• data/osm/DefaultNameFormatterTest.java (modified) (4 diffs)
• gui/preferences/server/ApiUrlTestTaskTest.java (modified) (6 diffs)
• io/OsmServerHistoryReaderTest.java (modified) (3 diffs)
• io/OverpassDownloadReaderTest.java (modified) (5 diffs)
• io/imagery/WMSImageryTest.java (modified) (11 diffs)
• tools/bugreport/BugReportSenderTest.java (modified) (4 diffs)

trunk/test/unit/org/openstreetmap/josm/actions/AddImageryLayerActionTest.java

@@ -10 +10 @@
 import java.util.List;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
 import org.junit.jupiter.api.Test;
 import org.openstreetmap.gui.jmapviewer.FeatureAdapter;
@@ -21 +22 @@
 import org.openstreetmap.josm.testutils.annotations.Projection;
 
-import com.github.tomakehurst.wiremock.WireMockServer;
-
 /**
  * Unit tests for class {@link AddImageryLayerAction}.
@@ -31 +30 @@
 @Projection
 final class AddImageryLayerActionTest {
-    /**
-     * HTTP mock.
-     */
-    @BasicWiremock
-    WireMockServer wireMockServer;
-
     /**
      * Unit test of {@link AddImageryLayerAction#updateEnabledState}.
@@ -65 +58 @@
      */
     @Test
-    void testActionPerformedEnabledWms() {
-        wireMockServer.stubFor(get(urlEqualTo("/wms?apikey=random_key&SERVICE=WMS&REQUEST=GetCapabilities&VERSION=1.1.1"))
+    void testActionPerformedEnabledWms(WireMockRuntimeInfo wireMockRuntimeInfo) {
+        wireMockRuntimeInfo.getWireMock().register(get(urlEqualTo("/wms?apikey=random_key&SERVICE=WMS&REQUEST=GetCapabilities&VERSION=1.1.1"))
                 .willReturn(aResponse()
                         .withStatus(200)
                         .withHeader("Content-Type", "text/xml")
                         .withBodyFile("imagery/wms-capabilities.xml")));
-        wireMockServer.stubFor(get(urlEqualTo("/wms?apikey=random_key&SERVICE=WMS&REQUEST=GetCapabilities"))
+        wireMockRuntimeInfo.getWireMock().register(get(urlEqualTo("/wms?apikey=random_key&SERVICE=WMS&REQUEST=GetCapabilities"))
                 .willReturn(aResponse()
                         .withStatus(404)));
-        wireMockServer.stubFor(get(urlEqualTo("/wms?apikey=random_key&SERVICE=WMS&REQUEST=GetCapabilities&VERSION=1.3.0"))
+        wireMockRuntimeInfo.getWireMock().register(get(urlEqualTo("/wms?apikey=random_key&SERVICE=WMS&REQUEST=GetCapabilities&VERSION=1.3.0"))
                 .willReturn(aResponse()
                         .withStatus(404)));
@@ -80 +73 @@
         try {
             FeatureAdapter.registerApiKeyAdapter(id -> "random_key");
-            final ImageryInfo imageryInfo = new ImageryInfo("localhost", wireMockServer.url("/wms?apikey={apikey}"),
+            final ImageryInfo imageryInfo = new ImageryInfo("localhost", wireMockRuntimeInfo.getHttpBaseUrl() + "/wms?apikey={apikey}",
                     "wms_endpoint", null, null);
             imageryInfo.setId("testActionPerformedEnabledWms");

trunk/test/unit/org/openstreetmap/josm/actions/downloadtasks/AbstractDownloadTaskTestParent.java

@@ -6 +6 @@
 import static com.github.tomakehurst.wiremock.client.WireMock.urlEqualTo;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
+import org.junit.jupiter.api.BeforeEach;
 import org.openstreetmap.josm.testutils.annotations.BasicWiremock;
 import org.openstreetmap.josm.testutils.annotations.HTTPS;
-
-import com.github.tomakehurst.wiremock.WireMockServer;
 
 /**
@@ -17 +17 @@
 @HTTPS
 public abstract class AbstractDownloadTaskTestParent {
-    /**
-     * HTTP mock.
-     */
-    @BasicWiremock
-    WireMockServer wireMockServer;
+    WireMockRuntimeInfo wireMockServer;
+    @BeforeEach
+    void setup(WireMockRuntimeInfo wireMockRuntimeInfo) {
+        this.wireMockServer = wireMockRuntimeInfo;
+    }
 
     /**
@@ -43 +43 @@
      */
     protected final String getRemoteFileUrl() {
-        return wireMockServer.url(getRemoteFile());
+        final String remote = getRemoteFile();
+        if (remote.startsWith("/")) {
+            return wireMockServer.getHttpBaseUrl() + getRemoteFile();
+        }
+        return wireMockServer.getHttpBaseUrl() + '/' + getRemoteFile();
     }
 
@@ -50 +54 @@
      */
     protected final void mockHttp() {
-        wireMockServer.stubFor(get(urlEqualTo("/" + getRemoteFile()))
+        wireMockServer.getWireMock().register(get(urlEqualTo("/" + getRemoteFile()))
                 .willReturn(aResponse()
                     .withStatus(200)

trunk/test/unit/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJobTest.java

@@ -21 +21 @@
 import java.util.concurrent.TimeUnit;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
 import org.apache.commons.jcs3.access.behavior.ICacheAccess;
 import org.apache.commons.jcs3.engine.behavior.ICacheElement;
@@ -33 +34 @@
 import org.openstreetmap.josm.tools.Logging;
 
-import com.github.tomakehurst.wiremock.WireMockServer;
 import com.github.tomakehurst.wiremock.matching.UrlPattern;
 
@@ -47 +47 @@
      * mocked tile server
      */
-    @BasicWiremock
-    WireMockServer tileServer;
+    WireMockRuntimeInfo tileServer;
 
     private static class TestCachedTileLoaderJob extends JCSCachedTileLoaderJob<String, CacheEntry> {
@@ -105 +104 @@
     /**
      * Always clear cache before tests
-     * @throws Exception when clearing fails
      */
     @BeforeEach
-    void clearCache() throws Exception {
+    void clearCache() {
         getCache().clear();
     }
 
+    @BeforeEach
+    void setup(WireMockRuntimeInfo wireMockRuntimeInfo) {
+        this.tileServer = wireMockRuntimeInfo;
+    }
+
     /**
      * Test status codes
-     * @throws InterruptedException in case of thread interruption
      * @throws IOException in case of I/O error
      */
     @Test
-    void testStatusCodes() throws IOException, InterruptedException {
+    void testStatusCodes() throws IOException {
         doTestStatusCode(200);
         doTestStatusCode(401);
@@ -155 +157 @@
 
     private void doTestStatusCode(int responseCode) throws IOException {
-        tileServer.stubFor(get(urlEqualTo("/httpstat/" + responseCode)).willReturn(aResponse().withStatus(responseCode)));
+        tileServer.getWireMock().register(get(urlEqualTo("/httpstat/" + responseCode)).willReturn(aResponse().withStatus(responseCode)));
         TestCachedTileLoaderJob job = getStatusLoaderJob(responseCode);
         Listener listener = submitJob(job);
@@ -196 +198 @@
         createHeadGetStub(urlEqualTo("/test"), expires, testStart, "eTag", "mock entry");
 
-        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.url("/test"), "test");
-        Listener listener = submitJob(job, false);
-        tileServer.verify(0, getRequestedFor(anyUrl()));
+        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.getHttpBaseUrl() + "/test", "test");
+        Listener listener = submitJob(job, false);
+        tileServer.getWireMock().verifyThat(0, getRequestedFor(anyUrl()));
         assertArrayEquals("cached entry".getBytes(StandardCharsets.UTF_8), listener.data);
     }
@@ -217 +219 @@
         createHeadGetStub(urlEqualTo("/test"), expires, testStart, "eTag", "mock entry");
 
-        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.url("/test"), "test");
+        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.getHttpBaseUrl() + "/test", "test");
         Listener listener = submitJob(job, true);
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), listener.data);
     }
@@ -231 +233 @@
     void testSettingMinimumExpiryWhenNoExpires() throws IOException {
         long testStart = System.currentTimeMillis();
-        tileServer.stubFor(get(urlEqualTo("/test")).willReturn(aResponse().withBody("mock entry")));
-
-        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.url("/test"), "test");
-        Listener listener = submitJob(job, false);
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
+        tileServer.getWireMock().register(get(urlEqualTo("/test")).willReturn(aResponse().withBody("mock entry")));
+
+        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.getHttpBaseUrl() + "/test", "test");
+        Listener listener = submitJob(job, false);
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
 
         assertTrue(listener.attributes.getExpirationTime() >= testStart + JCSCachedTileLoaderJob.DEFAULT_EXPIRE_TIME,
@@ -260 +262 @@
         long testStart = System.currentTimeMillis();
         long expires = TimeUnit.DAYS.toSeconds(1);
-        tileServer.stubFor(get(urlEqualTo("/test"))
+        tileServer.getWireMock().register(get(urlEqualTo("/test"))
                 .willReturn(aResponse()
                         .withHeader("Cache-control", "max-age=" + expires)
@@ -267 +269 @@
                 );
 
-        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.url("/test"), "test");
-        Listener listener = submitJob(job, false);
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
+        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.getHttpBaseUrl() + "/test", "test");
+        Listener listener = submitJob(job, false);
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
 
         assertTrue(listener.attributes.getExpirationTime() >= testStart + TimeUnit.SECONDS.toMillis(expires),
@@ -298 +300 @@
         createHeadGetStub(urlEqualTo("/test"), (JCSCachedTileLoaderJob.DEFAULT_EXPIRE_TIME / 10), testStart, "eTag", "mock entry");
 
-        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.url("/test"), "test", minimumExpiryTimeSeconds);
-        Listener listener = submitJob(job, false);
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
+        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.getHttpBaseUrl() + "/test", "test", minimumExpiryTimeSeconds);
+        Listener listener = submitJob(job, false);
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), listener.data);
 
@@ -331 +333 @@
         createHeadGetStub(urlEqualTo("/test"), (JCSCachedTileLoaderJob.DEFAULT_EXPIRE_TIME / 10), testStart, "eTag", "mock entry");
 
-        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.url("/test"), "test", minimumExpiryTimeSeconds);
-        Listener listener = submitJob(job, false);
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
+        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.getHttpBaseUrl() + "/test", "test", minimumExpiryTimeSeconds);
+        Listener listener = submitJob(job, false);
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), listener.data);
 
@@ -367 +369 @@
         int minimumExpiryTimeSeconds = 0;
 
-        tileServer.stubFor(get(urlEqualTo("/test"))
+        tileServer.getWireMock().register(get(urlEqualTo("/test"))
                 .willReturn(aResponse()
                         .withHeader("Expires", TestUtils.getHTTPDate(testStart + (JCSCachedTileLoaderJob.DEFAULT_EXPIRE_TIME / 10)))
@@ -375 +377 @@
                         )
                 );
-        tileServer.stubFor(head(urlEqualTo("/test"))
+        tileServer.getWireMock().register(head(urlEqualTo("/test"))
                 .willReturn(aResponse()
                         .withHeader("Expires", TestUtils.getHTTPDate(testStart + (JCSCachedTileLoaderJob.DEFAULT_EXPIRE_TIME / 10)))
@@ -382 +384 @@
                         )
                 );
-        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.url("/test"), "test", minimumExpiryTimeSeconds);
-        Listener listener = submitJob(job, false);
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
+        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.getHttpBaseUrl() + "/test", "test", minimumExpiryTimeSeconds);
+        Listener listener = submitJob(job, false);
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), listener.data);
 
@@ -413 +415 @@
         int minimumExpiryTimeSeconds = 0;
 
-        tileServer.stubFor(get(urlEqualTo("/test"))
+        tileServer.getWireMock().register(get(urlEqualTo("/test"))
                 .willReturn(aResponse()
                         .withHeader("Cache-Control", "" +
@@ -422 +424 @@
                         )
                 );
-        tileServer.stubFor(head(urlEqualTo("/test"))
+        tileServer.getWireMock().register(head(urlEqualTo("/test"))
                 .willReturn(aResponse()
                         .withHeader("Cache-Control", "" +
@@ -429 +431 @@
                         )
                 ));
-        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.url("/test"), "test", minimumExpiryTimeSeconds);
-        Listener listener = submitJob(job, false);
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
+        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.getHttpBaseUrl() + "/test", "test", minimumExpiryTimeSeconds);
+        Listener listener = submitJob(job, false);
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), listener.data);
 
@@ -461 +463 @@
                 );
 
-        tileServer.stubFor(get(urlEqualTo("/test"))
+        tileServer.getWireMock().register(get(urlEqualTo("/test"))
                 .willReturn(aResponse()
                         .withHeader("Expires", TestUtils.getHTTPDate(testStart + expires))
@@ -469 +471 @@
                         )
                 );
-        tileServer.stubFor(head(urlEqualTo("/test"))
+        tileServer.getWireMock().register(head(urlEqualTo("/test"))
                 .willReturn(aResponse()
                         .withHeader("Expires", TestUtils.getHTTPDate(testStart + expires))
@@ -477 +479 @@
                 );
 
-        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.url("/test"), "test");
+        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.getHttpBaseUrl() + "/test", "test");
         Listener listener = submitJob(job, false); // cache entry is expired, no need to force refetch
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), listener.data);
 
         // cache entry should be retrieved from cache
         listener = submitJob(job, false);
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), listener.data);
 
@@ -494 +496 @@
 
         // because cache entry is invalid - HEAD request shall be made
-        tileServer.verify(0, headRequestedFor(urlEqualTo("/test"))); // no head requests were made until now
+        tileServer.getWireMock().verifyThat(0, headRequestedFor(urlEqualTo("/test"))); // no head requests were made until now
         listener = submitJob(job, false);
-        tileServer.verify(1, headRequestedFor(urlEqualTo("/test"))); // verify head requests were made
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test"))); // verify no more get requests were made
+        tileServer.getWireMock().verifyThat(1, headRequestedFor(urlEqualTo("/test"))); // verify head requests were made
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test"))); // verify no more get requests were made
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), listener.data);
         assertTrue(listener.attributes.getExpirationTime() >= testStart + expires);
@@ -503 +505 @@
         // cache entry should be retrieved from cache
         listener = submitJob(job, false); // cache entry is expired, no need to force refetch
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), listener.data);
     }
@@ -522 +524 @@
                 );
 
-        tileServer.stubFor(get(urlEqualTo("/test"))
+        tileServer.getWireMock().register(get(urlEqualTo("/test"))
                 .willReturn(status(304)
                         .withHeader("Expires", TestUtils.getHTTPDate(testStart + expires))
@@ -530 +532 @@
                 );
 
-        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.url("/test"), "test");
-        Listener listener = submitJob(job, false);
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test")));
+        TestCachedTileLoaderJob job = new TestCachedTileLoaderJob(tileServer.getHttpBaseUrl() + "/test", "test");
+        Listener listener = submitJob(job, false);
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test")));
         assertArrayEquals("cached dummy".getBytes(StandardCharsets.UTF_8), listener.data);
         assertTrue(testStart + expires <= listener.attributes.getExpirationTime());
         submitJob(job, false);
-        tileServer.verify(1, getRequestedFor(urlEqualTo("/test"))); // no more requests were made
+        tileServer.getWireMock().verifyThat(1, getRequestedFor(urlEqualTo("/test"))); // no more requests were made
     }
 
     private void createHeadGetStub(UrlPattern url, long expires, long lastModified, String eTag, String body) {
-        tileServer.stubFor(get(url)
+        tileServer.getWireMock().register(get(url)
                 .willReturn(aResponse()
                         .withHeader("Expires", TestUtils.getHTTPDate(lastModified + expires))
@@ -548 +550 @@
                         )
                 );
-        tileServer.stubFor(head(url)
+        tileServer.getWireMock().register(head(url)
                 .willReturn(aResponse()
                         .withHeader("Expires", TestUtils.getHTTPDate(lastModified + expires))
@@ -567 +569 @@
 
     private TestCachedTileLoaderJob getStatusLoaderJob(int responseCode) {
-        return new TestCachedTileLoaderJob(tileServer.url("/httpstat/" + responseCode), "key_" + responseCode);
+        return new TestCachedTileLoaderJob(tileServer.getHttpBaseUrl() + "/httpstat/" + responseCode, "key_" + responseCode);
     }
 

trunk/test/unit/org/openstreetmap/josm/data/imagery/TMSCachedTileLoaderJobTest.java

@@ -16 +16 @@
 import java.util.regex.Pattern;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
 import org.apache.commons.jcs3.access.behavior.ICacheAccess;
 import org.junit.jupiter.api.BeforeEach;
@@ -31 +32 @@
 import org.openstreetmap.josm.tools.Utils;
 
-import com.github.tomakehurst.wiremock.WireMockServer;
 import com.github.tomakehurst.wiremock.client.WireMock;
 
@@ -43 +43 @@
      * mocked tile server
      */
-    @BasicWiremock
-    WireMockServer tileServer;
+    private WireMockRuntimeInfo wireMockRuntimeInfo;
 
     @BeforeEach
-    void clearCache() throws Exception {
+    void clearCache() {
         getCache().clear();
+    }
+
+    @BeforeEach
+    void setup(WireMockRuntimeInfo wireMockRuntimeInfo) {
+        this.wireMockRuntimeInfo = wireMockRuntimeInfo;
     }
 
@@ -126 +130 @@
 
         @Override
-        public String getTileUrl(int zoom, int tilex, int tiley) throws IOException {
+        public String getTileUrl(int zoom, int tilex, int tiley) {
             return url;
         }
@@ -219 +223 @@
     void testNoCacheHeaders() throws IOException {
         long testStart = System.currentTimeMillis();
-        tileServer.stubFor(
+        wireMockRuntimeInfo.getWireMock().register(
                 WireMock.get(WireMock.urlEqualTo("/test"))
                 .willReturn(WireMock.aResponse()
@@ -226 +230 @@
                 );
 
-        TestCachedTileLoaderJob job = submitJob(new MockTile(tileServer.url("/test")), "test", false);
+        TestCachedTileLoaderJob job = submitJob(new MockTile(wireMockRuntimeInfo.getHttpBaseUrl() + "/test"), "test", false);
         assertExpirationAtLeast(testStart + TMSCachedTileLoaderJob.MINIMUM_EXPIRES.get(), job);
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), job.get().getContent());
-        job = submitJob(new MockTile(tileServer.url("/test")), "test", false); // submit another job for the same tile
+        job = submitJob(new MockTile(wireMockRuntimeInfo.getHttpBaseUrl() + "/test"), "test", false); // submit another job for the same tile
         // only one request to tile server should be made, second should come from cache
-        tileServer.verify(1, WireMock.getRequestedFor(WireMock.urlEqualTo("/test")));
+        wireMockRuntimeInfo.getWireMock().verifyThat(1, WireMock.getRequestedFor(WireMock.urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), job.get().getContent());
     }
@@ -257 +261 @@
     private void noCacheHeadersMinimumExpires(int minimumExpires) throws IOException {
         long testStart = System.currentTimeMillis();
-        tileServer.stubFor(
+        wireMockRuntimeInfo.getWireMock().register(
                 WireMock.get(WireMock.urlEqualTo("/test"))
                 .willReturn(WireMock.aResponse()
@@ -263 +267 @@
                         )
                 );
-        TestCachedTileLoaderJob job = submitJob(new MockTile(tileServer.url("/test")), "test", minimumExpires, false);
+        TestCachedTileLoaderJob job = submitJob(new MockTile(wireMockRuntimeInfo.getHttpBaseUrl() + "/test"), "test", minimumExpires, false);
         assertExpirationAtLeast(testStart + minimumExpires, job);
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), job.get().getContent());
-        job = submitJob(new MockTile(tileServer.url("/test")), "test", false); // submit another job for the same tile
+        job = submitJob(new MockTile(wireMockRuntimeInfo.getHttpBaseUrl() + "/test"), "test", false); // submit another job for the same tile
         // only one request to tile server should be made, second should come from cache
-        tileServer.verify(1, WireMock.getRequestedFor(WireMock.urlEqualTo("/test")));
+        wireMockRuntimeInfo.getWireMock().verifyThat(1, WireMock.getRequestedFor(WireMock.urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), job.get().getContent());
     }
@@ -280 +284 @@
         long testStart = System.currentTimeMillis();
         long expires = TMSCachedTileLoaderJob.MINIMUM_EXPIRES.get() / 2;
-        tileServer.stubFor(
+        wireMockRuntimeInfo.getWireMock().register(
                 WireMock.get(WireMock.urlEqualTo("/test"))
                 .willReturn(WireMock.aResponse()
@@ -287 +291 @@
                         )
                 );
-        TestCachedTileLoaderJob job = submitJob(new MockTile(tileServer.url("/test")), "test", false);
+        TestCachedTileLoaderJob job = submitJob(new MockTile(wireMockRuntimeInfo.getHttpBaseUrl() + "/test"), "test", false);
         assertExpirationAtLeast(testStart + TMSCachedTileLoaderJob.MINIMUM_EXPIRES.get(), job);
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), job.get().getContent());
-        job = submitJob(new MockTile(tileServer.url("/test")), "test", false); // submit another job for the same tile
+        job = submitJob(new MockTile(wireMockRuntimeInfo.getHttpBaseUrl() + "/test"), "test", false); // submit another job for the same tile
         // only one request to tile server should be made, second should come from cache
-        tileServer.verify(1, WireMock.getRequestedFor(WireMock.urlEqualTo("/test")));
+        wireMockRuntimeInfo.getWireMock().verifyThat(1, WireMock.getRequestedFor(WireMock.urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), job.get().getContent());
     }
@@ -312 +316 @@
         long testStart = System.currentTimeMillis();
         long expires = TMSCachedTileLoaderJob.MAXIMUM_EXPIRES.get() * 2;
-        tileServer.stubFor(
+        wireMockRuntimeInfo.getWireMock().register(
                 WireMock.get(WireMock.urlEqualTo("/test"))
                 .willReturn(WireMock.aResponse()
@@ -319 +323 @@
                         )
                 );
-        TestCachedTileLoaderJob job = submitJob(new MockTile(tileServer.url("/test")), "test", false);
+        TestCachedTileLoaderJob job = submitJob(new MockTile(wireMockRuntimeInfo.getHttpBaseUrl() + "/test"), "test", false);
         // give 1 second margin
         assertExpirationAtMost(testStart + TMSCachedTileLoaderJob.MAXIMUM_EXPIRES.get() + TimeUnit.SECONDS.toMillis(1), job);
 
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), job.get().getContent());
-        job = submitJob(new MockTile(tileServer.url("/test")), "test", false); // submit another job for the same tile
+        job = submitJob(new MockTile(wireMockRuntimeInfo.getHttpBaseUrl() + "/test"), "test", false); // submit another job for the same tile
         // only one request to tile server should be made, second should come from cache
-        tileServer.verify(1, WireMock.getRequestedFor(WireMock.urlEqualTo("/test")));
+        wireMockRuntimeInfo.getWireMock().verifyThat(1, WireMock.getRequestedFor(WireMock.urlEqualTo("/test")));
         assertArrayEquals("mock entry".getBytes(StandardCharsets.UTF_8), job.get().getContent());
     }

trunk/test/unit/org/openstreetmap/josm/data/imagery/WMSEndpointTileSourceTest.java

@@ -11 +11 @@
 import java.util.Collections;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
+import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.openstreetmap.gui.jmapviewer.interfaces.TemplatedTileSource;
@@ -22 +24 @@
 import org.openstreetmap.josm.testutils.annotations.Projection;
 
-import com.github.tomakehurst.wiremock.WireMockServer;
 import com.github.tomakehurst.wiremock.client.WireMock;
 
@@ -29 +30 @@
 @Projection
 class WMSEndpointTileSourceTest implements TileSourceTest {
-    @BasicWiremock
-    WireMockServer tileServer;
+    WireMockRuntimeInfo tileServer;
+    @BeforeEach
+    void setup(WireMockRuntimeInfo wireMockRuntimeInfo) {
+        this.tileServer = wireMockRuntimeInfo;
+    }
 
     private void basicMock() {
         final byte[] response = assertDoesNotThrow(() -> Files.readAllBytes(
                 Paths.get(TestUtils.getTestDataRoot() + "wms/geofabrik-osm-inspector.xml")));
-        tileServer.stubFor(
+        tileServer.getWireMock().register(
                 WireMock.get(WireMock.urlEqualTo("/capabilities?SERVICE=WMS&REQUEST=GetCapabilities"))
                         .willReturn(WireMock.aResponse().withBody(response))
@@ -45 +49 @@
         this.basicMock();
         final ImageryInfo info = new ImageryInfo("WMSEndpointTileSourceTest");
-        info.setExtendedUrl(tileServer.url("/capabilities"));
+        info.setExtendedUrl(tileServer.getHttpBaseUrl() + "/capabilities");
         info.setDefaultLayers(Collections.singletonList(new DefaultLayer(ImageryType.WMS_ENDPOINT,
                 "single_node_in_way", "default", null)));
@@ -60 +64 @@
     void testDefaultLayerSetInMaps() throws Exception {
 
-        tileServer.stubFor(
+        tileServer.getWireMock().register(
                 WireMock.get(WireMock.urlEqualTo("/capabilities?SERVICE=WMS&REQUEST=GetCapabilities"))
                 .willReturn(
@@ -68 +72 @@
                 );
 
-        tileServer.stubFor(WireMock.get(WireMock.urlEqualTo("/other/maps")).willReturn(WireMock.aResponse().withBody(
+        tileServer.getWireMock().register(WireMock.get(WireMock.urlEqualTo("/other/maps")).willReturn(WireMock.aResponse().withBody(
                 "<?xml version='1.0' encoding='UTF-8'?>\n" +
                 "<imagery xmlns=\"http://josm.openstreetmap.de/maps-1.0\">\n" +
@@ -76 +80 @@
                 "<type>wms_endpoint</type>\n" +
                 "<category>qa</category>\n" +
-                "<url><![CDATA[" + tileServer.url("/capabilities") + "]]></url>\n" +
+                "<url><![CDATA[" + tileServer.getHttpBaseUrl() + "/capabilities]]></url>\n" +
                 "<icon>data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAACXBIWXMAAAsSAAALEgHS3X78AAAB5UlEQVQ4y4WTwWsTURDG" +
                 "fy8W1yYmXZOqtGJJFyGw6KF7CEigwYuS0kthrYUi4i0iORS9BU9hQdA/ILcixVBrwENKLz1FUBB0wWOwYFAqxUNYTZq6BfM8yC5d05iBObz3vfnmm3kz4sqDh/zP" +
@@ -95 +99 @@
                 )));
 
-        Config.getPref().putList("imagery.layers.sites", Collections.singletonList(tileServer.url("/other/maps")));
+        Config.getPref().putList("imagery.layers.sites", Collections.singletonList(tileServer.getHttpBaseUrl() + "/other/maps"));
         ImageryLayerInfo.instance.loadDefaults(true, null, false);
         assertEquals(1, ImageryLayerInfo.instance.getDefaultLayers().size());
@@ -110 +114 @@
     @Test
     void testCustomHeadersServerSide() throws IOException {
-        tileServer.stubFor(
+        tileServer.getWireMock().register(
                 WireMock.get(WireMock.urlEqualTo("/capabilities?SERVICE=WMS&REQUEST=GetCapabilities"))
                 .willReturn(
@@ -118 +122 @@
                 );
 
-        tileServer.stubFor(WireMock.get(WireMock.urlEqualTo("/other/maps")).willReturn(WireMock.aResponse().withBody(
+        tileServer.getWireMock().register(WireMock.get(WireMock.urlEqualTo("/other/maps")).willReturn(WireMock.aResponse().withBody(
                 "<?xml version='1.0' encoding='UTF-8'?>\n" +
                 "<imagery xmlns=\"http://josm.openstreetmap.de/maps-1.0\">\n" +
@@ -129 +133 @@
                 "        <country-code>NO</country-code>\n" +
                 "        <description lang=\"en\">Historic Norwegian orthophotos and maps, courtesy of Geovekst and Norkart.</description>\n" +
-                "        <url><![CDATA[" + tileServer.url("/capabilities?SERVICE=WMS&REQUEST=GetCapabilities") + "]]></url>\n" +
+                "        <url><![CDATA[" + tileServer.getHttpBaseUrl() + "/capabilities?SERVICE=WMS&REQUEST=GetCapabilities]]></url>\n" +
                 "        <custom-http-header header-name=\"X-WAAPI-TOKEN\" header-value=\"b8e36d51-119a-423b-b156-d744d54123d5\" />\n" +
                 "        <attribution-text>© Geovekst</attribution-text>\n" +
@@ -141 +145 @@
                 )));
 
-        Config.getPref().putList("imagery.layers.sites", Collections.singletonList(tileServer.url("/other/maps")));
+        Config.getPref().putList("imagery.layers.sites", Collections.singletonList(tileServer.getHttpBaseUrl() + "/other/maps"));
         ImageryLayerInfo.instance.loadDefaults(true, null, false);
         ImageryInfo wmsImageryInfo = ImageryLayerInfo.instance.getDefaultLayers().get(0);
@@ -150 +154 @@
         assertEquals("b8e36d51-119a-423b-b156-d744d54123d5", tileSource.getHeaders().get("X-WAAPI-TOKEN"));
         assertTrue(wmsImageryInfo.isGeoreferenceValid());
-        tileServer.verify(
+        tileServer.getWireMock().verifyThat(
                 WireMock.getRequestedFor(WireMock.urlEqualTo("/capabilities?SERVICE=WMS&REQUEST=GetCapabilities"))
                 .withHeader("X-WAAPI-TOKEN", WireMock.equalTo("b8e36d51-119a-423b-b156-d744d54123d5")));

trunk/test/unit/org/openstreetmap/josm/data/imagery/WMTSTileSourceTest.java

@@ -23 +23 @@
 import java.util.concurrent.TimeUnit;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
 import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
@@ -45 +46 @@
 import org.openstreetmap.josm.tools.ReflectionUtils;
 
-import com.github.tomakehurst.wiremock.WireMockServer;
 import com.github.tomakehurst.wiremock.client.WireMock;
 
@@ -56 +56 @@
 @Timeout(value = 5, unit = TimeUnit.MINUTES)
 class WMTSTileSourceTest {
-    @BasicWiremock
-    WireMockServer tileServer;
-
     private final ImageryInfo testImageryTMS = new ImageryInfo("test imagery", "http://localhost", "tms", null, null);
     private final ImageryInfo testImageryPSEUDO_MERCATOR = getImagery(TestUtils.getTestDataRoot() + "wmts/getcapabilities-pseudo-mercator.xml");
@@ -367 +364 @@
 
     @Test
-    void testDefaultLayer() throws Exception {
+    void testDefaultLayer(WireMockRuntimeInfo wireMockRuntimeInfo) throws Exception {
         // https://gibs.earthdata.nasa.gov/wmts/epsg3857/best/1.0.0/WMTSCapabilities.xml
         // do not use withFileBody as it needs different directory layout :(
 
-        tileServer.stubFor(
+        wireMockRuntimeInfo.getWireMock().register(
                 WireMock.get("/getcapabilities.xml")
                 .willReturn(
@@ -381 +378 @@
                 );
 
-        tileServer.stubFor(
+        wireMockRuntimeInfo.getWireMock().register(
                 WireMock.get("/other/maps")
                 .willReturn(
@@ -392 +389 @@
                 "<type>wmts</type>\n" +
                 "<category>photo</category>\n" +
-                "<url><![CDATA[" + tileServer.url("/getcapabilities.xml") + "]]></url>\n" +
+                "<url><![CDATA[" + wireMockRuntimeInfo.getHttpBaseUrl() + "/getcapabilities.xml]]></url>\n" +
                 "<default-layers>" +
                 "<layer name=\"GEOGRAPHICALGRIDSYSTEMS.MAPS\" />" +
@@ -400 +397 @@
                 )));
 
-        Config.getPref().putList("imagery.layers.sites", Collections.singletonList(tileServer.url("/other/maps")));
+        Config.getPref().putList("imagery.layers.sites", Collections.singletonList(wireMockRuntimeInfo.getHttpBaseUrl() + "/other/maps"));
         ImageryLayerInfo.instance.loadDefaults(true, null, false);
 

trunk/test/unit/org/openstreetmap/josm/data/osm/DefaultNameFormatterTest.java

@@ -15 +15 @@
 import java.util.stream.IntStream;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
 import org.junit.jupiter.api.Test;
 import org.openstreetmap.josm.TestUtils;
@@ -25 +26 @@
 import org.openstreetmap.josm.testutils.annotations.HTTP;
 import org.xml.sax.SAXException;
-
-import com.github.tomakehurst.wiremock.WireMockServer;
 
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
@@ -39 +38 @@
 class DefaultNameFormatterTest {
     /**
-     * HTTP mock.
-     */
-    @BasicWiremock
-    WireMockServer wireMockServer;
-
-    /**
      * Non-regression test for ticket <a href="https://josm.openstreetmap.de/ticket/9632">#9632</a>.
+     * @param wireMockRuntimeInfo Wiremock information
      * @throws IllegalDataException if an error was found while parsing the data from the source
      * @throws IOException if any I/O error occurs
@@ -52 +46 @@
     @Test
     @SuppressFBWarnings(value = "ITA_INEFFICIENT_TO_ARRAY")
-    void testTicket9632() throws IllegalDataException, IOException, SAXException {
+    void testTicket9632(WireMockRuntimeInfo wireMockRuntimeInfo) throws IllegalDataException, IOException, SAXException {
         String source = "presets/Presets_BicycleJunction-preset.xml";
-        wireMockServer.stubFor(get(urlEqualTo("/" + source))
+        wireMockRuntimeInfo.getWireMock().register(get(urlEqualTo("/" + source))
                 .willReturn(aResponse()
                     .withStatus(200)
                     .withHeader("Content-Type", "text/xml")
                     .withBodyFile(source)));
-        TaggingPresets.addTaggingPresets(TaggingPresetReader.readAll(wireMockServer.url(source), true));
+        TaggingPresets.addTaggingPresets(TaggingPresetReader.readAll(wireMockRuntimeInfo.getHttpBaseUrl() + '/' + source, true));
 
         Comparator<IRelation<?>> comparator = DefaultNameFormatter.getInstance().getRelationComparator();

trunk/test/unit/org/openstreetmap/josm/gui/preferences/server/ApiUrlTestTaskTest.java

@@ -14 +14 @@
 import java.awt.Component;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
+import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.Timeout;
@@ -20 +22 @@
 import org.openstreetmap.josm.testutils.annotations.HTTP;
 import org.openstreetmap.josm.tools.Logging;
-
-import com.github.tomakehurst.wiremock.WireMockServer;
 
 /**
@@ -31 +31 @@
 @HTTP
 class ApiUrlTestTaskTest {
-    /**
-     * HTTP mock.
-     */
-    @BasicWiremock
-    WireMockServer wireMockServer;
+    private WireMockRuntimeInfo wireMockServer;
+
 
     private static final Component PARENT = new JLabel();
+
+    @BeforeEach
+    void setup(WireMockRuntimeInfo wireMockServer) {
+        this.wireMockServer = wireMockServer;
+    }
 
     /**
@@ -52 +54 @@
     @Test
     void testNominalUrl() {
-        ApiUrlTestTask task = new ApiUrlTestTask(PARENT, wireMockServer.url("/__files/api"));
+        ApiUrlTestTask task = new ApiUrlTestTask(PARENT, wireMockServer.getHttpBaseUrl() + "/__files/api");
         task.run();
         assertTrue(task.isSuccess());
@@ -89 +91 @@
     void testAlertInvalidServerResult() {
         Logging.clearLastErrorAndWarnings();
-        wireMockServer.stubFor(get(urlEqualTo("/does-not-exist/0.6/capabilities"))
+        wireMockServer.getWireMock().register(get(urlEqualTo("/does-not-exist/0.6/capabilities"))
                 .willReturn(aResponse().withStatus(404)));
 
-        ApiUrlTestTask task = new ApiUrlTestTask(PARENT, wireMockServer.url("/does-not-exist"));
+        ApiUrlTestTask task = new ApiUrlTestTask(PARENT, wireMockServer.getHttpBaseUrl() + "/does-not-exist");
         task.run();
         assertFalse(task.isSuccess());
@@ -105 +107 @@
     void testAlertInvalidCapabilities() {
         Logging.clearLastErrorAndWarnings();
-        ApiUrlTestTask task = new ApiUrlTestTask(PARENT, wireMockServer.url("/__files/invalid_api"));
+        final String url = wireMockServer.getHttpBaseUrl() + "/__files/invalid_api";
+        ApiUrlTestTask task = new ApiUrlTestTask(PARENT, url);
         task.run();
         assertFalse(task.isSuccess());
         assertThat(Logging.getLastErrorAndWarnings().toString(), containsString(
                 "The OSM API server at 'XXX' did not return a valid response.<br>It is likely that 'XXX' is not an OSM API server."
-                        .replace("XXX", wireMockServer.url("/__files/invalid_api"))));
+                        .replace("XXX", url)));
     }
 }

trunk/test/unit/org/openstreetmap/josm/io/OsmServerHistoryReaderTest.java

@@ -6 +6 @@
 import java.time.Instant;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
@@ -17 +18 @@
 import org.openstreetmap.josm.testutils.annotations.HTTP;
 
-import com.github.tomakehurst.wiremock.WireMockServer;
-
 /**
  * Unit tests of {@link OsmServerHistoryReader} class.
@@ -27 +26 @@
 class OsmServerHistoryReaderTest {
     /**
-     * HTTP mock.
-     */
-    @BasicWiremock
-    WireMockServer wireMockServer;
-
-    /**
      * Setup tests.
      */
     @BeforeEach
-    void setUp() {
-        Config.getPref().put("osm-server.url", wireMockServer.url("/__files/api"));
+    void setUp(WireMockRuntimeInfo wireMockRuntimeInfo) {
+        Config.getPref().put("osm-server.url", wireMockRuntimeInfo.getHttpBaseUrl() + "/__files/api");
     }
 

trunk/test/unit/org/openstreetmap/josm/io/OverpassDownloadReaderTest.java

@@ -14 +14 @@
 import java.util.regex.Matcher;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
@@ -25 +26 @@
 import org.openstreetmap.josm.tools.date.DateUtils;
 
-import com.github.tomakehurst.wiremock.WireMockServer;
-
 /**
  * Unit tests of {@link OverpassDownloadReader} class.
@@ -34 +33 @@
 @HTTP
 class OverpassDownloadReaderTest {
-    /**
-     * HTTP mock.
-     */
-    @BasicWiremock
-    WireMockServer wireMockServer;
+    private WireMockRuntimeInfo wireMockServer;
 
     private static final String NOMINATIM_URL_PATH = "/search?format=xml&q=";
@@ -46 +41 @@
      */
     @BeforeEach
-    public void setUp() {
-        NameFinder.NOMINATIM_URL_PROP.put(wireMockServer.url(NOMINATIM_URL_PATH));
+    public void setUp(WireMockRuntimeInfo wireMockRuntimeInfo) {
+        this.wireMockServer = wireMockRuntimeInfo;
+        NameFinder.NOMINATIM_URL_PROP.put(wireMockServer.getHttpBaseUrl() + NOMINATIM_URL_PATH);
     }
 
@@ -74 +70 @@
 
     private void stubNominatim(String query) {
-        wireMockServer.stubFor(get(urlEqualTo(NOMINATIM_URL_PATH + query))
+        wireMockServer.getWireMock().register(get(urlEqualTo(NOMINATIM_URL_PATH + query))
                 .willReturn(aResponse()
                     .withStatus(200)

trunk/test/unit/org/openstreetmap/josm/io/imagery/WMSImageryTest.java

@@ -11 +11 @@
 import java.util.List;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
+import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.openstreetmap.josm.TestUtils;
@@ -18 +20 @@
 import org.openstreetmap.josm.testutils.annotations.Projection;
 
-import com.github.tomakehurst.wiremock.WireMockServer;
 import com.github.tomakehurst.wiremock.client.WireMock;
 
@@ -28 +29 @@
 @Projection
 class WMSImageryTest {
-    @BasicWiremock
-    WireMockServer tileServer;
+
+    private WireMockRuntimeInfo tileServer;
+
+    @BeforeEach
+    void setup(WireMockRuntimeInfo wireMockRuntimeRule) {
+        this.tileServer = wireMockRuntimeRule;
+    }
 
     /**
@@ -52 +58 @@
     @Test
     void testTicket15730() throws IOException, WMSGetCapabilitiesException {
-        tileServer.stubFor(WireMock.get(WireMock.anyUrl()).willReturn(WireMock.aResponse().withBody(
+        tileServer.getWireMock().register(WireMock.get(WireMock.anyUrl()).willReturn(WireMock.aResponse().withBody(
                 Files.readAllBytes(Paths.get(TestUtils.getRegressionDataDir(15730), "capabilities.xml"))
                 )));
 
-        WMSImagery wms = new WMSImagery(tileServer.url("capabilities.xml"));
+        WMSImagery wms = new WMSImagery(tileServer.getHttpBaseUrl() + "/capabilities.xml");
         assertEquals(1, wms.getLayers().size());
         assertTrue(wms.getLayers().get(0).getAbstract().startsWith("South Carolina  NAIP Imagery 2017    Resolution: 100CM "));
@@ -63 +69 @@
     @Test
     void testNestedLayers() throws Exception {
-        tileServer.stubFor(WireMock.get(WireMock.anyUrl()).willReturn(WireMock.aResponse().withBody(
+        tileServer.getWireMock().register(WireMock.get(WireMock.anyUrl()).willReturn(WireMock.aResponse().withBody(
                 Files.readAllBytes(Paths.get(TestUtils.getTestDataRoot() + "wms/mapa-um-warszawa-pl.xml")))));
-        WMSImagery wmsi = new WMSImagery(tileServer.url("/serwis"));
+        WMSImagery wmsi = new WMSImagery(tileServer.getHttpBaseUrl() + "/serwis");
         assertEquals(1, wmsi.getLayers().size());
         assertEquals("Server WMS m.st. Warszawy", wmsi.getLayers().get(0).toString());
@@ -79 +85 @@
     void testTicket16248() throws IOException, WMSGetCapabilitiesException {
         byte[] capabilities = Files.readAllBytes(Paths.get(TestUtils.getRegressionDataFile(16248, "capabilities.xml")));
-        tileServer.stubFor(WireMock.get(WireMock.anyUrl()).willReturn(WireMock.aResponse().withBody(capabilities)));
-        WMSImagery wms = new WMSImagery(tileServer.url("any"));
+        tileServer.getWireMock().register(WireMock.get(WireMock.anyUrl()).willReturn(WireMock.aResponse().withBody(capabilities)));
+        WMSImagery wms = new WMSImagery(tileServer.getHttpBaseUrl() + "/any");
         assertEquals("http://wms.hgis.cartomatic.pl/topo/3857/m25k?", wms.buildRootUrl());
         assertEquals("wms.hgis.cartomatic.pl", wms.getLayers().get(0).getName());
@@ -96 +102 @@
     void testTicket19193() throws IOException, WMSGetCapabilitiesException {
         byte[] capabilities = Files.readAllBytes(Paths.get(TestUtils.getRegressionDataFile(19193, "capabilities.xml")));
-        tileServer.stubFor(WireMock.get(WireMock.anyUrl()).willReturn(WireMock.aResponse().withBody(capabilities)));
-        WMSImagery wms = new WMSImagery(tileServer.url("any"));
+        tileServer.getWireMock().register(WireMock.get(WireMock.anyUrl()).willReturn(WireMock.aResponse().withBody(capabilities)));
+        WMSImagery wms = new WMSImagery(tileServer.getHttpBaseUrl() + "/any");
         assertEquals("https://inspire.brandenburg.de/services/gn_alkis_wms?", wms.buildRootUrl());
         assertEquals(1, wms.getLayers().size());
@@ -114 +120 @@
     @Test
     void testTicket16333() throws IOException, WMSGetCapabilitiesException {
-        tileServer.stubFor(
+        tileServer.getWireMock().register(
                 WireMock.get(WireMock.anyUrl())
                 .willReturn(WireMock.aResponse().withBody(
@@ -120 +126 @@
                 ))
         );
-        WMSImagery wms = new WMSImagery(tileServer.url("any"));
+        WMSImagery wms = new WMSImagery(tileServer.getHttpBaseUrl() + "/any");
         assertEquals("https://duinoord.xs4all.nl/geoserver/ows?SERVICE=WMS&", wms.buildRootUrl());
         assertNull(wms.getLayers().get(0).getName());
@@ -131 +137 @@
     @Test
     void testForTitleWithinAttribution_ticket16940() throws IOException, WMSGetCapabilitiesException {
-        tileServer.stubFor(
+        tileServer.getWireMock().register(
                 WireMock.get(WireMock.anyUrl())
                 .willReturn(WireMock.aResponse().withBody(
@@ -137 +143 @@
                 ))
         );
-        WMSImagery wms = new WMSImagery(tileServer.url("any"));
+        WMSImagery wms = new WMSImagery(tileServer.getHttpBaseUrl() + "/any");
         assertEquals("Hipsográfico", wms.getLayers().stream().findFirst().get().getTitle());
     }

trunk/test/unit/org/openstreetmap/josm/tools/bugreport/BugReportSenderTest.java

@@ -16 +16 @@
 import java.util.List;
 
+import com.github.tomakehurst.wiremock.junit5.WireMockRuntimeInfo;
 import org.junit.jupiter.api.Test;
 import org.openstreetmap.josm.actions.ShowStatusReportAction;
@@ -23 +24 @@
 import org.openstreetmap.josm.testutils.annotations.HTTP;
 import org.openstreetmap.josm.testutils.mockers.OpenBrowserMocker;
-
-import com.github.tomakehurst.wiremock.WireMockServer;
 
 /**
@@ -34 +33 @@
 class BugReportSenderTest {
     /**
-     * HTTP mock
-     */
-    @BasicWiremock
-    WireMockServer wireMockServer;
-
-    /**
      * Unit test for {@link BugReportSender#BugReportSender}.
      * @throws InterruptedException if the thread is interrupted
      */
     @Test
-    void testBugReportSender() throws InterruptedException {
-        Config.getPref().put("josm.url", wireMockServer.baseUrl());
-        wireMockServer.stubFor(post(urlEqualTo("/josmticket"))
+    void testBugReportSender(WireMockRuntimeInfo wireMockRuntimeInfo) throws InterruptedException {
+        Config.getPref().put("josm.url", wireMockRuntimeInfo.getHttpBaseUrl());
+        wireMockRuntimeInfo.getWireMock().register(post(urlEqualTo("/josmticket"))
                 .willReturn(aResponse()
                         .withStatus(200)
@@ -66 +59 @@
         assertFalse(sender.isAlive());
         assertNull(sender.getErrorMessage(), sender.getErrorMessage());
-        wireMockServer.verify(exactly(1), postRequestedFor(urlEqualTo("/josmticket")).withRequestBody(containing("pdata=")));
+        wireMockRuntimeInfo.getWireMock().verifyThat(exactly(1),
+                postRequestedFor(urlEqualTo("/josmticket")).withRequestBody(containing("pdata=")));
 
         List<URI> calledURIs = OpenBrowserMocker.getCalledURIs();
         assertEquals(1, calledURIs.size());
-        assertEquals(wireMockServer.url("/josmticket?pdata_stored=6bccff5c0417217bfbbe5fff"), calledURIs.get(0).toString());
+        assertEquals(wireMockRuntimeInfo.getHttpBaseUrl() + "/josmticket?pdata_stored=6bccff5c0417217bfbbe5fff", calledURIs.get(0).toString());
     }
 }