fakefs 0.6.7 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fc6809d0301720e85b2242f8e37100be5e77395b
-  data.tar.gz: 63eaa2bfd50a523613ec164fdbcfbbd14195aeae
+  metadata.gz: e41d41305c22bf0ef89f9e3e00e3b1888db74f0c
+  data.tar.gz: 23cf2892e9ef0c928b72436863cda9a22a0e4692
 SHA512:
-  metadata.gz: cd81047918fe4c89615fc92c5e5b57c5c7953c367c61c2e9b823c7b4bc8079fc82fa08048dca9cb17cffefd17b19a7b1a0f69740d1345658d047b2ceb34d16a4
-  data.tar.gz: 0a7cddc534cf1a1430392d95b8c17301ddb3f5cb233da7481da62401ef97e8995cd11522a030f04899aa932630bb976e68506fe36decac49c3d5c55db93d56e2
+  metadata.gz: 082a1ae98f913c68711d60091f6554fc8a87bf286b57fc7a909e25f5890cdece9851ee25ed3912f7ab9a398ea9fe920723ccf3437d26f1041252e84cec6beb2a
+  data.tar.gz: 817fb7bc5bd120f4f5d774a020fbc25dc0334311eb5fdcdaf391b4c089afa83cdf62c6c7cd298a1bec27c982a263cf971fc87be4a3681290690f344dec16ecea

data/.rubocop.yml

@@ -1,4 +1,5 @@
 AllCops:
+  UseCache: true
   Include:
     - '**/Gemfile'
     - lib/**/*
@@ -8,17 +9,19 @@ AllCops:
     - files/**/*
     - templates/**/*
     - etc/**/*
-MethodLength:
+Metrics/MethodLength:
   Exclude:
     - lib/**/*
     - test/**/*.rb
     - spec/**/*.rb
-ClassLength:
+Metrics/ClassLength:
   Enabled: false
-LineLength:
+Metrics/ModuleLength:
+  Enabled: false
+Metrics/LineLength:
   Exclude:
     - test/**/*
-ParameterLists:
+Metrics/ParameterLists:
   Enabled: false
 Metrics/CyclomaticComplexity:
   Severity: warning
@@ -33,6 +36,10 @@ Metrics/BlockNesting:
   Max: 4
 Metrics/AbcSize:
   Enabled: false
+Performance/StringReplacement:
+  Enabled: false
+Style/ParallelAssignment:
+  Enabled: false
 Style/PredicateName:
   Exclude:
     - lib/fakefs/file.rb
@@ -40,6 +47,8 @@ Style/MethodName:
   Enabled: false
 Style/ModuleFunction:
   Enabled: false
-TrivialAccessors:
+Style/SymbolProc:
+  Enabled: false
+Style/TrivialAccessors:
   Enabled: true
   ExactNameMatch: true

data/CONTRIBUTORS

@@ -7,12 +7,12 @@ Brian Donovan <donovan@squareup.com>
 Myles Eftos <myles@madpilot.com.au>
 Jeff Hodges <jeff@somethingsimilar.com>
 Jakub Jirutka <jakub@jirutka.cz>
+Brian Donovan <me@brian-donovan.com>
 Pierre RAMBAUD <pierre.rambaud86@gmail.com>
 Morten Møller Riis <mortenmoellerriis@gmail.com>
 Matt Freels <matt@freels.name>
 Eero Saynatkari <projects@kittensoft.org>
 Andres Riofrio <riofrios@gmail.com>
-Brian Donovan <me@brian-donovan.com>
 Jacob Evans <jacob@dekz.net>
 Víctor Martínez <knoopx@gmail.com>
 Mariusz Pietrzyk <wijet@wijet.pl>
@@ -34,8 +34,9 @@ Jon Yurek <jyurek@thoughtbot.com>
 Jared Luxenberg <jared@jaredlux.com>
 DSIW <dsiw@dsiw-it.de>
 doc75 <github@virlet.org>
-Greg Campbell <gtcampbell@gmail.com>
 Lars Gierth <lars.gierth@gmail.com>
+Greg Campbell <gtcampbell@gmail.com>
+Benjamin Quorning <benjamin@quorning.net>
 marano <thiagomarano@gmail.com>
 Ben Mabey <ben@benmabey.com>
 Jorge Orlando Munoz <jmunoz@altavistaed.com>
@@ -44,10 +45,10 @@ Sam Goldstein <sam@aboutus.org>
 Adam Alboyadjian <adam@vistahigherlearning.com>
 Rick Salevsky <rsalevsky@suse.com>
 Scott Petersen <petersen@centtech.com>
-Matt Todd <chiology@gmail.com>
 Noah Paessel <knowuh@gmail.com>
-Emil Soman <emil.soman@gmail.com>
 Ryan McGeary <ryan@mcgeary.org>
+Emil Soman <emil.soman@gmail.com>
+Matt Todd <chiology@gmail.com>
 dmathieu <42@dmathieu.com>
 Oleg Sukhodolsky <os97673@gmail.com>
 Winston Lee <lee.winston@gmail.com>
@@ -57,36 +58,40 @@ Zequez <zequez@gmail.com>
 jameswilding <james@jameswilding.net>
 Ed Ruder <ed@squareup.com>
 Tymon Tobolski <i@teamon.eu>
-Benjamin Oakes <hello@benjaminoakes.com>
 Scott Barron <scott@elitists.net>
+Benjamin Oakes <hello@benjaminoakes.com>
 Andrius Chamentauskas <andrius.chamentauskas@gmail.com>
 Eric Daspet <eric.daspet@survol.fr>
-W. Andrew Loe III <andrew@andrewloe.com>
 Matt Hoyle <matt@deployable.co>
+W. Andrew Loe III <andrew@andrewloe.com>
+Misha Gorodnitzky <misaka@pobox.com>
 Xavier Shay <xavier@squareup.com>
-Travis Herrick <travis@carbonfive.com>
 Keita Urashima <ursm@ursm.jp>
 rambutan <usul@usul-HP-EliteBook-8460p.(none)>
-Matthew Morgan <lytithwyn@gmail.com>
+Travis Herrick <travis@carbonfive.com>
+msassak <msassak@gmail.com>
 = <gokulnath@mobme.in>
 andrea longhi <andrea@spaghetticode.it>
-msassak <msassak@gmail.com>
+Matthew Morgan <lytithwyn@gmail.com>
 David Reese <david@whatcould.com>
-timo3377 <tim.linquist@gmail.com>
 Andrius Chamentauskas <sinsiliux@gmail.com>
-Paolo Gianrossi <paolino.gianrossi@gmail.com>
+timo3377 <tim.linquist@gmail.com>
 Matt Rogers <mattrogers@sbcglobal.net>
 Michael Scherer <misc@zarb.org>
+Paolo Gianrossi <paolino.gianrossi@gmail.com>
 Mislav Marohnić <mislav.marohnic@gmail.com>
 Andrew Ryan <nerdrew@gmail.com>
 Rob Sanheim <rsanheim@gmail.com>
-Chris Wanstrath <chris@github.com>
-Chris Knadler <takeshi91k@gmail.com>
+Jacob Atzen <jatzen@gmail.com>
 Toon Willems <willemstoon@gmail.com>
+Chris Knadler <takeshi91k@gmail.com>
 Ryan Scott Lewis <ryanscottlewis@lewis-software.com>
-Sven Riedel <sr@gimp.org>
-Dane O'Connor <dane.oconnor@gmail.com>
+Chris Wanstrath <chris@github.com>
 Yuta Shimizu <pachirel@gmail.com>
+Benjamin Quorning <bquorning@zendesk.com>
+Dane O'Connor <dane.oconnor@gmail.com>
+Sven Riedel <sr@gimp.org>
 Benjamin Fleischer <github@benjaminfleischer.com>
 Jordi Massaguer Pla <jmassaguerpla@suse.de>
 Maria Shaldibina <mariash@pivotallabs.com>
+Vít Ondruch <vondruch@redhat.com>

data/Rakefile

@@ -30,7 +30,7 @@ end
 task default: [:test, :spec, :rubocop]
 
 desc 'Push a new version to rubygems.org'
-task :publish => [:test, :spec, :rubocop, :update_contributors, :tag, :release, :push]
+task :publish => [:rubocop, :test, :spec, :rubocop, :update_contributors, :tag, :release, :push]
 
 desc 'Update contributors'
 task :update_contributors do

data/fakefs.gemspec

@@ -21,7 +21,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake", "~> 10.3"
   spec.add_development_dependency "rspec", "~> 3.1"
-  spec.add_development_dependency 'rubocop', '~>0.25'
+  spec.add_development_dependency 'rubocop', '~> 0.35.0'
   spec.add_development_dependency 'minitest', '~> 5.5'
   spec.add_development_dependency 'minitest-rg', '~> 5.1'
 end

data/lib/fakefs/dir.rb

@@ -97,11 +97,11 @@ module FakeFS
     def self.glob(pattern, _flags = 0, &block)
       matches_for_pattern = lambda do |matcher|
         [FileSystem.find(matcher) || []].flatten.map do |e|
-          if Dir.pwd.match(/\A\/?\z/) ||
-             !e.to_s.match(/\A#{Dir.pwd}\/?/)
+          if Dir.pwd.match(%r{\A/?\z}) ||
+             !e.to_s.match(%r{\A#{Dir.pwd}/?})
             e.to_s
           else
-            e.to_s.match(/\A#{Dir.pwd}\/?/).post_match
+            e.to_s.match(%r{\A#{Dir.pwd}/?}).post_match
           end
         end.sort
       end

data/lib/fakefs/file.rb

@@ -14,15 +14,15 @@ module FakeFS
 
     FILE_CREATION_MODES = MODES - [READ_ONLY, READ_WRITE]
 
-    MODE_BITMASK = RealFile::RDONLY   |
-                   RealFile::WRONLY   |
-                   RealFile::RDWR     |
-                   RealFile::APPEND   |
-                   RealFile::CREAT    |
-                   RealFile::EXCL     |
+    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::TRUNC |
+                   (RealFile.const_defined?(:NOCTTY) ? RealFile::NOCTTY : 0) |
                    (RealFile.const_defined?(:SYNC) ? RealFile::SYNC : 0)
 
     FILE_CREATION_BITMASK = RealFile::CREAT

data/lib/fakefs/file_system.rb

@@ -24,7 +24,10 @@ module FakeFS
       parts = path_parts(normalize_path(path))
       return fs if parts.empty? # '/'
 
-      entries = find_recurser(fs, parts).flatten
+      entries = Globber.expand(path).flat_map do |pattern|
+        parts = path_parts(normalize_path(pattern))
+        find_recurser(fs, parts).flatten
+      end
 
       case entries.length
       when 0 then nil
@@ -93,7 +96,7 @@ module FakeFS
     end
 
     def path_parts(path)
-      drop_root(path.split(File::SEPARATOR)).reject(&:empty?)
+      Globber.path_components(path)
     end
 
     def normalize_path(path)
@@ -113,14 +116,6 @@ module FakeFS
 
     private
 
-    def drop_root(path_parts)
-      # we need to remove parts from root dir at least for windows and jruby
-      return path_parts if path_parts.nil? || path_parts.empty?
-      root = RealFile.expand_path('/').split(File::SEPARATOR).first
-      path_parts.shift if path_parts.first == root
-      path_parts
-    end
-
     def find_recurser(dir, parts)
       return [] unless dir.respond_to? :[]
 
@@ -143,16 +138,9 @@ module FakeFS
                     directories_under(dir)
                   end
                 else
-                  regex_body = pattern.gsub('.', '\.')
-                               .gsub('?', '.')
-                               .gsub('*', '.*')
-                               .gsub('(', '\(')
-                               .gsub(')', '\)')
-                               .gsub(/\{(.*?)\}/) do
-                                 "(#{Regexp.last_match[1].gsub(',', '|')})"
-                               end
-                               .gsub(/\A\./, '(?!\.).')
-                  dir.matches(/\A#{regex_body}\Z/)
+                  Globber.expand(pattern).flat_map do |subpattern|
+                    dir.matches(Globber.regexp(subpattern))
+                  end
                 end
 
       if parts.empty? # we're done recursing

data/lib/fakefs/fileutils.rb

@@ -199,12 +199,10 @@ module FakeFS
       list.each do |f|
         if File.exist?(f)
           uid = if user
-                  user.to_s.match(/[0-9]+/) ? user.to_i :
-                    Etc.getpwnam(user).uid
+                  user.to_s.match(/\d+/) ? user.to_i : Etc.getpwnam(user).uid
                 end
           gid = if group
-                  group.to_s.match(/[0-9]+/) ? group.to_i :
-                    Etc.getgrnam(group).gid
+                  group.to_s.match(/\d+/) ? group.to_i : Etc.getgrnam(group).gid
                 end
           File.chown(uid, gid, f)
         else

data/lib/fakefs/globber.rb

@@ -0,0 +1,97 @@
+module FakeFS
+  # Handles globbing for FakeFS.
+  module Globber
+    extend self
+
+    def expand(pattern)
+      return [pattern] if pattern[0] != '{' || pattern[-1] != '}'
+
+      part = ''
+      result = []
+
+      each_char_with_levels pattern, '{', '}' do |chr, level|
+        case level
+        when 0
+          case chr
+          when '{'
+            # noop
+          else
+            part << chr
+          end
+        when 1
+          case chr
+          when ','
+            result << part
+            part = ''
+          when '}'
+            # noop
+          else
+            part << chr
+          end
+        else
+          part << chr
+        end
+      end
+
+      result << part
+
+      result
+    end
+
+    def path_components(pattern)
+      part = ''
+      result = []
+
+      each_char_with_levels pattern, '{', '}' do |chr, level|
+        if level == 0 && chr == File::SEPARATOR
+          result << part
+          part = ''
+        else
+          part << chr
+        end
+      end
+
+      result << part
+
+      drop_root(result).reject(&:empty?)
+    end
+
+    def regexp(pattern)
+      regex_body = pattern.gsub('.', '\.')
+                   .gsub('?', '.')
+                   .gsub('*', '.*')
+                   .gsub('(', '\(')
+                   .gsub(')', '\)')
+                   .gsub(/\{(.*?)\}/) do
+                     "(#{Regexp.last_match[1].gsub(',', '|')})"
+                   end
+                   .gsub(/\A\./, '(?!\.).')
+      /\A#{regex_body}\Z/
+    end
+
+    private
+
+    def each_char_with_levels(string, level_start, level_end)
+      level = 0
+
+      string.each_char do |chr|
+        yield chr, level
+
+        case chr
+        when level_start
+          level += 1
+        when level_end
+          level -= 1
+        end
+      end
+    end
+
+    def drop_root(path_parts)
+      # we need to remove parts from root dir at least for windows and jruby
+      return path_parts if path_parts.nil? || path_parts.empty?
+      root = RealFile.expand_path('/').split(File::SEPARATOR).first
+      path_parts.shift if path_parts.first == root
+      path_parts
+    end
+  end
+end

data/lib/fakefs/kernel.rb

@@ -11,6 +11,7 @@ module FakeFS
       captives[:hijacked].each do |name, prc|
         ::Kernel.send(:remove_method, name.to_sym)
         ::Kernel.send(:define_method, name.to_sym, &prc)
+        ::Kernel.send(:private, name.to_sym)
       end
     end
 
@@ -20,6 +21,7 @@ module FakeFS
         ::Kernel.send(:define_method, name.to_sym, proc do |*args, &block|
           ::FakeFS::Kernel.captives[:original][name].call(*args, &block)
         end)
+        ::Kernel.send(:private, name.to_sym)
       end
     end
 

data/lib/fakefs/pathname.rb

@@ -621,7 +621,7 @@ module FakeFS
     end
 
     # Pathname class
-    class Pathname    # * IO *
+    class Pathname # * IO *
       #
       # #each_line iterates over the line in the file.
       # It yields a String object for each line.
@@ -662,7 +662,7 @@ module FakeFS
     end
 
     # Pathname class
-    class Pathname    # * File *
+    class Pathname # * File *
       # See <tt>File.atime</tt>.  Returns last access time.
       def atime
         File.atime(@path)
@@ -790,7 +790,7 @@ module FakeFS
     end
 
     # Pathname class
-    class Pathname    # * FileTest *
+    class Pathname # * FileTest *
       # See <tt>FileTest.blockdev?</tt>.
       def blockdev?
         FileTest.blockdev?(@path)
@@ -913,7 +913,7 @@ module FakeFS
     end
 
     # Pathname class
-    class Pathname    # * Dir *
+    class Pathname # * Dir *
       # See <tt>Dir.glob</tt>.  Returns or yields Pathname objects.
       def self.glob(*args) # :yield: pathname
         if block_given?
@@ -962,7 +962,7 @@ module FakeFS
     end
 
     # Pathname class
-    class Pathname    # * Find *
+    class Pathname # * Find *
       #
       # Pathname#find is an iterator to traverse a directory tree
       # in a depth first manner.
@@ -985,7 +985,7 @@ module FakeFS
     end
 
     # Pathname class
-    class Pathname    # * FileUtils *
+    class Pathname # * FileUtils *
       # See <tt>FileUtils.mkpath</tt>.  Creates a full path, including any
       # intermediate directories that don't yet exist.
       def mkpath
@@ -1005,7 +1005,7 @@ module FakeFS
     end
 
     # Pathname class
-    class Pathname    # * mixed *
+    class Pathname # * mixed *
       # Removes a file or directory, using <tt>File.unlink</tt> or
       # <tt>Dir.unlink</tt> as necessary.
       def unlink

data/lib/fakefs/safe.rb

@@ -9,5 +9,6 @@ require 'fakefs/fileutils'
 require 'fakefs/file'
 require 'fakefs/file_test'
 require 'fakefs/dir'
+require 'fakefs/globber'
 require 'fakefs/pathname' if RUBY_VERSION >= '1.9.3'
 require 'fakefs/kernel'

data/lib/fakefs/version.rb

@@ -1,7 +1,7 @@
 module FakeFS
   # Version module
   module Version
-    VERSION = '0.6.7'
+    VERSION = '0.7.0'
 
     def self.to_s
       VERSION

data/test/fake/file/join_test.rb

@@ -11,7 +11,7 @@ class FileJoin < Minitest::Test
   end
 
   [
-    %w(a b),  %w(a/ b), %w(a /b), %w(a/ /b), %w(a / b)
+    %w(a b), %w(a/ b), %w(a /b), %w(a/ /b), %w(a / b)
   ].each_with_index do |args, i|
     define_method "test_file_join_#{i}" do
       assert_equal RealFile.join(args), File.join(args)

data/test/fakefs_test.rb

@@ -13,6 +13,9 @@ class FakeFSTest < Minitest::Test
 
     FakeFS.activate!
     FakeFS::FileSystem.clear
+    # Create /tmp so that Minitest can create files for diffing when an
+    # assertion fails. See https://github.com/defunkt/fakefs/issues/143
+    FileUtils.mkdir_p('/tmp')
   end
 
   def teardown
@@ -24,6 +27,7 @@ class FakeFSTest < Minitest::Test
   end
 
   def test_can_be_initialized_empty
+    FakeFS::FileSystem.clear
     fs = FakeFS::FileSystem
     assert_equal 0, fs.files.size
   end
@@ -831,6 +835,8 @@ class FakeFSTest < Minitest::Test
   end
 
   def test_file_object_initialization_with_brackets_in_filename
+    skip 'TODO'
+
     filename = 'bracket[1](2).txt'
     expected_contents = 'Yokudekimashita'
     # nothing raised
@@ -842,11 +848,11 @@ class FakeFSTest < Minitest::Test
     assert_equal contents, expected_contents
   end
 
-  def test_file_object_initialization_with_brackets_in_filename
+  def test_file_object_initialization_with_utf_chars
     filename = "\u65e5\u672c\u8a9e.txt"
     expected_contents = 'Yokudekimashita'
     # nothing raised
-    File.open(filename,  mode: 'w') { |f| f.write "#{expected_contents}" }
+    File.open(filename, mode: 'w') { |f| f.write "#{expected_contents}" }
     contents = File.open("/#{filename}").read
     assert_equal contents, expected_contents
   end
@@ -1051,7 +1057,7 @@ class FakeFSTest < Minitest::Test
     assert_equal ['/path/foo', '/path/foobar'], Dir['/p??h/foo*']
 
     assert_equal ['/path/bar', '/path/bar/baz', '/path/bar2', '/path/bar2/baz', '/path/foo', '/path/foobar'], Dir['/path/**/*']
-    assert_equal ['/path', '/path/bar', '/path/bar/baz', '/path/bar2', '/path/bar2/baz', '/path/foo', '/path/foobar'], Dir['/**/*']
+    assert_equal ['/path', '/path/bar', '/path/bar/baz', '/path/bar2', '/path/bar2/baz', '/path/foo', '/path/foobar', '/tmp'], Dir['/**/*']
 
     assert_equal ['/path/bar', '/path/bar/baz', '/path/bar2', '/path/bar2/baz', '/path/foo', '/path/foobar'], Dir['/path/**/*']
     assert_equal ['/path/bar/baz'], Dir['/path/bar/**/*']
@@ -1067,6 +1073,9 @@ class FakeFSTest < Minitest::Test
 
     assert_equal ['/path/bar', '/path/bar2'], Dir['/path/bar{2,}']
 
+    assert_equal ['/path/bar', '/path/foo'], Dir['{/nowhere,/path/{foo,bar}}']
+    assert_equal ['/path/bar', '/path/foo'], Dir['{/nowhere,/path/{foo,{bar,bar2/baz}}}']
+
     Dir.chdir '/path' do
       assert_equal ['foo'], Dir['foo']
     end
@@ -1109,7 +1118,7 @@ class FakeFSTest < Minitest::Test
 
   def test_dir_glob_takes_optional_flags
     FileUtils.touch '/foo'
-    assert_equal Dir.glob('/*', 0), ['/foo']
+    assert_equal Dir.glob('/*', 0), ['/foo', '/tmp']
   end
 
   def test_dir_glob_handles_recursive_globs
@@ -1142,7 +1151,7 @@ class FakeFSTest < Minitest::Test
     yielded = []
     Dir.glob('*') { |file| yielded << file }
 
-    assert_equal 2, yielded.size
+    assert_equal 3, yielded.size
   end
 
   def test_copy_with_subdirectory
@@ -1266,7 +1275,7 @@ class FakeFSTest < Minitest::Test
       File.open('/foobar', 'w') { |f| f.write 'foo' }
     end
     assert_equal ['/path/foo'], Dir.glob('/path/*').sort
-    assert_equal ['/foobar', '/path'], Dir.glob('/*').sort
+    assert_equal ['/foobar', '/path', '/tmp'], Dir.glob('/*').sort
 
     Dir.chdir '/path' do
       FileUtils.rm('foo')
@@ -1274,7 +1283,7 @@ class FakeFSTest < Minitest::Test
     end
 
     assert_equal [], Dir.glob('/path/*').sort
-    assert_equal ['/path'], Dir.glob('/*').sort
+    assert_equal ['/path', '/tmp'], Dir.glob('/*').sort
   end
 
   def test_chdir_should_be_nestable
@@ -2062,7 +2071,6 @@ class FakeFSTest < Minitest::Test
   end
 
   def test_directory_mkdir_nested
-    Dir.mkdir('/tmp')
     Dir.mkdir('/tmp/stream20120103-11847-xc8pb.lock')
     assert File.exist?('/tmp/stream20120103-11847-xc8pb.lock')
   end
@@ -2436,7 +2444,7 @@ class FakeFSTest < Minitest::Test
   end
 
   def test_dir_mktmpdir
-    FileUtils.mkdir '/tmp'
+    # FileUtils.mkdir '/tmpdir'
 
     tmpdir = Dir.mktmpdir
     assert File.directory?(tmpdir)

data/test/file/stat_test.rb

@@ -132,7 +132,7 @@ class FileStatTest < Minitest::Test
     assert File::Stat.new('/foo').world_readable? == 0777, "#{File::Stat.new('/foo').world_readable?}"
   end
 
-  def test_responds_to_world_readable
+  def test_can_open_tempfile
     FakeFS do
       require 'tempfile'
       FileUtils.mkdir_p('/tmp')

data/test/globber_test.rb

@@ -0,0 +1,28 @@
+require 'test_helper'
+
+# Globber test class
+class GlobberTest < Minitest::Test
+  def test_expand_without_brace_groups_returns_single_entry
+    assert_equal ['*.rb'], FakeFS::Globber.expand('*.rb')
+  end
+
+  def test_expand_with_brace_group_with_one_entry_returns_single_entry
+    assert_equal ['abc'], FakeFS::Globber.expand('{abc}')
+  end
+
+  def test_expand_with_brace_group_with_multiple_entries_returns_all_entries
+    assert_equal %w(a b c), FakeFS::Globber.expand('{a,b,c}')
+  end
+
+  def test_expand_with_brace_group_with_nested_entries_expands_only_first_level
+    assert_equal ['a', 'b', '{c,d}'], FakeFS::Globber.expand('{a,b,{c,d}}')
+  end
+
+  def test_path_components_with_no_globbing_splits_on_path_separator
+    assert_equal %w(a b c), FakeFS::Globber.path_components('/a/b/c')
+  end
+
+  def test_path_components_with_path_separator_inside_brace_group
+    assert_equal ['a', '{b,c/d}', 'e'], FakeFS::Globber.path_components('/a/{b,c/d}/e')
+  end
+end

data/test/kernel_test.rb

@@ -46,6 +46,10 @@ class KernelTest < Minitest::Test
     end
   end
 
+  def test_kernel_open_remains_private
+    refute 'foo'.respond_to?(:open), 'String#open should be private'
+  end
+
   def test_can_exec_normally2
     out = open("|echo 'foo'")
     assert_equal "foo\n", out.gets

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fakefs
 version: !ruby/object:Gem::Version
-  version: 0.6.7
+  version: 0.7.0
 platform: ruby
 authors:
 - Chris Wanstrath
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-15 00:00:00.000000000 Z
+date: 2016-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -62,14 +62,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.25'
+        version: 0.35.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.25'
+        version: 0.35.0
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -127,6 +127,7 @@ files:
 - lib/fakefs/file_system.rb
 - lib/fakefs/file_test.rb
 - lib/fakefs/fileutils.rb
+- lib/fakefs/globber.rb
 - lib/fakefs/kernel.rb
 - lib/fakefs/pathname.rb
 - lib/fakefs/safe.rb
@@ -146,6 +147,7 @@ files:
 - test/fake/symlink_test.rb
 - test/fakefs_test.rb
 - test/file/stat_test.rb
+- test/globber_test.rb
 - test/kernel_test.rb
 - test/pathname_test.rb
 - test/safe_test.rb
@@ -171,7 +173,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: A fake filesystem. Use it in your tests.
@@ -190,6 +192,7 @@ test_files:
 - test/fake/symlink_test.rb
 - test/fakefs_test.rb
 - test/file/stat_test.rb
+- test/globber_test.rb
 - test/kernel_test.rb
 - test/pathname_test.rb
 - test/safe_test.rb