Merge pull request #153 from dometto/fix_direntries

Fix bug in PathUtils::entries when dir is empty

Author: rafaelfranca

lib/sprockets/path_utils.rb

@@ -55,9 +55,12 @@ def directory?(path)
     # Returns an empty `Array` if the directory does not exist.
     def entries(path)
       if File.directory?(path)
-        Dir.entries(path, encoding: Encoding.default_internal).reject! { |entry|
+        entries = Dir.entries(path, encoding: Encoding.default_internal)
+        entries.reject! { |entry|
           entry =~ /^\.|~$|^\#.*\#$/
-        }.sort!
+        }
+        entries.sort!
+        entries
       else
         []
       end

test/test_path_utils.rb

@@ -43,6 +43,12 @@ def test_entries
       "source-maps",
       "symlink"
     ], entries(File.expand_path("../fixtures", __FILE__))
+    
+    [ ['a', 'b'], ['a', 'b', '.', '..'] ].each do |dir_contents|
+      Dir.stub :entries, dir_contents do
+        assert_equal ['a', 'b'], entries(Dir.tmpdir)
+      end
+    end
 
     assert_equal [], entries("/tmp/sprockets/missingdir")
   end