Add test for unknown node identifiers on modify peer command

[jFCPlib.git] / src / main / java / net / pterodactylus / fcp / quelaton / ListPeersCommandImpl.java

diff --git a/src/main/java/net/pterodactylus/fcp/quelaton/ListPeersCommandImpl.java b/src/main/java/net/pterodactylus/fcp/quelaton/ListPeersCommandImpl.java
index 97f3842..921c0fa 100644 (file)
--- a/src/main/java/net/pterodactylus/fcp/quelaton/ListPeersCommandImpl.java
+++ b/src/main/java/net/pterodactylus/fcp/quelaton/ListPeersCommandImpl.java
@@ -3,19 +3,20 @@ package net.pterodactylus.fcp.quelaton;
 import java.io.IOException;
 import java.util.Collection;
 import java.util.HashSet;
+import java.util.concurrent.ExecutionException;
 import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Future;
 import java.util.concurrent.atomic.AtomicBoolean;
 
 import net.pterodactylus.fcp.EndListPeers;
 import net.pterodactylus.fcp.ListPeers;
 import net.pterodactylus.fcp.Peer;
 
+import com.google.common.util.concurrent.ListenableFuture;
 import com.google.common.util.concurrent.ListeningExecutorService;
 import com.google.common.util.concurrent.MoreExecutors;
 
 /**
- * Default {@link ListPeersCommand} implementation based on {@link FcpReplySequence}.
+ * Default {@link ListPeersCommand} implementation based on {@link FcpDialog}.
  *
  * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
  */
@@ -44,18 +45,24 @@ public class ListPeersCommandImpl implements ListPeersCommand {
        }
 
        @Override
-       public Future<Collection<Peer>> execute() {
+       public ListenableFuture<Collection<Peer>> execute() {
+               return threadPool.submit(this::executeSequence);
+       }
+
+       private Collection<Peer> executeSequence() throws InterruptedException, ExecutionException, IOException {
                String identifier = new RandomIdentifierGenerator().generate();
                ListPeers listPeers = new ListPeers(identifier, includeMetadata.get(), includeVolatile.get());
-               return threadPool.submit(() -> new ListPeersReplySequence().send(listPeers).get());
+               try (ListPeersDialog listPeersDialog = new ListPeersDialog()) {
+                       return listPeersDialog.send(listPeers).get();
+               }
        }
 
-       private class ListPeersReplySequence extends FcpReplySequence<Collection<Peer>> {
+       private class ListPeersDialog extends FcpDialog<Collection<Peer>> {
 
                private final Collection<Peer> peers = new HashSet<>();
                private final AtomicBoolean finished = new AtomicBoolean(false);
 
-               public ListPeersReplySequence() throws IOException {
+               public ListPeersDialog() throws IOException {
                        super(threadPool, connectionSupplier.get());
                }