Add in a findbugs suppression to see if it works

Change-Id: I4ca6b65046df35e6d773b515ae819fffb81c43ba

Remove test suppression of findbugs error

Change-Id: I277ac9f5ff20c83e0208cb9adadc12d5a13723d2
diff --git a/core/store/dist/src/main/java/org/onlab/onos/store/service/impl/ClusterMessagingProtocolClient.java b/core/store/dist/src/main/java/org/onlab/onos/store/service/impl/ClusterMessagingProtocolClient.java
index 070ae8c..36bee56 100644
--- a/core/store/dist/src/main/java/org/onlab/onos/store/service/impl/ClusterMessagingProtocolClient.java
+++ b/core/store/dist/src/main/java/org/onlab/onos/store/service/impl/ClusterMessagingProtocolClient.java
@@ -89,9 +89,6 @@
     }
 
     @Override
-//    @edu.umd.cs.findbugs.annotations.SuppressWarnings
-//            (value="NP_NONNULL_PARAM_VIOLATION",
-//             justification="False positives on completedFuture call, passing null as a parameter is allowed")
     public synchronized CompletableFuture<Void> connect() {
         if (remoteNode != null) {
             // done