Merge pull request pimterry#71 from ingkebil/fix-completion

Fix autocompletion for search, ls & short-form commands
Patys · Jul 2, 2019 · 00f9e66 · 00f9e66
2 parents 1fb3d4f + 1ec25b7
commit 00f9e66
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/notes.bash_completion b/notes.bash_completion
@@ -21,7 +21,7 @@ _notes_complete_notes() {
 }
 
 _notes_complete_commands() {
-    local valid_commands="new find grep open ls rm cat"
+    local valid_commands="new find grep open ls rm cat search"
     COMPREPLY=($(compgen -W "${valid_commands}" -- "${1}"))
 }
 
@@ -34,18 +34,21 @@ _notes()
 
     if [[ $COMP_CWORD -gt 1 ]]; then
         case "${COMP_WORDS[1]}" in
-            new)
+            new|n)
                 _notes_complete_notes "$cur"
                 ;;
-            find)
+            find|f)
                 _notes_complete_notes "$cur"
                 ;;
             grep)
                 ;;
-            open)
+            open|o)
                 _notes_complete_notes "$cur"
                 ;;
-            cat)
+            cat|c)
+                _notes_complete_notes "$cur"
+                ;;
+            ls)
                 _notes_complete_notes "$cur"
                 ;;
         esac

diff --git a/test/test-bash-completion.bats b/test/test-bash-completion.bats
@@ -21,6 +21,7 @@ teardown() {
   assert_contains "open" "${COMPREPLY[@]}"
   assert_contains "ls"   "${COMPREPLY[@]}"
   assert_contains "rm"   "${COMPREPLY[@]}"
+  assert_contains "search" "${COMPREPLY[@]}"
 }
 
 @test "Should show matching note when found" {