From 232fbe1d9ad162bef49b695d035ac31bd4d3df4f Mon Sep 17 00:00:00 2001
From: Axel Uhl <axel.uhl@sap.com>
Date: Tue, 29 Nov 2016 09:34:05 +0100
Subject: [PATCH] moved ColorMapTest out of the never executed gwt.ui.test
 project into com.sap.sailing.domain.test

Change-Id: I0a0962fa8b32182eb29f9107325d6f1e44c338e0
---
 .../sap/sailing/domain/common/impl/ColorMapImpl.java   | 10 ++++------
 .../sap/sailing/domain/common}/test/ColorMapTest.java  |  2 +-
 2 files changed, 5 insertions(+), 7 deletions(-)
 rename java/{com.sap.sailing.gwt.ui.test/src/com/sap/sailing/gwt/ui => com.sap.sailing.domain.test/src/com/sap/sailing/domain/common}/test/ColorMapTest.java (95%)

diff --git a/java/com.sap.sailing.domain.common/src/com/sap/sailing/domain/common/impl/ColorMapImpl.java b/java/com.sap.sailing.domain.common/src/com/sap/sailing/domain/common/impl/ColorMapImpl.java
index 0955c87a368..7faf8fc71a8 100755
--- a/java/com.sap.sailing.domain.common/src/com/sap/sailing/domain/common/impl/ColorMapImpl.java
+++ b/java/com.sap.sailing.domain.common/src/com/sap/sailing/domain/common/impl/ColorMapImpl.java
@@ -55,10 +55,6 @@ public class ColorMapImpl<T> implements ColorMap<T> {
     /** a list of already used colors which should be excluded from the automatic color assignment */
     private List<HSVColor> blockedColors;
 
-    // public ColorMapImpl() {
-    // this(new ArrayList<Color>());
-    // }
-
     public ColorMapImpl(List<Color> initialBlockedColors) {
         baseColors = new HSVColor[11];
         idColor = new HashMap<>();
@@ -122,9 +118,11 @@ public class ColorMapImpl<T> implements ColorMap<T> {
     }
 
     public boolean addBlockedColor(Color color) {
-        boolean result = false;
-        if(color != null && !blockedColors.contains(color)) {
+        final boolean result;
+        if (color != null && !blockedColors.contains(color)) {
             result = blockedColors.add(convertFromColorToHSV(color));
+        } else {
+            result = false;
         }
         return result;
     }
diff --git a/java/com.sap.sailing.gwt.ui.test/src/com/sap/sailing/gwt/ui/test/ColorMapTest.java b/java/com.sap.sailing.domain.test/src/com/sap/sailing/domain/common/test/ColorMapTest.java
similarity index 95%
rename from java/com.sap.sailing.gwt.ui.test/src/com/sap/sailing/gwt/ui/test/ColorMapTest.java
rename to java/com.sap.sailing.domain.test/src/com/sap/sailing/domain/common/test/ColorMapTest.java
index 59a72d1f66f..f7299635f09 100644
--- a/java/com.sap.sailing.gwt.ui.test/src/com/sap/sailing/gwt/ui/test/ColorMapTest.java
+++ b/java/com.sap.sailing.domain.test/src/com/sap/sailing/domain/common/test/ColorMapTest.java
@@ -1,4 +1,4 @@
-package com.sap.sailing.gwt.ui.test;
+package com.sap.sailing.domain.common.test;
 
 import static org.junit.Assert.assertEquals;