bfs 0.6.1 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 13963edd5c68d92d4f5ce4ef5c3f3ef0131ddb318b36f4d7114e3e49e4a16a3b
-  data.tar.gz: b805a46a30c795dd6dc9c8761876a4af5bd73dcc117a2f7c76c8acea7a957ed9
+  metadata.gz: 7e9c5b7b27a3bca8975dac3ccce6d99fc3e22a012758adf5a82baceeafd12aa7
+  data.tar.gz: 930feef7182ef50e2106e8b48f0af0cf388b8cfa8cb39d3bf6a2dc0bb556568f
 SHA512:
-  metadata.gz: cd9c768fed6d9efa36379f9151c4066931cc92267d032b1b0e920c7150b7db108acb739ecd46b5e8c5853b0ff090767b259e35b0d98b1707fa41491c3eb9dd9e
-  data.tar.gz: 123205693490cd8ee1394be308f28944d84e10098dae690cd54d3bf073710d2f538d397960b19d4d0c33ec271bdbbea3df796bd086733d57f9d28db2c52e7bd6
+  metadata.gz: 496a92c2ac19fb0fb11a4fafbf20db067df87b678dbfea6d90fdf53e8f9088c8b1f059d87a93cd2da3de6a610f033a5ed6dec2eba0cffedad31a76a518130071
+  data.tar.gz: 56991e1f1088e1b361635ba17b3c235e464bd98e587db34a28e7e7d01c3b60c692b4b508646686a008e38cd731413dfd13174e87fe9b6e135317d24558fbee39

data/lib/bfs.rb

@@ -1,7 +1,36 @@
 require 'uri'
 
 module BFS
-  FileInfo = Struct.new(:path, :size, :mtime, :content_type, :metadata)
+  class FileInfo < Hash
+    def initialize(**attrs)
+      update(size: 0, mtime: Time.at(0), mode: 0, metadata: {})
+      update(attrs)
+    end
+
+    def path
+      fetch(:path, nil)
+    end
+
+    def size
+      fetch(:size, 0)
+    end
+
+    def content_type
+      fetch(:content_type, nil)
+    end
+
+    def mtime
+      fetch(:mtime, Time.at(0))
+    end
+
+    def mode
+      fetch(:mode, 0)
+    end
+
+    def metadata
+      fetch(:metadata, {})
+    end
+  end
 
   def self.register(*schemes, &resolver)
     @registry ||= {}
@@ -25,6 +54,11 @@ module BFS
     path.sub!(%r{/+$}, '')
     path
   end
+
+  def self.norm_mode(mode)
+    mode = mode.to_i(8) if mode.is_a?(String)
+    mode & 0o000777
+  end
 end
 
 require 'bfs/helpers'

data/lib/bfs/bucket/abstract.rb

@@ -6,8 +6,8 @@ module BFS
       # Initializes a new bucket
       # @param [Hash] opts options
       # @option opts [String] :encoding Custom encoding. Default: Encoding.default_external.
-      def initialize(**opts)
-        @encoding = opts.delete(:encoding) || Encoding.default_external
+      def initialize(encoding: Encoding.default_external, **_opts)
+        @encoding = encoding
       end
 
       # Lists the contents of a bucket using a glob pattern

data/lib/bfs/bucket/fs.rb

@@ -26,7 +26,8 @@ module BFS
       def info(path, **_opts)
         full = @root.join(norm_path(path))
         path = trim_prefix(full.to_s)
-        BFS::FileInfo.new(path, full.size, full.mtime, nil, {})
+        stat = full.stat
+        BFS::FileInfo.new(path: path, size: stat.size, mtime: stat.mtime, mode: BFS.norm_mode(stat.mode))
       rescue Errno::ENOENT
         raise BFS::FileNotFound, path
       end
@@ -37,13 +38,12 @@ module BFS
       # @param [Hash] opts Additional options.
       # @option opts [String] :encoding Custom encoding.
       # @option opts [Integer] :perm Custom file permission, default: 0600.
-      def create(path, **opts, &block)
+      def create(path, encoding: nil, perm: nil, **_opts, &block)
         full = @root.join(norm_path(path))
         FileUtils.mkdir_p(full.dirname.to_s)
 
-        perm = opts[:perm]
-        enc  = opts[:encoding] || @encoding
-        temp = BFS::TempWriter.new(full, perm: perm, encoding: enc) {|t| FileUtils.mv t, full.to_s }
+        enc  = encoding || @encoding
+        temp = BFS::TempWriter.new(full, encoding: enc, perm: perm) {|t| FileUtils.mv t, full.to_s }
         return temp unless block
 
         begin

data/lib/bfs/bucket/in_mem.rb

@@ -32,7 +32,7 @@ module BFS
         raise BFS::FileNotFound, path unless @files.key?(path)
 
         entry = @files[path]
-        BFS::FileInfo.new(path, entry.io.size, entry.mtime, entry.content_type, entry.metadata)
+        BFS::FileInfo.new(path: path, size: entry.io.size, mtime: entry.mtime, content_type: entry.content_type, metadata: entry.metadata)
       end
 
       # Creates a new file and opens it for writing.
@@ -44,9 +44,10 @@ module BFS
       # @option opts [Hash] :metadata Metadata key-value pairs.
       def create(path, **opts, &block)
         io = StringIO.new
-        io.set_encoding(opts[:encoding] || @encoding)
+        io.set_encoding(opts.delete(:encoding) || @encoding)
 
-        @files[norm_path(path)] = Entry.new(io, Time.now, opts[:content_type], norm_meta(opts[:metadata]))
+        entry = Entry.new(io, Time.now, opts.delete(:content_type), norm_meta(opts.delete(:metadata)))
+        @files[norm_path(path)] = entry
         return io unless block
 
         begin

data/spec/bfs/blob_spec.rb

@@ -21,7 +21,14 @@ RSpec.describe BFS::Blob do
       subject.write('TESTDATA', content_type: 'text/plain', metadata: { 'x-key' => 'val' })
 
       info = subject.info
-      expect(info).to eq(BFS::FileInfo.new('path/to/file.txt', 8, info.mtime, 'text/plain', 'X-Key' => 'val'))
+      expect(info).to eq(
+        path: 'path/to/file.txt',
+        size: 8,
+        mtime: info.mtime,
+        content_type: 'text/plain',
+        mode: 0,
+        metadata: { 'X-Key' => 'val' },
+      )
       expect(info.mtime).to be_within(1).of(Time.now)
 
       expect(subject.read).to eq('TESTDATA')
@@ -54,7 +61,13 @@ RSpec.describe BFS::Blob do
       expect(subject.read).to eq('TESTDATA')
 
       info = subject.info
-      expect(info).to eq(BFS::FileInfo.new(path, 8, info.mtime, nil, {}))
+      expect(info).to eq(
+        path: path,
+        size: 8,
+        mtime: info.mtime,
+        mode: 0o600,
+        metadata: {},
+      )
       expect(info.mtime).to be_within(1).of(Time.now)
 
       expect(Pathname.glob("#{tmpdir}/**/*").select(&:file?).map(&:to_s)).to eq [

data/spec/bfs/bucket/fs_spec.rb

@@ -20,7 +20,7 @@ RSpec.describe BFS::Bucket::FS do
   it 'should support custom perms' do
     blob = BFS::Blob.new("file://#{tmpdir}/test.txt")
     blob.create(perm: 0o666) {|w| w.write 'foo' }
-    expect(File.stat(File.join(tmpdir, 'test.txt')).mode).to eq(0o100666)
+    expect(blob.info.mode).to eq(0o666)
     blob.close
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bfs
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.6.2
 platform: ruby
 authors:
 - Dimitrij Denissenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-02-11 00:00:00.000000000 Z
+date: 2020-02-12 00:00:00.000000000 Z
 dependencies: []
 description: Minimalist abstraction for bucket storage
 email: dimitrij@blacksquaremedia.com
@@ -48,7 +48,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.0.6
 signing_key: 
 specification_version: 4
 summary: Multi-platform cloud bucket adapter