memfs 0.4.2 → 0.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 70a8628ef207a62b0e968c52b562a999eb9b17eb
-  data.tar.gz: da534038352ee4290851b8dc4f2f7aef7bed7cf5
+  metadata.gz: 71cd9903100239967550a02a2f803264604c076d
+  data.tar.gz: 65c68e74aab6519b99841afa80c8bf8ec22a4907
 SHA512:
-  metadata.gz: c47dc6fe5eb179d3ee6d4e1e439e714d19920af0d474fa6166c1c8d75eb0c6a9c37d5115cf9db627e1801e0318dcb077620f05c87387428698061282610541a5
-  data.tar.gz: 218c397cacf5071ea1f83cfa06b12b8420bfef293390c652066f5548f6f9b7e9fba78e6d19af76f6d73a59bf0939a3ffdbcb66f7b649a9073ce9406c04fd5da0
+  metadata.gz: 4b5b2d13af10abc53b81c33fef8dcabe4998da21faebcabeb7f50a2ca01334fef69054d2e1c22804aaf6d7202f75b80465be6db1d3d0d3f7513707635ed05950
+  data.tar.gz: 3c9ee2c3e9435c03c53c1777814cd42183774c914e78d2e30d2b51605df89db3a174d79a3c177cd4c911a0ea2e5e07355785f65c375d9c992986196376dbebb7

data/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## HEAD
 
+## 0.4.3
+
+* ADD: `File::SEPARATOR` and `File::ALT_SEPARATOR`
+* FIX: Support `YAML.load_file` by handling `r:bom|utf-8` open mode
+
 ## 0.4.2
 
 * ADD: `File#external_encoding`

data/README.md

@@ -43,6 +43,22 @@ end
 File.exists?('/test-file') #=> false
 ```
 
+## Why you may prefer MemFs over FakeFS?
+
+While FakeFS is pretty cool it overrides classes like `FileUtils`. This kind of override is problematic when you rely on real behavior from this kind of tool.
+
+For instance, trying to test the following with FakeFS will not work, the `noop` option will be ignored:
+
+``` ruby
+FileUtils.touch('somefile.txt', noop: true)
+```
+
+MemFs tries to be **compliant with the Ruby API** by overriding only the low level classes (C classes) like File, Dir or File::Stat leaving the stdlib classes untouched and still working, being less intrusive that way.
+
+Some stdlib classes may be overriden at some point if they don't use `File` or `Dir`, like `Pathname`, etc.
+
+Another key point is that MemFs **aims to implement every single method provided by Ruby classes** (when possible) and to behave and return **exactly** the same way as the original classes.
+
 ## Installation
 
 Add this line to your application's Gemfile:

data/lib/memfs/file.rb

@@ -13,6 +13,8 @@ module MemFs
     include OriginalFile::Constants
     include IO::InstanceMethods
 
+    ALT_SEPARATOR = nil
+
     MODE_MAP = {
       'r'  => RDONLY,
       'r+' => RDWR,
@@ -22,6 +24,7 @@ module MemFs
       'a+' => CREAT | APPEND | RDWR
     }
 
+    SEPARATOR = '/'
     SUCCESS = 0
 
     @umask = nil

data/lib/memfs/io.rb

@@ -129,6 +129,9 @@ module MemFs
       end
 
       def read(length = nil, buffer = '')
+        unless entry
+          fail(Errno::ENOENT, path)
+        end
         default = length ? nil : ''
         content.read(length, buffer) || default
       end
@@ -180,7 +183,7 @@ module MemFs
       def str_to_mode_int(mode)
         return mode unless mode.is_a?(String)
 
-        unless mode =~ /\A([rwa]\+?)([bt])?\z/
+        unless mode =~ /\A([rwa]\+?)([bt])?(:bom)?(\|.+)?\z/
           fail ArgumentError, "invalid access mode #{mode}"
         end
 

data/lib/memfs/version.rb

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

data/spec/memfs/file_spec.rb

@@ -20,6 +20,16 @@ module MemFs
       expect(subject_class.ancestors).to include Enumerable
     end
 
+    describe 'constants' do
+      it 'expose SEPARATOR' do
+        expect(MemFs::File::SEPARATOR).to eq '/'
+      end
+
+      it 'expose ALT_SEPARATOR' do
+        expect(MemFs::File::ALT_SEPARATOR).to be_nil
+      end
+    end
+
     describe '.absolute_path' do
       before { MemFs::Dir.chdir('/test-dir') }
 
@@ -803,11 +813,50 @@ module MemFs
         end
 
         context 'and it is a string' do
-          subject { subject_class.new('/test-file', 'r+') }
+          it 'sets the read mode for "r"' do
+            subject = subject_class.new('/test-file', 'r')
+            expect(subject.send(:opening_mode)).to eq File::RDONLY
+          end
 
-          it 'sets the mode to the integer value' do
+          it 'sets the write+create+truncate mode for "w"' do
+            subject = subject_class.new('/test-file', 'w')
+            expect(subject.send(:opening_mode)).to eq File::CREAT|File::TRUNC|File::WRONLY
+          end
+
+          it 'sets the read+write mode for "r+"' do
+            subject = subject_class.new('/test-file', 'r+')
             expect(subject.send(:opening_mode)).to eq File::RDWR
           end
+
+          it 'sets the read+write+create+truncate mode for "w+"' do
+            subject = subject_class.new('/test-file', 'w+')
+            expect(subject.send(:opening_mode)).to eq File::CREAT|File::TRUNC|File::RDWR
+          end
+
+          it 'sets the write+create+append mode for "a"' do
+            subject = subject_class.new('/test-file', 'a')
+            expect(subject.send(:opening_mode)).to eq File::CREAT|File::APPEND|File::WRONLY
+          end
+
+          it 'sets the read+write+create+append mode for "a+"' do
+            subject = subject_class.new('/test-file', 'a+')
+            expect(subject.send(:opening_mode)).to eq File::CREAT|File::APPEND|File::RDWR
+          end
+
+          it 'handles the :bom option' do
+            subject = subject_class.new('/test-file', 'r:bom')
+            expect(subject.send(:opening_mode)).to eq File::RDONLY
+          end
+
+          it 'handles the |utf-8 option' do
+            subject = subject_class.new('/test-file', 'r|utf-8')
+            expect(subject.send(:opening_mode)).to eq File::RDONLY
+          end
+
+          it 'handles the :bom|utf-8 option' do
+            subject = subject_class.new('/test-file', 'r:bom|utf-8')
+            expect(subject.send(:opening_mode)).to eq File::RDONLY
+          end
         end
 
         context 'and it specifies that the file must be created' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: memfs
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.4.3
 platform: ruby
 authors:
 - Simon COURTOIS
@@ -190,7 +190,7 @@ rubyforge_project:
 rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
-summary: memfs-0.4.2
+summary: memfs-0.4.3
 test_files:
 - spec/fileutils_spec.rb
 - spec/memfs/dir_spec.rb