summaryrefslogtreecommitdiffstats
path: root/controlloop/m2/appclcm
diff options
context:
space:
mode:
authorTaka Cho <takamune.cho@att.com>2020-06-16 14:08:33 -0400
committerTaka Cho <takamune.cho@att.com>2020-06-16 14:10:19 -0400
commit0fdfca329f8ade0bad9fd9509034098fed8a9f24 (patch)
tree6561c161cc21c1d7dd603767656eadc66d9468aa /controlloop/m2/appclcm
parenteb1d814c4e6a83222bc9a61f5f5e15d0ad85a492 (diff)
m2/appclcm sonar issue fix
swap the arguments use assertNull Issue-ID: POLICY-2616 Change-Id: Id5bd0b7acd0914566dfdebdb42fdcaabf9eb3012 Signed-off-by: Taka Cho <takamune.cho@att.com>
Diffstat (limited to 'controlloop/m2/appclcm')
-rw-r--r--controlloop/m2/appclcm/src/test/java/appclcm/AppcLcmOperationTest.java24
1 files changed, 12 insertions, 12 deletions
diff --git a/controlloop/m2/appclcm/src/test/java/appclcm/AppcLcmOperationTest.java b/controlloop/m2/appclcm/src/test/java/appclcm/AppcLcmOperationTest.java
index cd28ab408..780039978 100644
--- a/controlloop/m2/appclcm/src/test/java/appclcm/AppcLcmOperationTest.java
+++ b/controlloop/m2/appclcm/src/test/java/appclcm/AppcLcmOperationTest.java
@@ -438,7 +438,7 @@ public class AppcLcmOperationTest {
assertEquals("Start", appcRequest.getAction());
assertNotNull(appcRequest.getActionIdentifiers());
assertEquals(event.getAai().get("generic-vnf.vnf-id"), appcRequest.getActionIdentifiers().get("vnf-id"));
- assertEquals(appcRequest.getActionIdentifiers().get("vserver-id"), null);
+ assertNull(appcRequest.getActionIdentifiers().get("vserver-id"));
assertNull(appcRequest.getPayload());
logger.info("vnf start request: {}", Serialization.gson.toJson(request, AppcLcmDmaapWrapper.class));
@@ -498,7 +498,7 @@ public class AppcLcmOperationTest {
assertEquals("Stop", appcRequest.getAction());
assertNotNull(appcRequest.getActionIdentifiers());
assertEquals(event.getAai().get("generic-vnf.vnf-id"), appcRequest.getActionIdentifiers().get("vnf-id"));
- assertEquals(appcRequest.getActionIdentifiers().get("vserver-id"), null);
+ assertNull(appcRequest.getActionIdentifiers().get("vserver-id"));
assertNull(appcRequest.getPayload());
logger.info("vnf stop request: {}", Serialization.gson.toJson(request, AppcLcmDmaapWrapper.class));
@@ -553,7 +553,7 @@ public class AppcLcmOperationTest {
AppcLcmDmaapWrapper restartResponse = Serialization.gson.fromJson(lcmRespJson, AppcLcmDmaapWrapper.class);
operation.incomingMessage(restartResponse);
- assertEquals(operation.getResult(), PolicyResult.SUCCESS);
+ assertEquals(PolicyResult.SUCCESS, operation.getResult());
}
@Test
@@ -568,13 +568,13 @@ public class AppcLcmOperationTest {
/* Send in several partial success messages */
for (int i = 0; i < 5; i++) {
operation.incomingMessage(restartResponse);
- assertEquals(operation.getResult(), null);
+ assertNull(operation.getResult());
}
/* Send in an operation success */
restartResponse.getBody().getOutput().getStatus().setCode(400);
operation.incomingMessage(restartResponse);
- assertEquals(operation.getResult(), PolicyResult.SUCCESS);
+ assertEquals(PolicyResult.SUCCESS, operation.getResult());
}
@Test
@@ -587,7 +587,7 @@ public class AppcLcmOperationTest {
AppcLcmDmaapWrapper restartResponse = Serialization.gson.fromJson(lcmRespJson, AppcLcmDmaapWrapper.class);
operation.incomingMessage(restartResponse);
- assertEquals(operation.getResult(), PolicyResult.FAILURE);
+ assertEquals(PolicyResult.FAILURE, operation.getResult());
}
@Test
@@ -602,7 +602,7 @@ public class AppcLcmOperationTest {
/* Send in ALL failure messages */
for (int i = 0; i < 5; i++) {
operation.incomingMessage(restartResponse);
- assertEquals(operation.getResult(), null);
+ assertNull(operation.getResult());
}
/* Send in an operation failure */
@@ -610,7 +610,7 @@ public class AppcLcmOperationTest {
operation.incomingMessage(restartResponse);
/* Because every VM failed in the VNF, it should be failure result */
- assertEquals(operation.getResult(), PolicyResult.FAILURE);
+ assertEquals(PolicyResult.FAILURE, operation.getResult());
}
@Test
@@ -625,7 +625,7 @@ public class AppcLcmOperationTest {
/* Send in several partial success messages */
for (int i = 0; i < 5; i++) {
operation.incomingMessage(restartResponse);
- assertEquals(operation.getResult(), null);
+ assertNull(operation.getResult());
}
/* Change status to partial failure */
@@ -634,7 +634,7 @@ public class AppcLcmOperationTest {
/* Send in several partial failures messages */
for (int i = 0; i < 5; i++) {
operation.incomingMessage(restartResponse);
- assertEquals(operation.getResult(), null);
+ assertNull(operation.getResult());
}
/* Send in an operation failure */
@@ -645,7 +645,7 @@ public class AppcLcmOperationTest {
* Only a subset of VMs failed in the VNF so the
* result will be failure_exception
*/
- assertEquals(operation.getResult(), PolicyResult.FAILURE_EXCEPTION);
+ assertEquals(PolicyResult.FAILURE_EXCEPTION, operation.getResult());
}
/* ===================================================================== */
@@ -671,7 +671,7 @@ public class AppcLcmOperationTest {
VirtualControlLoopEvent noAaiTag = new VirtualControlLoopEvent();
noAaiTag.setAai(null);
assertFalse(AppcLcmOperation.isAaiValid(transaction, noAaiTag));
- assertEquals(transaction.getNotificationMessage(), "No A&AI Subtag");
+ assertEquals("No A&AI Subtag", transaction.getNotificationMessage());
}
@Test