summaryrefslogtreecommitdiffstats
path: root/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api
diff options
context:
space:
mode:
authorJulienBe <julien.bertozzi@intl.att.com>2020-06-18 13:33:05 +0200
committerOfir Sonsino <ofir.sonsino@intl.att.com>2020-06-21 07:04:15 +0000
commit886ffd112569eaa02cab85fb59a5a47db52c312d (patch)
tree39a43bc378debd7e54f8b7bea9e710a625b2d9d3 /openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api
parent473c167f6589b446e8c6491dc7940c3142beb740 (diff)
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>
Diffstat (limited to 'openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api')
-rw-r--r--openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api/src/main/java/org/openecomp/core/validation/types/MessageContainer.java84
-rw-r--r--openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-api/src/test/java/org/openecomp/core/validation/types/MessageContainerTest.java69
2 files changed, 108 insertions, 45 deletions
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 977c5cc27d..eb57672315 100644
--- 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
index 0000000000..e71d9778a7
--- /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());
+ }
+}