fakefs 0.5.4 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5dc2593e0eff91e5df327b9e0e6009d629cfcb42
-  data.tar.gz: 2f28f7c3fe6f1a0b5c1e6a3268a4b49b6e343543
+  metadata.gz: 56ce8f3a0f75895e325781ae4b269b380b7edda5
+  data.tar.gz: b3ccf2e1c5a712ec7f05eb0f6da3f881f91f14a6
 SHA512:
-  metadata.gz: c3610a7311b20ef411ae1d9b486b30efad8d68fa7e5d5b4c0e4cad43537d57f6f9f6bd843683d2826111a2bb2884e8d354dc0174a12b8356dbc67abb8ed1ac11
-  data.tar.gz: 35fad1a65e7b3972d465f64dee7c54fd0c4f28c2b8043c28bcefe63a5ca891e038e5cd7eb83cc2aea118a90939c71518821424acdc899ca87641d6b6b72cddec
+  metadata.gz: f165dc35f777b6117c2db4c04c1e5e75649bcff6d9b67e9ad877a01fbd9aac0e358930c78b3961c3cda3ff8ca6feaba6f1af2410119c493b814ac10b04f19216
+  data.tar.gz: c1d29b78cb3716070a7c5e29095bcbfc5fe73941712660f171e6710d1a7d040b6c6515f4ec6d210ae7ea35cce1ccafa49a248f124e6065a8d37e7ef077cf93aa

data/.gitignore

@@ -7,4 +7,6 @@ pkg
 .rbx
 .idea
 .bundle
+.project
+.ruby-version
 Gemfile.lock

data/.rubocop.yml

@@ -0,0 +1,50 @@
+AllCops:
+  Include:
+    - '**/Gemfile'
+    - lib/**/*
+    - spec/**/*
+    - gemirro.gemspec
+  Exclude:
+    - files/**/*
+    - templates/**/*
+MethodLength:
+  Exclude:
+    - lib/**/*
+    - test/**/*.rb
+    - spec/**/*.rb
+ClassLength:
+  Exclude:
+    - lib/**/*
+    - test/**/*
+LineLength:
+  Exclude:
+    - test/**/*
+ParameterLists:
+  Exclude:
+    - lib/**/*
+Metrics/CyclomaticComplexity:
+  Severity: warning
+  Max: 20
+Metrics/PerceivedComplexity:
+  Severity: warning
+  Max: 20
+Lint/Eval:
+  Exclude:
+    - lib/**/*
+Metrics/BlockNesting:
+  Severity: warning
+  Max: 4
+Style/PredicateName:
+  Exclude:
+    - lib/fakefs/file.rb
+Style/MethodName:
+  Exclude:
+    - lib/**/*.rb
+    - test/**/*.rb
+Style/ModuleFunction:
+  Exclude:
+    - lib/**/*.rb
+    - test/**/*.rb
+TrivialAccessors:
+  Enabled: true
+  ExactNameMatch: true

data/.travis.yml

@@ -1,5 +1,4 @@
 rvm:
-  - 1.8.7
   - 1.9.3
   - 2.0.0
   - 2.1.1

data/Rakefile

@@ -1,33 +1,40 @@
 $LOAD_PATH.unshift File.join(File.dirname(__FILE__))
 $LOAD_PATH.unshift File.join(File.dirname(__FILE__), 'test')
 
-require "bundler/setup"
-
+require 'bundler/setup'
 require 'rake/testtask'
 
 Rake::TestTask.new do |t|
-  t.libs << "test"
+  t.libs << 'test'
   t.test_files = FileList['test/**/*test.rb']
   t.verbose = true
 end
 
 begin
   require 'rspec/core/rake_task'
-  desc "Run specs"
+  desc 'Run specs'
   RSpec::Core::RakeTask.new
 rescue LoadError
   puts "Spec task can't be loaded. `gem install rspec`"
 end
 
-task :default => [:test, :spec]
+begin
+  require 'rubocop/rake_task'
+  desc 'Run RuboCop'
+  RuboCop::RakeTask.new(:rubocop)
+rescue LoadError
+  puts "Rubocop task can't be loaded. `gem install rubocop`"
+end
+
+task default: [:rubocop, :test, :spec]
 
-desc "Push a new version to rubygems.org"
+desc 'Push a new version to rubygems.org'
 task :publish do
-  abort("Tests failed!") unless system("rake test")
+  abort('Tests failed!') unless system('rake test')
   Rake::Task[:release].invoke
 end
 
-desc "Update contributors"
+desc 'Update contributors'
 task :update_contributors do
-  sh "git-rank-contributors > CONTRIBUTORS"
+  sh 'git-rank-contributors > CONTRIBUTORS'
 end

data/fakefs.gemspec

@@ -21,4 +21,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake", "~> 10.1"
   spec.add_development_dependency "rspec", "~> 2.14"
+  spec.add_development_dependency 'rubocop', '~>0.25'
 end

data/lib/fakefs/base.rb

@@ -8,12 +8,13 @@ def RealPathname(*args)
   RealPathname.new(*args)
 end
 
-if RUBY_VERSION >= "1.9.3"
+if RUBY_VERSION >= '1.9.3'
   def Pathname(*args)
     Pathname.new(*args)
   end
 end
 
+# FakeFS module
 module FakeFS
   @activated = false
   class << self
@@ -28,13 +29,13 @@ module FakeFS
         remove_const(:File)
         remove_const(:FileTest)
         remove_const(:FileUtils)
-        remove_const(:Pathname) if RUBY_VERSION >= "1.9.3"
+        remove_const(:Pathname) if RUBY_VERSION >= '1.9.3'
 
         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) if RUBY_VERSION >= '1.9.3'
         ::FakeFS::Kernel.hijack!
       end
       true
@@ -48,13 +49,13 @@ module FakeFS
         remove_const(:File)
         remove_const(:FileTest)
         remove_const(:FileUtils)
-        remove_const(:Pathname) if RUBY_VERSION >= "1.9.3"
+        remove_const(:Pathname) if RUBY_VERSION >= '1.9.3'
 
         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) if RUBY_VERSION >= '1.9.3'
         ::FakeFS::Kernel.unhijack!
       end
       true

data/lib/fakefs/dir.rb

@@ -1,9 +1,11 @@
 module FakeFS
+  # FakeFs Dir class
   class Dir
     include Enumerable
+    attr_reader :path
 
     def self._check_for_valid_file(path)
-      raise Errno::ENOENT, path unless FileSystem.find(path)
+      fail Errno::ENOENT, path unless FileSystem.find(path)
     end
 
     def initialize(string)
@@ -12,7 +14,7 @@ module FakeFS
       @path = FileSystem.normalize_path(string)
       @open = true
       @pointer = 0
-      @contents = [ '.', '..', ] + FileSystem.find(@path).entries
+      @contents = ['.', '..'] + FileSystem.find(@path).entries
     end
 
     def close
@@ -22,16 +24,12 @@ module FakeFS
       nil
     end
 
-    def each(&block)
-      while f = read
+    def each(&_block)
+      while (f = read)
         yield f
       end
     end
 
-    def path
-      @path
-    end
-
     def pos
       @pointer
     end
@@ -41,15 +39,15 @@ module FakeFS
     end
 
     def read
-      raise IOError, "closed directory" if @pointer == nil
+      fail IOError, 'closed directory' if @pointer.nil?
       n = @contents[@pointer]
       @pointer += 1
-      if n
-        if n.to_s[0, path.size+1] == path+'/'
-          n.to_s[path.size+1..-1]
-        else
-          n.to_s
-        end
+      return unless n
+
+      if n.to_s[0, path.size + 1] == path + '/'
+        n.to_s[path.size + 1..-1]
+      else
+        n.to_s
       end
     end
 
@@ -58,7 +56,7 @@ module FakeFS
     end
 
     def seek(integer)
-      raise IOError, "closed directory" if @pointer == nil
+      fail IOError, 'closed directory' if @pointer.nil?
       @pointer = integer
       @contents[integer]
     end
@@ -68,59 +66,68 @@ module FakeFS
     end
 
     def self.exists?(path)
-      File.exists?(path) && File.directory?(path)
+      File.exist?(path) && File.directory?(path)
     end
 
     def self.chdir(dir, &blk)
       FileSystem.chdir(dir, &blk)
     end
 
-    def self.chroot(string)
-      raise NotImplementedError
+    def self.chroot(_string)
+      fail NotImplementedError
     end
 
     def self.delete(string)
       _check_for_valid_file(string)
-      raise Errno::ENOTEMPTY, string unless FileSystem.find(string).empty?
+      fail Errno::ENOTEMPTY, string unless FileSystem.find(string).empty?
 
       FileSystem.delete(string)
     end
 
-    def self.entries(dirname, opts = {})
+    def self.entries(dirname, _opts = {})
       _check_for_valid_file(dirname)
 
       Dir.new(dirname).map { |file| File.basename(file) }
     end
 
-    def self.foreach(dirname, &block)
+    def self.foreach(dirname, &_block)
       Dir.open(dirname) { |file| yield file }
     end
 
-    def self.glob(pattern, flags = 0, &block)
+    def self.glob(pattern, _flags = 0, &block)
       matches_for_pattern = lambda do |matcher|
-        [FileSystem.find(matcher) || []].flatten.map{|e|
-          Dir.pwd.match(%r[\A/?\z]) || !e.to_s.match(%r[\A#{Dir.pwd}/?]) ? e.to_s : e.to_s.match(%r[\A#{Dir.pwd}/?]).post_match}.sort
+        [FileSystem.find(matcher) || []].flatten.map do |e|
+          if Dir.pwd.match(/\A\/?\z/) ||
+            !e.to_s.match(/\A#{Dir.pwd}\/?/)
+            e.to_s
+          else
+            e.to_s.match(/\A#{Dir.pwd}\/?/).post_match
+          end
+        end.sort
       end
 
       if pattern.is_a? Array
-        files = pattern.collect { |matcher| matches_for_pattern.call matcher }.flatten
+        files = pattern.map do |matcher|
+          matches_for_pattern.call matcher
+        end.flatten
       else
         files = matches_for_pattern.call pattern
       end
-      return block_given? ? files.each { |file| block.call(file) } : files
+
+      block_given? ? files.each { |file| block.call(file) } : files
     end
 
-    if RUBY_VERSION >= "1.9"
+    if RUBY_VERSION >= '1.9'
       def self.home(user = nil)
         RealDir.home(user)
       end
     end
 
-    def self.mkdir(string, integer = 0)
+    def self.mkdir(string, _integer = 0)
       FileUtils.mkdir(string)
     end
 
-    def self.open(string, &block)
+    def self.open(string, &_block)
       if block_given?
         Dir.new(string).each { |file| yield(file) }
       else
@@ -137,6 +144,7 @@ module FakeFS
     end
 
     if RUBY_VERSION >= '2.1'
+      # Tmpname module
       module Tmpname # :nodoc:
         module_function
 
@@ -148,21 +156,22 @@ module FakeFS
           case prefix_suffix
           when String
             prefix = prefix_suffix
-            suffix = ""
+            suffix = ''
           when Array
             prefix = prefix_suffix[0]
             suffix = prefix_suffix[1]
           else
-            raise ArgumentError, "unexpected prefix_suffix: #{prefix_suffix.inspect}"
+            fail ArgumentError,
+                 "unexpected prefix_suffix: #{prefix_suffix.inspect}"
           end
-          t = Time.now.strftime("%Y%m%d")
-          path = "#{prefix}#{t}-#{$$}-#{rand(0x100000000).to_s(36)}"
+          t = Time.now.strftime('%Y%m%d')
+          path = "#{prefix}#{t}-#{$PID}-#{rand(0x100000000).to_s(36)}"
           path << "-#{n}" if n
           path << suffix
         end
 
         def create(basename, *rest)
-          if opts = Hash.try_convert(rest[-1])
+          if (opts = Hash.try_convert(rest[-1]))
             opts = opts.dup if rest.pop.equal?(opts)
             max_try = opts.delete(:max_try)
             opts = [opts]
@@ -170,10 +179,10 @@ module FakeFS
             opts = []
           end
           tmpdir, = *rest
-          if $SAFE > 0 and tmpdir.tainted?
+          if $SAFE > 0 && tmpdir.tainted?
             tmpdir = '/tmp'
           else
-            tmpdir ||= tmpdir()
+            tmpdir ||= self.tmpdir
           end
           n = nil
           begin
@@ -182,8 +191,9 @@ module FakeFS
           rescue Errno::EEXIST
             n ||= 0
             n += 1
-            retry if !max_try or n < max_try
-            raise "cannot generate temporary name using `#{basename}' under `#{tmpdir}'"
+            retry if !max_try || n < max_try
+            raise "cannot generate temporary name using `#{basename}' " \
+              "under `#{tmpdir}'"
           end
           path
         end
@@ -194,23 +204,23 @@ module FakeFS
     def self.mktmpdir(prefix_suffix = nil, tmpdir = nil)
       case prefix_suffix
       when nil
-        prefix = "d"
-        suffix = ""
+        prefix = 'd'
+        suffix = ''
       when String
         prefix = prefix_suffix
-        suffix = ""
+        suffix = ''
       when Array
         prefix = prefix_suffix[0]
         suffix = prefix_suffix[1]
       else
-        raise ArgumentError, "unexpected prefix_suffix: #{prefix_suffix.inspect}"
+        fail ArgumentError, "unexpected prefix_suffix: #{prefix_suffix.inspect}"
       end
 
-      t = Time.now.strftime("%Y%m%d")
+      t = Time.now.strftime('%Y%m%d')
       n = nil
 
       begin
-        path = "#{tmpdir}/#{prefix}#{t}-#{$$}-#{rand(0x100000000).to_s(36)}"
+        path = "#{tmpdir}/#{prefix}#{t}-#{$PID}-#{rand(0x100000000).to_s(36)}"
         path << "-#{n}" if n
         path << suffix
         mkdir(path, 0700)

data/lib/fakefs/fake/dir.rb

@@ -1,4 +1,5 @@
 module FakeFS
+  # Fake Dir class
   class FakeDir
     attr_accessor :name, :parent, :mode, :uid, :gid, :mtime, :atime
     attr_reader :ctime, :content
@@ -12,7 +13,7 @@ module FakeFS
       @mode    = 0100000 + (0777 - File.umask)
       @uid     = Process.uid
       @gid     = Process.gid
-      @content = ""
+      @content = ''
       @entries = {}
     end
 
@@ -21,7 +22,8 @@ module FakeFS
     end
 
     def inspect
-      "(FakeDir name:#{name.inspect} parent:#{parent.to_s.inspect} size:#{@entries.size})"
+      "(FakeDir name:#{name.inspect} " \
+      "parent:#{parent.to_s.inspect} size:#{@entries.size})"
     end
 
     def clone(parent = nil)
@@ -52,7 +54,7 @@ module FakeFS
     end
 
     def matches(pattern)
-      @entries.reject {|k,v| pattern !~ k }.values
+      @entries.reject { |k, _v| pattern !~ k }.values
     end
 
     def [](name)