Fix sonars in policy-api 33/121133/1
authorJim Hahn <jrh3@att.com>
Thu, 6 May 2021 13:52:08 +0000 (09:52 -0400)
committerJim Hahn <jrh3@att.com>
Thu, 6 May 2021 13:55:29 +0000 (09:55 -0400)
Fixed sonars:
- use "var" instead of actual type name

Issue-ID: POLICY-3285
Change-Id: I349e57f7abe84138f5aa3478e22652a593b59845
Signed-off-by: Jim Hahn <jrh3@att.com>
main/src/main/java/org/onap/policy/api/main/parameters/ApiParameterHandler.java
main/src/main/java/org/onap/policy/api/main/rest/ApiRestController.java
main/src/main/java/org/onap/policy/api/main/rest/PolicyFetchMode.java
main/src/main/java/org/onap/policy/api/main/rest/provider/CommonModelProvider.java
main/src/main/java/org/onap/policy/api/main/rest/provider/HealthCheckProvider.java
main/src/main/java/org/onap/policy/api/main/rest/provider/StatisticsProvider.java
main/src/main/java/org/onap/policy/api/main/startstop/ApiDatabaseInitializer.java
main/src/main/java/org/onap/policy/api/main/startstop/Main.java

index 813dd75..019a012 100644 (file)
@@ -23,7 +23,6 @@
 
 package org.onap.policy.api.main.parameters;
 
-import com.google.gson.Gson;
 import com.google.gson.GsonBuilder;
 import java.io.FileReader;
 import org.onap.policy.api.main.exception.PolicyApiException;
@@ -53,7 +52,7 @@ public class ApiParameterHandler {
         // Read the parameters
         try {
             // Read the parameters from JSON using Gson
-            final Gson gson = new GsonBuilder().create();
+            final var gson = new GsonBuilder().create();
             apiParameterGroup = gson.fromJson(new FileReader(arguments.getFullConfigurationFilePath()),
                     ApiParameterGroup.class);
         } catch (final Exception e) {
index e23604b..5e84508 100644 (file)
@@ -3,7 +3,7 @@
  * ONAP Policy API\r
  * ================================================================================\r
  * Copyright (C) 2018 Samsung Electronics Co., Ltd. All rights reserved.\r
- * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.\r
+ * Modifications Copyright (C) 2019-2021 AT&T Intellectual Property. All rights reserved.\r
  * Modifications Copyright (C) 2020 Nordix Foundation.\r
  * Modifications Copyright (C) 2020 Bell Canada.\r
  * ================================================================================\r
@@ -237,7 +237,7 @@ public class ApiRestController extends CommonRestController {
     public Response\r
         getAllPolicyTypes(@HeaderParam(REQUEST_ID_NAME) @ApiParam(REQUEST_ID_PARAM_DESCRIPTION) UUID requestId) {\r
 \r
-        try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
+        try (var policyTypeProvider = new PolicyTypeProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyTypeProvider.fetchPolicyTypes(null, null);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.SUCCESS, HttpMethod.GET);\r
             return makeOkResponse(requestId, serviceTemplate);\r
@@ -285,7 +285,7 @@ public class ApiRestController extends CommonRestController {
         @PathParam("policyTypeId") @ApiParam(value = "ID of policy type", required = true) String policyTypeId,\r
         @HeaderParam(REQUEST_ID_NAME) @ApiParam(REQUEST_ID_PARAM_DESCRIPTION) UUID requestId) {\r
 \r
-        try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
+        try (var policyTypeProvider = new PolicyTypeProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyTypeProvider.fetchPolicyTypes(policyTypeId, null);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.SUCCESS, HttpMethod.GET);\r
             return makeOkResponse(requestId, serviceTemplate);\r
@@ -334,7 +334,7 @@ public class ApiRestController extends CommonRestController {
         @PathParam("versionId") @ApiParam(value = "Version of policy type", required = true) String versionId,\r
         @HeaderParam(REQUEST_ID_NAME) @ApiParam(REQUEST_ID_PARAM_DESCRIPTION) UUID requestId) {\r
 \r
-        try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
+        try (var policyTypeProvider = new PolicyTypeProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyTypeProvider.fetchPolicyTypes(policyTypeId, versionId);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.SUCCESS, HttpMethod.GET);\r
             return makeOkResponse(requestId, serviceTemplate);\r
@@ -381,7 +381,7 @@ public class ApiRestController extends CommonRestController {
         @PathParam("policyTypeId") @ApiParam(value = "ID of policy type", required = true) String policyTypeId,\r
         @HeaderParam(REQUEST_ID_NAME) @ApiParam(REQUEST_ID_PARAM_DESCRIPTION) UUID requestId) {\r
 \r
-        try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
+        try (var policyTypeProvider = new PolicyTypeProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyTypeProvider.fetchLatestPolicyTypes(policyTypeId);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.SUCCESS, HttpMethod.GET);\r
             return makeOkResponse(requestId, serviceTemplate);\r
@@ -433,7 +433,7 @@ public class ApiRestController extends CommonRestController {
             NetLoggerUtil.log(EventType.IN, CommInfrastructure.REST, "/policytypes", toJson(body));\r
         }\r
 \r
-        try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
+        try (var policyTypeProvider = new PolicyTypeProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyTypeProvider.createPolicyType(body);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.SUCCESS, HttpMethod.POST);\r
             return makeOkResponse(requestId, serviceTemplate);\r
@@ -486,7 +486,7 @@ public class ApiRestController extends CommonRestController {
         @PathParam("versionId") @ApiParam(value = "Version of policy type", required = true) String versionId,\r
         @HeaderParam(REQUEST_ID_NAME) @ApiParam(REQUEST_ID_PARAM_DESCRIPTION) UUID requestId) {\r
 \r
-        try (PolicyTypeProvider policyTypeProvider = new PolicyTypeProvider()) {\r
+        try (var policyTypeProvider = new PolicyTypeProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyTypeProvider.deletePolicyType(policyTypeId, versionId);\r
             updateApiStatisticsCounter(Target.POLICY_TYPE, Result.SUCCESS, HttpMethod.DELETE);\r
             return makeOkResponse(requestId, serviceTemplate);\r
@@ -545,7 +545,7 @@ public class ApiRestController extends CommonRestController {
             + " REFERENCED for fully referenced policies") PolicyFetchMode mode\r
     ) {\r
 \r
-        try (PolicyProvider policyProvider = new PolicyProvider()) {\r
+        try (var policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate =\r
                 policyProvider.fetchPolicies(policyTypeId, policyTypeVersion, null, null, mode);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.GET);\r
@@ -604,7 +604,7 @@ public class ApiRestController extends CommonRestController {
         @QueryParam("mode") @DefaultValue("bare") @ApiParam("Fetch mode for policies, BARE for bare policies (default),"\r
             + " REFERENCED for fully referenced policies") PolicyFetchMode mode\r
     ) {\r
-        try (PolicyProvider policyProvider = new PolicyProvider()) {\r
+        try (var policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate =\r
                 policyProvider.fetchPolicies(policyTypeId, policyTypeVersion, policyId, null, mode);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.GET);\r
@@ -666,7 +666,7 @@ public class ApiRestController extends CommonRestController {
         @QueryParam("mode") @DefaultValue("bare") @ApiParam("Fetch mode for policies, BARE for bare policies (default),"\r
             + " REFERENCED for fully referenced policies") PolicyFetchMode mode\r
     ) {\r
-        try (PolicyProvider policyProvider = new PolicyProvider()) {\r
+        try (var policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate =\r
                 policyProvider.fetchPolicies(policyTypeId, policyTypeVersion, policyId, policyVersion, mode);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.GET);\r
@@ -722,7 +722,7 @@ public class ApiRestController extends CommonRestController {
         @QueryParam("mode") @ApiParam("Fetch mode for policies, TERSE for bare policies (default), "\r
             + "REFERENCED for fully referenced policies") PolicyFetchMode mode) {\r
 \r
-        try (PolicyProvider policyProvider = new PolicyProvider()) {\r
+        try (var policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate =\r
                 policyProvider.fetchLatestPolicies(policyTypeId, policyTypeVersion, policyId, mode);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.GET);\r
@@ -784,7 +784,7 @@ public class ApiRestController extends CommonRestController {
                 "/policytypes/" + policyTypeId + "/versions/" + policyTypeVersion + "/policies", toJson(body));\r
         }\r
 \r
-        try (PolicyProvider policyProvider = new PolicyProvider()) {\r
+        try (var policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyProvider.createPolicy(policyTypeId, policyTypeVersion, body);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.POST);\r
             return makeOkResponse(requestId, serviceTemplate);\r
@@ -840,7 +840,7 @@ public class ApiRestController extends CommonRestController {
         @PathParam("policyVersion") @ApiParam(value = "Version of policy", required = true) String policyVersion,\r
         @HeaderParam(REQUEST_ID_NAME) @ApiParam(REQUEST_ID_PARAM_DESCRIPTION) UUID requestId) {\r
 \r
-        try (PolicyProvider policyProvider = new PolicyProvider()) {\r
+        try (var policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate =\r
                 policyProvider.deletePolicy(policyTypeId, policyTypeVersion, policyId, policyVersion);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.DELETE);\r
@@ -893,7 +893,7 @@ public class ApiRestController extends CommonRestController {
         @QueryParam("mode") @DefaultValue("bare") @ApiParam("Fetch mode for policies, BARE for bare policies (default),"\r
             + " REFERENCED for fully referenced policies") PolicyFetchMode mode\r
     ) {\r
-        try (PolicyProvider policyProvider = new PolicyProvider()) {\r
+        try (var policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate =\r
                 policyProvider.fetchPolicies(null, null, null, null, mode);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.GET);\r
@@ -954,7 +954,7 @@ public class ApiRestController extends CommonRestController {
         @QueryParam("mode") @DefaultValue("bare") @ApiParam("Fetch mode for policies, BARE for bare policies (default),"\r
             + " REFERENCED for fully referenced policies") PolicyFetchMode mode\r
     ) {\r
-        try (PolicyProvider policyProvider = new PolicyProvider()) {\r
+        try (var policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate =\r
                 policyProvider.fetchPolicies(null, null, policyId, policyVersion, mode);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.GET);\r
@@ -1009,7 +1009,7 @@ public class ApiRestController extends CommonRestController {
             NetLoggerUtil.log(EventType.IN, CommInfrastructure.REST, "/policies", toJson(body));\r
         }\r
 \r
-        try (PolicyProvider policyProvider = new PolicyProvider()) {\r
+        try (var policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate = policyProvider.createPolicies(body);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.POST);\r
             return makeOkResponse(requestId, serviceTemplate);\r
@@ -1060,7 +1060,7 @@ public class ApiRestController extends CommonRestController {
         @PathParam("policyVersion") @ApiParam(value = "Version of policy", required = true) String policyVersion,\r
         @HeaderParam(REQUEST_ID_NAME) @ApiParam(REQUEST_ID_PARAM_DESCRIPTION) UUID requestId) {\r
 \r
-        try (PolicyProvider policyProvider = new PolicyProvider()) {\r
+        try (var policyProvider = new PolicyProvider()) {\r
             ToscaServiceTemplate serviceTemplate =\r
                 policyProvider.deletePolicy(null, null, policyId, policyVersion);\r
             updateApiStatisticsCounter(Target.POLICY, Result.SUCCESS, HttpMethod.DELETE);\r
@@ -1199,4 +1199,4 @@ public class ApiRestController extends CommonRestController {
             }\r
         }\r
     }\r
-}
\ No newline at end of file
+}\r
index 99678b8..7e201b6 100644 (file)
@@ -1,6 +1,7 @@
 /*-
  * ============LICENSE_START=======================================================
  * Copyright (C) 2020 Nordix Foundation.
+ * Modifications Copyright (C) 2021 AT&T Intellectual Property. 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.
@@ -31,7 +32,6 @@ public enum PolicyFetchMode {
      * @return the enum value of the string
      */
     public static PolicyFetchMode fromString(final String modeString) {
-        String modeStringUpper = modeString.toUpperCase();
-        return valueOf(modeStringUpper);
+        return valueOf(modeString.toUpperCase());
     }
 }
index cb20c04..b0825ea 100644 (file)
@@ -2,7 +2,7 @@
  * ============LICENSE_START=======================================================\r
  * ONAP Policy API\r
  * ================================================================================\r
- * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.\r
+ * Copyright (C) 2019-2021 AT&T Intellectual Property. All rights reserved.\r
  * Modifications Copyright (C) 2019-2021 Nordix Foundation.\r
  * ================================================================================\r
  * Licensed under the Apache License, Version 2.0 (the "License");\r
@@ -132,7 +132,7 @@ public class CommonModelProvider implements AutoCloseable {
 \r
         List<ToscaConceptIdentifier> policyTypes = new ArrayList<>();\r
         policyTypes.add(new ToscaConceptIdentifier(policyType.getName(), policyType.getVersion()));\r
-        PdpGroupFilter pdpGroupFilter = PdpGroupFilter.builder().policyTypeList(policyTypes).groupState(PdpState.ACTIVE)\r
+        var pdpGroupFilter = PdpGroupFilter.builder().policyTypeList(policyTypes).groupState(PdpState.ACTIVE)\r
                 .pdpState(PdpState.ACTIVE).build();\r
         return modelsProvider.getFilteredPdpGroups(pdpGroupFilter);\r
     }\r
@@ -158,7 +158,7 @@ public class CommonModelProvider implements AutoCloseable {
         Map<Pair<String, String>, T> deployedPolicyMap = new HashMap<>();\r
         for (PdpGroup pdpGroup : pdpGroups) {\r
             List<ToscaConceptIdentifier> policyIdentifiers = extractPolicyIdentifiers(policyId, pdpGroup, policyType);\r
-            T deployedPolicies = getDeployedPolicies(policyIdentifiers, policyType, getter, consumer, data);\r
+            var deployedPolicies = getDeployedPolicies(policyIdentifiers, policyType, getter, consumer, data);\r
             deployedPolicyMap.put(Pair.of(pdpGroup.getName(), pdpGroup.getVersion()), deployedPolicies);\r
         }\r
         return deployedPolicyMap;\r
@@ -210,7 +210,7 @@ public class CommonModelProvider implements AutoCloseable {
             BiFunctionWithEx<String, String, R> getter, BiConsumer<T, R> consumer, T data) throws PfModelException {\r
 \r
         for (ToscaConceptIdentifier policyIdentifier : policyIdentifiers) {\r
-            R result = getter.apply(policyIdentifier.getName(),\r
+            var result = getter.apply(policyIdentifier.getName(),\r
                     getTrimedVersionForLegacyType(policyIdentifier.getVersion(), policyType));\r
             consumer.accept(data, result);\r
         }\r
index a8edaa1..f4019d9 100644 (file)
@@ -3,7 +3,7 @@
  * ONAP Policy API
  * ================================================================================
  * Copyright (C) 2018 Samsung Electronics Co., Ltd. All rights reserved.
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2019, 2021 AT&T Intellectual Property. All rights reserved.
  * Modifications Copyright (C) 2019 Nordix Foundation.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
@@ -45,7 +45,7 @@ public class HealthCheckProvider {
      * @return Report containing health check status
      */
     public HealthCheckReport performHealthCheck() {
-        final HealthCheckReport report = new HealthCheckReport();
+        final var report = new HealthCheckReport();
         report.setName(NAME);
         report.setUrl(URL);
         report.setHealthy(ApiActivator.isAlive());
index 3a84222..dadc630 100644 (file)
@@ -3,7 +3,7 @@
  * ============LICENSE_START=======================================================
  * ONAP Policy API
  * ================================================================================
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2019, 2021 AT&T Intellectual Property. 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.
@@ -40,7 +40,7 @@ public class StatisticsProvider {
      * @return Report containing API statistics
      */
     public StatisticsReport fetchCurrentStatistics() {
-        final StatisticsReport report = new StatisticsReport();
+        final var report = new StatisticsReport();
         report.setCode(ApiActivator.isAlive() ? 200 : 500);
         report.setTotalApiCallCount(ApiStatisticsManager.getTotalApiCallCount());
         report.setApiCallSuccessCount(ApiStatisticsManager.getApiCallSuccessCount());
index d30b012..76ffd9f 100644 (file)
@@ -2,7 +2,7 @@
  * ============LICENSE_START=======================================================
  * ONAP Policy API
  * ================================================================================
- * Copyright (C) 2019-2020 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2019-2021 AT&T Intellectual Property. All rights reserved.
  * Modifications Copyright (C) 2019-2021 Nordix Foundation.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
@@ -69,7 +69,7 @@ public class ApiDatabaseInitializer {
      */
     public void initializeApiDatabase(final ApiParameterGroup apiParameterGroup) throws PolicyApiException {
 
-        try (PolicyModelsProvider databaseProvider =
+        try (var databaseProvider =
                 factory.createPolicyModelsProvider(apiParameterGroup.getDatabaseProviderParameters())) {
 
             if (alreadyExists(databaseProvider)) {
@@ -77,7 +77,7 @@ public class ApiDatabaseInitializer {
                 return;
             }
 
-            ToscaServiceTemplate serviceTemplate = new ToscaServiceTemplate();
+            var serviceTemplate = new ToscaServiceTemplate();
             serviceTemplate.setDataTypes(new LinkedHashMap<>());
             serviceTemplate.setPolicyTypes(new LinkedHashMap<>());
             serviceTemplate.setToscaDefinitionsVersion("tosca_simple_yaml_1_1_0");
@@ -111,7 +111,7 @@ public class ApiDatabaseInitializer {
             throws PolicyApiException, CoderException, PfModelException {
 
         for (String entity : entities) {
-            String entityAsStringYaml = ResourceUtils.getResourceAsString(entity);
+            var entityAsStringYaml = ResourceUtils.getResourceAsString(entity);
             if (entityAsStringYaml == null) {
                 LOGGER.warn("Preloading entity cannot be found: {}", entity);
                 continue;
@@ -131,7 +131,7 @@ public class ApiDatabaseInitializer {
             }
 
             // Consolidate policies
-            ToscaTopologyTemplate topologyTemplate = singleEntity.getToscaTopologyTemplate();
+            var topologyTemplate = singleEntity.getToscaTopologyTemplate();
             if (topologyTemplate != null && topologyTemplate.getPolicies() != null) {
                 serviceTemplate.setToscaTopologyTemplate(new ToscaTopologyTemplate());
                 serviceTemplate.getToscaTopologyTemplate().setPolicies(new LinkedList<>());
index 36734f3..76de017 100644 (file)
@@ -3,7 +3,7 @@
  * ONAP Policy API
  * ================================================================================
  * Copyright (C) 2018 Samsung Electronics Co., Ltd. All rights reserved.
- * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2019, 2021 AT&T Intellectual Property. All rights reserved.
  * Modifications Copyright (C) 2020 Bell Canada. All rights reserved.
  * Modifications Copyright (C) 2021 Nordix Foundation.
  * ================================================================================
@@ -55,11 +55,11 @@ public class Main {
      * @param args the command line arguments
      */
     public Main(final String[] args) {
-        final String argumentString = Arrays.toString(args);
+        final var argumentString = Arrays.toString(args);
         LOGGER.info("Starting policy api service with arguments - {}", argumentString);
 
         // Check the arguments
-        final ApiCommandLineArguments arguments = new ApiCommandLineArguments();
+        final var arguments = new ApiCommandLineArguments();
         try {
             // The arguments return a string if there is a message to print and we should exit
             final String argumentMessage = arguments.parse(args);
@@ -88,7 +88,7 @@ public class Main {
 
         // Add a shutdown hook to shut everything down in an orderly manner
         Runtime.getRuntime().addShutdownHook(new PolicyApiShutdownHookClass());
-        String successMsg = String.format(MessageConstants.START_SUCCESS_MSG, MessageConstants.POLICY_API);
+        var successMsg = String.format(MessageConstants.START_SUCCESS_MSG, MessageConstants.POLICY_API);
         LOGGER.info(successMsg);
     }