From: adheli.tavares <adheli.tavares@est.tech>
Date: Wed, 1 Oct 2025 09:51:09 +0000 (+0100)
Subject: Fix public modifiers in tests
X-Git-Tag: 4.2.1~1
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=f40bf2ba1689c93ae8c09767b7935873c12a02d1;p=policy%2Fmodels.git

Fix public modifiers in tests

Issue-ID: POLICY-5461
Change-Id: I6d27195b22d1a90532452884b3f655c5bfc7c85c
Signed-off-by: adheli.tavares <adheli.tavares@est.tech>
---

diff --git a/models-base/src/test/java/org/onap/policy/models/base/PfKeyImplTest.java b/models-base/src/test/java/org/onap/policy/models/base/PfKeyImplTest.java
index 5557c976e..78bb663af 100644
--- a/models-base/src/test/java/org/onap/policy/models/base/PfKeyImplTest.java
+++ b/models-base/src/test/java/org/onap/policy/models/base/PfKeyImplTest.java
@@ -68,7 +68,7 @@ class PfKeyImplTest {
      * Sets data in Keys for the tests.
      */
     @BeforeAll
-    public static void setUp() {
+    static void setUp() {
         someKey = new MyKey();
 
         someKey0 = new MyKey();
diff --git a/models-dao/src/test/java/org/onap/policy/models/dao/EntityTest.java b/models-dao/src/test/java/org/onap/policy/models/dao/EntityTest.java
index 35080d81c..98339fb52 100644
--- a/models-dao/src/test/java/org/onap/policy/models/dao/EntityTest.java
+++ b/models-dao/src/test/java/org/onap/policy/models/dao/EntityTest.java
@@ -71,7 +71,7 @@ class EntityTest {
      * Closes the DAO.
      */
     @AfterAll
-    public static void tearDown() {
+    static void tearDown() {
         if (pfDao != null) {
             pfDao.close();
             pfDao = null;
diff --git a/models-interactions/model-impl/rest/src/test/java/org/onap/policy/rest/RestTest.java b/models-interactions/model-impl/rest/src/test/java/org/onap/policy/rest/RestTest.java
index 1bc3d6e34..8eff9fcb1 100644
--- a/models-interactions/model-impl/rest/src/test/java/org/onap/policy/rest/RestTest.java
+++ b/models-interactions/model-impl/rest/src/test/java/org/onap/policy/rest/RestTest.java
@@ -56,7 +56,7 @@ import org.onap.policy.common.endpoints.http.server.HttpServletServerFactoryInst
 import org.onap.policy.common.utils.network.NetworkUtil;
 
 @Path("RestTest")
-public class RestTest {
+public class RestTest {   // NOSONAR - class needs to be public for Jersey to work
     private static final String MERGE_PATCH_PLUS_JSON = "application/merge-patch+json";
 
     private static final String NAME_PARAM = "Bob";
@@ -85,7 +85,7 @@ public class RestTest {
      * Sets server endpoint for the tests.
      */
     @BeforeAll
-    public static void setUp() throws Exception {
+    static void setUp() throws Exception {
 
         port = NetworkUtil.allocPort();
         baseUri = "http://" + LOCALHOST + ":" + port + "/" + BASE + "/";
@@ -110,7 +110,7 @@ public class RestTest {
      *
      */
     @AfterAll
-    public static void tearDown() {
+    static void tearDown() {
         HttpServletServerFactoryInstance.getServerFactory().destroy();
     }
 
diff --git a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AaiSimulatorTest.java b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AaiSimulatorTest.java
index 76fc92826..6eb7d1260 100644
--- a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AaiSimulatorTest.java
+++ b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AaiSimulatorTest.java
@@ -41,7 +41,7 @@ class AaiSimulatorTest {
      * Set up test class.
      */
     @BeforeAll
-    public static void setUpSimulator() {
+    static void setUpSimulator() {
         try {
             var testServer = Util.buildAaiSim();
             assertNotNull(testServer);
@@ -51,7 +51,7 @@ class AaiSimulatorTest {
     }
 
     @AfterAll
-    public static void tearDownSimulator() {
+    static void tearDownSimulator() {
         HttpServletServerFactoryInstance.getServerFactory().destroy();
     }
 
diff --git a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AppcLcmTopicServerTest.java b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AppcLcmTopicServerTest.java
index dc20355ad..9a5f89239 100644
--- a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AppcLcmTopicServerTest.java
+++ b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AppcLcmTopicServerTest.java
@@ -53,7 +53,7 @@ class AppcLcmTopicServerTest {
      * Sets up.
      */
     @BeforeEach
-    public void setUp() {
+    void setUp() {
         server = new AppcLcmTopicServer(sink, source);
     }
 
diff --git a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AppcLegacyTopicServerTest.java b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AppcLegacyTopicServerTest.java
index 9e836ac71..da1b8887e 100644
--- a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AppcLegacyTopicServerTest.java
+++ b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/AppcLegacyTopicServerTest.java
@@ -53,7 +53,7 @@ class AppcLegacyTopicServerTest {
      * Sets up.
      */
     @BeforeEach
-    public void setUp() {
+    void setUp() {
         server = new AppcLegacyTopicServer(sink, source);
     }
 
diff --git a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/SdnrTopicServerTest.java b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/SdnrTopicServerTest.java
index 019a93af1..2819ea037 100644
--- a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/SdnrTopicServerTest.java
+++ b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/SdnrTopicServerTest.java
@@ -53,7 +53,7 @@ class SdnrTopicServerTest {
      * Sets up.
      */
     @BeforeEach
-    public void setUp() {
+    void setUp() {
         server = new SdnrTopicServer(sink, source);
     }
 
diff --git a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/SoSimulatorTest.java b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/SoSimulatorTest.java
index 0a8edb54c..ee605288c 100644
--- a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/SoSimulatorTest.java
+++ b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/SoSimulatorTest.java
@@ -55,7 +55,7 @@ class SoSimulatorTest {
      * Set up test class.
      */
     @BeforeAll
-    public static void setUpSimulator() {
+    static void setUpSimulator() {
         try {
             var testServer = Util.buildSoSim();
             assertNotNull(testServer);
@@ -65,7 +65,7 @@ class SoSimulatorTest {
     }
 
     @AfterAll
-    public static void tearDownSimulator() {
+    static void tearDownSimulator() {
         HttpServletServerFactoryInstance.getServerFactory().destroy();
         SoSimulatorJaxRs.setRequirePolling(false);
     }
diff --git a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/TopicServerTest.java b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/TopicServerTest.java
index 425e6aeef..403880b46 100644
--- a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/TopicServerTest.java
+++ b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/TopicServerTest.java
@@ -54,7 +54,7 @@ class TopicServerTest {
      * Sets up.
      */
     @BeforeEach
-    public void setUp() {
+    void setUp() {
         server = new MyServer();
     }
 
diff --git a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/XacmlSimulatorTest.java b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/XacmlSimulatorTest.java
index 24e9c938f..45c51f0bf 100644
--- a/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/XacmlSimulatorTest.java
+++ b/models-interactions/model-simulators/src/test/java/org/onap/policy/simulators/XacmlSimulatorTest.java
@@ -46,7 +46,7 @@ class XacmlSimulatorTest {
      * Set up test class.
      */
     @BeforeAll
-    public static void setupSimulator() {
+    static void setupSimulator() {
         try {
             var testServer = Util.buildXacmlSim();
             assertNotNull(testServer);
@@ -56,7 +56,7 @@ class XacmlSimulatorTest {
     }
 
     @AfterAll
-    public static void tearDownSimulator() {
+    static void tearDownSimulator() {
         HttpServletServerFactoryInstance.getServerFactory().destroy();
     }
 
diff --git a/models-provider/src/test/java/org/onap/policy/models/provider/impl/DatabasePolicyModelsProviderTest.java b/models-provider/src/test/java/org/onap/policy/models/provider/impl/DatabasePolicyModelsProviderTest.java
index 0db2e0d3d..75a6ca06e 100644
--- a/models-provider/src/test/java/org/onap/policy/models/provider/impl/DatabasePolicyModelsProviderTest.java
+++ b/models-provider/src/test/java/org/onap/policy/models/provider/impl/DatabasePolicyModelsProviderTest.java
@@ -85,7 +85,7 @@ class DatabasePolicyModelsProviderTest {
      * Initialize parameters.
      */
     @BeforeAll
-    public static void setupParameters() {
+    static void setupParameters() {
         parameters = new PolicyModelsProviderParameters();
         parameters.setDatabaseDriver("org.h2.Driver");
         parameters.setDatabaseUrl("jdbc:h2:mem:DatabasePolicyModelsProviderTest");
@@ -98,7 +98,7 @@ class DatabasePolicyModelsProviderTest {
      * Closes the DB.
      */
     @AfterAll
-    public static void tearDown() throws PfModelException {
+    static void tearDown() throws PfModelException {
         if (databaseProvider != null) {
             databaseProvider.close();
         }
diff --git a/models-provider/src/test/java/org/onap/policy/models/provider/impl/PolicyToscaPersistenceTest.java b/models-provider/src/test/java/org/onap/policy/models/provider/impl/PolicyToscaPersistenceTest.java
index 55b9f9aae..430e4f07e 100644
--- a/models-provider/src/test/java/org/onap/policy/models/provider/impl/PolicyToscaPersistenceTest.java
+++ b/models-provider/src/test/java/org/onap/policy/models/provider/impl/PolicyToscaPersistenceTest.java
@@ -72,7 +72,7 @@ class PolicyToscaPersistenceTest {
      * @throws CoderException on JSON encoding and decoding errors
      */
     @BeforeAll
-    public static void setupParameters() throws Exception {
+    static void setupParameters() throws Exception {
 
         PolicyModelsProviderParameters parameters = new PolicyModelsProviderParameters();
 
@@ -90,7 +90,7 @@ class PolicyToscaPersistenceTest {
     }
 
     @AfterAll
-    public static void teardown() throws Exception {
+    static void teardown() throws Exception {
         databaseProvider.close();
     }
 
diff --git a/models-provider/src/test/java/org/onap/policy/models/provider/impl/PolicyTypePersistenceTest.java b/models-provider/src/test/java/org/onap/policy/models/provider/impl/PolicyTypePersistenceTest.java
index 057af7618..b3cdd7a75 100644
--- a/models-provider/src/test/java/org/onap/policy/models/provider/impl/PolicyTypePersistenceTest.java
+++ b/models-provider/src/test/java/org/onap/policy/models/provider/impl/PolicyTypePersistenceTest.java
@@ -58,7 +58,7 @@ class PolicyTypePersistenceTest {
      * @throws PfModelException on exceptions in the tests
      */
     @BeforeAll
-    public static void setupParameters() throws PfModelException {
+    static void setupParameters() throws PfModelException {
 
         PolicyModelsProviderParameters parameters = new PolicyModelsProviderParameters();
 
@@ -72,7 +72,7 @@ class PolicyTypePersistenceTest {
     }
 
     @AfterAll
-    public static void teardown() throws Exception {
+    static void teardown() throws Exception {
         databaseProvider.close();
     }
 
diff --git a/models-provider/src/test/java/org/onap/policy/models/provider/revisionhierarchy/HierarchyFetchTest.java b/models-provider/src/test/java/org/onap/policy/models/provider/revisionhierarchy/HierarchyFetchTest.java
index f005ba230..30552b388 100644
--- a/models-provider/src/test/java/org/onap/policy/models/provider/revisionhierarchy/HierarchyFetchTest.java
+++ b/models-provider/src/test/java/org/onap/policy/models/provider/revisionhierarchy/HierarchyFetchTest.java
@@ -48,7 +48,7 @@ class HierarchyFetchTest {
     private static PolicyModelsProvider databaseProvider;
 
     @BeforeAll
-    public static void beforeSetupParameters() {
+    static void beforeSetupParameters() {
         parameters = new PolicyModelsProviderParameters();
         parameters.setDatabaseDriver("org.h2.Driver");
         parameters.setDatabaseUrl("jdbc:h2:mem:HierarchyFetchTest");
@@ -61,7 +61,7 @@ class HierarchyFetchTest {
      * Closes the DB.
      */
     @AfterAll
-    public static void tearDown() throws PfModelException {
+    static void tearDown() throws PfModelException {
         if (databaseProvider != null) {
             databaseProvider.close();
         }
diff --git a/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/concepts/ToscaEntityFilterTest.java b/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/concepts/ToscaEntityFilterTest.java
index f2fc60c6f..e2b614be3 100644
--- a/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/concepts/ToscaEntityFilterTest.java
+++ b/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/concepts/ToscaEntityFilterTest.java
@@ -83,7 +83,7 @@ class ToscaEntityFilterTest {
      * @throws CoderException on JSON decoding errors
      */
     @BeforeAll
-    public static void setupTypeList() throws CoderException {
+    static void setupTypeList() throws CoderException {
         for (String policyTypeResourceName : policyTypeResourceNames) {
             String policyTypeString = ResourceUtils.getResourceAsString(policyTypeResourceName);
             Object yamlObject = new Yaml().load(policyTypeString);
diff --git a/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/concepts/ToscaTypedEntityFilterTest.java b/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/concepts/ToscaTypedEntityFilterTest.java
index ea9fab728..da4d971eb 100644
--- a/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/concepts/ToscaTypedEntityFilterTest.java
+++ b/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/concepts/ToscaTypedEntityFilterTest.java
@@ -78,7 +78,7 @@ class ToscaTypedEntityFilterTest {
      * @throws CoderException on JSON decoding errors
      */
     @BeforeAll
-    public static void setupTypeList() throws CoderException {
+    static void setupTypeList() throws CoderException {
         for (String policyResourceName : policyResourceNames) {
             String policyString = ResourceUtils.getResourceAsString(policyResourceName);
             if (policyResourceName.endsWith("yaml")) {
diff --git a/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/provider/AuthorativeToscaProviderGenericTest.java b/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/provider/AuthorativeToscaProviderGenericTest.java
index a6e84bbea..dd4e12931 100644
--- a/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/provider/AuthorativeToscaProviderGenericTest.java
+++ b/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/provider/AuthorativeToscaProviderGenericTest.java
@@ -64,7 +64,7 @@ class AuthorativeToscaProviderGenericTest {
      * Read the policy type definition.
      */
     @BeforeAll
-    public static void readPolicyDefinition() {
+    static void readPolicyDefinition() {
         String yamlString = ResourceUtils.getResourceAsString("src/test/resources/onap.policies.NoVersion.yaml");
 
         Object yamlObject = new Yaml().load(yamlString);
diff --git a/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/provider/AuthorativeToscaProviderPolicyTypeTest.java b/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/provider/AuthorativeToscaProviderPolicyTypeTest.java
index 9ae7842a6..512650c22 100644
--- a/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/provider/AuthorativeToscaProviderPolicyTypeTest.java
+++ b/models-tosca/src/test/java/org/onap/policy/models/tosca/authorative/provider/AuthorativeToscaProviderPolicyTypeTest.java
@@ -67,7 +67,7 @@ class AuthorativeToscaProviderPolicyTypeTest {
      *
      */
     @BeforeAll
-    public static void readPolicyDefinition() {
+    static void readPolicyDefinition() {
         String yamlString = ResourceUtils.getResourceAsString("src/test/resources/onap.policies.NoVersion.yaml");
 
         Object yamlObject = new Yaml().load(yamlString);
diff --git a/models-tosca/src/test/java/org/onap/policy/models/tosca/simple/concepts/JpaToscaWithToscaPropertiesTest.java b/models-tosca/src/test/java/org/onap/policy/models/tosca/simple/concepts/JpaToscaWithToscaPropertiesTest.java
index 4bfff20da..3d7c96b0a 100644
--- a/models-tosca/src/test/java/org/onap/policy/models/tosca/simple/concepts/JpaToscaWithToscaPropertiesTest.java
+++ b/models-tosca/src/test/java/org/onap/policy/models/tosca/simple/concepts/JpaToscaWithToscaPropertiesTest.java
@@ -63,7 +63,7 @@ class JpaToscaWithToscaPropertiesTest {
      * Initializes the properties.
      */
     @BeforeAll
-    public static void setUpBeforeClass() {
+    static void setUpBeforeClass() {
         JPA_PROP1.setDescription(DESCRIPT1);
         JPA_PROP2.setDescription(DESCRIPT2);