Index: trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java =================================================================== --- trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java (revision 14316) +++ trunk/src/org/openstreetmap/josm/plugins/PluginHandler.java (revision 14317) @@ -183,8 +183,6 @@ a.setText(text); a.setCaretPosition(0); - if (!GraphicsEnvironment.isHeadless()) { - JOptionPane.showMessageDialog(MainApplication.getMainFrame(), new JScrollPane(a), tr("Plugin information"), - JOptionPane.INFORMATION_MESSAGE); - } + JOptionPane.showMessageDialog(MainApplication.getMainFrame(), new JScrollPane(a), tr("Plugin information"), + JOptionPane.INFORMATION_MESSAGE); } } @@ -352,12 +350,10 @@ } sb.append(""); - if (!GraphicsEnvironment.isHeadless()) { - JOptionPane.showMessageDialog( - parent, - sb.toString(), - tr("Warning"), - JOptionPane.WARNING_MESSAGE - ); - } + JOptionPane.showMessageDialog( + parent, + sb.toString(), + tr("Warning"), + JOptionPane.WARNING_MESSAGE + ); } Index: trunk/test/unit/org/openstreetmap/josm/plugins/PluginHandlerTest.java =================================================================== --- trunk/test/unit/org/openstreetmap/josm/plugins/PluginHandlerTest.java (revision 14316) +++ trunk/test/unit/org/openstreetmap/josm/plugins/PluginHandlerTest.java (revision 14317) @@ -11,4 +11,6 @@ import java.util.List; +import javax.swing.JScrollPane; + import org.junit.Rule; import org.junit.Test; @@ -16,8 +18,13 @@ import org.openstreetmap.josm.gui.MainApplication; import org.openstreetmap.josm.gui.preferences.plugin.PluginPreferenceTest; +import org.openstreetmap.josm.gui.widgets.JosmTextArea; import org.openstreetmap.josm.plugins.PluginHandler.DeprecatedPlugin; import org.openstreetmap.josm.plugins.PluginHandler.PluginInformationAction; import org.openstreetmap.josm.testutils.JOSMTestRules; +import org.openstreetmap.josm.testutils.mockers.HelpAwareOptionPaneMocker; +import org.openstreetmap.josm.testutils.mockers.JOptionPaneSimpleMocker; import org.openstreetmap.josm.tools.Utils; + +import com.google.common.collect.ImmutableMap; import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; @@ -50,4 +57,15 @@ @Test public void testBuildListOfPluginsToLoad() { + TestUtils.assumeWorkingJMockit(); + final HelpAwareOptionPaneMocker haMocker = new HelpAwareOptionPaneMocker() { + @Override + public String getStringFromMessage(final Object message) { + return ((String) message).substring(0, 66); + } + }; + haMocker.getMockResultMap().put( + "JOSM could not find information about the following plugins:", + "OK" + ); final String old = System.getProperty("josm.plugins"); try { @@ -65,4 +83,9 @@ } } + + assertEquals(1, haMocker.getInvocationLog().size()); + Object[] invocationLogEntry = haMocker.getInvocationLog().get(0); + assertEquals(0, (int) invocationLogEntry[0]); + assertEquals("Warning", invocationLogEntry[2]); } @@ -72,8 +95,22 @@ @Test public void testFilterDeprecatedPlugins() { + TestUtils.assumeWorkingJMockit(); + final JOptionPaneSimpleMocker jopsMocker = new JOptionPaneSimpleMocker( + ImmutableMap.of( + "The following plugin is no longer necessary and has been deactivated:" + + "", + 0 + ) + ); + List plugins = new ArrayList<>(Arrays.asList("foo", "bar", "imagery")); PluginHandler.filterDeprecatedPlugins(MainApplication.getMainFrame(), plugins); assertEquals(2, plugins.size()); assertFalse(plugins.contains("imagery")); + + assertEquals(1, jopsMocker.getInvocationLog().size()); + Object[] invocationLogEntry = jopsMocker.getInvocationLog().get(0); + assertEquals(0, (int) invocationLogEntry[0]); + assertEquals("Warning", invocationLogEntry[2]); } @@ -83,8 +120,23 @@ @Test public void testFilterUnmaintainedPlugins() { + TestUtils.assumeWorkingJMockit(); + final HelpAwareOptionPaneMocker haMocker = new HelpAwareOptionPaneMocker( + ImmutableMap.of( + "Loading of the plugin \"gpsbabelgui\" was requested.
" + + "This plugin is no longer developed and very likely will produce errors.
" + + "It should be disabled.
Delete from preferences?", + "Disable plugin" + ) + ); + List plugins = new ArrayList<>(Arrays.asList("foo", "bar", "gpsbabelgui")); PluginHandler.filterUnmaintainedPlugins(MainApplication.getMainFrame(), plugins); assertEquals(2, plugins.size()); assertFalse(plugins.contains("gpsbabelgui")); + + assertEquals(1, haMocker.getInvocationLog().size()); + Object[] invocationLogEntry = haMocker.getInvocationLog().get(0); + assertEquals(0, (int) invocationLogEntry[0]); + assertEquals("Disable plugin", invocationLogEntry[2]); } @@ -95,19 +147,33 @@ @Test public void testPluginInformationAction() throws PluginException { + TestUtils.assumeWorkingJMockit(); + final String expectedText = "Ant-Version: Apache Ant 1.9.6\n" + + "Author: Don-vip\n" + + "Created-By: 1.7.0_91-b02 (Oracle Corporation)\n" + + "Manifest-Version: 1.0\n" + + "Plugin-Canloadatruntime: true\n" + + "Plugin-Class: org.openstreetmap.josm.plugins.fr.epci.EpciPlugin\n" + + "Plugin-Date: 2015-11-19T08:21:07.645033Z\n" + + "Plugin-Description: Handling of French EPCIs (boundary=local_authority)\n" + + "Plugin-Early: true\n" + + "Plugin-Link: http://wiki.openstreetmap.org/wiki/FR:JOSM/Fr:Plugin/EPCI-fr\n" + + "Plugin-Mainversion: 7001\n" + + "Plugin-Version: 31772\n"; + final JOptionPaneSimpleMocker jopsMocker = new JOptionPaneSimpleMocker() { + @Override + public String getStringFromMessage(final Object message) { + return ((JosmTextArea) ((JScrollPane) message).getViewport().getView()).getText(); + } + }; + jopsMocker.getMockResultMap().put(expectedText, 0); + PluginInformationAction action = new PluginInformationAction(PluginPreferenceTest.getDummyPluginInformation()); - assertEquals( - "Ant-Version: Apache Ant 1.9.6\n" + - "Author: Don-vip\n" + - "Created-By: 1.7.0_91-b02 (Oracle Corporation)\n" + - "Manifest-Version: 1.0\n" + - "Plugin-Canloadatruntime: true\n" + - "Plugin-Class: org.openstreetmap.josm.plugins.fr.epci.EpciPlugin\n" + - "Plugin-Date: 2015-11-19T08:21:07.645033Z\n" + - "Plugin-Description: Handling of French EPCIs (boundary=local_authority)\n" + - "Plugin-Early: true\n" + - "Plugin-Link: http://wiki.openstreetmap.org/wiki/FR:JOSM/Fr:Plugin/EPCI-fr\n" + - "Plugin-Mainversion: 7001\n" + - "Plugin-Version: 31772\n", action.getText()); + assertEquals(expectedText, action.getText()); action.actionPerformed(null); + + assertEquals(1, jopsMocker.getInvocationLog().size()); + Object[] invocationLogEntry = jopsMocker.getInvocationLog().get(0); + assertEquals(0, (int) invocationLogEntry[0]); + assertEquals("Plugin information", invocationLogEntry[2]); } }