memfs 0.4.1 → 0.4.2

data/lib/memfs/file/stat.rb

@@ -21,7 +21,7 @@ module MemFs
                      :uid
 
       def blockdev?
-        !!entry.block_device
+        !!(entry.block_device)
       end
 
       def chardev?
@@ -110,7 +110,7 @@ module MemFs
       end
 
       def zero?
-        entry.content && entry.content.empty?
+        !!(entry.content && entry.content.empty?)
       end
 
       private

data/lib/memfs/file_system.rb

@@ -23,15 +23,13 @@ module MemFs
 
       block.call if block
     ensure
-      if block
-        self.working_directory = previous_directory
-      end
+      self.working_directory = previous_directory if block
     end
 
     def clear!
       self.root = Fake::Directory.new('/')
-      self.mkdir('/tmp')
-      self.chdir('/')
+      mkdir '/tmp'
+      chdir '/'
     end
 
     def chmod(mode_int, file_name)
@@ -63,13 +61,13 @@ module MemFs
     end
 
     def find!(path)
-      find(path) || raise(Errno::ENOENT, path)
+      find(path) || fail(Errno::ENOENT, path)
     end
 
     def find_directory!(path)
       entry = find!(path).dereferenced
 
-      raise Errno::ENOTDIR, path unless entry.is_a?(Fake::Directory)
+      fail Errno::ENOTDIR, path unless entry.is_a?(Fake::Directory)
 
       entry
     end
@@ -82,7 +80,7 @@ module MemFs
     def getwd
       working_directory.path
     end
-    alias :pwd :getwd
+    alias_method :pwd, :getwd
 
     def initialize
       clear!
@@ -91,7 +89,7 @@ module MemFs
     def link(old_name, new_name)
       file = find!(old_name)
 
-      raise Errno::EEXIST, "(#{old_name}, #{new_name})" if find(new_name)
+      fail Errno::EEXIST, "(#{old_name}, #{new_name})" if find(new_name)
 
       link = file.dup
       link.name = basename(new_name)
@@ -99,7 +97,7 @@ module MemFs
     end
 
     def mkdir(path)
-      raise Errno::EEXIST, path if find(path)
+      fail Errno::EEXIST, path if find(path)
       find_parent!(path).add_entry Fake::Directory.new(path)
     end
 
@@ -118,13 +116,13 @@ module MemFs
     def rmdir(path)
       directory = find!(path)
 
-      raise Errno::ENOTEMPTY, path unless directory.empty?
+      fail Errno::ENOTEMPTY, path unless directory.empty?
 
       directory.delete
     end
 
     def symlink(old_name, new_name)
-      raise Errno::EEXIST, new_name if find(new_name)
+      fail Errno::EEXIST, new_name if find(new_name)
 
       find_parent!(new_name).add_entry Fake::Symlink.new(new_name, old_name)
     end
@@ -146,7 +144,7 @@ module MemFs
     def unlink(path)
       entry = find!(path)
 
-      raise Errno::EPERM, path if entry.is_a?(Fake::Directory)
+      fail Errno::EPERM, path if entry.is_a?(Fake::Directory)
 
       entry.delete
     end

data/lib/memfs/io.rb

@@ -0,0 +1,201 @@
+require 'forwardable'
+require 'memfs/filesystem_access'
+
+module MemFs
+  module IO
+    module ClassMethods
+      def read(path, *args)
+        options = args.last.is_a?(Hash) ? args.pop : {}
+        options = { mode: File::RDONLY, encoding: nil, open_args: nil }.merge(options)
+        open_args = options[:open_args] ||
+                    [options[:mode], encoding: options[:encoding]]
+
+        length, offset = args
+
+        file = open(path, *open_args)
+        file.seek(offset || 0)
+        file.read(length)
+      ensure
+        file.close if file
+      end
+    end
+
+    module InstanceMethods
+      attr_writer :autoclose,
+                  :close_on_exec
+
+      def <<(object)
+        fail IOError, 'not opened for writing' unless writable?
+
+        content << object.to_s
+      end
+
+      def advise(advice_type, offset = 0, len = 0)
+        advice_types = [
+          :dontneed,
+          :noreuse,
+          :normal,
+          :random,
+          :sequential,
+          :willneed
+        ]
+        unless advice_types.include?(advice_type)
+          fail NotImplementedError, "Unsupported advice: #{advice_type.inspect}"
+        end
+        nil
+      end
+
+      def autoclose?
+        @autoclose.nil? ? true : !!@autoclose
+      end
+
+      def binmode
+        @binmode = true
+        @external_encoding = Encoding::ASCII_8BIT
+        self
+      end
+
+      def binmode?
+        @binmode.nil? ? false : @binmode
+      end
+
+      def close
+        self.closed = true
+      end
+
+      def closed?
+        closed
+      end
+
+      def close_on_exec?
+        @close_on_exec.nil? ? true : !!@close_on_exec
+      end
+
+      def eof?
+        pos >= content.size
+      end
+      alias_method :eof, :eof?
+
+      def external_encoding
+        if writable?
+          @external_encoding
+        else
+          @external_encoding ||= Encoding.default_external
+        end
+      end
+
+      def each(sep = $/, &block)
+        return to_enum(__callee__) unless block_given?
+        fail IOError, 'not opened for reading' unless readable?
+        content.each_line(sep) { |line| block.call(line) }
+        self
+      end
+
+      def each_byte(&block)
+        return to_enum(__callee__) unless block_given?
+        fail IOError, 'not opened for reading' unless readable?
+        content.each_byte { |byte| block.call(byte) }
+        self
+      end
+      alias_method :bytes, :each_byte
+
+      def each_char(&block)
+        return to_enum(__callee__) unless block_given?
+        fail IOError, 'not opened for reading' unless readable?
+        content.each_char { |char| block.call(char) }
+        self
+      end
+      alias_method :chars, :each_char
+
+      def pos
+        entry.pos
+      end
+
+      def print(*objs)
+        $stdout.puts $_.inspect
+        objs << $_ if objs.empty?
+        self << objs.join($,) << $\.to_s
+        nil
+      end
+
+      def printf(format_string, *objs)
+        print format_string % objs
+      end
+
+      def puts(text)
+        fail IOError, 'not opened for writing' unless writable?
+
+        content.puts text
+      end
+
+      def read(length = nil, buffer = '')
+        default = length ? nil : ''
+        content.read(length, buffer) || default
+      end
+
+      def seek(amount, whence = ::IO::SEEK_SET)
+        new_pos = case whence
+                  when ::IO::SEEK_CUR then entry.pos + amount
+                  when ::IO::SEEK_END then content.to_s.length + amount
+                  when ::IO::SEEK_SET then amount
+                  end
+
+        fail Errno::EINVAL, path if new_pos.nil? || new_pos < 0
+
+        entry.pos = new_pos
+        0
+      end
+
+      def stat
+        File.stat(path)
+      end
+
+      def write(string)
+        fail IOError, 'not opened for writing' unless writable?
+
+        content.write(string.to_s)
+      end
+
+      private
+
+      attr_accessor :closed,
+                    :entry,
+                    :opening_mode
+
+      attr_reader :path
+
+      def content
+        entry.content
+      end
+
+      def create_file?
+        (opening_mode & File::CREAT).nonzero?
+      end
+
+      def readable?
+        (opening_mode & File::RDWR).nonzero? ||
+        (opening_mode | File::RDONLY).zero?
+      end
+
+      def str_to_mode_int(mode)
+        return mode unless mode.is_a?(String)
+
+        unless mode =~ /\A([rwa]\+?)([bt])?\z/
+          fail ArgumentError, "invalid access mode #{mode}"
+        end
+
+        mode_str = $~[1]
+        File::MODE_MAP[mode_str]
+      end
+
+      def truncate_file?
+        (opening_mode & File::TRUNC).nonzero?
+      end
+
+      def writable?
+        (opening_mode & File::WRONLY).nonzero? ||
+        (opening_mode & File::RDWR).nonzero?
+      end
+    end
+  end
+end

data/lib/memfs/version.rb

@@ -1,3 +1,3 @@
 module MemFs
-  VERSION = '0.4.1'
+  VERSION = '0.4.2'
 end

data/memfs.gemspec

@@ -4,28 +4,28 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'memfs/version'
 
 Gem::Specification.new do |gem|
-  gem.name          = "memfs"
+  gem.name          = 'memfs'
   gem.version       = MemFs::VERSION
-  gem.authors       = ["Simon COURTOIS"]
-  gem.email         = ["scourtois@cubyx.fr"]
-  gem.description   = "MemFs provides a fake file system that can be used " \
-                      "for tests. Strongly inspired by FakeFS."
+  gem.authors       = ['Simon COURTOIS']
+  gem.email         = ['scourtois@cubyx.fr']
+  gem.description   = 'MemFs provides a fake file system that can be used ' \
+                      'for tests. Strongly inspired by FakeFS.'
   gem.summary       = "memfs-#{MemFs::VERSION}"
-  gem.homepage      = "http://github.com/simonc/memfs"
+  gem.homepage      = 'http://github.com/simonc/memfs'
 
   gem.license       = 'MIT'
 
   gem.files         = `git ls-files`.split($/)
-  gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
-  gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
-  gem.require_paths = ["lib"]
+  gem.executables   = gem.files.grep(/^bin\//).map { |f| File.basename(f) }
+  gem.test_files    = gem.files.grep(/^(test|spec|features)\//)
+  gem.require_paths = ['lib']
 
-  gem.add_development_dependency "coveralls", "~> 0.6"
-  gem.add_development_dependency "rake", "~> 10.0"
-  gem.add_development_dependency "rspec", "~> 2.14"
-  gem.add_development_dependency "guard", "~> 1.5"
-  gem.add_development_dependency "guard-rspec", "~> 2.1"
-  gem.add_development_dependency "rb-inotify", "~> 0.8"
-  gem.add_development_dependency "rb-fsevent", "~> 0.9"
-  gem.add_development_dependency "rb-fchange", "~> 0.0"
+  gem.add_development_dependency 'coveralls', '~> 0.6'
+  gem.add_development_dependency 'rake', '~> 10.0'
+  gem.add_development_dependency 'rspec', '~> 3.0'
+  gem.add_development_dependency 'guard', '~> 2.6'
+  gem.add_development_dependency 'guard-rspec', '~> 4.3'
+  gem.add_development_dependency 'rb-inotify', '~> 0.8'
+  gem.add_development_dependency 'rb-fsevent', '~> 0.9'
+  gem.add_development_dependency 'rb-fchange', '~> 0.0'
 end

data/spec/fileutils_spec.rb

@@ -15,86 +15,84 @@ describe FileUtils do
   end
 
   describe '.cd' do
-    it "changes the current working directory" do
+    it 'changes the current working directory' do
       FileUtils.cd '/test'
       expect(FileUtils.pwd).to eq('/test')
     end
 
-    it "returns nil" do
+    it 'returns nil' do
       expect(FileUtils.cd('/test')).to be_nil
     end
 
     it "raises an error when the given path doesn't exist" do
-      expect { FileUtils.cd('/nowhere') }.to raise_error(Errno::ENOENT)
+      expect { FileUtils.cd('/nowhere') }.to raise_specific_error(Errno::ENOENT)
     end
 
-    it "raises an error when the given path is not a directory" do
+    it 'raises an error when the given path is not a directory' do
       FileUtils.touch('/test-file')
-      expect { FileUtils.cd('/test-file') }.to raise_error(Errno::ENOTDIR)
+      expect { FileUtils.cd('/test-file') }.to raise_specific_error(Errno::ENOTDIR)
     end
 
-    context "when called with a block" do
-      it "changes current working directory for the block execution" do
+    context 'when called with a block' do
+      it 'changes current working directory for the block execution' do
         FileUtils.cd '/test' do
           expect(FileUtils.pwd).to eq('/test')
         end
       end
 
-      it "resumes to the old working directory after the block execution finished" do
+      it 'resumes to the old working directory after the block execution finished' do
         FileUtils.cd '/'
-        expect {
-          FileUtils.cd('/test') {}
-        }.to_not change{FileUtils.pwd}
+        expect { FileUtils.cd('/test') {} }.to_not change { FileUtils.pwd }
       end
     end
 
-    context "when the destination is a symlink" do
+    context 'when the destination is a symlink' do
       before :each do
         FileUtils.symlink('/test', '/test-link')
       end
 
-      it "changes directory to the last target of the link chain" do
+      it 'changes directory to the last target of the link chain' do
         FileUtils.cd('/test-link')
         expect(FileUtils.pwd).to eq('/test')
       end
 
       it "raises an error if the last target of the link chain doesn't exist" do
-        expect { FileUtils.cd('/nowhere-link') }.to raise_error(Errno::ENOENT)
+        expect { FileUtils.cd('/nowhere-link') }.to raise_specific_error(Errno::ENOENT)
       end
     end
   end
 
   describe '.chmod' do
-    it "changes permission bits on the named file to the bit pattern represented by mode" do
+    it 'changes permission bits on the named file to the bit pattern represented by mode' do
       FileUtils.touch '/test-file'
       FileUtils.chmod 0777, '/test-file'
       expect(File.stat('/test-file').mode).to eq(0100777)
     end
 
-    it "changes permission bits on the named files (in list) to the bit pattern represented by mode" do
+    it 'changes permission bits on the named files (in list) to the bit pattern represented by mode' do
       FileUtils.touch ['/test-file', '/test-file2']
       FileUtils.chmod 0777, ['/test-file', '/test-file2']
       expect(File.stat('/test-file2').mode).to eq(0100777)
     end
 
-    it "returns an array containing the file names" do
+    it 'returns an array containing the file names' do
       file_names = %w[/test-file /test-file2]
       FileUtils.touch file_names
       expect(FileUtils.chmod(0777, file_names)).to eq(file_names)
     end
 
-    it "raises an error if an entry does not exist" do
-      expect { FileUtils.chmod(0777, '/test-file') }.to raise_error(Errno::ENOENT)
+    it 'raises an error if an entry does not exist' do
+      expect { FileUtils.chmod(0777, '/test-file') }.to raise_specific_error(Errno::ENOENT)
     end
 
-    context "when the named file is a symlink" do
+    context 'when the named file is a symlink' do
       before :each do
         FileUtils.touch '/test-file'
         FileUtils.symlink '/test-file', '/test-link'
       end
 
-      context "when File responds to lchmod" do
-        it "changes the mode on the link" do
+      context 'when File responds to lchmod' do
+        it 'changes the mode on the link' do
           FileUtils.chmod(0777, '/test-link')
           expect(File.lstat('/test-link').mode).to eq(0100777)
         end
@@ -107,8 +105,9 @@ describe FileUtils do
       end
 
       context "when File doesn't respond to lchmod" do
-        it "does nothing" do
-          FileUtils::Entry_.any_instance.stub(:have_lchmod?).and_return(false)
+        it 'does nothing' do
+          allow_any_instance_of(FileUtils::Entry_).to \
+            receive_messages(have_lchmod?: false)
           mode = File.lstat('/test-link').mode
           FileUtils.chmod(0777, '/test-link')
           expect(File.lstat('/test-link').mode).to eq(mode)
@@ -122,40 +121,40 @@ describe FileUtils do
       FileUtils.touch '/test/test-file'
     end
 
-    it "changes the permission bits on the named entry" do
+    it 'changes the permission bits on the named entry' do
       FileUtils.chmod_R(0777, '/test')
       expect(File.stat('/test').mode).to eq(0100777)
     end
 
-    it "changes the permission bits on any sub-directory of the named entry" do
+    it 'changes the permission bits on any sub-directory of the named entry' do
       FileUtils.chmod_R(0777, '/')
       expect(File.stat('/test').mode).to eq(0100777)
     end
 
-    it "changes the permission bits on any descendant file of the named entry" do
+    it 'changes the permission bits on any descendant file of the named entry' do
       FileUtils.chmod_R(0777, '/')
       expect(File.stat('/test/test-file').mode).to eq(0100777)
     end
   end
 
   describe '.chown' do
-    it "changes owner on the named file" do
+    it 'changes owner on the named file' do
       FileUtils.chown(42, nil, '/test')
       expect(File.stat('/test').uid).to eq(42)
     end
 
-    it "changes owner on the named files (in list)" do
+    it 'changes owner on the named files (in list)' do
       FileUtils.touch('/test-file')
       FileUtils.chown(42, nil, ['/test', '/test-file'])
       expect(File.stat('/test-file').uid).to eq(42)
     end
 
-    it "changes group on the named entry" do
+    it 'changes group on the named entry' do
       FileUtils.chown(nil, 42, '/test')
       expect(File.stat('/test').gid).to eq(42)
     end
 
-    it "changes group on the named entries in list" do
+    it 'changes group on the named entries in list' do
       FileUtils.touch('/test-file')
       FileUtils.chown(nil, 42, ['/test', '/test-file'])
       expect(File.stat('/test-file').gid).to eq(42)
@@ -171,18 +170,18 @@ describe FileUtils do
       expect(File.stat('/test').gid).not_to be(42)
     end
 
-    context "when the name entry is a symlink" do
+    context 'when the name entry is a symlink' do
       before :each do
         FileUtils.touch '/test-file'
         FileUtils.symlink '/test-file', '/test-link'
       end
 
-      it "changes the owner on the last target of the link chain" do
+      it 'changes the owner on the last target of the link chain' do
         FileUtils.chown(42, nil, '/test-link')
         expect(File.stat('/test-file').uid).to eq(42)
       end
 
-      it "changes the group on the last target of the link chain" do
+      it 'changes the group on the last target of the link chain' do
         FileUtils.chown(nil, 42, '/test-link')
         expect(File.stat('/test-file').gid).to eq(42)
       end
@@ -204,32 +203,32 @@ describe FileUtils do
       FileUtils.touch '/test/test-file'
     end
 
-    it "changes the owner on the named entry" do
+    it 'changes the owner on the named entry' do
       FileUtils.chown_R(42, nil, '/test')
       expect(File.stat('/test').uid).to eq(42)
     end
 
-    it "changes the group on the named entry" do
+    it 'changes the group on the named entry' do
       FileUtils.chown_R(nil, 42, '/test')
       expect(File.stat('/test').gid).to eq(42)
     end
 
-    it "changes the owner on any sub-directory of the named entry" do
+    it 'changes the owner on any sub-directory of the named entry' do
       FileUtils.chown_R(42, nil, '/')
       expect(File.stat('/test').uid).to eq(42)
     end
 
-    it "changes the group on any sub-directory of the named entry" do
+    it 'changes the group on any sub-directory of the named entry' do
       FileUtils.chown_R(nil, 42, '/')
       expect(File.stat('/test').gid).to eq(42)
     end
 
-    it "changes the owner on any descendant file of the named entry" do
+    it 'changes the owner on any descendant file of the named entry' do
       FileUtils.chown_R(42, nil, '/')
       expect(File.stat('/test/test-file').uid).to eq(42)
     end
 
-    it "changes the group on any descendant file of the named entry" do
+    it 'changes the group on any descendant file of the named entry' do
       FileUtils.chown_R(nil, 42, '/')
       expect(File.stat('/test/test-file').gid).to eq(42)
     end
@@ -240,40 +239,40 @@ describe FileUtils do
   end
 
   describe '.compare_file' do
-    it "returns true if the contents of a file A and a file B are identical" do
-      File.open('/test-file', 'w')  { |f| f.puts "this is a test" }
-      File.open('/test-file2', 'w') { |f| f.puts "this is a test" }
+    it 'returns true if the contents of a file A and a file B are identical' do
+      File.open('/test-file', 'w')  { |f| f.puts 'this is a test' }
+      File.open('/test-file2', 'w') { |f| f.puts 'this is a test' }
 
-      expect(FileUtils.compare_file('/test-file', '/test-file2')).to be_true
+      expect(FileUtils.compare_file('/test-file', '/test-file2')).to be true
     end
 
-    it "returns false if the contents of a file A and a file B are not identical" do
-      File.open('/test-file', 'w')  { |f| f.puts "this is a test" }
-      File.open('/test-file2', 'w') { |f| f.puts "this is not a test" }
+    it 'returns false if the contents of a file A and a file B are not identical' do
+      File.open('/test-file', 'w')  { |f| f.puts 'this is a test' }
+      File.open('/test-file2', 'w') { |f| f.puts 'this is not a test' }
 
-      expect(FileUtils.compare_file('/test-file', '/test-file2')).to be_false
+      expect(FileUtils.compare_file('/test-file', '/test-file2')).to be false
     end
   end
 
   describe '.compare_stream' do
-    it "returns true if the contents of a stream A and stream B are identical" do
-      File.open('/test-file', 'w')  { |f| f.puts "this is a test" }
-      File.open('/test-file2', 'w') { |f| f.puts "this is a test" }
+    it 'returns true if the contents of a stream A and stream B are identical' do
+      File.open('/test-file', 'w')  { |f| f.puts 'this is a test' }
+      File.open('/test-file2', 'w') { |f| f.puts 'this is a test' }
 
       file1 = File.open('/test-file')
       file2 = File.open('/test-file2')
 
-      expect(FileUtils.compare_stream(file1, file2)).to be_true
+      expect(FileUtils.compare_stream(file1, file2)).to be true
     end
 
-    it "returns false if the contents of a stream A and stream B are not identical" do
-      File.open('/test-file', 'w')  { |f| f.puts "this is a test" }
-      File.open('/test-file2', 'w') { |f| f.puts "this is not a test" }
+    it 'returns false if the contents of a stream A and stream B are not identical' do
+      File.open('/test-file', 'w')  { |f| f.puts 'this is a test' }
+      File.open('/test-file2', 'w') { |f| f.puts 'this is not a test' }
 
       file1 = File.open('/test-file')
       file2 = File.open('/test-file2')
 
-      expect(FileUtils.compare_stream(file1, file2)).to be_false
+      expect(FileUtils.compare_stream(file1, file2)).to be false
     end
   end
 
@@ -282,57 +281,57 @@ describe FileUtils do
   end
 
   describe '.copy_entry' do
-    it "copies a file system entry +src+ to +dest+" do
+    it 'copies a file system entry +src+ to +dest+' do
       File.open('/test-file', 'w') { |f| f.puts 'test' }
       FileUtils.copy_entry('/test-file', '/test-copy')
       expect(File.read('/test-copy')).to eq("test\n")
     end
 
-    it "preserves file types" do
+    it 'preserves file types' do
       FileUtils.touch('/test-file')
       FileUtils.symlink('/test-file', '/test-link')
       FileUtils.copy_entry('/test-link', '/test-copy')
-      expect(File.symlink?('/test-copy')).to be_true
+      expect(File.symlink?('/test-copy')).to be true
     end
 
-    context "when +src+ does not exist" do
-      it "raises an exception" do
+    context 'when +src+ does not exist' do
+      it 'raises an exception' do
         expect {
           FileUtils.copy_entry('/test-file', '/test-copy')
-        }.to raise_error(RuntimeError)
+        }.to raise_specific_error(RuntimeError)
       end
     end
 
-    context "when +preserve+ is true" do
+    context 'when +preserve+ is true' do
       let(:time) { Date.parse('2013-01-01') }
 
       before :each do
         FileUtils.touch('/test-file')
         FileUtils.chown(1042, 1042, '/test-file')
         FileUtils.chmod(0777, '/test-file')
-        fs.find('/test-file').mtime = time
+        _fs.find('/test-file').mtime = time
         FileUtils.copy_entry('/test-file', '/test-copy', true)
       end
 
-      it "preserves owner" do
+      it 'preserves owner' do
         expect(File.stat('/test-copy').uid).to eq(1042)
       end
 
-      it "preserves group" do
+      it 'preserves group' do
         expect(File.stat('/test-copy').gid).to eq(1042)
       end
 
-      it "preserves permissions" do
+      it 'preserves permissions' do
         expect(File.stat('/test-copy').mode).to eq(0100777)
       end
 
-      it "preserves modified time" do
+      it 'preserves modified time' do
         expect(File.stat('/test-copy').mtime).to eq(time)
       end
     end
 
-    context "when +dest+ already exists" do
-      it "overwrite it" do
+    context 'when +dest+ already exists' do
+      it 'overwrite it' do
         File.open('/test-file', 'w') { |f| f.puts 'test' }
         FileUtils.touch('/test-copy')
         FileUtils.copy_entry('/test-file', '/test-copy')
@@ -340,25 +339,25 @@ describe FileUtils do
       end
     end
 
-    context "when +remove_destination+ is true" do
-      it "removes each destination file before copy" do
+    context 'when +remove_destination+ is true' do
+      it 'removes each destination file before copy' do
         FileUtils.touch(['/test-file', '/test-copy'])
         expect(File).to receive(:unlink).with('/test-copy')
         FileUtils.copy_entry('/test-file', '/test-copy', false, false, true)
       end
     end
 
-    context "when +src+ is a directory" do
-      it "copies its contents recursively" do
+    context 'when +src+ is a directory' do
+      it 'copies its contents recursively' do
         FileUtils.mkdir_p('/test-dir/test-sub-dir')
         FileUtils.copy_entry('/test-dir', '/test-copy')
-        expect(Dir.exists?('/test-copy/test-sub-dir')).to be_true
+        expect(Dir.exist?('/test-copy/test-sub-dir')).to be true
       end
     end
   end
 
   describe '.copy_file' do
-    it "copies file contents of src to dest" do
+    it 'copies file contents of src to dest' do
       File.open('/test-file', 'w') { |f| f.puts 'test' }
       FileUtils.copy_file('/test-file', '/test-file2')
       expect(File.read('/test-file2')).to eq("test\n")
@@ -374,70 +373,74 @@ describe FileUtils do
       File.open('/test-file', 'w') { |f| f.puts 'test' }
     end
 
-    it "copies a file content +src+ to +dest+" do
+    it 'copies a file content +src+ to +dest+' do
       FileUtils.cp('/test-file', '/copy-file')
       expect(File.read('/copy-file')).to eq("test\n")
     end
 
-    context "when +src+ and +dest+ are the same file" do
-      it "raises an error" do
-        expect { FileUtils.cp('/test-file', '/test-file') }.to raise_error(ArgumentError)
+    context 'when +src+ and +dest+ are the same file' do
+      it 'raises an error' do
+        expect {
+          FileUtils.cp('/test-file', '/test-file')
+        }.to raise_specific_error(ArgumentError)
       end
     end
 
-    context "when +dest+ is a directory" do
-      it "copies +src+ to +dest/src+" do
+    context 'when +dest+ is a directory' do
+      it 'copies +src+ to +dest/src+' do
         FileUtils.mkdir('/dest')
         FileUtils.cp('/test-file', '/dest/copy-file')
         expect(File.read('/dest/copy-file')).to eq("test\n")
       end
     end
 
-    context "when src is a list of files" do
-      context "when +dest+ is not a directory" do
-        it "raises an error" do
+    context 'when src is a list of files' do
+      context 'when +dest+ is not a directory' do
+        it 'raises an error' do
           FileUtils.touch(['/dest', '/test-file2'])
-          expect { FileUtils.cp(['/test-file', '/test-file2'], '/dest') }.to raise_error(Errno::ENOTDIR)
+          expect {
+            FileUtils.cp(['/test-file', '/test-file2'], '/dest')
+          }.to raise_specific_error(Errno::ENOTDIR)
         end
       end
     end
   end
 
   describe '.cp_r' do
-    it "copies +src+ to +dest+" do
+    it 'copies +src+ to +dest+' do
       File.open('/test-file', 'w') { |f| f.puts 'test' }
 
       FileUtils.cp_r('/test-file', '/copy-file')
       expect(File.read('/copy-file')).to eq("test\n")
     end
 
-    context "when +src+ is a directory" do
-      it "copies all its contents recursively" do
+    context 'when +src+ is a directory' do
+      it 'copies all its contents recursively' do
         FileUtils.mkdir('/test/dir')
         FileUtils.touch('/test/dir/file')
 
         FileUtils.cp_r('/test', '/dest')
-        expect(File.exists?('/dest/dir/file')).to be_true
+        expect(File.exist?('/dest/dir/file')).to be true
       end
     end
 
-    context "when +dest+ is a directory" do
-      it "copies +src+ to +dest/src+" do
+    context 'when +dest+ is a directory' do
+      it 'copies +src+ to +dest/src+' do
         FileUtils.mkdir(['/test/dir', '/dest'])
         FileUtils.touch('/test/dir/file')
 
         FileUtils.cp_r('/test', '/dest')
-        expect(File.exists?('/dest/test/dir/file')).to be_true
+        expect(File.exist?('/dest/test/dir/file')).to be true
       end
     end
 
-    context "when +src+ is a list of files" do
-      it "copies each of them in +dest+" do
+    context 'when +src+ is a list of files' do
+      it 'copies each of them in +dest+' do
         FileUtils.mkdir(['/test/dir', '/test/dir2', '/dest'])
         FileUtils.touch(['/test/dir/file', '/test/dir2/file'])
 
         FileUtils.cp_r(['/test/dir', '/test/dir2'], '/dest')
-        expect(File.exists?('/dest/dir2/file')).to be_true
+        expect(File.exist?('/dest/dir2/file')).to be true
       end
     end
   end
@@ -455,34 +458,34 @@ describe FileUtils do
       File.open('/test-file', 'w') { |f| f.puts 'test' }
     end
 
-    it "copies +src+ to +dest+" do
+    it 'copies +src+ to +dest+' do
       FileUtils.install('/test-file', '/test-file2')
       expect(File.read('/test-file2')).to eq("test\n")
     end
 
-    context "when +:mode+ is set" do
-      it "changes the permission mode to +mode+" do
+    context 'when +:mode+ is set' do
+      it 'changes the permission mode to +mode+' do
         expect(File).to receive(:chmod).with(0777, '/test-file2')
         FileUtils.install('/test-file', '/test-file2', mode: 0777)
       end
     end
 
-    context "when +src+ and +dest+ are the same file" do
-      it "raises an exception" do
+    context 'when +src+ and +dest+ are the same file' do
+      it 'raises an exception' do
         expect {
           FileUtils.install('/test-file', '/test-file')
         }.to raise_exception(ArgumentError)
       end
     end
 
-    context "when +dest+ already exists" do
-      it "removes destination before copy" do
+    context 'when +dest+ already exists' do
+      it 'removes destination before copy' do
         expect(File).to receive(:unlink).with('/test-file2')
         FileUtils.install('/test-file', '/test-file2')
       end
 
-      context "and +dest+ is a directory" do
-        it "installs +src+ in dest/src" do
+      context 'and +dest+ is a directory' do
+        it 'installs +src+ in dest/src' do
           FileUtils.mkdir('/test-dir')
           FileUtils.install('/test-file', '/test-dir')
           expect(File.read('/test-dir/test-file')).to eq("test\n")
@@ -500,33 +503,35 @@ describe FileUtils do
       File.open('/test-file', 'w') { |f| f.puts 'test' }
     end
 
-    it "creates a hard link +dest+ which points to +src+" do
+    it 'creates a hard link +dest+ which points to +src+' do
       FileUtils.ln('/test-file', '/test-file2')
       expect(File.read('/test-file2')).to eq(File.read('/test-file'))
     end
 
-    it "creates a hard link, not a symlink" do
+    it 'creates a hard link, not a symlink' do
       FileUtils.ln('/test-file', '/test-file2')
-      expect(File.symlink?('/test-file2')).to be_false
+      expect(File.symlink?('/test-file2')).to be false
     end
 
-    context "when +dest+ already exists" do
-      context "and is a directory" do
-        it "creates a link dest/src" do
+    context 'when +dest+ already exists' do
+      context 'and is a directory' do
+        it 'creates a link dest/src' do
           FileUtils.mkdir('/test-dir')
           FileUtils.ln('/test-file', '/test-dir')
           expect(File.read('/test-dir/test-file')).to eq(File.read('/test-file'))
         end
       end
 
-      context "and it is not a directory" do
-        it "raises an exception" do
+      context 'and it is not a directory' do
+        it 'raises an exception' do
           FileUtils.touch('/test-file2')
-          expect { FileUtils.ln('/test-file', '/test-file2') }.to raise_error(SystemCallError)
+          expect {
+            FileUtils.ln('/test-file', '/test-file2')
+          }.to raise_specific_error(SystemCallError)
         end
 
-        context "and +:force+ is set" do
-          it "overwrites +dest+" do
+        context 'and +:force+ is set' do
+          it 'overwrites +dest+' do
             FileUtils.touch('/test-file2')
             FileUtils.ln('/test-file', '/test-file2', force: true)
             expect(File.read('/test-file2')).to eq(File.read('/test-file'))
@@ -535,19 +540,19 @@ describe FileUtils do
       end
     end
 
-    context "when passing a list of paths" do
-      it "creates a link for each path in +destdir+" do
+    context 'when passing a list of paths' do
+      it 'creates a link for each path in +destdir+' do
         FileUtils.touch('/test-file2')
         FileUtils.mkdir('/test-dir')
         FileUtils.ln(['/test-file', '/test-file2'], '/test-dir')
       end
 
-      context "and +destdir+ is not a directory" do
-        it "raises an exception" do
+      context 'and +destdir+ is not a directory' do
+        it 'raises an exception' do
           FileUtils.touch(['/test-file2', '/not-a-dir'])
           expect {
             FileUtils.ln(['/test-file', '/test-file2'], '/not-a-dir')
-          }.to raise_error(Errno::ENOTDIR)
+          }.to raise_specific_error(Errno::ENOTDIR)
         end
       end
     end
@@ -560,67 +565,67 @@ describe FileUtils do
       FileUtils.mkdir('/test-dir')
     end
 
-    it "creates a symbolic link +new+" do
+    it 'creates a symbolic link +new+' do
       FileUtils.ln_s('/test-file', '/test-link')
-      expect(File.symlink?('/test-link')).to be_true
+      expect(File.symlink?('/test-link')).to be true
     end
 
-    it "creates a symbolic link which points to +old+" do
+    it 'creates a symbolic link which points to +old+' do
       FileUtils.ln_s('/test-file', '/test-link')
       expect(File.read('/test-link')).to eq(File.read('/test-file'))
     end
 
-    context "when +new+ already exists" do
-      context "and it is a directory" do
-        it "creates a symbolic link +new/old+" do
+    context 'when +new+ already exists' do
+      context 'and it is a directory' do
+        it 'creates a symbolic link +new/old+' do
           FileUtils.ln_s('/test-file', '/test-dir')
-          expect(File.symlink?('/test-dir/test-file')).to be_true
+          expect(File.symlink?('/test-dir/test-file')).to be true
         end
       end
 
-      context "and it is not a directory" do
-        it "raises an exeption" do
+      context 'and it is not a directory' do
+        it 'raises an exeption' do
           expect {
             FileUtils.ln_s('/test-file', '/not-a-dir')
-          }.to raise_error(Errno::EEXIST)
+          }.to raise_specific_error(Errno::EEXIST)
         end
 
-        context "and +:force+ is set" do
-          it "overwrites +new+" do
+        context 'and +:force+ is set' do
+          it 'overwrites +new+' do
             FileUtils.ln_s('/test-file', '/not-a-dir', force: true)
-            expect(File.symlink?('/not-a-dir')).to be_true
+            expect(File.symlink?('/not-a-dir')).to be true
           end
         end
       end
     end
 
-    context "when passing a list of paths" do
+    context 'when passing a list of paths' do
       before :each do
         File.open('/test-file2', 'w') { |f| f.puts 'test2' }
       end
 
-      it "creates several symbolic links in +destdir+" do
+      it 'creates several symbolic links in +destdir+' do
         FileUtils.ln_s(['/test-file', '/test-file2'], '/test-dir')
-        expect(File.exists?('/test-dir/test-file2')).to be_true
+        expect(File.exist?('/test-dir/test-file2')).to be true
       end
 
-      it "creates symbolic links pointing to each item in the list" do
+      it 'creates symbolic links pointing to each item in the list' do
         FileUtils.ln_s(['/test-file', '/test-file2'], '/test-dir')
         expect(File.read('/test-dir/test-file2')).to eq(File.read('/test-file2'))
       end
 
-      context "when +destdir+ is not a directory" do
-        it "raises an error" do
+      context 'when +destdir+ is not a directory' do
+        it 'raises an error' do
           expect {
             FileUtils.ln_s(['/test-file', '/test-file2'], '/not-a-dir')
-          }.to raise_error(Errno::ENOTDIR)
+          }.to raise_specific_error(Errno::ENOTDIR)
         end
       end
     end
   end
 
   describe '.ln_sf' do
-    it "calls ln_s with +:force+ set to true" do
+    it 'calls ln_s with +:force+ set to true' do
       File.open('/test-file', 'w') { |f| f.puts 'test' }
       File.open('/test-file2', 'w') { |f| f.puts 'test2' }
       FileUtils.ln_sf('/test-file', '/test-file2')
@@ -633,39 +638,39 @@ describe FileUtils do
   end
 
   describe '.mkdir' do
-    it "creates one directory" do
+    it 'creates one directory' do
       FileUtils.mkdir('/test-dir')
-      expect(File.directory?('/test-dir')).to be_true
+      expect(File.directory?('/test-dir')).to be true
     end
 
-    context "when passing a list of paths" do
-      it "creates several directories" do
+    context 'when passing a list of paths' do
+      it 'creates several directories' do
         FileUtils.mkdir(['/test-dir', '/test-dir2'])
-        expect(File.directory?('/test-dir2')).to be_true
+        expect(File.directory?('/test-dir2')).to be true
       end
     end
   end
 
   describe '.mkdir_p' do
-    it "creates a directory" do
+    it 'creates a directory' do
       FileUtils.mkdir_p('/test-dir')
-      expect(File.directory?('/test-dir')).to be_true
+      expect(File.directory?('/test-dir')).to be true
     end
 
-    it "creates all the parent directories" do
+    it 'creates all the parent directories' do
       FileUtils.mkdir_p('/path/to/some/test-dir')
-      expect(File.directory?('/path/to/some')).to be_true
+      expect(File.directory?('/path/to/some')).to be true
     end
 
-    context "when passing a list of paths" do
-      it "creates each directory" do
+    context 'when passing a list of paths' do
+      it 'creates each directory' do
         FileUtils.mkdir_p(['/test-dir', '/test-dir'])
-        expect(File.directory?('/test-dir')).to be_true
+        expect(File.directory?('/test-dir')).to be true
       end
 
       it "creates each directory's parents" do
         FileUtils.mkdir_p(['/test-dir', '/path/to/some/test-dir'])
-        expect(File.directory?('/path/to/some')).to be_true
+        expect(File.directory?('/path/to/some')).to be true
       end
     end
   end
@@ -679,30 +684,30 @@ describe FileUtils do
   end
 
   describe '.mv' do
-    it "moves +src+ to +dest+" do
+    it 'moves +src+ to +dest+' do
       FileUtils.touch('/test-file')
       FileUtils.mv('/test-file', '/test-file2')
-      expect(File.exists?('/test-file2')).to be_true
+      expect(File.exist?('/test-file2')).to be true
     end
 
-    it "removes +src+" do
+    it 'removes +src+' do
       FileUtils.touch('/test-file')
       FileUtils.mv('/test-file', '/test-file2')
-      expect(File.exists?('/test-file')).to be_false
+      expect(File.exist?('/test-file')).to be false
     end
 
-    context "when +dest+ already exists" do
-      context "and is a directory" do
-        it "moves +src+ to dest/src" do
+    context 'when +dest+ already exists' do
+      context 'and is a directory' do
+        it 'moves +src+ to dest/src' do
           FileUtils.touch('/test-file')
           FileUtils.mkdir('/test-dir')
           FileUtils.mv('/test-file', '/test-dir')
-          expect(File.exists?('/test-dir/test-file')).to be_true
+          expect(File.exist?('/test-dir/test-file')).to be true
         end
       end
 
-      context "and +dest+ is not a directory" do
-        it "it overwrites +dest+" do
+      context 'and +dest+ is not a directory' do
+        it 'it overwrites +dest+' do
           File.open('/test-file', 'w') { |f| f.puts 'test' }
           FileUtils.touch('/test-file2')
           FileUtils.mv('/test-file', '/test-file2')
@@ -713,7 +718,7 @@ describe FileUtils do
   end
 
   describe '.pwd' do
-    it "returns the name of the current directory" do
+    it 'returns the name of the current directory' do
       FileUtils.cd '/test'
       expect(FileUtils.pwd).to eq('/test')
     end
@@ -724,37 +729,37 @@ describe FileUtils do
   end
 
   describe '.remove_dir' do
-    it "removes the given directory +dir+" do
+    it 'removes the given directory +dir+' do
       FileUtils.mkdir('/test-dir')
       FileUtils.remove_dir('/test-dir')
-      expect(File.exists?('/test-dir')).to be_false
+      expect(File.exist?('/test-dir')).to be false
     end
 
-    it "removes the contents of the given directory +dir+" do
+    it 'removes the contents of the given directory +dir+' do
       FileUtils.mkdir_p('/test-dir/test-sub-dir')
       FileUtils.remove_dir('/test-dir')
-      expect(File.exists?('/test-dir/test-sub-dir')).to be_false
+      expect(File.exist?('/test-dir/test-sub-dir')).to be false
     end
 
-    context "when +force+ is set" do
-      it "ignores standard errors" do
+    context 'when +force+ is set' do
+      it 'ignores standard errors' do
         expect { FileUtils.remove_dir('/test-dir', true) }.not_to raise_error
       end
     end
   end
 
   describe '.remove_entry' do
-    it "removes a file system entry +path+" do
+    it 'removes a file system entry +path+' do
       FileUtils.touch('/test-file')
       FileUtils.remove_entry('/test-file')
-      expect(File.exists?('/test-file')).to be_false
+      expect(File.exist?('/test-file')).to be false
     end
 
-    context "when +path+ is a directory" do
-      it "removes it recursively" do
+    context 'when +path+ is a directory' do
+      it 'removes it recursively' do
         FileUtils.mkdir_p('/test-dir/test-sub-dir')
         FileUtils.remove_entry('/test-dir')
-        expect(Dir.exists?('/test-dir')).to be_false
+        expect(Dir.exist?('/test-dir')).to be false
       end
     end
   end
@@ -764,30 +769,30 @@ describe FileUtils do
       FileUtils.mkdir_p('/test-dir/test-sub-dir')
     end
 
-    it "removes a file system entry +path+" do
+    it 'removes a file system entry +path+' do
       FileUtils.chmod(0755, '/')
       FileUtils.remove_entry_secure('/test-dir')
-      expect(Dir.exists?('/test-dir')).to be_false
+      expect(Dir.exist?('/test-dir')).to be false
     end
 
-    context "when +path+ is a directory" do
-      it "removes it recursively" do
+    context 'when +path+ is a directory' do
+      it 'removes it recursively' do
         FileUtils.chmod(0755, '/')
         FileUtils.remove_entry_secure('/test-dir')
-        expect(Dir.exists?('/test-dir/test-sub-dir')).to be_false
+        expect(Dir.exist?('/test-dir/test-sub-dir')).to be false
       end
 
-      context "and is word writable" do
-        it "calls chown(2) on it" do
+      context 'and is word writable' do
+        it 'calls chown(2) on it' do
           FileUtils.chmod(01777, '/')
-          directory = fs.find('/test-dir')
+          directory = _fs.find('/test-dir')
           expect(directory).to receive(:uid=).at_least(:once)
           FileUtils.remove_entry_secure('/test-dir')
         end
 
-        it "calls chmod(2) on all sub directories" do
+        it 'calls chmod(2) on all sub directories' do
           FileUtils.chmod(01777, '/')
-          directory = fs.find('/test-dir')
+          directory = _fs.find('/test-dir')
           expect(directory).to receive(:mode=).at_least(:once)
           FileUtils.remove_entry_secure('/test-dir')
         end
@@ -796,49 +801,49 @@ describe FileUtils do
   end
 
   describe '.remove_file' do
-    it "removes a file path" do
+    it 'removes a file path' do
       FileUtils.touch('/test-file')
       FileUtils.remove_file('/test-file')
-      expect(File.exists?('/test-file')).to be_false
+      expect(File.exist?('/test-file')).to be false
     end
 
-    context "when +force+ is set" do
-      it "ignores StandardError" do
+    context 'when +force+ is set' do
+      it 'ignores StandardError' do
         expect { FileUtils.remove_file('/no-file', true) }.not_to raise_error
       end
     end
   end
 
   describe '.rm' do
-    it "removes the specified file" do
+    it 'removes the specified file' do
       FileUtils.touch('/test-file')
       FileUtils.rm('/test-file')
-      expect(File.exists?('/test-file')).to be_false
+      expect(File.exist?('/test-file')).to be false
     end
 
-    it "removes files specified in list" do
+    it 'removes files specified in list' do
       FileUtils.touch(['/test-file', '/test-file2'])
       FileUtils.rm(['/test-file', '/test-file2'])
-      expect(File.exists?('/test-file2')).to be_false
+      expect(File.exist?('/test-file2')).to be false
     end
 
-    it "cannot remove a directory" do
+    it 'cannot remove a directory' do
       FileUtils.mkdir('/test-dir')
-      expect { FileUtils.rm('/test-dir') }.to raise_error(Errno::EPERM)
+      expect { FileUtils.rm('/test-dir') }.to raise_specific_error(Errno::EPERM)
     end
 
-    context "when +:force+ is set" do
-      it "ignores StandardError" do
+    context 'when +:force+ is set' do
+      it 'ignores StandardError' do
         FileUtils.mkdir('/test-dir')
         expect {
           FileUtils.rm('/test-dir', force: true)
-        }.not_to raise_error(Errno::EPERM)
+        }.not_to raise_error
       end
     end
   end
 
   describe '.rm_f' do
-    it "calls rm with +:force+ set to true" do
+    it 'calls rm with +:force+ set to true' do
       expect(FileUtils).to receive(:rm).with('test', force: true)
       FileUtils.rm_f('test')
     end
@@ -849,63 +854,63 @@ describe FileUtils do
       FileUtils.touch(['/test-file', '/test-file2'])
     end
 
-    it "removes a list of files" do
+    it 'removes a list of files' do
       FileUtils.rm_r(['/test-file', '/test-file2'])
-      expect(File.exists?('/test-file2')).to be_false
+      expect(File.exist?('/test-file2')).to be false
     end
 
-    context "when an item of the list is a directory" do
-      it "removes all its contents recursively" do
+    context 'when an item of the list is a directory' do
+      it 'removes all its contents recursively' do
         FileUtils.mkdir('/test-dir')
         FileUtils.touch('/test-dir/test-file')
         FileUtils.rm_r(['/test-file', '/test-file2', '/test-dir'])
-        expect(File.exists?('/test-dir/test-file')).to be_false
+        expect(File.exist?('/test-dir/test-file')).to be false
       end
     end
 
-    context "when +:force+ is set" do
-      it "ignores StandardError" do
+    context 'when +:force+ is set' do
+      it 'ignores StandardError' do
         expect {
           FileUtils.rm_r(['/no-file'], force: true)
-        }.not_to raise_error(Errno::ENOENT)
+        }.not_to raise_error
       end
     end
   end
 
   describe '.rm_rf' do
-    it "calls rm with +:force+ set to true" do
+    it 'calls rm with +:force+ set to true' do
       expect(FileUtils).to receive(:rm_r).with('test', force: true)
       FileUtils.rm_rf('test')
     end
   end
 
   describe '.rmdir' do
-    it "Removes a directory" do
+    it 'Removes a directory' do
       FileUtils.mkdir('/test-dir')
       FileUtils.rmdir('/test-dir')
-      expect(Dir.exists?('/test-dir')).to be_false
+      expect(Dir.exist?('/test-dir')).to be false
     end
 
-    it "Removes a list of directories" do
+    it 'Removes a list of directories' do
       FileUtils.mkdir('/test-dir')
       FileUtils.mkdir('/test-dir2')
       FileUtils.rmdir(['/test-dir', '/test-dir2'])
-      expect(Dir.exists?('/test-dir2')).to be_false
+      expect(Dir.exist?('/test-dir2')).to be false
     end
 
-    context "when a directory is not empty" do
+    context 'when a directory is not empty' do
       before :each do
         FileUtils.mkdir('/test-dir')
         FileUtils.touch('/test-dir/test-file')
       end
 
-      it "ignores errors" do
+      it 'ignores errors' do
         expect { FileUtils.rmdir('/test-dir') }.not_to raise_error
       end
 
       it "doesn't remove the directory" do
         FileUtils.rmdir('/test-dir')
-        expect(Dir.exists?('/test-dir')).to be_true
+        expect(Dir.exist?('/test-dir')).to be true
       end
     end
   end
@@ -925,12 +930,12 @@ describe FileUtils do
   describe '.touch' do
     it "creates a file if it doesn't exist" do
       FileUtils.touch('/test-file')
-      expect(fs.find('/test-file')).not_to be_nil
+      expect(_fs.find('/test-file')).not_to be_nil
     end
 
     it "creates a list of files if they don't exist" do
       FileUtils.touch(['/test-file', '/test-file2'])
-      expect(fs.find('/test-file2')).not_to be_nil
+      expect(_fs.find('/test-file2')).not_to be_nil
     end
   end
 
@@ -938,23 +943,23 @@ describe FileUtils do
     before :each do
       FileUtils.touch('/test-file')
       FileUtils.touch('/old-file')
-      fs.find!('/old-file').mtime = Time.now - 3600
+      _fs.find!('/old-file').mtime = Time.now - 3600
     end
 
-    it "returns true if +newer+ is newer than all +old_list+" do
-      expect(FileUtils.uptodate?('/test-file', ['/old-file'])).to be_true
+    it 'returns true if +newer+ is newer than all +old_list+' do
+      expect(FileUtils.uptodate?('/test-file', ['/old-file'])).to be true
     end
 
-    context "when +newer+ does not exist" do
-      it "consideres it as older" do
-        expect(FileUtils.uptodate?('/no-file', ['/old-file'])).to be_false
+    context 'when +newer+ does not exist' do
+      it 'consideres it as older' do
+        expect(FileUtils.uptodate?('/no-file', ['/old-file'])).to be false
       end
     end
 
-    context "when a item of +old_list+ does not exist" do
-      it "consideres it as older than +newer+" do
+    context 'when a item of +old_list+ does not exist' do
+      it 'consideres it as older than +newer+' do
         uptodate = FileUtils.uptodate?('/test-file', ['/old-file', '/no-file'])
-        expect(uptodate).to be_true
+        expect(uptodate).to be true
       end
     end
   end