Changeset 12156 in josm for trunk/test/unit/org

Timestamp:

2017-05-15T14:14:40+02:00 (7 years ago)

Author:

michael2402

Message:

See #14120: Don't make gpx tracks depend on the isChanged method, use a listener based approach instead.

Location:

trunk/test/unit/org/openstreetmap/josm

Files:

• gui/layer/gpx/GpxDrawHelperTest.java (modified) (3 diffs)
• io/GpxReaderTest.java (modified) (1 diff)

trunk/test/unit/org/openstreetmap/josm/gui/layer/gpx/GpxDrawHelperTest.java

@@ -6 +6 @@
 import java.io.FileNotFoundException;
 import java.io.IOException;
-import java.util.ArrayList;
-import java.util.Iterator;
 import java.util.List;
+import java.util.stream.Collectors;
 
 import org.junit.BeforeClass;
@@ -16 +15 @@
 import org.openstreetmap.josm.TestUtils;
 import org.openstreetmap.josm.data.gpx.GpxData;
-import org.openstreetmap.josm.data.gpx.WayPoint;
 import org.openstreetmap.josm.io.GpxReaderTest;
 import org.openstreetmap.josm.tools.ColorHelper;
@@ -128 +126 @@
         gdh.readPreferences(layerName);
         gdh.calculateColors();
-        final Iterator<WayPoint> wayPointIterator = data.tracks.iterator().next().getSegments().iterator().next().getWayPoints().iterator();
-        final List<String> colorCodes = new ArrayList<>(n);
-        while (colorCodes.size() < n) {
-            colorCodes.add(ColorHelper.color2html(wayPointIterator.next().customColoring));
-        }
-        return colorCodes;
+        return data.getTrackPoints().limit(n).map(p -> ColorHelper.color2html(p.customColoring)).collect(Collectors.toList());
     }
 }

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

@@ -47 +47 @@
     public void testMunich() throws Exception {
         final GpxData result = parseGpxData("data_nodist/munich.gpx");
-        assertEquals(2762, result.tracks.size());
-        assertEquals(0, result.routes.size());
-        assertEquals(903, result.waypoints.size());
+        assertEquals(2762, result.getTracks().size());
+        assertEquals(0, result.getRoutes().size());
+        assertEquals(903, result.getWaypoints().size());
 
         final WayPoint tenthWayPoint = ((List<WayPoint>) result.waypoints).get(10);