Changeset 16311 in josm for trunk/test

Timestamp:

2020-04-15T22:40:32+02:00 (6 years ago)

Author:

simon04

Message:

fix #19041 - GeoJSONReader: reuse node on same at same position to

Avoids multiple nodes on top of each other. Avoid creating unconnected polygons.

Location:

trunk/test

Files:

• data/geo.json (modified) (1 diff)
• unit/org/openstreetmap/josm/io/GeoJSONReaderTest.java (modified) (3 diffs)

trunk/test/data/geo.json

@@ -22 +22 @@
                     [
                         102.0,
-                        0.0
+                        0.5
                     ],
                     [

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

@@ -46 +46 @@
                 .doParseDataSet(in, null)
                 .getPrimitives(it -> true));
-            assertEquals(21, primitives.size());
+            assertEquals(20, primitives.size());
 
             final Node node1 = new Node(new LatLon(0.5, 102.0));
@@ -56 +56 @@
 
             final Way way1 = new Way();
-            way1.addNode(new Node(new LatLon(0, 102)));
+            way1.addNode(new Node(new LatLon(0.5, 102.0)));
             way1.addNode(new Node(new LatLon(1, 103)));
             way1.addNode(new Node(new LatLon(0, 104)));
@@ -66 +66 @@
             assertEquals("valueB", foundWay1.get().get("propB"));
             assertEquals("0.0", foundWay1.get().get("propB2"));
+            assertEquals(foundNode1.get(), ((Way) foundWay1.get()).firstNode());
+            assertEquals("valueA", ((Way) foundWay1.get()).firstNode().get("propA"));
 
             final Way way2 = new Way();