From ac97522c6eca08a4dd9c4b2486e07b89fb5d9d99 Mon Sep 17 00:00:00 2001 From: vidhyasree Date: Fri, 12 Jan 2018 15:23:34 -0500 Subject: Fix for resumetraffic to allow traffic ResumeTraffic action is added to LCM actions prior. In this ticket payload structure is added to resumetraffic in yang file. In ResumeTrafficService.java and ResumeTrafficServiceTest.java added payload parameters. Issue-ID: APPC-346 Signed-off-by: vidhyasree Change-Id: I9a3ca163bbd520e599416d03edc921a6cafb3059 --- .../lcm/service/ResumeTrafficServiceTest.java | 34 ++++++++++++++++++---- 1 file changed, 29 insertions(+), 5 deletions(-) (limited to 'appc-provider/appc-provider-bundle/src/test/java/org/onap') diff --git a/appc-provider/appc-provider-bundle/src/test/java/org/onap/appc/provider/lcm/service/ResumeTrafficServiceTest.java b/appc-provider/appc-provider-bundle/src/test/java/org/onap/appc/provider/lcm/service/ResumeTrafficServiceTest.java index 2ea7ee0d2..1372b7370 100644 --- a/appc-provider/appc-provider-bundle/src/test/java/org/onap/appc/provider/lcm/service/ResumeTrafficServiceTest.java +++ b/appc-provider/appc-provider-bundle/src/test/java/org/onap/appc/provider/lcm/service/ResumeTrafficServiceTest.java @@ -51,10 +51,11 @@ import static org.powermock.api.mockito.PowerMockito.whenNew; @PrepareForTest({ResumeTrafficService.class, RequestExecutor.class}) public class ResumeTrafficServiceTest { private final Action myAction = Action.ResumeTraffic; - + private final String PAYLOAD_STRING = "{\"A\":\"A-value\",\"B\":{\"C\":\"B.C-value\",\"D\":\"B.D-value\"}}"; private ResumeTrafficInput mockInput = mock(ResumeTrafficInput.class); private CommonHeader mockCommonHeader = mock(CommonHeader.class); private ActionIdentifiers mockAI = mock(ActionIdentifiers.class); + private Payload mockPayload = mock(Payload.class); private ResumeTrafficService resumeServiceAction; @Before @@ -126,7 +127,7 @@ public class ResumeTrafficServiceTest { @Test public void testValidate() throws Exception { - resumeServiceAction.validate(mockCommonHeader, Action.ResumeTraffic, mockAI); + resumeServiceAction.validate(mockCommonHeader, Action.ResumeTraffic, mockAI,mockPayload); Status status = (Status) Whitebox.getInternalState(resumeServiceAction, "status"); Assert.assertEquals("should return missing parameter", Integer.valueOf(LCMCommandStatus.MISSING_MANDATORY_PARAMETER.getResponseCode()), status.getCode()); @@ -140,23 +141,46 @@ public class ResumeTrafficServiceTest { Mockito.doReturn("request Id").when(mockCommonHeader).getRequestId(); // test empty action - resumeServiceAction.validate(mockCommonHeader, Action.ResumeTraffic, mockAI); + resumeServiceAction.validate(mockCommonHeader, Action.ResumeTraffic, mockAI,mockPayload); status = (Status) Whitebox.getInternalState(resumeServiceAction, "status"); Assert.assertEquals("Should return missing parameter for action", Integer.valueOf(LCMCommandStatus.MISSING_MANDATORY_PARAMETER.getResponseCode()), status.getCode()); // test empty ActionIdentifier - resumeServiceAction.validate(mockCommonHeader, Action.ResumeTraffic, mockAI); + resumeServiceAction.validate(mockCommonHeader, Action.ResumeTraffic, mockAI,mockPayload); status = (Status) Whitebox.getInternalState(resumeServiceAction, "status"); Assert.assertEquals("should return missing parameter", Integer.valueOf(LCMCommandStatus.MISSING_MANDATORY_PARAMETER.getResponseCode()), status.getCode()); // test Invalid VNF_ID Mockito.doReturn("").when(mockAI).getVnfId(); - resumeServiceAction.validate(mockCommonHeader, Action.ResumeTraffic, mockAI); + resumeServiceAction.validate(mockCommonHeader, Action.ResumeTraffic, mockAI,mockPayload); + status = (Status) Whitebox.getInternalState(resumeServiceAction, "status"); + Assert.assertEquals("should return invalid parameter", + Integer.valueOf(LCMCommandStatus.INVALID_INPUT_PARAMETER.getResponseCode()), status.getCode()); + // test null payload + Mockito.doReturn("vnfId").when(mockAI).getVnfId(); + resumeServiceAction.validate(mockCommonHeader, Action.ResumeTraffic, mockAI, null); + Mockito.verify(resumeServiceAction, times(1)).validateExcludedActIds(any(), any()); + status = (Status) Whitebox.getInternalState(resumeServiceAction, "status"); + Assert.assertEquals("should return missing parameter", + Integer.valueOf(LCMCommandStatus.MISSING_MANDATORY_PARAMETER.getResponseCode()), status.getCode()); + + // test empty payload + + Mockito.doReturn("").when(mockPayload).getValue(); + resumeServiceAction.validate(mockCommonHeader, Action.ResumeTraffic, mockAI, mockPayload); + status = (Status) Whitebox.getInternalState(resumeServiceAction, "status"); + Assert.assertEquals("should return invalid parameter", + Integer.valueOf(LCMCommandStatus.INVALID_INPUT_PARAMETER.getResponseCode()), status.getCode()); + + // test space payload + Mockito.doReturn(" ").when(mockPayload).getValue(); + resumeServiceAction.validate(mockCommonHeader, Action.ResumeTraffic, mockAI, mockPayload); status = (Status) Whitebox.getInternalState(resumeServiceAction, "status"); Assert.assertEquals("should return invalid parameter", Integer.valueOf(LCMCommandStatus.INVALID_INPUT_PARAMETER.getResponseCode()), status.getCode()); + } } -- cgit 1.2.3-korg