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

Timestamp:

2017-05-19T20:56:36+02:00 (8 years ago)

Author:

Don-vip

Message:

see #14817 - better assertion error message, include preset source url

File:

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

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

@@ -6 +6 @@
 
 import java.io.IOException;
-import java.util.ArrayList;
 import java.util.Collection;
+import java.util.HashMap;
 import java.util.HashSet;
+import java.util.Map;
 import java.util.Set;
 
@@ -58 +59 @@
         Main.pref.putInteger("socket.timeout.connect", 30);
         Main.pref.putInteger("socket.timeout.read", 60);
-        Collection<Throwable> allErrors = new ArrayList<>();
+        Map<Object, Throwable> allErrors = new HashMap<>();
         Set<String> allMessages = new HashSet<>();
         for (ExtendedSourceEntry source : sources) {
@@ -73 +74 @@
                     // ignore frequent network errors with www.freietonne.de causing too much Jenkins failures
                     if (!source.url.contains("www.freietonne.de")) {
-                        allErrors.add(e1);
+                        allErrors.put(source.url, e1);
                     }
                     System.out.println(" => KO");
@@ -81 +82 @@
                 if (!source.url.contains("yopaseopor/")) {
                     // ignore https://raw.githubusercontent.com/yopaseopor/traffic_signs_preset_JOSM cause too much errors
-                    allErrors.add(e);
+                    allErrors.put(source.url, e);
                 }
                 System.out.println(" => KO");