aboutsummaryrefslogtreecommitdiffstats
path: root/appc-event-listener/appc-event-listener-bundle
diff options
context:
space:
mode:
authorJakub Dudycz <jakub.dudycz@nokia.com>2018-03-16 16:44:51 +0100
committerTakamune Cho <tc012c@att.com>2018-03-17 12:53:01 +0000
commitf74c992c1ad8471303df62066ab5f9d778825e30 (patch)
tree70f8b2ce79b301b0ea6edba3029f1f3c60507161 /appc-event-listener/appc-event-listener-bundle
parentfc6964a2908732641ac2e1be37eb9d2a429d09f9 (diff)
InputBody and OutputBody unit tests
Improved code coverage. Change-Id: I8b083f3281f0057d14cdfa9ae3c47eb3da99614e Issue-ID: APPC-745 Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
Diffstat (limited to 'appc-event-listener/appc-event-listener-bundle')
-rw-r--r--appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/InputBody.java25
-rw-r--r--appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/OutputBody.java32
-rw-r--r--appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/ResponseStatus.java16
-rw-r--r--appc-event-listener/appc-event-listener-bundle/src/test/java/org/onap/appc/listener/LCM/model/InputBodyTest.java98
-rw-r--r--appc-event-listener/appc-event-listener-bundle/src/test/java/org/onap/appc/listener/LCM/model/OutputBodyTest.java113
5 files changed, 248 insertions, 36 deletions
diff --git a/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/InputBody.java b/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/InputBody.java
index 85aed42d8..d5f157d8d 100644
--- a/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/InputBody.java
+++ b/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/InputBody.java
@@ -50,34 +50,35 @@ public class InputBody {
return payload;
}
- public void setPayload(Object payload) {
- this.payload = payload;
+ public String getAction() {
+ return action;
}
+
+ public ActionIdentifiers getActionIdentifiers() {
+ return actionIdentifiers;
+ }
+
+ public CommonHeader getCommonHeader() {
+ return commonHeader;
+ }
+
@JsonIgnore
public void setPayloadAsString(String payload) {
this.payload = payload;
}
- public String getAction() {
- return action;
+ public void setPayload(Object payload) {
+ this.payload = payload;
}
public void setAction(String action) {
this.action = action;
}
- public ActionIdentifiers getActionIdentifiers() {
- return actionIdentifiers;
- }
-
public void setActionIdentifiers(ActionIdentifiers actionIdentifiers) {
this.actionIdentifiers = actionIdentifiers;
}
- public CommonHeader getCommonHeader() {
- return commonHeader;
- }
-
public void setCommonHeader(CommonHeader commonHeader) {
this.commonHeader = commonHeader;
}
diff --git a/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/OutputBody.java b/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/OutputBody.java
index 11714e8fb..54eab7914 100644
--- a/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/OutputBody.java
+++ b/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/OutputBody.java
@@ -32,7 +32,6 @@ import org.onap.appc.listener.util.Mapper;
/**
* This class represents a message being sent out to DMaaP by APPC to update listeners on the status of a request
- *
*/
@JsonSerialize(include = JsonSerialize.Inclusion.NON_NULL)
@JsonIgnoreProperties(ignoreUnknown = true)
@@ -53,38 +52,27 @@ public class OutputBody {
public OutputBody() {
}
+ public OutputBody(InputBody msg) {
+ this.header = new CommonHeader(msg.getCommonHeader());
+ }
+
public JSONObject toResponse() {
return Mapper.toJsonObject(this);
}
- public OutputBody(InputBody msg) {
- this.header = new CommonHeader(msg.getCommonHeader());
- }
-
public String getLocked() {
return locked;
}
- public void setLocked(String locked) {
- this.locked = locked;
- }
public String getPayload() {
return payload;
}
- public void setPayload(String payload) {
- this.payload = payload;
- }
-
public CommonHeader getHeader() {
return header;
}
- public void setHeader(CommonHeader header) {
- this.header = header;
- }
-
public ResponseStatus getStatus() {
return status;
}
@@ -93,6 +81,18 @@ public class OutputBody {
this.status = status;
}
+ public void setLocked(String locked) {
+ this.locked = locked;
+ }
+
+ public void setHeader(CommonHeader header) {
+ this.header = header;
+ }
+
+ public void setPayload(String payload) {
+ this.payload = payload;
+ }
+
@Override
public String toString() {
return String.format("%s - %s(%s)", getHeader().getRequestID(), getStatus().getCode(), getStatus().getValue());
diff --git a/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/ResponseStatus.java b/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/ResponseStatus.java
index 114b97b35..5c00295e4 100644
--- a/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/ResponseStatus.java
+++ b/appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/ResponseStatus.java
@@ -32,6 +32,7 @@ import com.fasterxml.jackson.databind.annotation.JsonSerialize;
@JsonSerialize(include = JsonSerialize.Inclusion.NON_NULL)
@JsonIgnoreProperties(ignoreUnknown = true)
public class ResponseStatus {
+
@JsonProperty("code")
private Integer code;
@JsonProperty("message")
@@ -49,25 +50,24 @@ public class ResponseStatus {
return code;
}
- public void setCode(Integer code) {
- code = code;
- }
public String getValue() {
return value;
}
-
+ public void setCode(Integer code) {
+ this.code = code;
+ }
public void setValue(String value) {
- value = value;
+ this.value = value;
}
@Override
public String toString() {
return "ResponseStatus{" +
- "code=" + code +
- ", value='" + value + '\'' +
- '}';
+ "code=" + code +
+ ", value='" + value + '\'' +
+ '}';
}
}
diff --git a/appc-event-listener/appc-event-listener-bundle/src/test/java/org/onap/appc/listener/LCM/model/InputBodyTest.java b/appc-event-listener/appc-event-listener-bundle/src/test/java/org/onap/appc/listener/LCM/model/InputBodyTest.java
new file mode 100644
index 000000000..0593eb2dc
--- /dev/null
+++ b/appc-event-listener/appc-event-listener-bundle/src/test/java/org/onap/appc/listener/LCM/model/InputBodyTest.java
@@ -0,0 +1,98 @@
+/*-
+ * ============LICENSE_START=======================================================
+ * ONAP : APPC
+ * ================================================================================
+ * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * Copyright (C) 2018 Nokia Solutions and Networks
+ * =============================================================================
+ * 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.
+ *
+ * ECOMP is a trademark and service mark of AT&T Intellectual Property.
+ * ============LICENSE_END=========================================================
+ */
+package org.onap.appc.listener.LCM.model;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
+import java.util.HashMap;
+import java.util.Map;
+import org.junit.Before;
+import org.junit.Test;
+
+public class InputBodyTest {
+
+ private InputBody inputBody;
+
+ @Before
+ public void setup() {
+ inputBody = new InputBody();
+ }
+
+ @Test
+ public void should_set_properties() {
+
+ CommonHeader testCommonHeader = buildCommonHeader();
+ ActionIdentifiers testActionIdentifiers = buildActionIdentifiers();
+
+ inputBody.setCommonHeader(testCommonHeader);
+ inputBody.setActionIdentifiers(testActionIdentifiers);
+ inputBody.setAction("test-action");
+ inputBody.setPayload("{\"payload\": \"value\"");
+
+ assertEquals(testCommonHeader, inputBody.getCommonHeader());
+ assertEquals(testActionIdentifiers, inputBody.getActionIdentifiers());
+ assertEquals("test-action", inputBody.getAction());
+ assertEquals("{\"payload\": \"value\"", inputBody.getPayload());
+ }
+
+ @Test
+ public void should_verify_if_is_valid() {
+
+ assertFalse(inputBody.isValid());
+ inputBody.setCommonHeader(buildCommonHeader());
+ assertTrue(inputBody.isValid());
+ }
+
+
+ private CommonHeader buildCommonHeader() {
+
+ CommonHeader commonHeader = new CommonHeader();
+ commonHeader.setTimeStamp("test-timestamp");
+ commonHeader.setApiVer("test-api-version");
+ commonHeader.setOriginatorId("test-originator-id");
+ commonHeader.setRequestID("test-request-id");
+ commonHeader.setSubRequestId("test-subrequest-id");
+
+ Map<String, String> flags = new HashMap<>();
+ flags.put("key1", "flag1");
+ flags.put("key2", "flag2");
+ flags.put("key3", "flag3");
+
+ commonHeader.setFlags(flags);
+ return commonHeader;
+ }
+
+ private ActionIdentifiers buildActionIdentifiers() {
+
+ ActionIdentifiers actionIdentifiers = new ActionIdentifiers();
+ actionIdentifiers.setServiceInstanceId("test-instance-id");
+ actionIdentifiers.setVnfID("test-vnf-id");
+ actionIdentifiers.setVnfcName("test-name");
+ actionIdentifiers.setVserverId("test-vserver-id");
+
+ return actionIdentifiers;
+ }
+}
diff --git a/appc-event-listener/appc-event-listener-bundle/src/test/java/org/onap/appc/listener/LCM/model/OutputBodyTest.java b/appc-event-listener/appc-event-listener-bundle/src/test/java/org/onap/appc/listener/LCM/model/OutputBodyTest.java
new file mode 100644
index 000000000..2d73aa5d8
--- /dev/null
+++ b/appc-event-listener/appc-event-listener-bundle/src/test/java/org/onap/appc/listener/LCM/model/OutputBodyTest.java
@@ -0,0 +1,113 @@
+/*-
+ * ============LICENSE_START=======================================================
+ * ONAP : APPC
+ * ================================================================================
+ * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * Copyright (C) 2018 Nokia Solutions and Networks
+ * =============================================================================
+ * 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.
+ *
+ * ECOMP is a trademark and service mark of AT&T Intellectual Property.
+ * ============LICENSE_END=========================================================
+ */
+package org.onap.appc.listener.LCM.model;
+
+import static junit.framework.TestCase.assertNotNull;
+import static org.junit.Assert.assertEquals;
+
+import java.util.HashMap;
+import java.util.Map;
+import org.json.JSONObject;
+import org.junit.Before;
+import org.junit.Test;
+
+public class OutputBodyTest {
+
+ private OutputBody outputBody;
+
+ @Before
+ public void setup() {
+ outputBody = new OutputBody();
+ }
+
+ @Test
+ public void should_set_properties() {
+
+ CommonHeader testCommonHeader = buildCommonHeader();
+ ResponseStatus testResponseStatus = new ResponseStatus(200, "OK");
+
+ outputBody.setHeader(testCommonHeader);
+ outputBody.setStatus(testResponseStatus);
+ outputBody.setLocked("test-locked");
+ outputBody.setPayload("{\"payload\": \"value\"");
+
+ assertEquals(testCommonHeader, outputBody.getHeader());
+ assertEquals(testResponseStatus, outputBody.getStatus());
+ assertEquals("test-locked", outputBody.getLocked());
+ assertEquals("{\"payload\": \"value\"", outputBody.getPayload());
+ }
+
+
+ @Test
+ public void should_inherit_input_body_header_when_initialized_from_constructor() {
+
+ InputBody testInputBody = new InputBody();
+ CommonHeader testCommonHeader = buildCommonHeader();
+ testInputBody.setCommonHeader(testCommonHeader);
+
+ outputBody = new OutputBody(testInputBody);
+
+ assertNotNull(outputBody.getHeader());
+ assertEquals(testCommonHeader.getFlags(), outputBody.getHeader().getFlags());
+ assertEquals(testCommonHeader.getSubRequestId(), outputBody.getHeader().getSubRequestId());
+ assertEquals(testCommonHeader.getRequestID(), outputBody.getHeader().getRequestID());
+ assertEquals(testCommonHeader.getOriginatorId(), outputBody.getHeader().getOriginatorId());
+ assertEquals(testCommonHeader.getApiVer(), outputBody.getHeader().getApiVer());
+ }
+
+ @Test
+ public void toResponse_should_convert_to_json_object() {
+ CommonHeader testCommonHeader = buildCommonHeader();
+ ResponseStatus testResponseStatus = new ResponseStatus(200, "OK");
+
+ outputBody.setHeader(testCommonHeader);
+ outputBody.setStatus(testResponseStatus);
+ outputBody.setLocked("test-locked");
+ outputBody.setPayload("{\"payload\": \"value\"");
+
+ JSONObject response = outputBody.toResponse();
+ assertNotNull(response);
+
+ assertEquals("test-locked", response.get("locked"));
+ assertEquals("{\"payload\": \"value\"", response.get("payload"));
+ }
+
+ private CommonHeader buildCommonHeader() {
+
+ CommonHeader commonHeader = new CommonHeader();
+ commonHeader.setTimeStamp("test-timestamp");
+ commonHeader.setApiVer("test-api-version");
+ commonHeader.setOriginatorId("test-originator-id");
+ commonHeader.setRequestID("test-request-id");
+ commonHeader.setSubRequestId("test-subrequest-id");
+
+ Map<String, String> flags = new HashMap<>();
+ flags.put("key1", "flag1");
+ flags.put("key2", "flag2");
+ flags.put("key3", "flag3");
+
+ commonHeader.setFlags(flags);
+ return commonHeader;
+ }
+}