fakefs 0.3.2 → 0.4.0

data/CONTRIBUTORS

@@ -4,27 +4,32 @@ Chris Wanstrath <chris@ozmm.org>
 Myles Eftos <myles@madpilot.com.au>
 Jeff Hodges <jeff@somethingsimilar.com>
 Matt Freels <matt@freels.name>
+Eero Saynatkari <projects@kittensoft.org>
 Víctor Martínez <knoopx@gmail.com>
 Nick Quaranto <nick@quaran.to>
 Aaron Suggs <aaron@ktheory.com>
 Victor Costan <costan@gmail.com>
 Eric MSP Veith <eveith@wwweb-library.net>
 Jon Yurek <jyurek@thoughtbot.com>
+Jared Luxenberg <jared@jaredlux.com>
+Lars Gierth <lars.gierth@gmail.com>
 Greg Campbell <gtcampbell@gmail.com>
 Ben Mabey <ben@benmabey.com>
 Mark <mark@amerine.net>
 Sam Goldstein <sam@aboutus.org>
+Ryan McGeary <ryan@mcgeary.org>
 Noah Paessel <knowuh@gmail.com>
 dmathieu <42@dmathieu.com>
 Mariusz Pietrzyk <wijet@wijet.pl>
-Lars Gierth <lars.gierth@gmail.com>
 marano <thiagomarano@gmail.com>
 jameswilding <james@jameswilding.net>
 Tymon Tobolski <i@teamon.eu>
 Scott Barron <scott@elitists.net>
+Andrius Chamentauskas <andrius.chamentauskas@gmail.com>
 Keita Urashima <ursm@ursm.jp>
 David Reese <david@whatcould.com>
 msassak <msassak@gmail.com>
+Andrius Chamentauskas <sinsiliux@gmail.com>
 timo3377 <tim.linquist@gmail.com>
 Mislav Marohnić <mislav.marohnic@gmail.com>
 Rob Sanheim <rsanheim@gmail.com>

data/README.markdown

@@ -47,6 +47,13 @@ Don't Fake the FS Immediately
       # your code
     end
 
+Rails
+-----
+
+If you are using fakefs in a rails project with bundler, you'll probably want to specify the following in your Gemfile:
+
+  gem "fakefs", :require => "fakefs/safe"
+
 
 RSpec
 -----

data/fakefs.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{fakefs}
-  s.version = "0.3.2"
+  s.version = "0.4.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Chris Wanstrath", "Scott Taylor", "Jeff Hodges", "Pat Nakajima"]
-  s.date = %q{2011-05-02}
+  s.date = %q{2011-09-05}
   s.description = %q{A fake filesystem. Use it in your tests.}
   s.email = %q{chris@ozmm.org}
   s.extra_rdoc_files = [
@@ -21,7 +21,6 @@ Gem::Specification.new do |s|
     ".rspec",
     "CONTRIBUTORS",
     "Gemfile",
-    "Gemfile.lock",
     "LICENSE",
     "README.markdown",
     "Rakefile",

data/lib/fakefs/dir.rb

@@ -2,8 +2,13 @@ module FakeFS
   class Dir
     include Enumerable
 
+    def self._check_for_valid_file(path)
+      raise Errno::ENOENT, "No such file or directory - #{path}" unless FileSystem.find(path)
+    end
+
     def initialize(string)
-      raise Errno::ENOENT, string unless FileSystem.find(string)
+      self.class._check_for_valid_file(string)
+
       @path = string
       @open = true
       @pointer = 0
@@ -56,21 +61,28 @@ module FakeFS
       glob(pattern)
     end
 
+    def self.exists?(path)
+      File.exists?(path) && File.directory?(path)
+    end
+
     def self.chdir(dir, &blk)
       FileSystem.chdir(dir, &blk)
     end
 
     def self.chroot(string)
-      # Not implemented yet
+      raise NotImplementedError
     end
 
     def self.delete(string)
-      raise SystemCallError, "No such file or directory - #{string}" unless FileSystem.find(string).values.empty?
+      _check_for_valid_file(string)
+      raise Errno::ENOTEMPTY, "Directory not empty - #{string}" unless FileSystem.find(string).values.empty?
+
       FileSystem.delete(string)
     end
 
     def self.entries(dirname)
-      raise SystemCallError, dirname unless FileSystem.find(dirname)
+      _check_for_valid_file(dirname)
+
       Dir.new(dirname).map { |file| File.basename(file) }
     end
 
@@ -86,8 +98,12 @@ module FakeFS
     def self.mkdir(string, integer = 0)
       parent = string.split('/')
       parent.pop
-      raise Errno::ENOENT, "No such file or directory - #{string}" unless parent.join == "" || FileSystem.find(parent.join('/'))
+
+      joined_parent_path = parent.join
+
+      _check_for_valid_file(joined_parent_path) unless joined_parent_path == ""
       raise Errno::EEXIST, "File exists - #{string}" if File.exists?(string)
+
       FileUtils.mkdir_p(string)
     end
 

data/lib/fakefs/fake/symlink.rb

@@ -1,7 +1,6 @@
 module FakeFS
   class FakeSymlink
-    attr_accessor :name, :target
-    alias_method  :to_s, :name
+    attr_accessor :name, :target, :parent
 
     def initialize(target)
       @target = target
@@ -19,6 +18,10 @@ module FakeFS
       parent.delete(self)
     end
 
+    def to_s
+      File.join(parent.to_s, name)
+    end
+
     def respond_to?(method)
       entry.respond_to?(method)
     end

data/lib/fakefs/file.rb

@@ -37,7 +37,12 @@ module FakeFS
     end
 
     def self.exist?(path)
-      !!FileSystem.find(path)
+      if(File.symlink?(path)) then
+        referent = File.expand_path(File.readlink(path), File.dirname(path))
+        exist?(referent)
+      else
+        !!FileSystem.find(path)
+      end
     end
 
     class << self
@@ -132,7 +137,7 @@ module FakeFS
 
     def self.readlink(path)
       symlink = FileSystem.find(path)
-      FileSystem.find(symlink.target).to_s
+      symlink.target
     end
 
     def self.read(path)
@@ -358,7 +363,7 @@ module FakeFS
       end
 
       def to_path
-        raise NotImplementedError
+        @path
       end
     end
 
@@ -398,8 +403,22 @@ module FakeFS
       (@mode & mask) != 0 if @mode.is_a?(Integer)
     end
 
+    # Create a missing file if the path is valid.
+    #
     def create_missing_file
-      if !File.exists?(@path)
+      raise Errno::EISDIR, "Is a directory - #{path}" if File.directory?(@path)
+
+      if !File.exists?(@path) # Unnecessary check, probably.
+        dirname = RealFile.dirname @path
+
+        unless dirname == "."
+          dir = FileSystem.find dirname
+
+          unless dir.kind_of? FakeDir
+            raise Errno::ENOENT, "No such file or directory - #{path}"
+          end
+        end
+
         @file = FileSystem.add(path, FakeFile.new)
       end
     end

data/lib/fakefs/file_system.rb

@@ -7,7 +7,7 @@ module FakeFS
     end
 
     def fs
-      @fs ||= FakeDir.new('.')
+      @fs ||= FakeDir.new('/')
     end
 
     def clear
@@ -68,6 +68,7 @@ module FakeFS
     def delete(path)
       if node = FileSystem.find(path)
         node.delete
+        true
       end
     end
 
@@ -121,7 +122,8 @@ module FakeFS
           directories_under(dir)
         end
       else
-        dir.reject {|k,v| /\A#{pattern.gsub('?','.').gsub('*', '.*')}\Z/ !~ k }.values
+        regexp_pattern = /\A#{pattern.gsub('?','.').gsub('*', '.*').gsub(/\{(.*?)\}/) { "(#{$1.gsub(',', '|')})" }}\Z/
+        dir.reject {|k,v| regexp_pattern !~ k }.values
       end
 
       if parts.empty? # we're done recursing

data/lib/fakefs/file_test.rb

@@ -7,5 +7,9 @@ module FakeFS
     def self.directory?(file_name)
       File.directory?(file_name)
     end
+
+    def self.file?(file_name)
+      File.file?(file_name)
+    end
   end
 end

data/lib/fakefs/fileutils.rb

@@ -30,7 +30,7 @@ module FakeFS
 
     def rm(list, options = {})
       Array(list).each do |path|
-        FileSystem.delete(path)
+        FileSystem.delete(path) or raise Errno::ENOENT.new(path)
       end
     end
 
@@ -40,9 +40,17 @@ module FakeFS
 
     def ln_s(target, path, options = {})
       options = { :force => false }.merge(options)
-      (FileSystem.find(path) and !options[:force]) ? raise(Errno::EEXIST, path) : FileSystem.delete(path)
+      (FileSystem.find(path) && !options[:force]) ?
+        raise(Errno::EEXIST, path) :
+        FileSystem.delete(path)
+
+      if !options[:force] && !Dir.exists?(File.dirname(path))
+        raise Errno::ENOENT, path
+      end
+
       FileSystem.add(path, FakeSymlink.new(target))
     end
+
     def ln_sf(target, path)
       ln_s(target, path, { :force => true })
     end

data/lib/fakefs/version.rb

@@ -1,6 +1,6 @@
 module FakeFS
   module Version
-    VERSION = "0.3.2"
+    VERSION = "0.4.0"
 
     def self.to_s
       VERSION

data/test/fakefs_test.rb

@@ -65,6 +65,12 @@ class FakeFSTest < Test::Unit::TestCase
     assert_kind_of FakeDir, FileSystem.fs['path']['to']['dir']
   end
 
+  def test_unlink_errors_on_file_not_found
+    assert_raise Errno::ENOENT do
+      FileUtils.rm("/foo")
+    end
+  end
+
   def test_can_delete_directories
     FileUtils.mkdir_p("/path/to/dir")
     FileUtils.rmdir("/path/to/dir")
@@ -111,6 +117,11 @@ class FakeFSTest < Test::Unit::TestCase
     assert File.exists?(path)
   end
 
+  def test_symlink_with_missing_refferent_does_not_exist
+    File.symlink('/foo', '/bar')
+    assert !File.exists?('/bar')
+  end
+
   def test_can_create_symlinks
     FileUtils.mkdir_p(target = "/path/to/target")
     FileUtils.ln_s(target, "/path/to/link")
@@ -141,14 +152,63 @@ class FakeFSTest < Test::Unit::TestCase
     assert_equal target, File.readlink(link)
   end
 
+  def test_symlinks_in_different_directories
+    FileUtils.mkdir_p("/path/to/bar")
+    FileUtils.mkdir_p(target = "/path/to/foo/target")
+
+    FileUtils.ln_s(target, link = "/path/to/bar/symlink")
+    assert_equal target, File.readlink(link)
+  end
+
+  def test_symlink_with_relative_path_exists
+    FileUtils.touch("/file")
+    FileUtils.mkdir_p("/a/b")
+    FileUtils.ln_s("../../file", link = "/a/b/symlink")
+    assert File.exist?('/a/b/symlink')
+  end
+
+  def test_symlink_with_relative_path_and_nonexistant_file_does_not_exist
+    FileUtils.touch("/file")
+    FileUtils.mkdir_p("/a/b")
+    FileUtils.ln_s("../../file_foo", link = "/a/b/symlink")
+    assert !File.exist?('/a/b/symlink')
+  end
+
+  def test_symlink_with_relative_path_has_correct_target
+    FileUtils.touch("/file")
+    FileUtils.mkdir_p("/a/b")
+    FileUtils.ln_s("../../file", link = "/a/b/symlink")
+    assert_equal "../../file", File.readlink(link)
+  end
+
+  def test_symlinks_to_symlinks
+    FileUtils.mkdir_p(target = "/path/to/foo/target")
+    FileUtils.mkdir_p("/path/to/bar")
+    FileUtils.mkdir_p("/path/to/bar2")
+
+    FileUtils.ln_s(target, link1 = "/path/to/bar/symlink")
+    FileUtils.ln_s(link1, link2 = "/path/to/bar2/symlink")
+    assert_equal link1, File.readlink(link2)
+  end
+
+  def test_symlink_to_symlinks_should_raise_error_if_dir_doesnt_exist
+    FileUtils.mkdir_p(target = "/path/to/foo/target")
+
+    assert !Dir.exists?("/path/to/bar")
+
+    assert_raise Errno::ENOENT do
+      FileUtils.ln_s(target, "/path/to/bar/symlink")
+    end
+  end
+
   def test_knows_symlinks_are_symlinks
     FileUtils.mkdir_p(target = "/path/to/target")
     FileUtils.ln_s(target, link = "/path/to/symlink")
     assert File.symlink?(link)
   end
 
-  def test_can_create_files
-    path = '/path/to/file.txt'
+  def test_can_create_files_in_current_dir
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f.write "Yatta!"
     end
@@ -158,7 +218,74 @@ class FakeFSTest < Test::Unit::TestCase
     assert File.writable?(path)
   end
 
+  def test_can_create_files_in_existing_dir
+    FileUtils.mkdir_p "/path/to"
+    path = "/path/to/file.txt"
+
+    File.open(path, 'w') do |f|
+      f.write "Yatta!"
+    end
+
+    assert File.exists?(path)
+    assert File.readable?(path)
+    assert File.writable?(path)
+  end
+
+  def test_raises_ENOENT_trying_to_create_files_in_nonexistent_dir
+    path = "/path/to/file.txt"
+
+    assert_raises(Errno::ENOENT) {
+      File.open(path, 'w') do |f|
+        f.write "Yatta!"
+      end
+    }
+  end
+
+  def test_raises_ENOENT_trying_to_create_files_in_relative_nonexistent_dir
+    FileUtils.mkdir_p "/some/path"
+
+    Dir.chdir("/some/path") {
+      assert_raises(Errno::ENOENT) {
+        File.open("../foo") {|f| f.write "moo" }
+      }
+    }
+  end
+
+  def test_raises_ENOENT_trying_to_create_files_in_obscured_nonexistent_dir
+    FileUtils.mkdir_p "/some/path"
+
+    assert_raises(Errno::ENOENT) {
+      File.open("/some/path/../foo") {|f| f.write "moo" }
+    }
+  end
+
+  def test_raises_ENOENT_trying_to_create_tilde_referenced_nonexistent_dir
+    path = "~/fakefs_test_#{$$}_0000"
+
+    while File.exist? path
+      path = path.succ
+    end
+
+    assert_raises(Errno::ENOENT) {
+      File.open("#{path}/foo") {|f| f.write "moo" }
+    }
+  end
+
+  def test_raises_EISDIR_if_trying_to_open_existing_directory_name
+    path = "/path/to"
+
+    FileUtils.mkdir_p path
+
+    assert_raises(Errno::EISDIR) {
+      File.open(path, 'w') do |f|
+        f.write "Yatta!"
+      end
+    }
+  end
+
   def test_can_create_files_with_bitmasks
+    FileUtils.mkdir_p("/path/to")
+
     path = '/path/to/file.txt'
     File.open(path, File::RDWR | File::CREAT) do |f|
       f.write "Yatta!"
@@ -277,7 +404,7 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_can_read_files_once_written
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f.write "Yatta!"
     end
@@ -286,7 +413,7 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_can_write_to_files
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f << 'Yada Yada'
     end
@@ -300,11 +427,12 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_can_open_file_in_binary_mode
-    File.open("/foo", "wb") { |x| x << "a" }
-    assert_equal "a", File.read("/foo")
+    File.open("foo", "wb") { |x| x << "a" }
+    assert_equal "a", File.read("foo")
   end
 
   def test_can_chunk_io_when_reading
+    FileUtils.mkdir_p "/path/to"
     path = '/path/to/file.txt'
     File.open(path, 'w') do |f|
       f << 'Yada Yada'
@@ -318,7 +446,7 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_can_get_size_of_files
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f << 'Yada Yada'
     end
@@ -326,20 +454,20 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_can_check_if_file_has_size?
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f << 'Yada Yada'
     end
     assert File.size?(path)
-    assert_nil File.size?("/path/to/other.txt")
+    assert_nil File.size?("other.txt")
   end
 
   def test_can_check_size_of_empty_file
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f << ''
     end
-    assert_nil File.size?("/path/to/file.txt")
+    assert_nil File.size?("file.txt")
   end
 
   def test_raises_error_on_mtime_if_file_does_not_exist
@@ -349,16 +477,16 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_can_return_mtime_on_existing_file
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f << ''
     end
-    assert File.mtime('/path/to/file.txt').is_a?(Time)
+    assert File.mtime('file.txt').is_a?(Time)
   end
 
   def test_raises_error_on_ctime_if_file_does_not_exist
     assert_raise Errno::ENOENT do
-      File.ctime('/path/to/file.txt')
+      File.ctime('file.txt')
     end
   end
 
@@ -418,16 +546,16 @@ class FakeFSTest < Test::Unit::TestCase
 
   def test_can_call_utime_on_an_existing_file
     time = Time.now - 300 # Not now
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f << ''
     end
     File.utime(time, time, path)
-    assert_equal time, File.mtime('/path/to/file.txt')
+    assert_equal time, File.mtime('file.txt')
   end
 
   def test_utime_returns_number_of_paths
-    path1, path2 = '/path/to/file.txt', '/path/to/another_file.txt'
+    path1, path2 = 'file.txt', 'another_file.txt'
     [path1, path2].each do |path|
       File.open(path, 'w') do |f|
         f << ''
@@ -437,7 +565,7 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_can_read_with_File_readlines
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f.puts "Yatta!", "Gatta!"
       f.puts ["woot","toot"]
@@ -447,7 +575,7 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_File_close_disallows_further_access
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     file = File.open(path, 'w')
     file.write 'Yada'
     file.close
@@ -457,7 +585,7 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_File_close_disallows_further_writes
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     file = File.open(path, 'w')
     file.write 'Yada'
     file.close
@@ -467,7 +595,7 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_can_read_from_file_objects
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f.write "Yatta!"
     end
@@ -482,7 +610,7 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_knows_files_are_files
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f.write "Yatta!"
     end
@@ -491,17 +619,17 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_File_io_returns_self
-    f = File.open("/foo", "w")
+    f = File.open("foo", "w")
     assert_equal f, f.to_io
   end
 
   def test_File_to_i_is_alias_for_filno
-    f = File.open("/foo", "w")
+    f = File.open("foo", "w")
     assert_equal f.method(:to_i), f.method(:fileno)
   end
 
   def test_knows_symlink_files_are_files
-    path = '/path/to/file.txt'
+    path = 'file.txt'
     File.open(path, 'w') do |f|
       f.write "Yatta!"
     end
@@ -573,16 +701,21 @@ class FakeFSTest < Test::Unit::TestCase
     FileUtils.cp_r '/path', '/otherpath'
 
     assert_equal %w( /otherpath/foo /otherpath/foobar /path/foo /path/foobar ), Dir['/*/foo*']
+
+    assert_equal ['/path/bar', '/path/foo'], Dir['/path/{foo,bar}']
+
+    assert_equal ['/path/bar', '/path/bar2'], Dir['/path/bar{2,}']
   end
 
   def test_dir_glob_handles_root
     FileUtils.mkdir_p '/path'
 
     # this fails. the root dir should be named '/' but it is '.'
-    #assert_equal ['/'], Dir['/']
+    assert_equal ['/'], Dir['/']
   end
 
   def test_dir_glob_handles_recursive_globs
+    FileUtils.mkdir_p "/one/two/three"
     File.open('/one/two/three/four.rb', 'w')
     File.open('/one/five.rb', 'w')
     assert_equal ['/one/five.rb', '/one/two/three/four.rb'], Dir['/one/**/*.rb']
@@ -591,6 +724,7 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_dir_recursive_glob_ending_in_wildcards_returns_both_files_and_dirs
+    FileUtils.mkdir_p "/one/two/three"
     File.open('/one/two/three/four.rb', 'w')
     File.open('/one/five.rb', 'w')
     assert_equal ['/one/five.rb', '/one/two', '/one/two/three', '/one/two/three/four.rb'], Dir['/one/**/*']
@@ -608,7 +742,7 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_should_report_pos_as_0_when_opening
-    File.open("/foo", "w") do |f|
+    File.open("foo", "w") do |f|
       f << "foobar"
       f.rewind
 
@@ -617,7 +751,7 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_should_report_pos_as_1_when_seeking_one_char
-    File.open("/foo", "w") do |f|
+    File.open("foo", "w") do |f|
       f << "foobar"
 
       f.rewind
@@ -628,22 +762,22 @@ class FakeFSTest < Test::Unit::TestCase
   end
 
   def test_should_set_pos
-    File.open("/foo", "w") do |f|
+    File.open("foo", "w") do |f|
       f << "foo"
     end
 
-    fp = File.open("/foo", "r")
+    fp = File.open("foo", "r")
     fp.pos = 1
 
     assert_equal 1, fp.pos
   end
 
   def test_should_set_pos_with_tell_method
-    File.open("/foo", "w") do |f|
+    File.open("foo", "w") do |f|
       f << "foo"
     end
 
-    fp = File.open("/foo", "r")
+    fp = File.open("foo", "r")
     fp.tell = 1
 
     assert_equal 1, fp.pos
@@ -673,14 +807,14 @@ class FakeFSTest < Test::Unit::TestCase
   def test_chdir_changes_directories_like_a_boss
     # I know memes!
     FileUtils.mkdir_p '/path'
-    assert_equal '.', FileSystem.fs.name
+    assert_equal '/', FileSystem.fs.name
     assert_equal({}, FileSystem.fs['path'])
     Dir.chdir '/path' do
       File.open('foo', 'w') { |f| f.write 'foo'}
       File.open('foobar', 'w') { |f| f.write 'foo'}
     end
 
-    assert_equal '.', FileSystem.fs.name
+    assert_equal '/', FileSystem.fs.name
     assert_equal(['foo', 'foobar'], FileSystem.fs['path'].keys.sort)
 
     c = nil
@@ -1211,11 +1345,17 @@ class FakeFSTest < Test::Unit::TestCase
       FileUtils.touch("/this/path/should/be/here/#{f}")
     end
 
-    assert_raises(SystemCallError) do
+    assert_raises(Errno::ENOTEMPTY) do
       Dir.delete('/this/path/should/be/here')
     end
   end
 
+  def test_directory_class_delete_does_not_work_if_dir_path_cannot_be_found
+    assert_raises(Errno::ENOENT) do
+      Dir.delete('/this/path/should/not/be/here')
+    end
+  end
+
   def test_directory_entries
     test = ['.', '..', 'file_1', 'file_2', 'file_3', 'file_4', 'file_5' ]
 
@@ -1244,6 +1384,12 @@ class FakeFSTest < Test::Unit::TestCase
     test.each { |t| assert yielded.include?(t) }
   end
 
+  def test_directory_entries_does_not_work_if_dir_path_cannot_be_found
+    assert_raises(Errno::ENOENT) do
+      Dir.delete('/this/path/should/not/be/here')
+    end
+  end
+
   def test_directory_foreach
     test = ['.', '..', 'file_1', 'file_2', 'file_3', 'file_4', 'file_5' ]
 
@@ -1468,14 +1614,6 @@ class FakeFSTest < Test::Unit::TestCase
     end
   end
 
-  def test_delete_does_not_raise_error_when_second_file_does_not_exist
-    FileUtils.touch("/foo")
-
-    assert_nothing_raised do
-      File.delete("/foo", "/bar")
-    end
-  end
-
   def test_unlink_is_alias_for_delete
     assert_equal File.method(:unlink), File.method(:delete)
   end
@@ -1584,6 +1722,20 @@ class FakeFSTest < Test::Unit::TestCase
     assert !FileTest.directory?('/path/to/somedir')
   end
 
+  def test_filetest_file_returns_correct_values
+    FileUtils.mkdir_p("/path/to")
+
+    path = '/path/to/file.txt'
+    File.open(path, 'w') { |f| f.write "Yatta!" }
+    assert FileTest.file?(path)
+
+    FileUtils.rm path
+    assert !FileTest.file?(path)
+
+    FileUtils.mkdir_p '/path/to/somedir'
+    assert !FileTest.file?('/path/to/somedir')
+  end
+
   def test_pathname_exists_returns_correct_value
     FileUtils.touch "foo"
     assert Pathname.new("foo").exist?
@@ -1651,5 +1803,11 @@ class FakeFSTest < Test::Unit::TestCase
         assert_equal false, f.autoclose?
       end
     end
+
+    def test_to_path
+      File.new("foo", 'w') do |f|
+        assert_equal "foo", f.to_path
+      end
+    end
   end
 end

data/test/safe_test.rb

@@ -10,7 +10,7 @@ class FakeFSSafeTest < Test::Unit::TestCase
   end
 
   def test_FakeFS_method_does_not_intrude_on_global_namespace
-    path = '/path/to/file.txt'
+    path = 'file.txt'
 
     FakeFS do
       File.open(path, 'w') { |f| f.write "Yatta!" }

data/test/verify.rb

@@ -24,7 +24,7 @@ class FakeFSVerifierTest < Test::Unit::TestCase
 
     real_class.instance_methods.each do |method|
       define_method("test #{method} instance method") do
-        assert fake_class.instance_methods.include?(method), "#{fake_class}##{name} not implemented"
+        assert fake_class.instance_methods.include?(method), "#{fake_class}##{method} not implemented"
       end
     end
   end

metadata

@@ -2,7 +2,7 @@
 name: fakefs
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.3.2
+  version: 0.4.0
 platform: ruby
 authors: 
 - Chris Wanstrath
@@ -13,7 +13,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-05-02 00:00:00 -04:00
+date: 2011-09-05 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -74,7 +74,6 @@ files:
 - .rspec
 - CONTRIBUTORS
 - Gemfile
-- Gemfile.lock
 - LICENSE
 - README.markdown
 - Rakefile

data/Gemfile.lock

@@ -1,35 +0,0 @@
-GEM
-  remote: http://rubygems.org/
-  specs:
-    diff-lcs (1.1.2)
-    git (1.2.5)
-    jeweler (1.6.0)
-      bundler (~> 1.0.0)
-      git (>= 1.2.5)
-      rake
-    json (1.5.1)
-    rake (0.8.7)
-    rdiscount (1.6.8)
-    rdoc (2.4.3)
-    rspec (2.5.0)
-      rspec-core (~> 2.5.0)
-      rspec-expectations (~> 2.5.0)
-      rspec-mocks (~> 2.5.0)
-    rspec-core (2.5.1)
-    rspec-expectations (2.5.0)
-      diff-lcs (~> 1.1.2)
-    rspec-mocks (2.5.0)
-    sdoc (0.2.20)
-      json (>= 1.1.3)
-      rdoc (= 2.4.3)
-    sdoc-helpers (0.1.4)
-      sdoc (~> 0.2)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  jeweler
-  rdiscount
-  rspec
-  sdoc-helpers