Index: trunk/test/unit/org/openstreetmap/josm/gui/dialogs/properties/PropertiesCellRendererTest.java =================================================================== --- trunk/test/unit/org/openstreetmap/josm/gui/dialogs/properties/PropertiesCellRendererTest.java (revision 16319) +++ trunk/test/unit/org/openstreetmap/josm/gui/dialogs/properties/PropertiesCellRendererTest.java (revision 16319) @@ -0,0 +1,55 @@ +// License: GPL. For details, see LICENSE file. +package org.openstreetmap.josm.gui.dialogs.properties; + +import static org.junit.Assert.assertEquals; + +import java.util.function.IntFunction; + +import javax.swing.JLabel; +import javax.swing.JTable; +import javax.swing.table.DefaultTableModel; + +import org.junit.Rule; +import org.junit.Test; +import org.openstreetmap.josm.testutils.JOSMTestRules; + +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; + +/** + * Unit tests of {@link PropertiesCellRenderer} class. + */ +public class PropertiesCellRendererTest { + + /** + * Setup test. + */ + @Rule + @SuppressFBWarnings(value = "URF_UNREAD_PUBLIC_OR_PROTECTED_FIELD") + public JOSMTestRules test = new JOSMTestRules().preferences(); + + /** + * Test of color rendering. + */ + @Test + public void testColorRendering() { + PropertiesCellRenderer renderer = new PropertiesCellRenderer(); + DefaultTableModel tableModel = new DefaultTableModel(new Object[][]{ + new Object[]{"colour", "red"}, + new Object[]{"colour", "green"}, + new Object[]{"colour", "#123"}, + new Object[]{"colour", "#123456"}, + new Object[]{"colour", "foobarbaz"}, + new Object[]{"elevation", "314159"}, + }, new Object[]{"key", "value"}); + JTable table = new JTable(tableModel); + IntFunction getLabel = row -> ((JLabel) renderer.getTableCellRendererComponent( + table, table.getValueAt(row, 1), false, false, row, 1)).getText(); + assertEquals("\u25A0 red", getLabel.apply(0)); + assertEquals("\u25A0 green", getLabel.apply(1)); + assertEquals("\u25A0 #123", getLabel.apply(2)); + assertEquals("\u25A0 #123456", getLabel.apply(3)); + assertEquals("foobarbaz", getLabel.apply(4)); + assertEquals("314159", getLabel.apply(5)); + } + +} Index: trunk/test/unit/org/openstreetmap/josm/tools/ColorHelperTest.java =================================================================== --- trunk/test/unit/org/openstreetmap/josm/tools/ColorHelperTest.java (revision 16311) +++ trunk/test/unit/org/openstreetmap/josm/tools/ColorHelperTest.java (revision 16319) @@ -26,7 +26,4 @@ assertEquals(Color.CYAN, ColorHelper.html2color("#00ffff")); assertEquals(Color.CYAN, ColorHelper.html2color("#00FFFF")); - assertEquals(Color.CYAN, ColorHelper.html2color("cyan")); - assertEquals(new Color(0xa52a2a), ColorHelper.html2color("brown")); - assertEquals(new Color(0x6495ed), ColorHelper.html2color("cornflowerblue")); assertEquals(new Color(0x12345678, true), ColorHelper.html2color("#34567812")); }