Changeset 18128 in josm for trunk/test/unit/org/openstreetmap

Timestamp:

2021-08-07T00:40:05+02:00 (3 years ago)

Author:

Don-vip

Message:

fix #21203 - Use Object.toString() to display history combobox items (patch by marcello)

File:

• trunk/test/unit/org/openstreetmap/josm/gui/widgets/HistoryComboBoxTest.java (modified) (3 diffs)

trunk/test/unit/org/openstreetmap/josm/gui/widgets/HistoryComboBoxTest.java

@@ -4 +4 @@
 import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertThrows;
 
 import java.util.stream.Stream;
@@ -22 +21 @@
 class HistoryComboBoxTest {
     static Stream<Arguments> testNonRegression21203() {
-        return Stream.of(Arguments.of("Hello world"), Arguments.of(new AutoCompletionItem("Hello world2")));
+        return Stream.of(Arguments.of("Hello world"), Arguments.of(new AutoCompletionItem("Hello world2")), Arguments.of(new Double(42)));
     }
 
@@ -39 +38 @@
     }
 
-    /**
-     * This ensures that we do throw on unknown objects for #21203
-     */
     @Test
-    void testNonRegression21203Throws() {
+    void testEmptyEditor() {
         final HistoryComboBox historyComboBox = new HistoryComboBox();
-        // Sanity check
-        assertEquals(0, historyComboBox.getModel().getSize());
-        historyComboBox.getEditor().setItem(new Object());
-        IllegalArgumentException illegalArgumentException = assertThrows(IllegalArgumentException.class,
-                historyComboBox::addCurrentItemToHistory);
-        assertEquals("Object is not supported in addCurrentItemToHistory", illegalArgumentException.getMessage());
+        assertDoesNotThrow(historyComboBox::addCurrentItemToHistory);
+        historyComboBox.getEditor().setItem(null);
+        assertDoesNotThrow(historyComboBox::addCurrentItemToHistory);
     }
 }