Changeset 18819 in josm for trunk/test/unit

Timestamp:

2023-08-23T13:53:32+02:00 (15 months ago)

Author:

taylor.smock

Message:

See #22652, r18817: Stop parsing gpx files when point elements have invalid coordinates

This adds back the wpt check that was temporarily removed in r18818 to get back
to the previous "known good" state and additionally adds checks for rtept and
trkpt elements.

This also removes a change to build.xml that was accidentally committed in r18818.
The specific change (adding Automatic-Module-Name to the JOSM manifest) will
probably be done in a separate commit in the future.

File:

• trunk/test/unit/org/openstreetmap/josm/io/GpxReaderTest.java (modified) (2 diffs)

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

@@ -104 +104 @@
 
     @ParameterizedTest
-    @ValueSource(strings = "<gpx><wpt></wpt></gpx>")
+    @ValueSource(strings = {
+            "<gpx><wpt></wpt></gpx>",
+            "<gpx><trk><trkseg><trkpt></trkpt></trkseg></trk></gpx>",
+            "<gpx><rte><rtept></rtept></rte></gpx>"
+    })
     void testIncompleteLocations(String gpx) {
         SAXException saxException = assertThrows(SAXException.class,
@@ -111 +115 @@
         if ("<wpt>".regionMatches(0, gpx, 5, 4)) {
             type = "wpt";
+        } else if ("<trkpt>".regionMatches(0, gpx, 18, 7)) {
+            type = "trkpt";
+        } else if ("<rtept>".regionMatches(0, gpx, 10, 7)) {
+            type = "rtept";
         } else {
             fail("You need to add code to tell us what the exception for \"" + gpx + "\" should be");