Merge remote-tracking branch 'origin/merge/Jalview-JS/develop_feature/JAL-3690_callba...

[jalview.git] / test / jalview / util / ColorUtilsTest.java

diff --git a/test/jalview/util/ColorUtilsTest.java b/test/jalview/util/ColorUtilsTest.java
index 0acd806..9d9482c 100644 (file)
--- a/test/jalview/util/ColorUtilsTest.java
+++ b/test/jalview/util/ColorUtilsTest.java
@@ -24,13 +24,13 @@ import static org.testng.AssertJUnit.assertEquals;
 import static org.testng.AssertJUnit.assertNull;
 import static org.testng.AssertJUnit.assertSame;
 
-import jalview.gui.JvOptionPane;
-
 import java.awt.Color;
 
 import org.testng.annotations.BeforeClass;
 import org.testng.annotations.Test;
 
+import jalview.gui.JvOptionPane;
+
 public class ColorUtilsTest
 {
 
@@ -206,7 +206,7 @@ public class ColorUtilsTest
      */
     assertNull(ColorUtils.parseColourString(null));
     assertNull(ColorUtils.parseColourString("rubbish"));
-    assertEquals(Color.WHITE, ColorUtils.parseColourString("-1"));
+    assertNull(ColorUtils.parseColourString("-1"));
     assertNull(ColorUtils.parseColourString(String
             .valueOf(Integer.MAX_VALUE)));
     assertNull(ColorUtils.parseColourString("100,200,300")); // out of range
@@ -243,48 +243,4 @@ public class ColorUtilsTest
     assertEquals(new Color(184, 184, 184),
             ColorUtils.createColourFromName("HELLO HELLO HELLO "));
   }
-
-  /**
-   * Tests for the method that returns a colour graduated between darker() and
-   * brighter()
-   */
-  @Test(groups = { "Functional" })
-  public void testGetGraduatedColour_darkerToBrighter()
-  {
-    final Color colour = new Color(180, 200, 220);
-  
-    /*
-     * value half-way between min and max does _not_ mean colour unchanged
-     * darker (*.7) is (126, 140, 154)
-     * brighter (*1/.7) is (255, 255, 255)
-     * midway is (190, 197, 204)
-     */
-    Color col = ColorUtils.getGraduatedColour(20f, 10f, 30f, colour);
-    assertEquals(190, col.getRed());
-    assertEquals(197, col.getGreen());
-    assertEquals(204, col.getBlue());
-
-    // minValue (or less) returns colour.darker()
-    // - or would do if Color.darker calculated better!
-    col = ColorUtils.getGraduatedColour(10f, 10f, 30f, colour);
-    assertEquals(col, new Color(126, 140, 154));
-    // Color.darker computes 125.999999 and rounds down!
-    assertEquals(new Color(125, 140, 154), colour.darker());
-    col = ColorUtils.getGraduatedColour(-10f, 10f, 30f, colour);
-    assertEquals(new Color(126, 140, 154), col);
-
-    // maxValue (or more) returns colour.brighter()
-    col = ColorUtils.getGraduatedColour(30f, 10f, 30f, colour);
-    assertEquals(colour.brighter(), col);
-    col = ColorUtils.getGraduatedColour(40f, 10f, 30f, colour);
-    assertEquals(colour.brighter(), col);
-
-    /*
-     * 'linear' mid-point between 0.7 and 1/0.7 is 1.057
-     * so the '
-     */
-    Color c = new Color(200, 200, 200);
-    col = ColorUtils.getGraduatedColour(106f, 0f, 200f, c);
-    assertEquals(c, col);
-  }
 }