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

Timestamp:

2015-10-27T20:02:56+01:00 (9 years ago)

Author:

Don-vip

Message:

improve unit test detecting invalid presets

File:

• trunk/test/unit/org/openstreetmap/josm/gui/preferences/map/TaggingPresetPreferenceTest.java (modified) (3 diffs)

trunk/test/unit/org/openstreetmap/josm/gui/preferences/map/TaggingPresetPreferenceTest.java

@@ -4 +4 @@
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
 
 import java.io.IOException;
@@ -47 +46 @@
             System.out.println(source.url);
             try {
-                Collection<TaggingPreset> presets = TaggingPresetReader.readAll(source.url, true);
-                assertFalse(presets.isEmpty());
-                System.out.println(" => OK");
-                allMessages.addAll(Main.getLastErrorAndWarnings());
-            } catch (SAXException | IOException e) {
+                testPresets(allMessages, source);
+            } catch (IOException e) {
+                try {
+                    Main.warn(e);
+                    // try again in case of temporary network error
+                    testPresets(allMessages, source);
+                } catch (SAXException | IOException e1) {
+                    e.printStackTrace();
+                    allErrors.add(e1);
+                    System.out.println(" => KO");
+                }
+            } catch (SAXException e) {
                 e.printStackTrace();
                 allErrors.add(e);
@@ -58 +64 @@
         }
         assertTrue(allErrors.isEmpty());
-        for (String message : allMessages) {
+        assertTrue(allMessages.isEmpty());
+    }
+
+    private static void testPresets(Set<String> allMessages, ExtendedSourceEntry source) throws SAXException, IOException {
+        Collection<TaggingPreset> presets = TaggingPresetReader.readAll(source.url, true);
+        assertFalse(presets.isEmpty());
+        Collection<String> errorsAndWarnings = Main.getLastErrorAndWarnings();
+        boolean error = false;
+        for (String message : errorsAndWarnings) {
             if (message.contains(TaggingPreset.PRESET_ICON_ERROR_MSG_PREFIX)) {
-                fail(message);
+                error = true;
+                allMessages.add(message);
             }
+        }
+        System.out.println(error ? " => KO" : " => OK");
+        if (error) {
+            Main.clearLastErrorAndWarnings();
         }
     }