fix sonar issues related to MessageContainer.java

Issue-ID: SDC-3127
Signed-off-by: JulienBe <julien.bertozzi@intl.att.com>
Change-Id: I02357fa3cbf7afd7845737ba01c72dc202401882
Signed-off-by: JulienBe <julien.bertozzi@intl.att.com>

diff --git a/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api/src/main/java/org/openecomp/core/validation/types/MessageContainer.java b/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api/src/main/java/org/openecomp/core/validation/types/MessageContainer.java
index 977c5cc..eb57672 100644 (file)
--- a/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api/src/main/java/org/openecomp/core/validation/types/MessageContainer.java
+++ b/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api/src/main/java/org/openecomp/core/validation/types/MessageContainer.java
@@ -7,9 +7,9 @@
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -25,59 +25,53 @@ import org.openecomp.sdc.datatypes.error.ErrorMessage;
 
 import java.util.ArrayList;
 import java.util.List;
-import java.util.function.Predicate;
-
+import java.util.stream.Collectors;
 
 public class MessageContainer {
 
-  private List<ErrorMessage> errorMessageList = new ArrayList<>();
-
-  public List<ErrorMessage> getErrorMessageList() {
-    return errorMessageList;
-  }
+    private final List<ErrorMessage> errorMessageList = new ArrayList<>();
 
-  public MessageBuilder getMessageBuilder() {
-    return new MessageBuilder();
-  }
+    public List<ErrorMessage> getErrorMessageList() {
+        return errorMessageList;
+    }
 
-  /**
-   * Gets error message list by level.
-   *
-   * @param level the level
-   * @return the error message list by level
-   */
-  public List<ErrorMessage> getErrorMessageListByLevel(ErrorLevel level) {
+    public MessageBuilder getMessageBuilder() {
+        return new MessageBuilder();
+    }
 
-    List<ErrorMessage> errors = new ArrayList<>();
-    errorMessageList.stream().filter(new Predicate<ErrorMessage>() {
-      @Override
-      public boolean test(ErrorMessage errorMessage) {
-        return errorMessage.getLevel().equals(level);
-      }
-    }).forEach(errorMessage -> errors.add(errorMessage));
-    return errors;
-  }
+    /**
+     * Gets error message list by level.
+     * Only this level, not this level and above
+     *
+     * @param level the level
+     * @return the error message list by level
+     */
+    public List<ErrorMessage> getErrorMessageListByLevel(ErrorLevel level) {
+        return errorMessageList.stream()
+                .filter(message -> message.getLevel().equals(level))
+                .collect(Collectors.toList());
+    }
 
-  public class MessageBuilder {
+    public class MessageBuilder {
 
-    String message;
-    ErrorLevel level;
+        String message;
+        ErrorLevel level;
 
-    MessageBuilder setMessage(String message) {
-      this.message = message;
-      return this;
-    }
+        MessageBuilder setMessage(String message) {
+            this.message = message;
+            return this;
+        }
 
-    MessageBuilder setLevel(ErrorLevel level) {
-      this.level = level;
-      return this;
-    }
+        MessageBuilder setLevel(ErrorLevel level) {
+            this.level = level;
+            return this;
+        }
 
-    void create() {
-      ErrorMessage errorMessage = new ErrorMessage(level, message);
-      if (!errorMessageList.contains(errorMessage)) {
-        errorMessageList.add(errorMessage);
-      }
+        void create() {
+            ErrorMessage errorMessage = new ErrorMessage(level, message);
+            if (!errorMessageList.contains(errorMessage)) {
+                errorMessageList.add(errorMessage);
+            }
+        }
     }
-  }
 }

diff --git a/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api/src/test/java/org/openecomp/core/validation/types/MessageContainerTest.java b/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api/src/test/java/org/openecomp/core/validation/types/MessageContainerTest.java
new file mode 100644 (file)
index 0000000..e71d977
--- /dev/null
+++ b/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api/src/test/java/org/openecomp/core/validation/types/MessageContainerTest.java
@@ -0,0 +1,69 @@
+/*-
+ * ============LICENSE_START=======================================================
+ * SDC
+ * ================================================================================
+ * Copyright (C) 2020 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ */
+
+package org.openecomp.core.validation.types;
+
+import static org.junit.jupiter.api.Assertions.*;
+
+import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Test;
+import org.openecomp.sdc.datatypes.error.ErrorLevel;
+
+public class MessageContainerTest {
+
+    final static MessageContainer container = new MessageContainer();
+    final static String mess1 = "Message 1";
+    final static String mess2 = "Message 2";
+    final static String mess3 = "Third Message";
+    final static String mess4 = "That's mess 4";
+
+    @BeforeAll
+    public static void setup() {
+        container.getMessageBuilder()
+                .setMessage(mess1)
+                .setLevel(ErrorLevel.ERROR).create();
+        container.getMessageBuilder()
+                .setMessage(mess2)
+                .setLevel(ErrorLevel.INFO).create();
+        container.getMessageBuilder()
+                .setMessage(mess3)
+                .setLevel(ErrorLevel.INFO).create();
+        container.getMessageBuilder()
+                .setMessage(mess4)
+                .setLevel(ErrorLevel.WARNING).create();
+    }
+
+    @Test
+    public void getErrorMessageListTest() {
+        assertEquals(4, container.getErrorMessageList().size());
+        assertEquals(mess1, container.getErrorMessageList().get(0).getMessage());
+        assertEquals(mess2, container.getErrorMessageList().get(1).getMessage());
+        assertEquals(ErrorLevel.ERROR, container.getErrorMessageList().get(0).getLevel());
+        assertEquals(ErrorLevel.INFO, container.getErrorMessageList().get(1).getLevel());
+    }
+    @Test
+    public void getErrorMessageListByLevelTest() {
+        assertEquals(1, container.getErrorMessageListByLevel(ErrorLevel.WARNING).size());
+        assertEquals(2, container.getErrorMessageListByLevel(ErrorLevel.INFO).size());
+        assertEquals(1, container.getErrorMessageListByLevel(ErrorLevel.ERROR).size());
+        assertEquals(mess1, container.getErrorMessageListByLevel(ErrorLevel.ERROR).get(0).getMessage());
+        assertEquals(ErrorLevel.ERROR, container.getErrorMessageListByLevel(ErrorLevel.ERROR).get(0).getLevel());
+    }
+}