InputBody and OutputBody unit tests 43/36243/4
authorJakub Dudycz <jakub.dudycz@nokia.com>
Fri, 16 Mar 2018 15:44:51 +0000 (16:44 +0100)
committerTakamune Cho <tc012c@att.com>
Sat, 17 Mar 2018 12:53:01 +0000 (12:53 +0000)
Improved code coverage.

Change-Id: I8b083f3281f0057d14cdfa9ae3c47eb3da99614e
Issue-ID: APPC-745
Signed-off-by: Jakub Dudycz <jakub.dudycz@nokia.com>
appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/InputBody.java
appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/OutputBody.java
appc-event-listener/appc-event-listener-bundle/src/main/java/org/onap/appc/listener/LCM/model/ResponseStatus.java
appc-event-listener/appc-event-listener-bundle/src/test/java/org/onap/appc/listener/LCM/model/InputBodyTest.java [new file with mode: 0644]
appc-event-listener/appc-event-listener-bundle/src/test/java/org/onap/appc/listener/LCM/model/OutputBodyTest.java [new file with mode: 0644]

index 85aed42..d5f157d 100644 (file)
@@ -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;
     }
index 11714e8..54eab79 100644 (file)
@@ -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());
index 114b97b..5c00295 100644 (file)
@@ -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 (file)
index 0000000..0593eb2
--- /dev/null
@@ -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 (file)
index 0000000..2d73aa5
--- /dev/null
@@ -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;
+    }
+}