Changeset 17090 in josm for trunk/test/unit/org/openstreetmap/josm/testutils

Timestamp:

2020-10-07T07:46:55+02:00 (4 years ago)

Author:

simon04

Message:

fix #18200 - Update to JMockit 1.49 (patch by taylor.smock, modified)

Location:

trunk/test/unit/org/openstreetmap/josm/testutils/mockers

Files:

• EDTAssertionMocker.java (modified) (1 diff)
• ExtendedDialogMocker.java (modified) (5 diffs)
• WindowlessMapViewStateMocker.java (modified) (1 diff)

trunk/test/unit/org/openstreetmap/josm/testutils/mockers/EDTAssertionMocker.java

@@ -14 +14 @@
 public class EDTAssertionMocker extends MockUp<GuiHelper> {
     @Mock
-    private static void handleEDTException(final Invocation invocation, final Throwable t) {
+    static void handleEDTException(final Invocation invocation, final Throwable t) throws Throwable {
         final Throwable cause = t.getCause();
         if (cause instanceof AssertionError) {

trunk/test/unit/org/openstreetmap/josm/testutils/mockers/ExtendedDialogMocker.java

@@ -6 +6 @@
 import java.awt.Component;
 import java.awt.GraphicsEnvironment;
+import java.lang.reflect.Field;
 import java.util.Arrays;
 import java.util.Map;
+import java.util.NoSuchElementException;
 import java.util.Optional;
 import java.util.WeakHashMap;
 
+import org.junit.platform.commons.util.ReflectionUtils;
 import org.openstreetmap.josm.TestUtils;
 import org.openstreetmap.josm.gui.ExtendedDialog;
 import org.openstreetmap.josm.tools.Logging;
 
-import mockit.Deencapsulation;
 import mockit.Invocation;
 import mockit.Mock;
-import mockit.internal.reflection.FieldReflection;
 
 /**
@@ -77 +78 @@
 
     protected int getButtonPositionFromLabel(final ExtendedDialog instance, final String label) {
-        final String[] bTexts = Deencapsulation.getField(instance, "bTexts");
+        final String[] bTexts = (String[]) ReflectionUtils.tryToReadFieldValue(ExtendedDialog.class, "bTexts", instance)
+                .toOptional().orElseThrow(NoSuchElementException::new);
         final int position = Arrays.asList(bTexts).indexOf(label);
         if (position == -1) {
@@ -151 +153 @@
 
     @Mock
-    private void setVisible(final Invocation invocation, final boolean value) {
+    private void setVisible(final Invocation invocation, final boolean value) throws Throwable {
         if (value == true) {
             try {
@@ -158 +160 @@
                 final int mockResult = this.getMockResult(instance);
                 // TODO check validity of mockResult?
-                FieldReflection.setField(instance.getClass(), instance, "result", mockResult);
+                Field resultField = instance.getClass().getDeclaredField("result");
+                resultField.setAccessible(true);
+                resultField.set(instance, mockResult);
                 Logging.info(
                     "{0} answering {1} to ExtendedDialog with content {2}",
@@ -166 +170 @@
                 );
                 this.getInvocationLogInternal().add(this.getInvocationLogEntry(instance, mockResult));
-            } catch (AssertionError e) {
+            } catch (AssertionError | NoSuchFieldException | IllegalAccessException e) {
                 // in case this exception gets ignored by the calling thread we want to signify this failure
                 // in the invocation log. it's hard to know what to add to the log in these cases as it's

trunk/test/unit/org/openstreetmap/josm/testutils/mockers/WindowlessMapViewStateMocker.java

@@ -17 +17 @@
 public class WindowlessMapViewStateMocker extends MockUp<MapViewState> {
     @Mock
-    private static Point findTopLeftInWindow(JComponent position) {
+    static Point findTopLeftInWindow(JComponent position) {
         return new Point();
     }
 
     @Mock
-    private static Point findTopLeftOnScreen(JComponent position) {
+    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);