Changeset 8937 in josm for trunk/src/org/openstreetmap/josm

Timestamp:

2015-10-24T00:54:52+02:00 (9 years ago)

Author:

Don-vip

Message:

add unit tests to check validity of all map paint styles and tagging presets

Location:

trunk/src/org/openstreetmap/josm/gui

Files:

• mappaint/MapPaintStyles.java (modified) (4 diffs)
• tagging/presets/TaggingPreset.java (modified) (3 diffs)

trunk/src/org/openstreetmap/josm/gui/mappaint/MapPaintStyles.java

@@ -11 +11 @@
 import java.util.Arrays;
 import java.util.Collection;
+import java.util.Collections;
 import java.util.HashSet;
 import java.util.LinkedList;
@@ -266 +267 @@
             }
         }
-        if (Main.isDebugEnabled()) {
+        if (Main.isDebugEnabled() || !source.getErrors().isEmpty()) {
             final long elapsedTime = System.currentTimeMillis() - startTime;
-            Main.debug("Initializing map style " + source.url + " completed in " + Utils.getDurationString(elapsedTime));
+            String message = "Initializing map style " + source.url + " completed in " + Utils.getDurationString(elapsedTime);
+            if (!source.getErrors().isEmpty()) {
+                Main.warn(message + " (" + source.getErrors().size() + " errors)");
+            } else {
+                Main.debug(message);
+            }
         }
     }
@@ -433 +439 @@
     }
 
-    public static void addStyle(SourceEntry entry) {
+    /**
+     * Add a new map paint style.
+     * @param entry map paint style
+     * @return list of errors that occured during loading
+     */
+    public static Collection<Throwable> addStyle(SourceEntry entry) {
         StyleSource source = fromSourceEntry(entry);
         if (source != null) {
@@ -441 +452 @@
             fireMapPaintSylesUpdated();
             styles.clearCached();
-            Main.map.mapView.repaint();
-        }
+            if (Main.isDisplayingMapView()) {
+                Main.map.mapView.repaint();
+            }
+            return source.getErrors();
+        }
+        return Collections.emptyList();
     }
 

trunk/src/org/openstreetmap/josm/gui/tagging/presets/TaggingPreset.java

@@ -83 +83 @@
     public static final String OPTIONAL_TOOLTIP_TEXT = "Optional tooltip text";
 
+    /** Prefix of preset icon loading failure error message */
+    public static final String PRESET_ICON_ERROR_MSG_PREFIX = "Could not get presets icon ";
+
     public TaggingPresetMenu group;
     public String name;
@@ -166 +169 @@
      * Called from the XML parser to set the icon.
      * The loading task is performed in the background in order to speedup startup.
+     * @param iconName icon name
      */
     public void setIcon(final String iconName) {
@@ -190 +194 @@
                     });
                 } else {
-                    Main.warn("Could not get presets icon " + iconName);
+                    Main.warn(PRESET_ICON_ERROR_MSG_PREFIX + iconName);
                 }
             }