Ticket #16010: v2-0022-plugin-preferences-small-additions-to-ease-testin.patch

src/org/openstreetmap/josm/gui/preferences/PreferenceTabbedPane.java

@@ -25 +25 @@
 import javax.swing.JPanel;
 import javax.swing.JScrollPane;
 import javax.swing.JTabbedPane;
-import javax.swing.SwingUtilities;
 import javax.swing.event.ChangeEvent;
 import javax.swing.event.ChangeListener;
 
@@ -59 +58 @@
 import org.openstreetmap.josm.gui.preferences.validator.ValidatorPreference;
 import org.openstreetmap.josm.gui.preferences.validator.ValidatorTagCheckerRulesPreference;
 import org.openstreetmap.josm.gui.preferences.validator.ValidatorTestsPreference;
+import org.openstreetmap.josm.gui.util.GuiHelper;
 import org.openstreetmap.josm.plugins.PluginDownloadTask;
 import org.openstreetmap.josm.plugins.PluginHandler;
 import org.openstreetmap.josm.plugins.PluginInformation;
@@ -174 +174 @@
                 }
             }
 
-            Main.parent.repaint();
+            if (Main.parent != null) {
+                Main.parent.repaint();
+            }
         }
     }
 
@@ -420 +422 @@
                 // if we have to launch a plugin download task we do it asynchronously, followed
                 // by the remaining "save preferences" activites run on the Swing EDT.
                 MainApplication.worker.submit(task);
-                MainApplication.worker.submit(() -> SwingUtilities.invokeLater(continuation));
+                MainApplication.worker.submit(() -> GuiHelper.runInEDT(continuation));
             } else {
                 // no need for asynchronous activities. Simply run the remaining "save preference"
                 // activities on this thread (we are already on the Swing EDT

src/org/openstreetmap/josm/gui/preferences/plugin/PluginPreference.java

@@ -211 +211 @@
     private JPanel buildActionPanel() {
         JPanel pnl = new JPanel(new GridLayout(1, 4));
 
-        pnl.add(new JButton(new DownloadAvailablePluginsAction()));
-        pnl.add(new JButton(new UpdateSelectedPluginsAction()));
-        ExpertToggleAction.addVisibilitySwitcher(pnl.add(new JButton(new SelectByListAction())));
-        ExpertToggleAction.addVisibilitySwitcher(pnl.add(new JButton(new ConfigureSitesAction())));
+        // assign some component names to these as we go to aid testing
+        final JButton downloadListButton = new JButton(new DownloadAvailablePluginsAction());
+        downloadListButton.setName("downloadListButton");
+        final JButton updatePluginsButton = new JButton(new UpdateSelectedPluginsAction());
+        updatePluginsButton.setName("updatePluginsButton");
+        final JButton loadFromListButton = new JButton(new SelectByListAction());
+        loadFromListButton.setName("loadFromListButton");
+        final JButton configureSitesButton = new JButton(new ConfigureSitesAction());
+        configureSitesButton.setName("configureSitesButton");
+
+        pnl.add(downloadListButton);
+        pnl.add(updatePluginsButton);
+        ExpertToggleAction.addVisibilitySwitcher(pnl.add(loadFromListButton));
+        ExpertToggleAction.addVisibilitySwitcher(pnl.add(configureSitesButton));
         return pnl;
     }