Merge pull request #76 from eshapard/patch-1

Make `ls` ignore vim backup files

Author: pimterry

README.md

@@ -109,7 +109,7 @@ Searches all note content and note filenames for the given string and returns al
 
 ### `notes ls <directory>`
 
-Lists note names and note directories at a single level. Lists all top level notes and directories if no path is provided, or the top-level contents of a directory if one is provided.
+Lists note names and note directories at a single level. Lists all top level notes and directories if no path is provided, or the top-level contents of a directory if one is provided. Automatically ignores hidden files or filenames ending with `~` (Vim backup files).
 
 ### `notes open`
 

notes

@@ -31,7 +31,7 @@ without_notes_dir() {
 }
 
 ls_notes() {
-    local ls_output=$(ls -p "$notes_dir/$*" 2>&1)
+    local ls_output=$(ls -p "$notes_dir/$*" 2>&1 | grep -v "~$")
     local ls_result=$?
     local formatted_output
 

test/test-ls.bats

@@ -30,6 +30,20 @@ notes="./notes"
   assert_line "note2.md"
 }
 
+@test "Should not list temporary files" {
+  touch $NOTES_DIRECTORY/note1.md
+  touch $NOTES_DIRECTORY/note1.md~
+  touch $NOTES_DIRECTORY/note2~with~tilde.md
+
+  run $notes ls
+  assert_success
+
+  assert_line "note1.md"
+  assert_line "note2~with~tilde.md"
+
+  refute_line "note1.md~"
+}
+
 @test "Should list subdirectories with trailing slash" {
   touch $NOTES_DIRECTORY/match-note1.md
   mkdir $NOTES_DIRECTORY/match-dir