From 0fdfca329f8ade0bad9fd9509034098fed8a9f24 Mon Sep 17 00:00:00 2001 From: Taka Cho Date: Tue, 16 Jun 2020 14:08:33 -0400 Subject: m2/appclcm sonar issue fix swap the arguments use assertNull Issue-ID: POLICY-2616 Change-Id: Id5bd0b7acd0914566dfdebdb42fdcaabf9eb3012 Signed-off-by: Taka Cho --- .../test/java/appclcm/AppcLcmOperationTest.java | 24 +++++++++++----------- 1 file changed, 12 insertions(+), 12 deletions(-) (limited to 'controlloop/m2') 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 -- cgit 1.2.3-korg