diff options
Diffstat (limited to 'models-interactions/model-actors')
5 files changed, 36 insertions, 49 deletions
diff --git a/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/AppcOperation.java b/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/AppcOperation.java index b8ec634e3..ffdbde831 100644 --- a/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/AppcOperation.java +++ b/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/AppcOperation.java @@ -3,6 +3,7 @@ * ONAP * ================================================================================ * Copyright (C) 2020-2021 AT&T Intellectual Property. All rights reserved. + * Modifications Copyright (C) 2023 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -74,11 +75,10 @@ public abstract class AppcOperation extends BidirectionalTopicOperation<Request, * Makes a request, given the target VNF. This is a support function for * {@link #makeRequest(int)}. * - * @param attempt attempt number * @param targetVnf target VNF * @return a new request */ - protected Request makeRequest(int attempt, GenericVnf targetVnf) { + protected Request makeRequest(GenericVnf targetVnf) { var request = new Request(); request.setCommonHeader(new CommonHeader()); request.getCommonHeader().setRequestId(params.getRequestId()); @@ -145,19 +145,13 @@ public abstract class AppcOperation extends BidirectionalTopicOperation<Request, "unknown APPC-C response status code: " + response.getStatus().getCode()); } - switch (code) { - case SUCCESS: - return Status.SUCCESS; - case FAILURE: - return Status.FAILURE; - case ERROR: - case REJECT: - throw new IllegalArgumentException("APP-C request was not accepted, code=" + code); - case ACCEPT: - default: - // awaiting a "final" response - return Status.STILL_WAITING; - } + return switch (code) { + case SUCCESS -> Status.SUCCESS; + case FAILURE -> Status.FAILURE; + case ERROR, REJECT -> throw new IllegalArgumentException("APP-C request was not accepted, code=" + code); + // awaiting a "final" response + default -> Status.STILL_WAITING; + }; } /** diff --git a/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/ModifyConfigOperation.java b/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/ModifyConfigOperation.java index 524c797b6..0cf004935 100644 --- a/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/ModifyConfigOperation.java +++ b/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/ModifyConfigOperation.java @@ -3,6 +3,7 @@ * ONAP * ================================================================================ * Copyright (C) 2020 AT&T Intellectual Property. All rights reserved. + * Modifications Copyright (C) 2023 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -43,6 +44,6 @@ public class ModifyConfigOperation extends AppcOperation { @Override protected Request makeRequest(int attempt) { - return makeRequest(attempt, getRequiredProperty(OperationProperties.AAI_RESOURCE_VNF, "resource VNF")); + return makeRequest(getRequiredProperty(OperationProperties.AAI_RESOURCE_VNF, "resource VNF")); } } diff --git a/models-interactions/model-actors/actor.appc/src/test/java/org/onap/policy/controlloop/actor/appc/AppcOperationTest.java b/models-interactions/model-actors/actor.appc/src/test/java/org/onap/policy/controlloop/actor/appc/AppcOperationTest.java index d40fa025d..b6e9b34e7 100644 --- a/models-interactions/model-actors/actor.appc/src/test/java/org/onap/policy/controlloop/actor/appc/AppcOperationTest.java +++ b/models-interactions/model-actors/actor.appc/src/test/java/org/onap/policy/controlloop/actor/appc/AppcOperationTest.java @@ -90,7 +90,7 @@ public class AppcOperationTest extends BasicAppcOperation { String subreq = oper.getSubRequestId(); assertNotNull(subreq); - Request request = oper.makeRequest(2, genvnf); + Request request = oper.makeRequest(genvnf); assertEquals(DEFAULT_OPERATION, request.getAction()); assertNotNull(request.getPayload()); @@ -101,18 +101,18 @@ public class AppcOperationTest extends BasicAppcOperation { assertEquals(subreq, header.getSubRequestId()); - request = oper.makeRequest(2, genvnf); + request = oper.makeRequest(genvnf); assertEquals(subreq, request.getCommonHeader().getSubRequestId()); // repeat using a null payload params = params.toBuilder().payload(null).build(); oper = new MyOper(params, config); - assertEquals(Map.of(AppcOperation.VNF_ID_KEY, MY_VNF), oper.makeRequest(2, genvnf).getPayload()); + assertEquals(Map.of(AppcOperation.VNF_ID_KEY, MY_VNF), oper.makeRequest(genvnf).getPayload()); } @Test public void testConvertPayload() { - Request request = oper.makeRequest(2, genvnf); + Request request = oper.makeRequest(genvnf); // @formatter:off assertEquals( @@ -132,7 +132,7 @@ public class AppcOperationTest extends BasicAppcOperation { params = params.toBuilder().payload(payload).build(); oper = new MyOper(params, config); - request = oper.makeRequest(2, genvnf); + request = oper.makeRequest(genvnf); // @formatter:off assertEquals( @@ -153,7 +153,7 @@ public class AppcOperationTest extends BasicAppcOperation { params = params.toBuilder().payload(payload).build(); oper = new MyOper(params, config); - request = oper.makeRequest(2, genvnf); + request = oper.makeRequest(genvnf); payload.put(AppcOperation.VNF_ID_KEY, MY_VNF); payload.put(KEY1, "abc"); @@ -166,7 +166,7 @@ public class AppcOperationTest extends BasicAppcOperation { @Test public void testGetExpectedKeyValues() { oper.generateSubRequestId(2); - Request request = oper.makeRequest(2, genvnf); + Request request = oper.makeRequest(genvnf); assertEquals(Arrays.asList(request.getCommonHeader().getSubRequestId()), oper.getExpectedKeyValues(50, request)); } @@ -239,7 +239,7 @@ public class AppcOperationTest extends BasicAppcOperation { @Override protected Request makeRequest(int attempt) { - return makeRequest(attempt, genvnf); + return makeRequest(genvnf); } } } diff --git a/models-interactions/model-actors/actor.appclcm/src/main/java/org/onap/policy/controlloop/actor/appclcm/AppcLcmOperation.java b/models-interactions/model-actors/actor.appclcm/src/main/java/org/onap/policy/controlloop/actor/appclcm/AppcLcmOperation.java index 5d256e8ae..4e8f59008 100644 --- a/models-interactions/model-actors/actor.appclcm/src/main/java/org/onap/policy/controlloop/actor/appclcm/AppcLcmOperation.java +++ b/models-interactions/model-actors/actor.appclcm/src/main/java/org/onap/policy/controlloop/actor/appclcm/AppcLcmOperation.java @@ -3,6 +3,7 @@ * ONAP * ================================================================================ * Copyright (C) 2020-2021 AT&T Intellectual Property. All rights reserved. + * Modifications Copyright (C) 2023 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -150,18 +151,13 @@ public class AppcLcmOperation extends BidirectionalTopicOperation<AppcLcmDmaapWr throw new IllegalArgumentException("unknown APPC-LCM response status code: " + status.getCode()); } - switch (code) { - case AppcLcmResponseCode.SUCCESS: - return Status.SUCCESS; - case AppcLcmResponseCode.FAILURE: - return Status.FAILURE; - case AppcLcmResponseCode.ERROR: - case AppcLcmResponseCode.REJECT: + return switch (code) { + case AppcLcmResponseCode.SUCCESS -> Status.SUCCESS; + case AppcLcmResponseCode.FAILURE -> Status.FAILURE; + case AppcLcmResponseCode.ERROR, AppcLcmResponseCode.REJECT -> throw new IllegalArgumentException("APPC-LCM request was not accepted, code=" + code); - case AppcLcmResponseCode.ACCEPTED: - default: - return Status.STILL_WAITING; - } + default -> Status.STILL_WAITING; + }; } /** diff --git a/models-interactions/model-actors/actor.sdnr/src/main/java/org/onap/policy/controlloop/actor/sdnr/SdnrOperation.java b/models-interactions/model-actors/actor.sdnr/src/main/java/org/onap/policy/controlloop/actor/sdnr/SdnrOperation.java index 5b30c673a..7e57bcccd 100644 --- a/models-interactions/model-actors/actor.sdnr/src/main/java/org/onap/policy/controlloop/actor/sdnr/SdnrOperation.java +++ b/models-interactions/model-actors/actor.sdnr/src/main/java/org/onap/policy/controlloop/actor/sdnr/SdnrOperation.java @@ -3,6 +3,7 @@ * SdnrOperation * ================================================================================ * Copyright (C) 2020-2021 AT&T Intellectual Property. All rights reserved. + * Modifications Copyright (C) 2023 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -90,22 +91,17 @@ public class SdnrOperation extends BidirectionalTopicOperation<PciMessage, PciMe return Status.FAILURE; } - switch (code) { - case SUCCESS: - case PARTIAL_SUCCESS: - return Status.SUCCESS; - case FAILURE: - case PARTIAL_FAILURE: - return Status.FAILURE; - case ERROR: - case REJECT: + return switch (code) { + case SUCCESS, PARTIAL_SUCCESS -> Status.SUCCESS; + case FAILURE, PARTIAL_FAILURE -> Status.FAILURE; + case ERROR, REJECT -> { logger.warn("SDNR request was not accepted, code={}", code); - return Status.FAILURE; - case ACCEPTED: - default: + yield Status.FAILURE; + } + default -> // awaiting a "final" response - return Status.STILL_WAITING; - } + Status.STILL_WAITING; + }; } /** |