Changeset 19646 in josm for trunk/test/unit

Timestamp:

2026-10-09T18:21:45+02:00 (5 days ago)

Author:

gaben

Message:

see #24927 - report non-OSM JSON (e.g. GeoJSON from Postpass) as illegal data instead of crashing

File:

• trunk/test/unit/org/openstreetmap/josm/io/OsmJsonReaderTest.java (modified) (1 diff)

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

@@ -260 +260 @@
     }
 
+    /**
+     * Non-regression test for ticket #24927. GeoJSON served as {@code application/json} (e.g. by Postpass)
+     * must be reported as illegal data, not crash the reader.
+     */
+    @Test
+    void testTicket24927() {
+        byte[] geoJson = ("{\"type\": \"FeatureCollection\", \"features\": [{\"type\": \"Feature\", " +
+                "\"geometry\": {\"type\": \"Point\", \"coordinates\": [10.1, 20.2]}, " +
+                "\"properties\": {\"tags\": {\"fixme\": \"yes\"}}}]}").getBytes(StandardCharsets.UTF_8);
+        IllegalDataException exception = assertThrows(IllegalDataException.class,
+                () -> OsmJsonReader.parseDataSet(new ByteArrayInputStream(geoJson), NullProgressMonitor.INSTANCE));
+        assertEquals("Missing mandatory attribute 'version'.", exception.getMessage());
+    }
+
     static Stream<Arguments> testException() {
         final byte[] smallJson = "{\"type\", \"node\", \"id\": 1, \"lat\": 1.0, \"lon\": 2.0}".getBytes(StandardCharsets.UTF_8);