picocli fish completion tests

.gitignore

@@ -21,5 +21,7 @@ picocli.iml
 /src/test/dejagnu.tests/testrun.log
 /src/test/dejagnu.tests/testrun.sum
 /src/test/dejagnu.tests/tmp/
+/src/test/dejagnu.fishtests/log/completion.sum
+/src/test/dejagnu.fishtests/log/completion.log
 /picocli-legacy-tests/gradle/wrapper/dists/**/*.lck
 /picocli-legacy-tests/gradle/wrapper/dists/**/*.ok

src/main/java/picocli/AutoComplete.java

@@ -554,22 +554,18 @@ public static String fish(String scriptName, CommandLine commandLine) {
                 continue;
             }
             if (!descriptor.parentFunctionName.equals(parentFunction)) {
-                if (!currentLevelCommands.isEmpty()) {
-                    processLevel(scriptName, result, currentLevel, currentLevelCommands, parentFunction, rootDescriptor);
-                    rootDescriptor = null;
+                processLevel(scriptName, result, currentLevel, currentLevelCommands, parentFunction, rootDescriptor);
+                rootDescriptor = null;
 
-                    currentLevel.clear();
-                    currentLevelCommands.clear();
-                }
+                currentLevel.clear();
+                currentLevelCommands.clear();
                 parentFunction = descriptor.parentFunctionName;
             }
 
             currentLevel.add(descriptor);
             currentLevelCommands.add(descriptor.commandName);
         }
-        if (!currentLevelCommands.isEmpty()) {
-            processLevel(scriptName, result, currentLevel, currentLevelCommands, parentFunction, rootDescriptor);
-        }
+        processLevel(scriptName, result, currentLevel, currentLevelCommands, parentFunction, rootDescriptor);
 
 
         return result.toString();
@@ -578,7 +574,10 @@ public static String fish(String scriptName, CommandLine commandLine) {
     private static void processLevel(String scriptName, StringBuilder result, List<CommandDescriptor> currentLevel,
                                      List<String> currentLevelCommands, String levelName,
                                      CommandDescriptor rootDescriptor) {
-        result.append("\n# ").append(levelName).append(" completion \n");
+        if (levelName.equals("")) {
+            levelName = "root";
+        }
+        result.append("\n# ").append(levelName).append(" completion\n");
         result.append("set -l ").append(levelName).append(" ").append(String.join(" ", currentLevelCommands)).append(
                 "\n");
         if (rootDescriptor != null) {

src/test/dejagnu.fishtests/README.adoc

@@ -0,0 +1,4 @@
+Run
+```
+runtest --outdir log --tool completion
+```

src/test/dejagnu.fishtests/completion/simple.exp

@@ -0,0 +1,29 @@
+exp_internal 1
+set timeout 1
+
+proc run_completion_test {cmd test candidates} {
+    send "${cmd}\t"
+    expect {
+        -re "(\n${candidates}\u001b)" { pass $test }
+        timeout { fail $test }
+    }
+    puts "###Output"
+    puts "$expect_out(1,string)"
+    send "\x03"
+    expect ">"
+}
+
+set cmd "basicExample -"
+set test "Tab should show options for '$cmd'"
+set candidates "-t  -u  --timeout  --timeUnit"
+run_completion_test $cmd $test $candidates
+
+set cmd "basicExample --"
+set test "Tab should show options for '$cmd'"
+set candidates "--timeout  --timeUnit"
+run_completion_test $cmd $test $candidates
+
+set cmd "basicExample --timeUnit "
+set test "Tab should show time unit enum values for '$cmd'"
+set candidates "DAYS\\s*HOURS\\s*MICROSECONDS\\s*MILLISECONDS\\s*MINUTES\\s*NANOSECONDS\\s*SECONDS\r\n/@${cmd}"
+run_completion_test $cmd $test $candidates

src/test/dejagnu.fishtests/lib/completion.exp

@@ -0,0 +1,8 @@
+exp_spawn fish --no-config
+expect -re "(.+>)"
+
+send "source ../resources/basic.fish\r"
+expect -re "(.+>)"
+
+send "function basicExample; echo 'do'; end\r"
+expect -re "(.+>)"

src/test/java/picocli/AutoCompleteTest.java

@@ -102,6 +102,14 @@ public void basic() throws Exception {
         assertEquals(expected, script);
     }
 
+    @Test
+    public void basicFish() throws Exception {
+        String script = AutoComplete.fish("basicExample", new CommandLine(new BasicExample()));
+        System.out.println(script);
+        String expected = loadTextFromClasspath("/basic.fish");
+        assertEquals(expected, script);
+    }
+
     public static class TopLevel {
         @Option(names = {"-V", "--version"}, help = true) boolean versionRequested;
         @Option(names = {"-h", "--help"}, help = true) boolean helpRequested;

src/test/resources/basic.fish

@@ -0,0 +1,7 @@
+
+# root completion
+set -l root 
+complete -c basicExample -n "not __fish_seen_subcommand_from $root" -l timeUnit -d ''
+complete -c basicExample -n "not __fish_seen_subcommand_from $root" -s u -d ''
+complete -c basicExample -n "not __fish_seen_subcommand_from $root" -l timeout -d ''
+complete -c basicExample -n "not __fish_seen_subcommand_from $root" -s t -d ''