fixed some Sonar critical issues 41/28541/2
authorTaka <tc012c@att.com>
Thu, 18 Jan 2018 16:51:47 +0000 (11:51 -0500)
committerPatrick Brady <pb071s@att.com>
Thu, 18 Jan 2018 21:20:50 +0000 (21:20 +0000)
Change-Id: I9dfe73a45b43919ff7b01ab3217d957a40ac2018
Issue-ID: APPC-408
Signed-off-by: Taka <tc012c@att.com>
appc-adapters/appc-chef-adapter/appc-chef-adapter-bundle/src/main/java/org/onap/appc/adapter/chef/chefclient/Utils.java
appc-adapters/appc-dmaap-adapter/appc-message-adapter-factory/src/main/java/org/onap/appc/adapter/factory/DmaapMessageAdapterFactoryImpl.java
appc-adapters/appc-dmaap-adapter/appc-message-adapter-factory/src/main/java/org/onap/appc/adapter/factory/MessageService.java
appc-adapters/appc-iaas-adapter/appc-iaas-adapter-bundle/src/main/java/org/onap/appc/adapter/iaas/provider/operation/common/constants/Constants.java
appc-adapters/appc-iaas-adapter/appc-iaas-adapter-bundle/src/main/java/org/onap/appc/adapter/iaas/provider/operation/common/constants/Property.java
appc-adapters/appc-iaas-adapter/appc-iaas-adapter-bundle/src/main/java/org/onap/appc/adapter/iaas/provider/operation/common/enums/Operation.java
appc-adapters/appc-iaas-adapter/appc-iaas-adapter-bundle/src/main/java/org/onap/appc/adapter/iaas/provider/operation/common/enums/Outcome.java
appc-adapters/appc-netconf-adapter/appc-netconf-adapter-bundle/src/main/java/org/onap/appc/adapter/netconf/exception/DataAccessException.java
appc-adapters/appc-netconf-adapter/appc-netconf-adapter-bundle/src/main/java/org/onap/appc/adapter/netconf/exception/NetconfDAOException.java
appc-adapters/appc-netconf-adapter/appc-netconf-adapter-bundle/src/main/java/org/onap/appc/adapter/netconf/util/Constants.java
appc-adapters/appc-rest-healthcheck-adapter/appc-rest-healthcheck-adapter-bundle/src/main/java/org/onap/appc/adapter/restHealthcheck/RestHealthcheckActivator.java

index dbaf1d3..33b20dc 100644 (file)
@@ -25,7 +25,6 @@
 package org.onap.appc.adapter.chef.chefclient;
 
 import java.io.BufferedReader;
-import java.io.FileNotFoundException;
 import java.io.FileReader;
 import java.io.IOException;
 import java.security.InvalidKeyException;
@@ -48,7 +47,6 @@ public class Utils {
     
     public static String sha1AndBase64(String inStr) {
         MessageDigest md = null;
-        String outStr = null;
         byte[] outbty = null;
         try {
             md = MessageDigest.getInstance("SHA-1");
index cb23f5f..5fe2d03 100644 (file)
@@ -36,16 +36,19 @@ import org.onap.appc.adapter.messaging.dmaap.impl.DmaapProducerImpl;
 public class DmaapMessageAdapterFactoryImpl implements MessageAdapterFactory {
 
     
+    @Override
     public Producer createProducer(Collection<String> pools, String writeTopic, String apiKey, String apiSecret) {
         return  new  DmaapProducerImpl(pools, writeTopic,apiKey, apiSecret);
     }
 
+    @Override
     public Producer createProducer(Collection<String> pools, Set<String> writeTopics, String apiKey, String apiSecret) {
         return new DmaapProducerImpl(pools, writeTopics, apiKey, apiSecret);
     }
 
+    @Override
     public Consumer createConsumer(Collection<String> pool, String readTopic, 
-            String clientName, String clientId, String filter_json, String apiKey, String apiSecret) {
-        return new DmaapConsumerImpl(pool, readTopic, clientName, clientId, apiKey, apiSecret, filter_json);
+            String clientName, String clientId, String filterJson, String apiKey, String apiSecret) {
+        return new DmaapConsumerImpl(pool, readTopic, clientName, clientId, apiKey, apiSecret, filterJson);
     }
 }
index 952f903..9418a40 100644 (file)
@@ -49,7 +49,7 @@ public enum MessageService {
     public static MessageService parse(String input) {
         if (input != null) {
             for (MessageService ms : MessageService.values()) {
-                if (ms.getValue().equals(input.toLowerCase())) {
+                if (input.equalsIgnoreCase(ms.getValue())) {
                     return ms;
                 }
             }
index 90532b9..363b7a6 100644 (file)
@@ -29,6 +29,8 @@ package org.onap.appc.adapter.iaas.provider.operation.common.constants;
  */
 public class Property {
 
+    private Property() {
+    }
     public static final String PROVIDER = "provider";
     public static final String PROVIDER_IDENTITY = "identity";
     public static final String PROVIDER_TENANT = "tenant";
index 34f7631..81e1c92 100644 (file)
@@ -27,76 +27,91 @@ package org.onap.appc.adapter.iaas.provider.operation.common.enums;
  */
 public enum Operation {
     EVACUATE_SERVICE {
+        @Override
         public String toString() {
             return "evacuateServer";
         }
     },
     MIGRATE_SERVICE {
+        @Override
         public String toString() {
             return "migrateServer";
         }
     },
     REBUILD_SERVICE {
+        @Override
         public String toString() {
             return "rebuildServer";
         }
     },
     RESTART_SERVICE {
+        @Override
         public String toString() {
             return "restartServer";
         }
     },
     VMSTATUSCHECK_SERVICE {
+        @Override
         public String toString() {
             return "vmStatuschecker";
         }
     },
     SNAPSHOT_SERVICE {
+        @Override
         public String toString() {
             return "createSnapshot";
         }
     },
     TERMINATE_STACK {
+        @Override
         public String toString() {
             return "terminateStack";
         }
     },
     SNAPSHOT_STACK {
+        @Override
         public String toString() {
             return "snapshotStack";
         }
     },
     START_SERVICE {
+        @Override
         public String toString() {
             return "startServer";
         }
     },
     STOP_SERVICE {
+        @Override
         public String toString() {
             return "stopServer";
         }
     },
     TERMINATE_SERVICE {
+        @Override
         public String toString() {
             return "terminateServer";
         }
     },
     LOOKUP_SERVICE {
+        @Override
         public String toString() {
             return "lookupServer";
         }
     },
     RESTORE_STACK {
+        @Override
         public String toString() {
             return "restoreStack";
         }
     },
     ATTACHVOLUME_SERVICE {
+        @Override
         public String toString(){
             return "attachVolume";
         }
     },
     DETACHVOLUME_SERVICE {
+        @Override
         public String toString(){
             return "dettachVolume";
         }
index 9724091..5760879 100644 (file)
@@ -29,11 +29,13 @@ package org.onap.appc.adapter.iaas.provider.operation.common.enums;
  */
 public enum Outcome {
     FAILURE {
+        @Override
         public String toString() {
             return "failure";
         }
     },
     SUCCESS {
+        @Override
         public String toString() {
             return "success";
         }
index 5ff1a33..dccab85 100644 (file)
@@ -29,6 +29,7 @@ public class NetconfDAOException extends RuntimeException {
        private static final long serialVersionUID = -155423437162622414L;
 
        public NetconfDAOException(){
+            // for NetconfDAOException
        }
 
        public NetconfDAOException(String message){
index e8b8e93..86d1750 100644 (file)
@@ -26,6 +26,9 @@ package org.onap.appc.adapter.netconf.util;
 
 public class Constants {
 
+    private Constants() {
+    }
+
     public static final String CONFIGURE_PATH = "/restconf/config/opendaylight-inventory:nodes/node/";
     public static final String CONNECT_PATH = "/restconf/config/opendaylight-inventory:nodes/node/controller-config/yang-ext:mount/config:modules";
     public static final String CHECK_CONNECTION_PATH = "/restconf/operational/opendaylight-inventory:nodes/node/";
index 6dc4134..296ef60 100644 (file)
  */
 package org.onap.appc.adapter.restHealthcheck;
 
-import org.onap.appc.Constants;
 import org.onap.appc.adapter.restHealthcheck.impl.RestHealthcheckAdapterImpl;
 import org.onap.appc.configuration.Configuration;
-import org.onap.appc.configuration.ConfigurationFactory;
-import org.onap.appc.i18n.Msg;
 import com.att.eelf.configuration.EELFLogger;
 import com.att.eelf.configuration.EELFManager;
 import org.osgi.framework.BundleActivator;
@@ -42,11 +39,6 @@ public class RestHealthcheckActivator implements BundleActivator {
     private ServiceRegistration registration = null;
     private static final EELFLogger logger = EELFManager.getInstance().getLogger(RestHealthcheckActivator.class);
 
-    /**
-     * The configuration object used to configure this bundle
-     */
-    private Configuration configuration;
-
     /**
      * Called when this bundle is started so the Framework can perform the bundle-specific activities necessary to start
      * this bundle. This method can be used to register services or to allocate any resources that this bundle needs.