Changeset 31828 in osm for applications/editors/josm/plugins/mapillary/test

Timestamp:

2015-12-15T16:32:30+01:00 (9 years ago)

Author:

floscher

Message:

[mapillary] Move creation of Mapillary URLs in separate utility class

File:

• applications/editors/josm/plugins/mapillary/test/unit/org/openstreetmap/josm/plugins/mapillary/io/download/MapillarySequenceDownloadThreadTest.java (modified) (3 diffs)

applications/editors/josm/plugins/mapillary/test/unit/org/openstreetmap/josm/plugins/mapillary/io/download/MapillarySequenceDownloadThreadTest.java

@@ -43 +43 @@
 
     ExecutorService ex = Executors.newSingleThreadExecutor();
-    String queryString = String
-        .format(
-            Locale.UK,
-            "?max_lat=%.8f&max_lon=%.8f&min_lat=%.8f&min_lon=%.8f&limit=10&client_id=%s",
-            maxLatLon.lat(), maxLatLon.lon(), minLatLon.lat(), minLatLon.lon(),
-            MapillaryPlugin.CLIENT_ID);
+    Bounds bounds = new Bounds(minLatLon, maxLatLon);
     MapillaryLayer.getInstance().getData().bounds.add(new Bounds(minLatLon,
         maxLatLon));
@@ -58 +53 @@
       System.out.println("Sending sequence-request " + page
           + " to Mapillary-servers…");
-      Thread downloadThread = new MapillarySequenceDownloadThread(ex,
-          queryString + "&page=" + page);
+      Thread downloadThread = new MapillarySequenceDownloadThread(ex, bounds, page);
       downloadThread.start();
       downloadThread.join();
@@ -66 +60 @@
     }
     assertTrue(MapillaryLayer.getInstance().getData().getImages().size() >= 1);
-    System.out
-        .println("One or more images were added to the MapillaryLayer within the given bounds.");
+    System.out.println("One or more images were added to the MapillaryLayer within the given bounds.");
   }