Include new ls and rm commands in bash completion

Patys · Dec 8, 2016 · d7ff4f3 · d7ff4f3
1 parent 9586aa3
commit d7ff4f3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/notes.bash_completion b/notes.bash_completion
@@ -16,7 +16,7 @@ _notes_complete_notes() {
 }
 
 _notes_complete_commands() {
-    local valid_commands="new find grep open"
+    local valid_commands="new find grep open ls rm"
     COMPREPLY=($(compgen -W "${valid_commands}" -- "${1}"))
 }
 

diff --git a/test/test-bash-completion.bats b/test/test-bash-completion.bats
@@ -17,10 +17,12 @@ teardown() {
 @test "Should list all commands" {
   touch $NOTES_DIRECTORY/note1.md
   _notes_complete_commands ""
-  assert_contains "new" ${COMPREPLY[@]}
-  assert_contains "find" ${COMPREPLY[@]}
-  assert_contains "grep" ${COMPREPLY[@]}
-  assert_contains "open" ${COMPREPLY[@]}
+  assert_contains "new"  "${COMPREPLY[@]}"
+  assert_contains "find" "${COMPREPLY[@]}"
+  assert_contains "grep" "${COMPREPLY[@]}"
+  assert_contains "open" "${COMPREPLY[@]}"
+  assert_contains "ls"   "${COMPREPLY[@]}"
+  assert_contains "rm"   "${COMPREPLY[@]}"
 }
 
 @test "Should show matching note when found" {