Changeset 11698 in josm for trunk/test/unit/org/openstreetmap/josm/gui/mappaint/MapCSSRendererTest.java

Timestamp:

2017-03-08T16:07:36+01:00 (7 years ago)

Author:

michael2402

Message:

Add test that tests if elements are ordered correctly.

File:

• trunk/test/unit/org/openstreetmap/josm/gui/mappaint/MapCSSRendererTest.java (modified) (7 diffs)

trunk/test/unit/org/openstreetmap/josm/gui/mappaint/MapCSSRendererTest.java

@@ -34 +34 @@
 import org.openstreetmap.josm.data.Bounds;
 import org.openstreetmap.josm.data.osm.DataSet;
+import org.openstreetmap.josm.data.osm.OsmPrimitive;
 import org.openstreetmap.josm.data.osm.visitor.paint.StyledMapRenderer;
 import org.openstreetmap.josm.gui.NavigatableComponent;
@@ -70 +71 @@
     /**
      * The different configurations of this test.
+     *
      * @return The parameters.
      */
@@ -88 +90 @@
 
                 /** Tests dashed ways. */
-                new TestConfig("way-dashes", AREA_DEFAULT)
+                new TestConfig("way-dashes", AREA_DEFAULT),
+
+                /** Tests if all styles are sorted correctly. Tests {@link StyleRecord#compareTo(StyleRecord)} */
+                new TestConfig("order", AREA_DEFAULT)
 
                 ).map(e -> new Object[] {e, e.testDirectory})
@@ -113 +118 @@
 
         List<String> fonts = Arrays.asList(GraphicsEnvironment.getLocalGraphicsEnvironment().getAvailableFontFamilyNames());
-        for(String font : testConfig.fonts) {
+        for (String font : testConfig.fonts) {
             Assume.assumeTrue("Test requires font: " + font, fonts.contains(font));
         }
@@ -138 +143 @@
             }
             MapPaintStyles.getStyles().setStyleSources(Arrays.asList(source));
+            MapPaintStyles.fireMapPaintSylesUpdated();
+            MapPaintStyles.getStyles().clearCached();
 
         } finally {
@@ -162 +169 @@
         };
         nc.zoomTo(testConfig.testArea);
-        dataSet.allPrimitives().stream().forEach(n -> n.setHighlighted(n.isKeyTrue("highlight")));
-        dataSet.allPrimitives().stream().filter(n -> n.isKeyTrue("disabled")).forEach(n -> n.setDisabledState(false));
+        dataSet.allPrimitives().stream().forEach(this::loadPrimitiveStyle);
+        dataSet.setSelected(dataSet.allPrimitives().stream().filter(n -> n.isKeyTrue("selected")).collect(Collectors.toList()));
+
         Graphics2D g = image.createGraphics();
         // Force all render hints to be defaults - do not use platform values
@@ -217 +225 @@
             ImageIO.write(diffImage, "png", new File(testConfig.getTestDirectory() + "/test-differences.png"));
 
-            fail(MessageFormat.format("Images for test {1} differ at {2} points: {3}",
+            fail(MessageFormat.format("Images for test {0} differ at {1} points: {2}",
                     testConfig.testDirectory, differencePoints.size(), differences.toString()));
+        }
+    }
+
+    private void loadPrimitiveStyle(OsmPrimitive n) {
+        n.setHighlighted(n.isKeyTrue("highlight"));
+        if (n.isKeyTrue("disabled")) {
+            n.setDisabledState(false);
         }
     }