Add in a findbugs suppression to see if it works

Change-Id: I4ca6b65046df35e6d773b515ae819fffb81c43ba

Remove test suppression of findbugs error

Change-Id: I277ac9f5ff20c83e0208cb9adadc12d5a13723d2

@@ -89,9 +89,6 @@ public class ClusterMessagingProtocolClient implements ProtocolClient {
     }
     }
 
 
     @Override
     @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() {
     public synchronized CompletableFuture<Void> connect() {
         if (remoteNode != null) {
         if (remoteNode != null) {
             // done
             // done