Changeset 18283 in josm for trunk/test/unit/org

Timestamp:

2021-10-17T15:17:42+02:00 (3 years ago)

Author:

Don-vip

Message:

fix #21427 - further simplify UploadDialog (patch by marcello, modified)

• The dialog was simplified by combining the function of two radiobuttons and one combobox into one combobox.
• When an open changeset was selected on tab 2, existing tags on the open changeset could overwrite the data the user entered on tab 1. The user might spot this by looking closely at the tag table on tab 2, but then he may not. This non-obvious behaviour was removed.
• The exception thrown when closing an already closed changeset was fixed.
• More cosmetic changes to the dialog.
• Maybe also a solution to #19319, #21387 (added revalidate()).

Location:

trunk/test/unit/org/openstreetmap/josm

Files:

• data/osm/ChangesetCacheTest.java (modified) (2 diffs)
• gui/io/ChangesetManagementPanelTest.java (modified) (1 diff)

trunk/test/unit/org/openstreetmap/josm/data/osm/ChangesetCacheTest.java

@@ -19 +19 @@
 import org.openstreetmap.josm.data.UserIdentityManager;
 import org.openstreetmap.josm.testutils.JOSMTestRules;
+import org.openstreetmap.josm.testutils.annotations.BasicPreferences;
 import org.openstreetmap.josm.tools.Logging;
 
@@ -26 +27 @@
  * Unit test of {@link ChangesetCache}
  */
+@BasicPreferences
 class ChangesetCacheTest {
 

trunk/test/unit/org/openstreetmap/josm/gui/io/ChangesetManagementPanelTest.java

@@ -18 +18 @@
     @Test
     void testChangesetManagementPanel() {
-        assertNotNull(new ChangesetManagementPanel(new UploadDialogModel()));
+        assertNotNull(new ChangesetManagementPanel());
     }
 }