From de18be8467912348ea73c7f5557397bab9ba86b3 Mon Sep 17 00:00:00 2001 From: liamfallon Date: Thu, 3 Feb 2022 12:01:57 +0000 Subject: Remove direct DB Load/Save from apex-pdp This review is part of a series of reviews to move the apex-pdp state machine model to use the base model types in policy models. Persistence of policies for apex-pdp is now implemented in PAP and API. The historic direct load and save functionality for APEX policy state machine models to and from databases is no longer used and no longer supported. This revmoves the DAO code from apex-pdp and the associated handling and test code. The next review will remove the JPA annotations. Subsequent reviews will mvoe the mdoel across to use the policy models base model types. Issue-ID: POLICY-1820 Change-Id: Iabf034b40a413d32fe62091fe3446f52ec30d3af Signed-off-by: liamfallon --- .../context/distribution/ContextUpdate.java | 3 +- .../src/test/resources/META-INF/persistence.xml | 51 ---------------- .../handling/TestApexSamplePolicyDbWrite.java | 64 --------------------- .../handling/TestApexSamplePolicyModel.java | 24 +------- .../src/test/resources/META-INF/persistence.xml | 67 ---------------------- 5 files changed, 2 insertions(+), 207 deletions(-) delete mode 100644 testsuites/integration/integration-context-test/src/test/resources/META-INF/persistence.xml delete mode 100644 testsuites/integration/integration-executor-test/src/test/java/org/onap/policy/apex/testsuites/integration/executor/handling/TestApexSamplePolicyDbWrite.java delete mode 100644 testsuites/integration/integration-executor-test/src/test/resources/META-INF/persistence.xml (limited to 'testsuites') diff --git a/testsuites/integration/integration-context-test/src/test/java/org/onap/policy/apex/testsuites/integration/context/distribution/ContextUpdate.java b/testsuites/integration/integration-context-test/src/test/java/org/onap/policy/apex/testsuites/integration/context/distribution/ContextUpdate.java index b0370c5d9..8d5b702b0 100644 --- a/testsuites/integration/integration-context-test/src/test/java/org/onap/policy/apex/testsuites/integration/context/distribution/ContextUpdate.java +++ b/testsuites/integration/integration-context-test/src/test/java/org/onap/policy/apex/testsuites/integration/context/distribution/ContextUpdate.java @@ -1,7 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2016-2018 Ericsson. All rights reserved. - * Modifications Copyright (C) 2019-2020 Nordix Foundation. + * Modifications Copyright (C) 2019-2020,2022 Nordix Foundation. * Modifications Copyright (C) 2021 AT&T Intellectual Property. All rights reserved. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); @@ -26,7 +26,6 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThatThrownBy; import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertNull; -import static org.junit.Assert.assertTrue; import static org.onap.policy.apex.testsuites.integration.context.utils.Constants.APEX_DISTRIBUTOR; import static org.onap.policy.apex.testsuites.integration.context.utils.Constants.DATE_CONTEXT_ALBUM; import static org.onap.policy.apex.testsuites.integration.context.utils.Constants.LONG_CONTEXT_ALBUM; diff --git a/testsuites/integration/integration-context-test/src/test/resources/META-INF/persistence.xml b/testsuites/integration/integration-context-test/src/test/resources/META-INF/persistence.xml deleted file mode 100644 index 341c33e89..000000000 --- a/testsuites/integration/integration-context-test/src/test/resources/META-INF/persistence.xml +++ /dev/null @@ -1,51 +0,0 @@ - - - - - - org.eclipse.persistence.jpa.PersistenceProvider - - org.onap.policy.apex.model.basicmodel.dao.converters.CDataConditioner - org.onap.policy.apex.model.basicmodel.dao.converters.Uuid2String - org.onap.policy.apex.model.basicmodel.concepts.AxArtifactKey - org.onap.policy.apex.model.basicmodel.concepts.AxConcept - org.onap.policy.apex.model.basicmodel.concepts.AxKeyInfo - org.onap.policy.apex.model.basicmodel.concepts.AxKeyInformation - org.onap.policy.apex.model.basicmodel.concepts.AxModel - org.onap.policy.apex.model.basicmodel.concepts.TestEntity - org.onap.policy.apex.model.contextmodel.concepts.AxContextSchema - org.onap.policy.apex.model.contextmodel.concepts.AxContextSchemas - org.onap.policy.apex.model.contextmodel.concepts.AxContextAlbum - org.onap.policy.apex.model.contextmodel.concepts.AxContextAlbums - org.onap.policy.apex.model.contextmodel.concepts.AxContextModel - - - - - - - - - - - - diff --git a/testsuites/integration/integration-executor-test/src/test/java/org/onap/policy/apex/testsuites/integration/executor/handling/TestApexSamplePolicyDbWrite.java b/testsuites/integration/integration-executor-test/src/test/java/org/onap/policy/apex/testsuites/integration/executor/handling/TestApexSamplePolicyDbWrite.java deleted file mode 100644 index 413b57880..000000000 --- a/testsuites/integration/integration-executor-test/src/test/java/org/onap/policy/apex/testsuites/integration/executor/handling/TestApexSamplePolicyDbWrite.java +++ /dev/null @@ -1,64 +0,0 @@ -/*- - * ============LICENSE_START======================================================= - * Copyright (C) 2016-2018 Ericsson. All rights reserved. - * ================================================================================ - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - * - * SPDX-License-Identifier: Apache-2.0 - * ============LICENSE_END========================================================= - */ - -package org.onap.policy.apex.testsuites.integration.executor.handling; - -import java.sql.Connection; -import java.sql.DriverManager; -import org.junit.Before; -import org.junit.Test; -import org.onap.policy.apex.model.basicmodel.dao.DaoParameters; -import org.onap.policy.apex.model.basicmodel.dao.impl.DefaultApexDao; -import org.onap.policy.apex.model.basicmodel.test.TestApexModel; -import org.onap.policy.apex.model.policymodel.concepts.AxPolicyModel; - -/** - * The Class TestApexSamplePolicyDbWrite. - */ -public class TestApexSamplePolicyDbWrite { - TestApexModel testApexModel; - - /** - * Setup. - * - * @throws Exception the exception - */ - @Before - public void setup() throws Exception { - final TestApexSamplePolicyModelCreator apexPolicyModelCreator = new TestApexSamplePolicyModelCreator("MVEL"); - testApexModel = new TestApexModel(AxPolicyModel.class, apexPolicyModelCreator); - } - - /** - * Test model write read jpa. - * - * @throws Exception the exception - */ - @Test - public void testModelWriteReadJpa() throws Exception { - final DaoParameters DaoParameters = new DaoParameters(); - DaoParameters.setPluginClass(DefaultApexDao.class.getName()); - DaoParameters.setPersistenceUnit("SampleModelTest"); - - Connection connection = DriverManager.getConnection("jdbc:h2:mem:testdb"); - testApexModel.testApexModelWriteReadJpa(DaoParameters); - connection.close(); - } -} diff --git a/testsuites/integration/integration-executor-test/src/test/java/org/onap/policy/apex/testsuites/integration/executor/handling/TestApexSamplePolicyModel.java b/testsuites/integration/integration-executor-test/src/test/java/org/onap/policy/apex/testsuites/integration/executor/handling/TestApexSamplePolicyModel.java index 0561162c8..54adc166d 100644 --- a/testsuites/integration/integration-executor-test/src/test/java/org/onap/policy/apex/testsuites/integration/executor/handling/TestApexSamplePolicyModel.java +++ b/testsuites/integration/integration-executor-test/src/test/java/org/onap/policy/apex/testsuites/integration/executor/handling/TestApexSamplePolicyModel.java @@ -1,7 +1,7 @@ /*- * ============LICENSE_START======================================================= * Copyright (C) 2016-2018 Ericsson. All rights reserved. - * Modifications Copyright (C) 2020 Nordix Foundation. + * Modifications Copyright (C) 2020,2022 Nordix Foundation. * ================================================================================ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -23,13 +23,9 @@ package org.onap.policy.apex.testsuites.integration.executor.handling; import static org.junit.Assert.assertEquals; -import java.sql.Connection; -import java.sql.DriverManager; import org.junit.Before; import org.junit.Test; import org.onap.policy.apex.model.basicmodel.concepts.AxValidationResult; -import org.onap.policy.apex.model.basicmodel.dao.DaoParameters; -import org.onap.policy.apex.model.basicmodel.dao.impl.DefaultApexDao; import org.onap.policy.apex.model.basicmodel.test.TestApexModel; import org.onap.policy.apex.model.policymodel.concepts.AxPolicyModel; @@ -81,22 +77,4 @@ public class TestApexSamplePolicyModel { public void testModelWriteReadJson() throws Exception { testApexModel.testApexModelWriteReadJson(); } - - /** - * Test model write read jpa. - * - * @throws Exception the exception - */ - @Test - public void testModelWriteReadJpa() throws Exception { - final DaoParameters DaoParameters = new DaoParameters(); - DaoParameters.setPluginClass(DefaultApexDao.class.getName()); - DaoParameters.setPersistenceUnit("SampleModelTest"); - - Connection connection = DriverManager.getConnection("jdbc:h2:mem:testdb"); - testApexModel.testApexModelWriteReadJpa(DaoParameters); - connection.close(); - } - - } diff --git a/testsuites/integration/integration-executor-test/src/test/resources/META-INF/persistence.xml b/testsuites/integration/integration-executor-test/src/test/resources/META-INF/persistence.xml deleted file mode 100644 index b2d6a96ed..000000000 --- a/testsuites/integration/integration-executor-test/src/test/resources/META-INF/persistence.xml +++ /dev/null @@ -1,67 +0,0 @@ - - - - - - org.eclipse.persistence.jpa.PersistenceProvider - - org.onap.policy.apex.model.basicmodel.dao.converters.CDataConditioner - org.onap.policy.apex.model.basicmodel.dao.converters.Uuid2String - org.onap.policy.apex.model.basicmodel.concepts.AxArtifactKey - org.onap.policy.apex.model.basicmodel.concepts.AxConcept - org.onap.policy.apex.model.basicmodel.concepts.AxKeyInfo - org.onap.policy.apex.model.basicmodel.concepts.AxKeyInformation - org.onap.policy.apex.model.basicmodel.concepts.AxModel - org.onap.policy.apex.model.basicmodel.concepts.TestEntity - org.onap.policy.apex.model.contextmodel.concepts.AxContextSchema - org.onap.policy.apex.model.contextmodel.concepts.AxContextSchemas - org.onap.policy.apex.model.contextmodel.concepts.AxContextAlbum - org.onap.policy.apex.model.contextmodel.concepts.AxContextAlbums - org.onap.policy.apex.model.contextmodel.concepts.AxContextModel - org.onap.policy.apex.model.eventmodel.concepts.AxField - org.onap.policy.apex.model.eventmodel.concepts.AxInputField - org.onap.policy.apex.model.eventmodel.concepts.AxOutputField - org.onap.policy.apex.model.eventmodel.concepts.AxEvent - org.onap.policy.apex.model.eventmodel.concepts.AxEvents - org.onap.policy.apex.model.eventmodel.concepts.AxEventModel - org.onap.policy.apex.model.policymodel.concepts.AxLogic - org.onap.policy.apex.model.policymodel.concepts.AxTaskParameter - org.onap.policy.apex.model.policymodel.concepts.AxTaskLogic - org.onap.policy.apex.model.policymodel.concepts.AxTask - org.onap.policy.apex.model.policymodel.concepts.AxTasks - org.onap.policy.apex.model.policymodel.concepts.AxTaskSelectionLogic - org.onap.policy.apex.model.policymodel.concepts.AxStateFinalizerLogic - org.onap.policy.apex.model.policymodel.concepts.AxStateOutput - org.onap.policy.apex.model.policymodel.concepts.AxStateTaskReference - org.onap.policy.apex.model.policymodel.concepts.AxState - org.onap.policy.apex.model.policymodel.concepts.AxPolicy - org.onap.policy.apex.model.policymodel.concepts.AxPolicies - org.onap.policy.apex.model.policymodel.concepts.AxPolicyModel - - - - - - - - - - -- cgit 1.2.3-korg