Changeset 12989 in josm for trunk/test

Timestamp:

2017-10-12T19:56:13+02:00 (7 years ago)

Author:

bastiK

Message:

closes #15410 - update tests + minor fixes

Location:

trunk/test/unit/org/openstreetmap/josm

Files:

• data/PreferencesTest.java (modified) (2 diffs)
• data/preferences/NamedColorPropertyTest.java (moved) (moved from trunk/test/unit/org/openstreetmap/josm/data/preferences/ColorPropertyTest.java ) (7 diffs)
• gui/layer/LayerTest.java (modified) (3 diffs)

trunk/test/unit/org/openstreetmap/josm/data/PreferencesTest.java

@@ -3 +3 @@
 
 import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
-
-import java.awt.Color;
 
 import org.junit.Rule;
 import org.junit.Test;
 import org.openstreetmap.josm.Main;
-import org.openstreetmap.josm.data.preferences.ColorProperty;
 import org.openstreetmap.josm.testutils.JOSMTestRules;
 
@@ -28 +24 @@
 
     /**
-     * Test color name.
-     */
-    @Test
-    public void testColorName() {
-        assertEquals("Layer: {5DE308C0-916F-4B5A-B3DB-D45E17F30172}.gpx",
-                Main.pref.getColorName("layer.{5DE308C0-916F-4B5A-B3DB-D45E17F30172}.gpx"));
-    }
-
-    /**
-     * Test color alpha.
-     */
-    @Test
-    public void testColorAlpha() {
-        assertEquals(0x12, new ColorProperty("foo", new Color(0x12345678, true)).get().getAlpha());
-        assertTrue(Main.pref.putColor("bar", new Color(0x12345678, true)));
-        assertEquals(0x12, new ColorProperty("bar", Color.RED).get().getAlpha());
-    }
-
-    /**
-     * Test color name and alpha.
-     */
-    @Test
-    public void testColorNameAlpha() {
-        assertEquals(0x12, new ColorProperty("foo", new Color(0x12345678, true)).get().getAlpha());
-        assertEquals(new Color(0x34, 0x56, 0x78, 0x12), Main.pref.getDefaultColor("foo"));
-        assertEquals(0x12, Main.pref.getDefaultColor("foo").getAlpha());
-    }
-
-    /**
      * Test {@link Preferences#toXML}.
      */

trunk/test/unit/org/openstreetmap/josm/data/preferences/NamedColorPropertyTest.java

@@ -3 +3 @@
 
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertTrue;
 
 import java.awt.Color;
+import java.util.Arrays;
 
 import org.junit.Before;
 import org.junit.Rule;
 import org.junit.Test;
+import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.spi.preferences.Config;
 import org.openstreetmap.josm.testutils.JOSMTestRules;
@@ -18 +21 @@
  * @author Michael Zangl
  */
-public class ColorPropertyTest {
+public class NamedColorPropertyTest {
     /**
      * This is a preference test.
@@ -25 +28 @@
     @SuppressFBWarnings(value = "URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD")
     public JOSMTestRules test = new JOSMTestRules().preferences();
-    private ColorProperty base;
+    private NamedColorProperty base;
 
     /**
@@ -32 +35 @@
     @Before
     public void createTestProperty() {
-        base = new ColorProperty("test", Color.RED);
+        base = new NamedColorProperty("test", Color.RED);
     }
 
@@ -41 +44 @@
     public void testGet() {
         assertEquals(Color.RED, base.get());
-
-        Config.getPref().put("color.test", "#00ff00");
-        assertEquals(new Color(0xff00ff00), base.get());
     }
 
@@ -53 +53 @@
         assertEquals(Color.RED, base.get());
 
-        base.put(new Color(0xff00ff00));
-        assertEquals(new Color(0xff00ff00), base.get());
-        assertEquals("#00ff00", Config.getPref().get("color.test").toLowerCase());
+        base.put(new Color(0xff00af00));
+        assertEquals(new Color(0xff00af00), base.get());
+        assertEquals("#00af00", Config.getPref().getList("clr.general.test").get(0).toLowerCase());
 
         base.put(null);
         assertEquals(Color.RED, base.get());
+    }
+
+    /**
+     * Test color alpha.
+     */
+    @Test
+    public void testColorAlpha() {
+        assertEquals(0x12, new NamedColorProperty("foo", new Color(0x12345678, true)).get().getAlpha());
+        assertTrue(Main.pref.putList("clr.general.bar", Arrays.asList("#34567812", "general", "", "bar")));
+        assertEquals(0x12, new NamedColorProperty("bar", Color.RED).get().getAlpha());
+    }
+
+    /**
+     * Test color name and alpha.
+     */
+    @Test
+    public void testColorNameAlpha() {
+        assertEquals(0x12, new NamedColorProperty("foo", new Color(0x12345678, true)).get().getAlpha());
     }
 
@@ -66 +84 @@
     @Test
     public void testGetChildColor() {
-        AbstractToStringProperty<Color> child = base.getChildColor("test2");
+        AbstractProperty<Color> child = base.getChildColor("test2");
 
         assertEquals(Color.RED, child.get());

trunk/test/unit/org/openstreetmap/josm/gui/layer/LayerTest.java

@@ -16 +16 @@
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.preferences.AbstractProperty;
-import org.openstreetmap.josm.data.preferences.ColorProperty;
+import org.openstreetmap.josm.data.preferences.NamedColorProperty;
 import org.openstreetmap.josm.testutils.JOSMTestRules;
 
@@ -52 +52 @@
         AbstractProperty<Color> color = new LayerManagerTest.TestLayer() {
             @Override
-            protected ColorProperty getBaseColorProperty() {
-                return new ColorProperty("x", Color.BLACK);
+            protected NamedColorProperty getBaseColorProperty() {
+                return new NamedColorProperty("x", Color.BLACK);
             }
         }.getColorProperty();
@@ -101 +101 @@
             @Override
             public AbstractProperty<Color> getColorProperty() {
-                return new ColorProperty("test", Color.RED);
+                return new NamedColorProperty("test", Color.RED);
             }
         };