Ticket #16010: v1-0001-unit-tests-replace-isHeadless-checks-in-Navigatab.patch

src/org/openstreetmap/josm/gui/MapViewState.java

@@ -2 +2 @@
 package org.openstreetmap.josm.gui;
 
 import java.awt.Container;
-import java.awt.GraphicsEnvironment;
 import java.awt.Point;
 import java.awt.geom.AffineTransform;
 import java.awt.geom.Area;
@@ -145 +144 @@
     }
 
     private static Point findTopLeftOnScreen(JComponent position) {
-        if (GraphicsEnvironment.isHeadless()) {
-            // in our imaginary universe the window is always (10, 10) from the top left of the screen
-            Point topLeftInWindow = findTopLeftInWindow(position);
-            return new Point(topLeftInWindow.x + 10, topLeftInWindow.y + 10);
-        } else {
-            try {
-                return position.getLocationOnScreen();
-            } catch (JosmRuntimeException | IllegalArgumentException | IllegalStateException e) {
-                throw BugReport.intercept(e).put("position", position).put("parent", position::getParent);
-            }
+        try {
+            return position.getLocationOnScreen();
+        } catch (JosmRuntimeException | IllegalArgumentException | IllegalStateException e) {
+            throw BugReport.intercept(e).put("position", position).put("parent", position::getParent);
         }
     }
 

src/org/openstreetmap/josm/gui/NavigatableComponent.java

@@ -2 +2 @@
 package org.openstreetmap.josm.gui;
 
 import java.awt.Cursor;
-import java.awt.GraphicsEnvironment;
 import java.awt.Point;
 import java.awt.Rectangle;
 import java.awt.event.ComponentAdapter;
@@ -323 +322 @@
     }
 
     protected boolean isVisibleOnScreen() {
-        return GraphicsEnvironment.isHeadless() || (
-            SwingUtilities.getWindowAncestor(this) != null && isShowing()
-        );
+        return SwingUtilities.getWindowAncestor(this) != null && isShowing();
     }
 
     /**

test/unit/org/openstreetmap/josm/testutils/JOSMTestRules.java

@@ -63 +63 @@
     private TileSourceRule tileSourceRule;
     private String assumeRevisionString;
     private Version originalVersion;
+    private Runnable mapViewStateMockingRunnable;
+    private Runnable navigableComponentMockingRunnable;
     private boolean platform;
     private boolean useProjection;
     private boolean useProjectionNadGrids;
@@ -293 +295 @@
      * Use the {@link Main#main}, {@code Main.contentPanePrivate}, {@code Main.mainPanel},
      *         global variables in this test.
      * @return this instance, for easy chaining
-     * @since 12557
      */
     public JOSMTestRules main() {
+        return this.main(WindowlessMapViewStateMocker::new, WindowlessNavigatableComponentMocker::new);
+    }
+
+    /**
+     * Use the {@link Main#main}, {@code Main.contentPanePrivate}, {@code Main.mainPanel},
+     *         global variables in this test.
+     * @return this instance, for easy chaining
+     * @since 12557
+     */
+    public JOSMTestRules main(Runnable mapViewStateMockingRunnable, Runnable navigableComponentMockingRunnable) {
         platform();
-        main = true;
+        this.main = true;
+        this.mapViewStateMockingRunnable = mapViewStateMockingRunnable;
+        this.navigableComponentMockingRunnable = navigableComponentMockingRunnable;
         return this;
     }
 
@@ -452 +465 @@
             JOSMFixture.createUnitTestFixture().init(true);
         } else {
             if (main) {
+                // apply mockers to MapViewState and NavigableComponent whether we're headless or not
+                // as we generally don't create the josm main window even in non-headless mode.
+                if (this.mapViewStateMockingRunnable != null) {
+                    this.mapViewStateMockingRunnable.run();
+                }
+                if (this.navigableComponentMockingRunnable != null) {
+                    this.navigableComponentMockingRunnable.run();
+                }
+
                 new MainApplication();
                 JOSMFixture.initContentPane();
                 JOSMFixture.initMainPanel(true);

new file test/unit/org/openstreetmap/josm/testutils/mockers/WindowlessMapViewStateMocker.java

@@ +1 @@
+// License: GPL. For details, see LICENSE file.
+package org.openstreetmap.josm.testutils;
+
+import javax.swing.JComponent;
+import java.awt.Point;
+
+import org.openstreetmap.josm.gui.MapViewState;
+
+import mockit.Mock;
+import mockit.MockUp;
+
+/**
+ * MockUp for allowing a {@link MapViewState} to be fully initialized in either headless or
+ * windowless tests
+ */
+public class WindowlessMapViewStateMocker extends MockUp<MapViewState> {
+    @Mock
+    private static Point findTopLeftInWindow(JComponent position) {
+        return new Point();
+    }
+
+    @Mock
+    private static Point findTopLeftOnScreen(JComponent position) {
+        // in our imaginary universe the window is always (10, 10) from the top left of the screen
+        Point topLeftInWindow = findTopLeftInWindow(position);
+        return new Point(topLeftInWindow.x + 10, topLeftInWindow.y + 10);
+    }
+}

new file test/unit/org/openstreetmap/josm/testutils/mockers/WindowlessNavigatableComponentMocker.java

@@ +1 @@
+// License: GPL. For details, see LICENSE file.
+package org.openstreetmap.josm.testutils;
+
+import org.openstreetmap.josm.gui.NavigatableComponent;
+
+import mockit.Mock;
+import mockit.MockUp;
+
+/**
+ * MockUp for allowing a {@link NavigatableComponent} to be used in either headless or windowless
+ * tests.
+ */
+public class WindowlessNavigatableComponentMocker extends MockUp<NavigatableComponent> {
+    @Mock
+    private boolean isVisibleOnScreen() {
+        return true;
+    }
+}