fakefs 0.13.3 → 0.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bfda62318697df45ab410fa56165a7c9d86587279c8898d7db8a780d94b0ad1e
-  data.tar.gz: 29cf7eb95ecf9ee562a6939fb4a8d1143242b86f829347805f9abb4e43993e10
+  metadata.gz: 31db99cb7de1e5f339b6c41d82f61680c648e36f319e5d1973299a2b3effe347
+  data.tar.gz: b18014abd60aa12ba08bc037164cec52ddc005c1a780d17ed6a7a6e9906af399
 SHA512:
-  metadata.gz: 2678f051b418e296f4936255962bfd8d846edf02d12d46071caf6b0920f6af0855ef3adf3b8ac2f13b42e4ed2bd0048d1ed4e646de761f69ddf3f4f0c9ca4d4b
-  data.tar.gz: a6d876f196eebf667db10898c3c184c4a3c91a0bcc5b43172842d1a6bf333fba88dd9f9f22d38aefb8e5f7a6ef93f69c8603d1c63daf7bf16e7c0cfde6761fcc
+  metadata.gz: 814c3fe4b2eb600b674adbb0d903550a3a3fdf1a6209660b0f9dea7f4d90657981e2078e5e584aca8972c738fbddd73f39b20c3704970a368ee3b17a174e2d2c
+  data.tar.gz: 70892ade1983519bdb0900005dadd09e1d620ed457754c02bc12f5505dac61a74d8dfdeca8d801c22db4d2c24445fbed9b6cad343d16f8beb87867c1692a0a1c

data/lib/fakefs/base.rb

@@ -8,10 +8,8 @@ def RealPathname(*args)
   RealPathname.new(*args)
 end
 
-if RUBY_VERSION >= '1.9.3'
-  def Pathname(*args)
-    Pathname.new(*args)
-  end
+def Pathname(*args)
+  Pathname.new(*args)
 end
 
 # FakeFS module
@@ -28,13 +26,13 @@ module FakeFS
         remove_const(:File)
         remove_const(:FileTest)
         remove_const(:FileUtils)
-        remove_const(:Pathname) if RUBY_VERSION >= '1.9.3'
+        remove_const(:Pathname)
 
         const_set(:Dir,       FakeFS::Dir)
         const_set(:File,      FakeFS::File)
         const_set(:FileUtils, FakeFS::FileUtils)
         const_set(:FileTest,  FakeFS::FileTest)
-        const_set(:Pathname,  FakeFS::Pathname) if RUBY_VERSION >= '1.9.3'
+        const_set(:Pathname,  FakeFS::Pathname)
         ::FakeFS::Kernel.hijack!
       end
 
@@ -50,13 +48,13 @@ module FakeFS
         remove_const(:File)
         remove_const(:FileTest)
         remove_const(:FileUtils)
-        remove_const(:Pathname) if RUBY_VERSION >= '1.9.3'
+        remove_const(:Pathname)
 
         const_set(:Dir,       RealDir)
         const_set(:File,      RealFile)
         const_set(:FileTest,  RealFileTest)
         const_set(:FileUtils, RealFileUtils)
-        const_set(:Pathname,  RealPathname) if RUBY_VERSION >= '1.9.3'
+        const_set(:Pathname,  RealPathname)
         ::FakeFS::Kernel.unhijack!
       end
 

data/lib/fakefs/dir.rb

@@ -7,7 +7,7 @@ module FakeFS
     attr_reader :path
 
     def self._check_for_valid_file(path)
-      fail Errno::ENOENT, path unless FileSystem.find(path)
+      raise Errno::ENOENT, path unless FileSystem.find(path)
     end
 
     def initialize(string)
@@ -45,7 +45,7 @@ module FakeFS
     end
 
     def read
-      fail IOError, 'closed directory' unless @pointer
+      raise IOError, 'closed directory' unless @pointer
       entry = @contents[@pointer]
       @pointer += 1
       entry_to_relative_path(entry) if entry
@@ -56,7 +56,7 @@ module FakeFS
     end
 
     def seek(integer)
-      fail IOError, 'closed directory' if @pointer.nil?
+      raise IOError, 'closed directory' if @pointer.nil?
       @pointer = integer
       @contents[integer]
     end
@@ -74,12 +74,12 @@ module FakeFS
     end
 
     def self.chroot(_string)
-      fail NotImplementedError
+      raise NotImplementedError
     end
 
     def self.delete(string)
       _check_for_valid_file(string)
-      fail Errno::ENOTEMPTY, string unless FileSystem.find(string).empty?
+      raise Errno::ENOTEMPTY, string unless FileSystem.find(string).empty?
 
       FileSystem.delete(string)
     end
@@ -91,12 +91,12 @@ module FakeFS
     end
 
     def self.children(dirname, opts = {})
-      entries(dirname, opts) - %w(. ..)
+      entries(dirname, opts) - ['.', '..']
     end
 
     def self.each_child(dirname, &_block)
       Dir.open(dirname) do |file|
-        next if %w(. ..).include?(file)
+        next if ['.', '..'].include?(file)
         yield file
       end
     end
@@ -140,10 +140,8 @@ module FakeFS
       block_given? ? files.each { |file| block.call(file) } : files
     end
 
-    if RUBY_VERSION >= '1.9'
-      def self.home(user = nil)
-        RealDir.home(user)
-      end
+    def self.home(user = nil)
+      RealDir.home(user)
     end
 
     def self.mkdir(string, _integer = 0)
@@ -166,70 +164,60 @@ module FakeFS
       FileSystem.current_dir.to_s
     end
 
-    if RUBY_VERSION >= '2.1'
-      # Tmpname module
-      module Tmpname # :nodoc:
-        module_function
+    # Tmpname module
+    module Tmpname # :nodoc:
+      module_function
 
-        def tmpdir
-          Dir.tmpdir
-        end
+      def tmpdir
+        Dir.tmpdir
+      end
 
-        def make_tmpname(prefix_suffix, n)
-          case prefix_suffix
-          when String
-            prefix = prefix_suffix
-            suffix = ''
-          when Array
-            prefix = prefix_suffix[0]
-            suffix = prefix_suffix[1]
-          else
-            fail ArgumentError,
-                 "unexpected prefix_suffix: #{prefix_suffix.inspect}"
-          end
-          t = Time.now.strftime('%Y%m%d')
-          path = "#{prefix}#{t}-#{$PID}-#{rand(0x100000000).to_s(36)}"
-          path << "-#{n}" if n
-          path << suffix
+      def make_tmpname(prefix_suffix, suffix)
+        case prefix_suffix
+        when String
+          prefix = prefix_suffix
+          suffix = ''
+        when Array
+          prefix = prefix_suffix[0]
+          suffix = prefix_suffix[1]
+        else
+          raise ArgumentError, "unexpected prefix_suffix: #{prefix_suffix.inspect}"
         end
+        t = Time.now.strftime('%Y%m%d')
+        path = "#{prefix}#{t}-#{$$}-#{rand(0x100000000).to_s(36)}"
+        path << "-#{suffix}" if suffix
+        path << suffix
+      end
 
-        def create(basename, *rest)
-          if (opts = Hash.try_convert(rest[-1]))
-            opts = opts.dup if rest.pop.equal?(opts)
-            max_try = opts.delete(:max_try)
-            opts = [opts]
-          else
-            opts = []
-          end
-          tmpdir, = *rest
-          if $SAFE > 0 && tmpdir.tainted?
-            tmpdir = '/tmp'
-          else
-            tmpdir ||= self.tmpdir
-          end
-          n = nil
-          begin
-            path = File.join(tmpdir, make_tmpname(basename, n))
-            yield(path, n, *opts)
-          rescue Errno::EEXIST
-            n ||= 0
-            n += 1
-            retry if !max_try || n < max_try
-            raise "cannot generate temporary name using `#{basename}' " \
-              "under `#{tmpdir}'"
-          end
-          path
+      def create(basename, *rest)
+        if (opts = Hash.try_convert(rest[-1]))
+          opts = opts.dup if rest.pop.equal?(opts)
+          max_try = opts.delete(:max_try)
+          opts = [opts]
+        else
+          opts = []
+        end
+        tmpdir, = *rest
+        if $SAFE > 0 && tmpdir.tainted?
+          tmpdir = '/tmp'
+        else
+          tmpdir ||= self.tmpdir
         end
+        n = nil
+        begin
+          path = File.join(tmpdir, make_tmpname(basename, n))
+          yield(path, n, *opts)
+        rescue Errno::EEXIST
+          n ||= 0
+          n += 1
+          retry if !max_try || n < max_try
+          raise "cannot generate temporary name using `#{basename}' " \
+            "under `#{tmpdir}'"
+        end
+        path
       end
     end
 
-    private
-
-    def entry_to_relative_path(entry)
-      filename = entry.to_s
-      filename.start_with?("#{path}/") ? filename[path.size + 1..-1] : filename
-    end
-
     # This code has been borrowed from Rubinius
     def self.mktmpdir(prefix_suffix = nil, tmpdir = nil)
       case prefix_suffix
@@ -243,17 +231,17 @@ module FakeFS
         prefix = prefix_suffix[0]
         suffix = prefix_suffix[1]
       else
-        fail ArgumentError, "unexpected prefix_suffix: #{prefix_suffix.inspect}"
+        raise ArgumentError, "unexpected prefix_suffix: #{prefix_suffix.inspect}"
       end
 
       t = Time.now.strftime('%Y%m%d')
       n = nil
 
       begin
-        path = "#{tmpdir}/#{prefix}#{t}-#{$PID}-#{rand(0x100000000).to_s(36)}"
+        path = "#{tmpdir}/#{prefix}#{t}-#{$$}-#{rand(0x100000000).to_s(36)}"
         path << "-#{n}" if n
         path << suffix
-        mkdir(path, 0700)
+        mkdir(path, 0o700)
       rescue Errno::EEXIST
         n ||= 0
         n += 1
@@ -276,11 +264,18 @@ module FakeFS
       end
     end
 
+    private
+
+    def entry_to_relative_path(entry)
+      filename = entry.to_s
+      filename.start_with?("#{path}/") ? filename[path.size + 1..-1] : filename
+    end
+
     class << self
-      alias_method :getwd, :pwd
-      alias_method :rmdir, :delete
-      alias_method :unlink, :delete
-      alias_method :exist?, :exists?
+      alias getwd pwd
+      alias rmdir delete
+      alias unlink delete
+      alias exist? exists?
     end
   end
 end

data/lib/fakefs/fake/dir.rb

@@ -10,7 +10,7 @@ module FakeFS
       @ctime   = Time.now
       @mtime   = @ctime
       @atime   = @ctime
-      @mode    = 0100000 + (0777 - File.umask)
+      @mode    = 0o100000 + (0o777 - File.umask)
       @uid     = Process.uid
       @gid     = Process.gid
       @content = ''
@@ -54,7 +54,7 @@ module FakeFS
     end
 
     def matches(pattern)
-      @entries.reject { |k, _v| pattern !~ k }.values
+      @entries.select { |k, _v| pattern =~ k }.values
     end
 
     def [](name)

data/lib/fakefs/fake/file.rb

@@ -7,11 +7,8 @@ module FakeFS
     # Inode class
     class Inode
       def initialize(file_owner)
-        # 1.9.3 when possible set default external encoding
-        @content = ''
-        @content = ''.encode(
-          Encoding.default_external) if ''.respond_to?(:encode)
-        @links   = [file_owner]
+        @content = ''.encode(Encoding.default_external)
+        @links = [file_owner]
       end
 
       attr_accessor :content
@@ -41,7 +38,7 @@ module FakeFS
       @mtime     = @ctime
       @atime     = @ctime
       @birthtime = @ctime
-      @mode      = 0100000 + (0666 - File.umask)
+      @mode      = 0o100000 + (0o666 - File.umask)
       @uid       = Process.uid
       @gid       = Process.gid
     end

data/lib/fakefs/fake/symlink.rb

@@ -23,13 +23,13 @@ module FakeFS
       File.join(parent.to_s, name)
     end
 
-    def respond_to?(method, include_private = false)
+    def respond_to_missing?(method, include_private = false)
       entry.respond_to?(method, include_private)
     end
 
     private
 
-    def method_missing(*args, &block)
+    def method_missing(*args, &block) # rubocop:disable Style/MethodMissingSuper
       entry.send(*args, &block)
     end
   end

data/lib/fakefs/file.rb

@@ -14,16 +14,18 @@ module FakeFS
 
     FILE_CREATION_MODES = (MODES - [READ_ONLY, READ_WRITE]).freeze
 
-    MODE_BITMASK = RealFile::RDONLY |
-                   RealFile::WRONLY |
-                   RealFile::RDWR |
-                   RealFile::APPEND |
-                   RealFile::CREAT |
-                   RealFile::EXCL |
-                   RealFile::NONBLOCK |
-                   RealFile::TRUNC |
-                   (RealFile.const_defined?(:NOCTTY) ? RealFile::NOCTTY : 0) |
-                   (RealFile.const_defined?(:SYNC) ? RealFile::SYNC : 0)
+    MODE_BITMASK = (
+      RealFile::RDONLY |
+      RealFile::WRONLY |
+      RealFile::RDWR |
+      RealFile::APPEND |
+      RealFile::CREAT |
+      RealFile::EXCL |
+      RealFile::NONBLOCK |
+      RealFile::TRUNC |
+      (RealFile.const_defined?(:NOCTTY) ? RealFile::NOCTTY : 0) |
+      (RealFile.const_defined?(:SYNC) ? RealFile::SYNC : 0)
+    )
 
     FILE_CREATION_BITMASK = RealFile::CREAT
 
@@ -49,11 +51,11 @@ module FakeFS
     end
 
     class << self
-      alias_method :exists?, :exist?
+      alias exists? exist?
 
       # Assuming that everyone can read and write files
-      alias_method :readable?, :exist?
-      alias_method :writable?, :exist?
+      alias readable? exist?
+      alias writable? exist?
 
       # Assume nothing is sticky.
       def sticky?(_path)
@@ -65,7 +67,7 @@ module FakeFS
       if exists?(path)
         FileSystem.find(path).mtime
       else
-        fail Errno::ENOENT
+        raise Errno::ENOENT
       end
     end
 
@@ -73,7 +75,7 @@ module FakeFS
       if exists?(path)
         FileSystem.find(path).ctime
       else
-        fail Errno::ENOENT
+        raise Errno::ENOENT
       end
     end
 
@@ -81,7 +83,7 @@ module FakeFS
       if exists?(path)
         FileSystem.find(path).atime
       else
-        fail Errno::ENOENT
+        raise Errno::ENOENT
       end
     end
 
@@ -91,7 +93,7 @@ module FakeFS
           FileSystem.find(path).atime = atime
           FileSystem.find(path).mtime = mtime
         else
-          fail Errno::ENOENT
+          raise Errno::ENOENT
         end
       end
 
@@ -112,7 +114,7 @@ module FakeFS
 
     if RUBY_VERSION >= '2.4'
       class << self
-        alias_method :empty?, :zero?
+        alias empty? zero?
       end
     end
 
@@ -169,12 +171,12 @@ module FakeFS
 
     def self.read(path, *args)
       options = args[-1].is_a?(Hash) ? args.pop : {}
-      length = args.size > 0 ? args.shift : nil
-      offset = args.size > 0 ? args.shift : 0
+      length = args.empty? ? nil : args.shift
+      offset = args.empty? ? 0 : args.shift
       file = new(path, options)
 
-      fail Errno::ENOENT unless file.exists?
-      fail Errno::EISDIR, path if directory?(path)
+      raise Errno::ENOENT unless file.exists?
+      raise Errno::EISDIR, path if directory?(path)
 
       FileSystem.find(path).atime = Time.now
       file.seek(offset)
@@ -187,7 +189,7 @@ module FakeFS
         FileSystem.find(path).atime = Time.now
         file.readlines
       else
-        fail Errno::ENOENT
+        raise Errno::ENOENT
       end
     end
 
@@ -201,17 +203,17 @@ module FakeFS
           file.each_line(*args)
         end
       else
-        fail Errno::ENOENT
+        raise Errno::ENOENT
       end
     end
 
     def self.rename(source, dest)
       if directory?(source) && file?(dest)
-        fail Errno::ENOTDIR, "#{source} or #{dest}"
+        raise Errno::ENOTDIR, "#{source} or #{dest}"
       elsif file?(source) && directory?(dest)
-        fail Errno::EISDIR, "#{source} or #{dest}"
+        raise Errno::EISDIR, "#{source} or #{dest}"
       elsif !exist?(dirname(dest))
-        fail Errno::ENOENT, "#{source} or #{dest}"
+        raise Errno::ENOENT, "#{source} or #{dest}"
       end
 
       if (target = FileSystem.find(source))
@@ -223,16 +225,16 @@ module FakeFS
 
         FileSystem.delete(source)
       else
-        fail Errno::ENOENT, "#{source} or #{dest}"
+        raise Errno::ENOENT, "#{source} or #{dest}"
       end
 
       0
     end
 
     def self.link(source, dest)
-      fail Errno::EPERM, "#{source} or #{dest}" if directory?(source)
-      fail Errno::ENOENT, "#{source} or #{dest}" unless exists?(source)
-      fail Errno::EEXIST, "#{source} or #{dest}" if exists?(dest)
+      raise Errno::EPERM, "#{source} or #{dest}" if directory?(source)
+      raise Errno::ENOENT, "#{source} or #{dest}" unless exists?(source)
+      raise Errno::EEXIST, "#{source} or #{dest}" if exists?(dest)
 
       source = FileSystem.find(source)
       dest = FileSystem.add(dest, source.entry.clone)
@@ -243,7 +245,7 @@ module FakeFS
 
     def self.delete(*file_names)
       file_names.each do |file_name|
-        fail Errno::ENOENT, file_name unless exists?(file_name)
+        raise Errno::ENOENT, file_name unless exists?(file_name)
 
         FileUtils.rm(file_name)
       end
@@ -252,7 +254,7 @@ module FakeFS
     end
 
     class << self
-      alias_method :unlink, :delete
+      alias unlink delete
     end
 
     def self.symlink(source, dest)
@@ -272,7 +274,7 @@ module FakeFS
     end
 
     def self.chmod(mode_int, filename)
-      FileSystem.find(filename).mode = 0100000 + mode_int
+      FileSystem.find(filename).mode = 0o100000 + mode_int
     end
 
     # Not exactly right, returns true if the file is chmod +x for owner. In the
@@ -280,20 +282,18 @@ module FakeFS
     def self.executable?(filename)
       file = FileSystem.find(filename)
       return false unless file
-      (file.mode - 0100000) & 0100 != 0
+      (file.mode - 0o100000) & 0o100 != 0
     end
 
     def self.chown(owner_int, group_int, filename)
       file = FileSystem.find(filename)
 
       if owner_int && owner_int != -1
-        owner_int.is_a?(Integer) || fail(TypeError,
-                                         "can't convert String into Integer")
+        owner_int.is_a?(Integer) || raise(TypeError, "can't convert String into Integer")
         file.uid = owner_int
       end
       if group_int && group_int != -1
-        group_int.is_a?(Integer) || fail(TypeError,
-                                         "can't convert String into Integer")
+        group_int.is_a?(Integer) || raise(TypeError, "can't convert String into Integer")
         file.gid = group_int
       end
     end
@@ -311,16 +311,16 @@ module FakeFS
     end
 
     class << self
-      alias_method :fnmatch, :fnmatch?
+      alias fnmatch fnmatch?
     end
 
     # FakeFS Stat class
     class Stat
       attr_reader :ctime, :mtime, :atime, :mode, :uid, :gid
-      attr_reader :birthtime if RUBY_VERSION >= '2.2.0'
+      attr_reader :birthtime
 
       def initialize(file, lstat = false)
-        fail(Errno::ENOENT, file) unless File.exist?(file)
+        raise(Errno::ENOENT, file) unless File.exist?(file)
 
         @file      = file
         @fake_file = FileSystem.find(@file)
@@ -374,11 +374,11 @@ module FakeFS
       # World_writable and readable are platform dependent
       # usually comparing with S_IROTH defined on compilation (MRI)
       def world_writable?
-        0777
+        0o777
       end
 
       def world_readable?
-        0777
+        0o777
       end
 
       def nlink
@@ -429,9 +429,9 @@ module FakeFS
       val
     end
 
-    alias_method :tell=,    :pos=
-    alias_method :sysread,  :read
-    alias_method :syswrite, :write
+    alias tell= pos=
+    alias sysread read
+    alias syswrite write
 
     undef_method :closed_read?
     undef_method :closed_write?
@@ -453,11 +453,11 @@ module FakeFS
     end
 
     def ioctl(*)
-      fail NotImplementedError
+      raise NotImplementedError
     end
 
     def read_nonblock
-      fail NotImplementedError
+      raise NotImplementedError
     end
 
     def stat
@@ -473,18 +473,18 @@ module FakeFS
       pos
     end
 
-    alias_method :to_i, :fileno
+    alias to_i fileno
 
     def to_io
       self
     end
 
     def write_nonblock(*)
-      fail NotImplementedError
+      raise NotImplementedError
     end
 
     def readpartial(*)
-      fail NotImplementedError
+      raise NotImplementedError
     end
 
     def atime
@@ -496,7 +496,7 @@ module FakeFS
     end
 
     def flock(*)
-      fail NotImplementedError
+      raise NotImplementedError
     end
 
     def mtime
@@ -504,119 +504,107 @@ module FakeFS
     end
 
     def chmod(mode_int)
-      @file.mode = 0100000 + mode_int
+      @file.mode = 0o100000 + mode_int
     end
 
     def chown(owner_int, group_int)
       return unless group_int && group_int != -1
 
-      owner_int.is_a?(Fixnum) || fail(
-        TypeError, "can't convert String into Integer")
+      owner_int.is_a?(Integer) || raise(
+        TypeError, "can't convert String into Integer"
+      )
       @file.uid = owner_int
 
-      group_int.is_a?(Fixnum) || fail(
-        TypeError, "can't convert String into Integer")
+      group_int.is_a?(Integer) || raise(
+        TypeError, "can't convert String into Integer"
+      )
       @file.gid = group_int
     end
 
-    if RUBY_VERSION >= '1.9'
-      def self.realpath(*args)
-        RealFile.realpath(*args)
-      end
+    def self.realpath(*args)
+      RealFile.realpath(*args)
+    end
 
-      def binmode?
-        fail NotImplementedError
-      end
+    def binmode?
+      raise NotImplementedError
+    end
 
-      def close_on_exec=(_bool)
-        fail NotImplementedError
-      end
+    def close_on_exec=(_bool)
+      raise NotImplementedError
+    end
 
-      def close_on_exec?
-        fail NotImplementedError
-      end
+    def close_on_exec?
+      raise NotImplementedError
+    end
 
-      def to_path
-        @path
-      end
+    def to_path
+      @path
     end
 
-    if RUBY_VERSION >= '1.9.1'
-      def self.absolute_path(file_name, dir_name = Dir.getwd)
-        RealFile.absolute_path(file_name, dir_name)
-      end
+    def self.absolute_path(file_name, dir_name = Dir.getwd)
+      RealFile.absolute_path(file_name, dir_name)
     end
 
-    if RUBY_VERSION >= '1.9.2'
-      attr_accessor :autoclose
+    attr_accessor :autoclose
 
-      def autoclose?
-        @autoclose ? true : false
-      end
+    def autoclose?
+      @autoclose ? true : false
+    end
 
-      alias_method :fdatasync, :flush
+    alias fdatasync flush
 
-      def size
-        File.size(@path)
-      end
+    def size
+      File.size(@path)
+    end
 
-      def self.realdirpath(*args)
-        RealFile.realdirpath(*args)
-      end
+    def self.realdirpath(*args)
+      RealFile.realdirpath(*args)
     end
 
-    if RUBY_VERSION >= '1.9.3'
-      def advise(_advice, _offset = 0, _len = 0)
-      end
+    def advise(_advice, _offset = 0, _len = 0); end
 
-      def self.write(filename, contents, offset = nil, open_args = {})
-        offset, open_args = nil, offset if offset.is_a?(Hash)
-        mode = offset ? 'a' : 'w'
-        if open_args.size > 0
-          if open_args[:open_args]
-            args = [filename, *open_args[:open_args]]
-          else
-            mode = open_args[:mode] || mode
-            args = [filename, mode, open_args]
-          end
+    def self.write(filename, contents, offset = nil, open_args = {})
+      offset, open_args = nil, offset if offset.is_a?(Hash)
+      mode = offset ? 'a' : 'w'
+      if open_args.any?
+        if open_args[:open_args]
+          args = [filename, *open_args[:open_args]]
         else
-          args = [filename, mode]
+          mode = open_args[:mode] || mode
+          args = [filename, mode, open_args]
         end
-        if offset
-          open(*args) do |f|
-            f.seek(offset)
-            f.write(contents)
-          end
-        else
-          open(*args) do |f|
-            f << contents
-          end
+      else
+        args = [filename, mode]
+      end
+      if offset
+        open(*args) do |f| # rubocop:disable Security/Open
+          f.seek(offset)
+          f.write(contents)
+        end
+      else
+        open(*args) do |f| # rubocop:disable Security/Open
+          f << contents
         end
-
-        contents.length
       end
+
+      contents.length
     end
 
-    if RUBY_VERSION >= '2.2.0'
-      def self.birthtime(path)
-        if exists?(path)
-          FileSystem.find(path).birthtime
-        else
-          fail Errno::ENOENT
-        end
+    def self.birthtime(path)
+      if exists?(path)
+        FileSystem.find(path).birthtime
+      else
+        raise Errno::ENOENT
       end
+    end
 
-      def birthtime
-        self.class.birthtime(@path)
-      end
+    def birthtime
+      self.class.birthtime(@path)
     end
 
     def read(length = nil, buf = '')
       read_buf = super(length, buf)
-      # change to binary only for ruby 1.9.3
-      if read_buf.respond_to?(:force_encoding) && binary_mode?
-        read_buf = read_buf.force_encoding('ASCII-8BIT')
-      end
+      read_buf.force_encoding('ASCII-8BIT') if binary_mode?
       read_buf
     end
 
@@ -627,13 +615,14 @@ module FakeFS
     end
 
     def binary_mode?
-      @mode.is_a?(String) && (@mode.include?('b') ||
-                              @mode.include?('binary')) &&
-        !@mode.include?('bom')
+      @mode.is_a?(String) && (
+        @mode.include?('b') ||
+        @mode.include?('binary')
+      ) && !@mode.include?('bom')
     end
 
     def check_file_existence!
-      fail Errno::ENOENT, @path unless @file
+      raise Errno::ENOENT, @path unless @file
     end
 
     def file_creation_mode?
@@ -641,9 +630,11 @@ module FakeFS
     end
 
     def mode_in?(list)
-      list.any? do |element|
-        @mode.include?(element)
-      end if @mode.respond_to?(:include?)
+      if @mode.respond_to?(:include?)
+        list.any? do |element|
+          @mode.include?(element)
+        end
+      end
     end
 
     def mode_in_bitmask?(mask)
@@ -653,7 +644,7 @@ module FakeFS
     # Create a missing file if the path is valid.
     #
     def create_missing_file
-      fail Errno::EISDIR, path if File.directory?(@path)
+      raise Errno::EISDIR, path if File.directory?(@path)
 
       return if File.exist?(@path) # Unnecessary check, probably.
       dirname = RealFile.dirname @path
@@ -661,7 +652,7 @@ module FakeFS
       unless dirname == '.'
         dir = FileSystem.find dirname
 
-        fail Errno::ENOENT, path unless dir.is_a? FakeDir
+        raise Errno::ENOENT, path unless dir.is_a? FakeDir
       end
 
       @file = FileSystem.add(path, FakeFile.new)