✅ Add first test for download command

diff --git a/src/main/java/net/pterodactylus/xdcc/ui/stdin/DownloadCommand.java b/src/main/java/net/pterodactylus/xdcc/ui/stdin/DownloadCommand.java
index 2fdec0d..75e92ca 100644 (file)
--- a/src/main/java/net/pterodactylus/xdcc/ui/stdin/DownloadCommand.java
+++ b/src/main/java/net/pterodactylus/xdcc/ui/stdin/DownloadCommand.java
@@ -65,6 +65,9 @@ public class DownloadCommand implements Command {
 
        @Override
        public State execute(State state, List<String> parameters, Writer outputWriter) throws IOException {
+               if (parameters.isEmpty()) {
+                       return state;
+               }
                Integer index = Ints.tryParse(parameters.get(0));
                if ((index != null) && (index < state.getLastResults().size())) {
                        core.fetch(state.getLastResults().get(index).bot(), state.getLastResults().get(index).pack());

diff --git a/src/test/java/net/pterodactylus/xdcc/ui/stdin/DownloadCommandTest.kt b/src/test/java/net/pterodactylus/xdcc/ui/stdin/DownloadCommandTest.kt
new file mode 100644 (file)
index 0000000..2d3aa0a
--- /dev/null
+++ b/src/test/java/net/pterodactylus/xdcc/ui/stdin/DownloadCommandTest.kt
@@ -0,0 +1,31 @@
+package net.pterodactylus.xdcc.ui.stdin
+
+import net.pterodactylus.xdcc.core.Core
+import org.hamcrest.MatcherAssert.assertThat
+import org.hamcrest.Matchers.equalTo
+import org.mockito.Matchers.any
+import org.mockito.Mockito
+import org.mockito.Mockito.never
+import org.mockito.Mockito.verify
+import java.io.StringWriter
+import kotlin.test.Test
+
+class DownloadCommandTest {
+
+       private val core = Mockito.mock(Core::class.java)
+       private val command = DownloadCommand(core)
+       private val state = State()
+
+       @Test
+       fun `executing command without parameters will not fetch anything`() {
+               command.execute(state, emptyList(), StringWriter())
+               verify(core, never()).fetch(any(), any())
+       }
+
+       @Test
+       fun `executing command without parameters will return old state`() {
+               val newState = command.execute(state, emptyList(), StringWriter())
+               assertThat(newState, equalTo(state))
+       }
+
+}