Make TextMate snippets loading immune to hidden files & invalid plist

closes #143

spec/fixtures/packages/textmate-package.tmbundle/Snippets/.hidden-file

@@ -0,0 +1 @@
+I am hidden so I shouldn't be loaded

spec/fixtures/packages/textmate-package.tmbundle/Snippets/invalid.plist

@@ -0,0 +1 @@
+I am not a valid plist but that shouldn't cause a crisis

src/packages/snippets/spec/snippets-spec.coffee

@@ -228,6 +228,7 @@ describe "Snippets extension", ->
         expect(console.warn.calls.length).toBeGreaterThan 0
 
     it "loads snippets from all TextMate packages with snippets", ->
+      spyOn(console, 'warn').andCallThrough()
       jasmine.unspy(LoadSnippetsTask.prototype, 'start')
       snippets.loaded = false
       snippets.loadAll()
@@ -245,6 +246,10 @@ describe "Snippets extension", ->
           }
         """
 
+        # warn about junk-file, but don't even try to parse a hidden file
+        expect(console.warn).toHaveBeenCalled()
+        expect(console.warn.calls.length).toBeGreaterThan 0
+
   describe "Snippets parser", ->
     it "breaks a snippet body into lines, with each line containing tab stops at the appropriate position", ->
       bodyTree = snippets.parser.parse """

src/packages/snippets/src/load-snippets-handler.coffee

@@ -6,8 +6,21 @@ module.exports =
 
   loadTextmateSnippets: (path) ->
     snippetsDirPath = fs.join(path, 'Snippets')
-    snippets = fs.list(snippetsDirPath).map (snippetPath) ->
-      fs.readPlist(snippetPath)
+    snippets = []
+
+    for snippetsPath in fs.list(snippetsDirPath)
+      logWarning = ->
+        console.warn "Error reading TextMate snippets file '#{snippetsPath}'"
+
+      continue if fs.base(snippetsPath).indexOf('.') is 0
+      try
+        if object = fs.readPlist(snippetsPath)
+          snippets.push(object) if object
+        else
+          logWarning()
+      catch e
+        logWarning()
+
     @snippetsLoaded(@translateTextmateSnippets(snippets))
 
   loadAtomSnippets: (path) ->