From f0b1c5bf1662edf6a2a07b79c1ffe7e42e1bb054 Mon Sep 17 00:00:00 2001 From: "adheli.tavares" Date: Thu, 12 Oct 2023 13:07:28 +0100 Subject: Remove AAF references from distribution. Issue-ID: POLICY-4592 Change-Id: Ifa924a99a9429494052a1d36d4b7480b9ddce37d Signed-off-by: adheli.tavares --- .../api/LifecycleApiAutomationCompositionForwarderTest.java | 2 +- .../lifecycle/api/LifecycleApiPolicyForwarderTest.java | 4 ++-- .../LifecycleApiAutomationCompositionSimulatorMain.java | 11 +++++------ .../forwarding/testclasses/LifecycleApiSimulatorMain.java | 12 ++++++------ .../reception/handling/sdc/SdcReceptionHandler.java | 12 +++++++----- 5 files changed, 21 insertions(+), 20 deletions(-) (limited to 'plugins') diff --git a/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/lifecycle/api/LifecycleApiAutomationCompositionForwarderTest.java b/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/lifecycle/api/LifecycleApiAutomationCompositionForwarderTest.java index f28b52e5..6069b667 100644 --- a/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/lifecycle/api/LifecycleApiAutomationCompositionForwarderTest.java +++ b/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/lifecycle/api/LifecycleApiAutomationCompositionForwarderTest.java @@ -65,7 +65,7 @@ class LifecycleApiAutomationCompositionForwarderTest { "src/test/resources/parameters/LifecycleApiAutomationCompositionForwarderParameters.json", LifecycleApiAutomationCompositionForwarderParameters.class); ParameterService.register(parameterGroup); - simulator.startLifecycycleApiSimulator(); + simulator.startLifecycleApiSimulator(); if (!NetworkUtil.isTcpPortOpen("localhost", 6969, 50, 200L)) { throw new IllegalStateException("cannot connect to port 6969"); } diff --git a/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/lifecycle/api/LifecycleApiPolicyForwarderTest.java b/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/lifecycle/api/LifecycleApiPolicyForwarderTest.java index ba85505d..a78debee 100644 --- a/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/lifecycle/api/LifecycleApiPolicyForwarderTest.java +++ b/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/lifecycle/api/LifecycleApiPolicyForwarderTest.java @@ -66,7 +66,7 @@ class LifecycleApiPolicyForwarderTest { "src/test/resources/parameters/LifecycleApiPolicyForwarderParameters.json", LifecycleApiForwarderParameters.class); ParameterService.register(parameterGroup); - simulator.startLifecycycleApiSimulator(); + simulator.startLifecycleApiSimulator(); if (!NetworkUtil.isTcpPortOpen("localhost", 6969, 50, 200L)) { throw new IllegalStateException("cannot connect to port 6969"); } @@ -78,7 +78,7 @@ class LifecycleApiPolicyForwarderTest { @AfterAll static void tearDown() { ParameterService.deregister(LifecycleApiForwarderParameters.class.getSimpleName()); - simulator.stopLifecycycleApiSimulator(); + simulator.stopLifecycleApiSimulator(); } @Test diff --git a/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/testclasses/LifecycleApiAutomationCompositionSimulatorMain.java b/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/testclasses/LifecycleApiAutomationCompositionSimulatorMain.java index 98615af0..be8cc827 100644 --- a/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/testclasses/LifecycleApiAutomationCompositionSimulatorMain.java +++ b/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/testclasses/LifecycleApiAutomationCompositionSimulatorMain.java @@ -1,7 +1,6 @@ /*- * ============LICENSE_START======================================================= - * Copyright (C) 2022 Nordix Foundation. - * Modifications Copyright (C) 2022 Nordix Foundation. + * Copyright (C) 2022-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. @@ -21,6 +20,7 @@ package org.onap.policy.distribution.forwarding.testclasses; +import java.util.List; import org.onap.policy.common.endpoints.http.server.RestServer; import org.onap.policy.common.endpoints.parameters.RestServerParameters; import org.onap.policy.common.utils.coder.CoderException; @@ -28,7 +28,6 @@ import org.onap.policy.common.utils.coder.StandardCoder; import org.onap.policy.common.utils.resources.ResourceUtils; import org.onap.policy.distribution.forwarding.PolicyForwardingException; import org.onap.policy.distribution.forwarding.lifecycle.api.LifecycleApiAutomationCompositionForwarder; -import org.onap.policy.distribution.main.rest.aaf.AafDistributionFilter; /** * The class for starting/stopping simulator for testing {@link LifecycleApiAutomationCompositionForwarder} . @@ -44,13 +43,13 @@ public class LifecycleApiAutomationCompositionSimulatorMain { * @throws PolicyForwardingException if error occurs * @throws CoderException if error occurs */ - public void startLifecycycleApiSimulator() throws PolicyForwardingException, CoderException { + public void startLifecycleApiSimulator() throws PolicyForwardingException, CoderException { final StandardCoder standardCoder = new StandardCoder(); final RestServerParameters restServerParameters = standardCoder.decode( ResourceUtils.getResourceAsString("src/test/resources/parameters/RestServerParameters.json"), RestServerParameters.class); - restServer = new RestServer(restServerParameters, AafDistributionFilter.class, - LifecycleApiAutomationCompositionSimulatorEndpoint.class); + restServer = new RestServer(restServerParameters, List.of(), + List.of(LifecycleApiAutomationCompositionSimulatorEndpoint.class)); if (!restServer.start()) { throw new PolicyForwardingException("Failed to start rest simulator. Check log for more details..."); } diff --git a/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/testclasses/LifecycleApiSimulatorMain.java b/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/testclasses/LifecycleApiSimulatorMain.java index 07c4b6c6..ef89149c 100644 --- a/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/testclasses/LifecycleApiSimulatorMain.java +++ b/plugins/forwarding-plugins/src/test/java/org/onap/policy/distribution/forwarding/testclasses/LifecycleApiSimulatorMain.java @@ -1,6 +1,6 @@ /*- * ============LICENSE_START======================================================= - * Copyright (C) 2019 Nordix Foundation. + * Copyright (C) 2019, 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. @@ -20,6 +20,7 @@ package org.onap.policy.distribution.forwarding.testclasses; +import java.util.List; import org.onap.policy.common.endpoints.http.server.RestServer; import org.onap.policy.common.endpoints.parameters.RestServerParameters; import org.onap.policy.common.utils.coder.CoderException; @@ -27,7 +28,6 @@ import org.onap.policy.common.utils.coder.StandardCoder; import org.onap.policy.common.utils.resources.ResourceUtils; import org.onap.policy.distribution.forwarding.PolicyForwardingException; import org.onap.policy.distribution.forwarding.lifecycle.api.LifecycleApiPolicyForwarder; -import org.onap.policy.distribution.main.rest.aaf.AafDistributionFilter; /** * The class for starting/stopping simulator for testing {@link LifecycleApiPolicyForwarder} . @@ -43,13 +43,13 @@ public class LifecycleApiSimulatorMain { * @throws PolicyForwardingException if error occurs * @throws CoderException if error occurs */ - public void startLifecycycleApiSimulator() throws PolicyForwardingException, CoderException { + public void startLifecycleApiSimulator() throws PolicyForwardingException, CoderException { final StandardCoder standardCoder = new StandardCoder(); final RestServerParameters restServerParameters = standardCoder.decode( ResourceUtils.getResourceAsString("src/test/resources/parameters/RestServerParameters.json"), RestServerParameters.class); - restServer = new RestServer(restServerParameters, AafDistributionFilter.class, - LifecycycleApiSimulatorEndpoint.class); + restServer = new RestServer(restServerParameters, List.of(), + List.of(LifecycycleApiSimulatorEndpoint.class)); if (!restServer.start()) { throw new PolicyForwardingException("Failed to start rest simulator. Check log for more details..."); } @@ -58,7 +58,7 @@ public class LifecycleApiSimulatorMain { /** * Shut down Execution. */ - public void stopLifecycycleApiSimulator() { + public void stopLifecycleApiSimulator() { if (restServer != null) { restServer.stop(); } diff --git a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandler.java b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandler.java index ff915613..de44dc85 100644 --- a/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandler.java +++ b/plugins/reception-plugins/src/main/java/org/onap/policy/distribution/reception/handling/sdc/SdcReceptionHandler.java @@ -1,7 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2018 Ericsson. All rights reserved. - * Copyright (C) 2019, 2022 Nordix Foundation. + * Copyright (C) 2019, 2022-2023 Nordix Foundation. * Modifications Copyright (C) 2020-2021 AT&T Intellectual Property. All rights reserved. * Modifications Copyright (C) 2021 Bell Canada. All rights reserved. * ================================================================================ @@ -104,8 +104,7 @@ public class SdcReceptionHandler extends AbstractReceptionHandler implements INo */ private synchronized void changeSdcReceptionHandlerStatus(final SdcReceptionHandlerStatus newStatus) { switch (newStatus) { - case INIT: - case STOPPED: + case INIT, STOPPED: sdcReceptionHandlerStatus = newStatus; break; case IDLE: @@ -212,7 +211,10 @@ public class SdcReceptionHandler extends AbstractReceptionHandler implements INo "Failed to deploy the artifact due to: " + exp.getMessage()); } } - if (artifactsProcessedSuccessfully) { + + // NoSonar here for complaining about var not changing, when, in fact, + // can change to false when Exceptions are triggered. + if (artifactsProcessedSuccessfully) { // NOSONAR DistributionStatisticsManager.updateDistributionSuccessCount(); sendComponentDoneStatus(notificationData.getDistributionID(), DistributionStatusEnum.COMPONENT_DONE_OK, null); @@ -379,7 +381,7 @@ public class SdcReceptionHandler extends AbstractReceptionHandler implements INo } private File safelyCreateFile(String prefix) throws IOException { - File file = Files.createTempFile(prefix, ".csar").toFile(); + File file = Files.createTempFile(prefix, ".csar").toFile(); // NOSONAR if (file.setReadable(true, false) && file.setWritable(true, true)) { return file; -- cgit 1.2.3-korg