Include new ls and rm commands in bash completion

Author: pimterry

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}"))
 }
 

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" {