Changeset 18050 in josm for trunk/test/unit/org/openstreetmap/josm

Timestamp:

2021-07-17T17:16:14+02:00 (3 years ago)

Author:

Don-vip

Message:

see #17606 - see #21126 - fixes

Location:

trunk/test/unit/org/openstreetmap/josm

Files:

• gui/layer/geoimage/WikimediaCommonsLoaderTest.java (modified) (4 diffs)
• tools/MediawikiTest.java (modified) (1 diff)

trunk/test/unit/org/openstreetmap/josm/gui/layer/geoimage/WikimediaCommonsLoaderTest.java

@@ -11 +11 @@
 import java.util.List;
 
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
 import org.openstreetmap.josm.data.Bounds;
 import org.openstreetmap.josm.testutils.annotations.BasicPreferences;
+import org.openstreetmap.josm.tools.Http1Client;
+import org.openstreetmap.josm.tools.HttpClient;
 
 import com.github.tomakehurst.wiremock.WireMockServer;
-import org.junit.jupiter.api.Test;
 
 /**
@@ -22 +25 @@
 @BasicPreferences
 class WikimediaCommonsLoaderTest {
+
+    @BeforeAll
+    static void beforeAll() {
+        HttpClient.setFactory(Http1Client::new);
+    }
+
     /**
      * Unit test of {@link WikimediaCommonsLoader}
@@ -45 +54 @@
     }
 
-    private WireMockServer mockHttp() {
-        String xml = "" +
+    private static WireMockServer mockHttp() {
+        String xml =
                 "<api batchcomplete=\"\">\n" +
                 "<query>\n" +
@@ -57 +66 @@
         final WireMockServer wireMock = new WireMockServer(options().dynamicPort());
         wireMock.stubFor(get(urlEqualTo("/w/api.php?format=xml&action=query&list=geosearch" +
-                "&gsnamespace=6&gslimit=500&gsprop=type|name&gsbbox=48.0|11.0|47.0|12.0"))
+                "&gsnamespace=6&gslimit=500&gsprop=type%7Cname&gsbbox=48.0%7C11.0%7C47.0%7C12.0"))
                 .willReturn(aResponse().withBody(xml)));
         return wireMock;

trunk/test/unit/org/openstreetmap/josm/tools/MediawikiTest.java

@@ -34 +34 @@
         // See https://josm.openstreetmap.de/ticket/21126
         // Checks that URL can be converted to URI, needed for HTTP/2
+        // CHECKSTYLE.OFF: LineLength
         assertEquals(new URI("https://commons.wikimedia.org/w/api.php?format=xml&action=query&list=geosearch&gsnamespace=6&gslimit=500&gsprop=type%7Cname&gsbbox=48.8623665%7C2.3913497%7C48.8600879%7C2.3967605"),
-                new URL(Mediawiki.getGeoImagesUrl("https://commons.wikimedia.org/w/api.php", new Bounds(48.8600879, 2.3913497, 48.8623665, 2.3967605))).toURI());
+                new URL(Mediawiki.getGeoImagesUrl("https://commons.wikimedia.org/w/api.php",
+                        new Bounds(48.8600879, 2.3913497, 48.8623665, 2.3967605))).toURI());
+        // CHECKSTYLE.ON: LineLength
     }
 }