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

Timestamp:

2015-07-26T17:03:37+02:00 (9 years ago)

Author:

wiktorn

Message:

Fix calculation of World Bounds for WMS when no projection has no bounds defined, and bogus results are returned on boundaries.

Closes: #11697, #11701

File:

• trunk/test/unit/org/openstreetmap/josm/data/imagery/TemplatedWMSTileSourceTest.java (modified) (6 diffs)

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

@@ -15 +15 @@
 import org.openstreetmap.josm.data.coor.LatLon;
 import org.openstreetmap.josm.data.osm.BBox;
+import org.openstreetmap.josm.data.projection.CustomProjection;
 import org.openstreetmap.josm.data.projection.Projection;
 import org.openstreetmap.josm.data.projection.Projections;
@@ -67 +68 @@
 
     @Test
+    public void testEPSG4326_widebounds() {
+        Main.setProjection(new CustomProjection("+proj=lonlat +datum=WGS84 +axis=neu +bounds=-180,53,180,54"));
+        TemplatedWMSTileSource source = getSource();
+
+        verifyLocation(source, new LatLon(53.5937132, 19.5652017));
+        verifyLocation(source, new LatLon(53.501565692302854, 18.54455233898721));
+    }
+
+    @Test
+    public void testEPSG4326_narrowbounds() {
+        Main.setProjection(new CustomProjection("+proj=lonlat +datum=WGS84 +axis=neu +bounds=18,-90,20,90"));
+        TemplatedWMSTileSource source = getSource();
+
+        verifyLocation(source, new LatLon(53.5937132, 19.5652017));
+        verifyLocation(source, new LatLon(53.501565692302854, 18.54455233898721));
+    }
+
+    @Test
     public void testEPSG2180() {
         Main.setProjection(Projections.getProjectionByCode("EPSG:2180"));
@@ -77 +96 @@
         verifyTileSquarness(source, 150, 20, 18);
         verifyTileSquarness(source, 2270, 1323, 12);
+    }
+
+    @Test
+    public void testEPSG3006_withbounds() {
+        Main.setProjection(
+                new CustomProjection("+proj=utm +zone=33 +ellps=GRS80 +towgs84=0,0,0,0,0,0,0 "
+                        + "+units=m +no_defs +axis=neu +wmssrs=EPSG:3006 +bounds=10.5700,55.2000,24.1800,69.1000 "));
+        TemplatedWMSTileSource source = getSource();
+
+        verifyLocation(source, new LatLon(60, 18), 3);
+        verifyLocation(source, new LatLon(60, 18));
+
+    }
+
+    @Test
+    public void testEPSG3006_withoutbounds() {
+        Main.setProjection(
+                new CustomProjection("+proj=utm +zone=33 +ellps=GRS80 +towgs84=0,0,0,0,0,0,0 "
+                        + "+units=m +no_defs +axis=neu +wmssrs=EPSG:3006"));
+        TemplatedWMSTileSource source = getSource();
+
+        verifyTileSquarness(source, 0, 1, 4);
+        verifyLocation(source, new LatLon(60, 18.1), 3);
+        verifyLocation(source, new LatLon(60, 18.1));
+
     }
 
@@ -83 +127 @@
         LatLon result = getTileLatLon(source, x, y, z);
         LatLon expected = new LatLon(verifier.tileYToLat(y, z - 1), verifier.tileXToLon(x, z - 1)); //
-        System.out.println(z + "/" + x + "/" + y + " - result: " + result.toDisplayString() + " osmMercator: " +  expected.toDisplayString());
         assertTrue("result: " + result.toDisplayString() + " osmMercator: " +  expected.toDisplayString(), result.equalsEpsilon(expected));
         LatLon tileCenter = new Bounds(result, getTileLatLon(source, x+1, y+1, z)).getCenter();
@@ -92 +135 @@
 
     private void verifyLocation(TemplatedWMSTileSource source, LatLon location) {
-        for (int z = 1; z < 22; z++) {
-            TileXY tileIndex = source.latLonToTileXY(location.toCoordinate(), z);
-            BBox bbox = new BBox(
-                    getTileLatLon(source, tileIndex, z),
-                    getTileLatLon(source, tileIndex.getXIndex() + 1, tileIndex.getYIndex() + 1, z)
-                    );
-            assertTrue(location.toDisplayString() + " not within " + bbox.toString() +
-                    " for tile " + z + "/" + tileIndex.getXIndex() + "/" + tileIndex.getYIndex(),
-                    bbox.bounds(location));
+        for (int z = 22; z > 1; z--) {
+            verifyLocation(source, location, z);
         }
+    }
+
+    private void verifyLocation(TemplatedWMSTileSource source, LatLon location, int z) {
+        assertTrue(
+                "Point outside world bounds",
+                Main.getProjection().getWorldBoundsLatLon().contains(location)
+                );
+
+        TileXY tileIndex = source.latLonToTileXY(location.toCoordinate(), z);
+
+        assertTrue("X index: " + tileIndex.getXIndex() + " greater than tileXmax: " + source.getTileXMax(z) + " at zoom: " + z,
+                tileIndex.getXIndex() <= source.getTileXMax(z));
+
+        assertTrue("Y index: " + tileIndex.getYIndex() + " greater than tileYmax: " + source.getTileYMax(z) + " at zoom: " + z,
+                tileIndex.getYIndex() <= source.getTileYMax(z));
+
+        // test that location is within tile bounds
+        BBox bbox = new BBox(
+                getTileLatLon(source, tileIndex, z),
+                getTileLatLon(source, tileIndex.getXIndex() + 1, tileIndex.getYIndex() + 1, z)
+                );
+        assertTrue(location.toDisplayString() + " not within " + bbox.toString() +
+                " for tile " + z + "/" + tileIndex.getXIndex() + "/" + tileIndex.getYIndex(),
+                bbox.bounds(location));
+        verifyTileSquarness(source, tileIndex.getXIndex(), tileIndex.getYIndex(), z);
     }
 
@@ -119 +180 @@
          * t3 | t4
          */
-        EastNorth t1 = proj.latlon2eastNorth(getTileLatLon(source, x, y, z));
-        EastNorth t2 = proj.latlon2eastNorth(getTileLatLon(source, x + 1, y, z));
-        EastNorth t3 = proj.latlon2eastNorth(getTileLatLon(source, x, y + 1, z));
-        EastNorth t4 = proj.latlon2eastNorth(getTileLatLon(source, x + 1, y + 1, z));
+        EastNorth t1 = source.getTileEastNorth(x, y, z);
+        EastNorth t2 = source.getTileEastNorth(x + 1, y, z);
+        EastNorth t3 = source.getTileEastNorth(x, y + 1, z);
+        EastNorth t4 = source.getTileEastNorth(x + 1, y + 1, z);
         double y_size = Math.abs(t1.getY() - t4.getY());
         double x_size = Math.abs(t1.getX() - t4.getX());
 
-        assertEquals(x_size, y_size, Math.max(x_size, y_size) * 1e-05);
-        assertEquals(y_size, Math.abs(t1.getY() - t3.getY()), y_size * 1e-05);
-        assertEquals(x_size, Math.abs(t1.getX() - t2.getX()), x_size * 1e-05);
+        assertEquals(x_size, y_size, Math.max(x_size, y_size) * 1e-06);
+        assertEquals(y_size, Math.abs(t1.getY() - t3.getY()), y_size * 1e-06);
+        assertEquals(x_size, Math.abs(t1.getX() - t2.getX()), x_size * 1e-06);
 
         t1 = source.getTileEastNorth(x, y, z);