From c73bd6ca851463b47facad70d9296215f410f660 Mon Sep 17 00:00:00 2001
From: Robert Scott <code@humanleg.org.uk>
Date: Mon, 29 Oct 2018 23:03:50 +0000
Subject: [PATCH v1 1/2] RelationEditorActionsTest: add forgotten
assumeWorkingJMockit() check to relevant tests
---
.../josm/gui/dialogs/relation/actions/RelationEditorActionsTest.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/test/unit/org/openstreetmap/josm/gui/dialogs/relation/actions/RelationEditorActionsTest.java b/test/unit/org/openstreetmap/josm/gui/dialogs/relation/actions/RelationEditorActionsTest.java
index 88e87d1ee..db4050909 100644
|
a
|
b
|
import javax.swing.Icon;
|
| 10 | 10 | import javax.swing.JOptionPane; |
| 11 | 11 | import javax.swing.text.JTextComponent; |
| 12 | 12 | |
| 13 | | import org.openstreetmap.josm.testutils.mockers.JOptionPaneSimpleMocker; |
| | 13 | import org.openstreetmap.josm.TestUtils; |
| 14 | 14 | import org.openstreetmap.josm.gui.ConditionalOptionPaneUtil; |
| | 15 | import org.openstreetmap.josm.testutils.mockers.JOptionPaneSimpleMocker; |
| 15 | 16 | |
| 16 | 17 | import com.google.common.collect.ImmutableMap; |
| 17 | 18 | |
| … |
… |
public class RelationEditorActionsTest extends AbstractRelationEditorActionTest
|
| 70 | 71 | */ |
| 71 | 72 | @Test |
| 72 | 73 | public void testDeleteCurrentRelationAction() { |
| | 74 | TestUtils.assumeWorkingJMockit(); |
| 73 | 75 | final JOptionPaneSimpleMocker jopsMocker = new JOptionPaneSimpleMocker( |
| 74 | 76 | ImmutableMap.<String, Object>of( |
| 75 | 77 | "<html>\n <head>\n \n </head>\n <body>\n You are about to delete 1 " |
| … |
… |
public class RelationEditorActionsTest extends AbstractRelationEditorActionTest
|
| 106 | 108 | */ |
| 107 | 109 | @Test |
| 108 | 110 | public void testSetRoleAction() { |
| | 111 | TestUtils.assumeWorkingJMockit(); |
| 109 | 112 | final JOptionPaneSimpleMocker.MessagePanelMocker mpMocker = new JOptionPaneSimpleMocker.MessagePanelMocker(); |
| 110 | 113 | // JOptionPaneSimpleMocker doesn't handle showOptionDialog calls because of their potential |
| 111 | 114 | // complexity, but this is quite a simple use of showOptionDialog which we can mock from scratch. |