Changeset 18963 in josm for trunk/test

Timestamp:

2024-01-30T14:33:14+01:00 (10 months ago)

Author:

taylor.smock

Message:

Fix #23444: NPE in org.openstreetmap.josm.gui.MapViewState$MapViewEastNorthPoint.<init>

This occurs when an incomplete way is loaded into JOSM (by an .osm file, for
example). This is fixed by checking to see if the way is usable (not incomplete,
not deleted, and more importantly for this ticket, no nodes are incomplete).

This additionally fixes some pmd issues and sonarlint issues.

File:

• trunk/test/unit/org/openstreetmap/josm/actions/mapmode/ImproveWayAccuracyActionTest.java (modified) (2 diffs)

trunk/test/unit/org/openstreetmap/josm/actions/mapmode/ImproveWayAccuracyActionTest.java

@@ -2 +2 @@
 package org.openstreetmap.josm.actions.mapmode;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
+import java.awt.Graphics2D;
+import java.awt.Point;
+import java.awt.event.MouseEvent;
+import java.awt.image.BufferedImage;
+
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.RegisterExtension;
 import org.openstreetmap.josm.TestUtils;
 import org.openstreetmap.josm.actions.mapmode.ImproveWayAccuracyAction.State;
+import org.openstreetmap.josm.data.Bounds;
+import org.openstreetmap.josm.data.coor.ILatLon;
+import org.openstreetmap.josm.data.coor.LatLon;
 import org.openstreetmap.josm.data.osm.DataSet;
+import org.openstreetmap.josm.data.osm.Node;
+import org.openstreetmap.josm.data.osm.Way;
 import org.openstreetmap.josm.gui.MainApplication;
 import org.openstreetmap.josm.gui.MapFrame;
+import org.openstreetmap.josm.gui.MapView;
 import org.openstreetmap.josm.gui.layer.OsmDataLayer;
+import org.openstreetmap.josm.gui.util.GuiHelper;
 import org.openstreetmap.josm.testutils.annotations.Main;
 import org.openstreetmap.josm.testutils.annotations.Projection;
+import org.openstreetmap.josm.testutils.mockers.WindowlessMapViewStateMocker;
+import org.openstreetmap.josm.testutils.mockers.WindowlessNavigatableComponentMocker;
+
+import mockit.Mock;
 
 /**
  * Unit tests for class {@link ImproveWayAccuracyAction}.
  */
-@Main
 @Projection
 class ImproveWayAccuracyActionTest {
+    @RegisterExtension
+    Main.MainExtension mainExtension = new Main.MainExtension().setMapViewMocker(SizedWindowlessMapViewStateMocker::new)
+            .setNavigableComponentMocker(SizedWindowlessNavigatableComponentMocker::new);
+
+    private static final int WIDTH = 800;
+    private static final int HEIGHT = 600;
+    private static class SizedWindowlessMapViewStateMocker extends WindowlessMapViewStateMocker {
+        @Mock
+        public int getWidth() {
+            return WIDTH;
+        }
+
+        @Mock
+        public int getHeight() {
+            return HEIGHT;
+        }
+    }
+
+    private static class SizedWindowlessNavigatableComponentMocker extends WindowlessNavigatableComponentMocker {
+        @Mock
+        public int getWidth() {
+            return WIDTH;
+        }
+
+        @Mock
+        public int getHeight() {
+            return HEIGHT;
+        }
+    }
+
+    private static MouseEvent generateEvent(MapView mapView, ILatLon location) {
+        final Point p = mapView.getPoint(location);
+        return new MouseEvent(mapView, 0, 0, 0, p.x, p.y, p.x, p.y, 1, false, MouseEvent.BUTTON1);
+    }
+
     /**
      * Unit test of {@link ImproveWayAccuracyAction#enterMode} and {@link ImproveWayAccuracyAction#exitMode}.
@@ -45 +98 @@
     @Test
     void testEnumState() {
-        TestUtils.superficialEnumCodeCoverage(State.class);
+        assertDoesNotThrow(() -> TestUtils.superficialEnumCodeCoverage(State.class));
+    }
+
+    @Test
+    void testNonRegression23444() {
+        final DataSet dataSet = new DataSet();
+        final OsmDataLayer layer = new OsmDataLayer(dataSet, "ImproveWayAccuracyActionT", null);
+        MainApplication.getLayerManager().addLayer(layer);
+        final ImproveWayAccuracyAction mapMode = new ImproveWayAccuracyAction();
+        final MapFrame map = MainApplication.getMap();
+        assertTrue(map.selectMapMode(mapMode));
+        assertEquals(mapMode, map.mapMode);
+        final Way testWay = TestUtils.newWay("", new Node(1, 1), new Node(2, 1),
+                new Node(3), new Node(4, 1), new Node(5, 1));
+        testWay.firstNode().setCoor(new LatLon(0, 0));
+        testWay.lastNode().setCoor(new LatLon(0.001, 0.001));
+        testWay.getNode(1).setCoor(new LatLon(0.0001, 0.0001));
+        testWay.getNode(3).setCoor(new LatLon(0.0009, 0.0009));
+        dataSet.addPrimitiveRecursive(testWay);
+        assertFalse(testWay.getNode(2).isLatLonKnown(), "The second node should not have valid coordinates");
+        final Graphics2D g2d = new BufferedImage(WIDTH, HEIGHT, BufferedImage.TYPE_INT_ARGB).createGraphics();
+        g2d.setClip(0, 0, WIDTH, HEIGHT);
+        try {
+            // If this fails, something else is wrong
+            assertDoesNotThrow(() -> map.mapView.paint(g2d), "The mapview should be able to handle a null coordinate node");
+            // Ensure that the test way is selected (and use the methods from the action to do so)
+            GuiHelper.runInEDTAndWaitWithException(() -> {
+                map.mapView.zoomTo(new Bounds(0, 0, 0.001, 0.001));
+                // Get the target way selected (note: not selected in dataset -- use mapMode.mapReleased for that)
+                mapMode.mouseMoved(generateEvent(map.mapView, testWay.getNode(1)));
+            });
+            // mouseMoved shouldn't cause the way to get selected
+            assertFalse(dataSet.getAllSelected().contains(testWay));
+
+            // Now check painting (where the problem should occur; the mapMode.paint call should be called as part of the map.mapView.paint call)
+            assertDoesNotThrow(() -> map.mapView.paint(g2d));
+            assertDoesNotThrow(() -> mapMode.paint(g2d, map.mapView, new Bounds(0, 0, 0.001, 0.001)));
+
+            // Finally, check painting during selection
+            GuiHelper.runInEDTAndWaitWithException(() -> {
+                // Set the way as selected
+                mapMode.mouseReleased(generateEvent(map.mapView, new LatLon(0.0001, 0.0001)));
+                // Set the mouse location (unset in mouseReleased call)
+                mapMode.mouseMoved(generateEvent(map.mapView, new LatLon(0.0001, 0.0001)));
+            });
+            // The way shouldn't be selected, since it isn't usable for the improve way tool
+            assertFalse(dataSet.getAllSelected().contains(testWay));
+            // Now check painting again (just in case)
+            assertDoesNotThrow(() -> map.paint(g2d));
+            assertDoesNotThrow(() -> mapMode.paint(g2d, map.mapView, new Bounds(0, 0, 0.001, 0.001)));
+        } finally {
+            g2d.dispose();
+        }
     }
 }