bfs-s3 0.7.3 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 439b5c3e0e7cc280d527e238e6dc0d67404e720b8899b79f8cfd201950e059ea
-  data.tar.gz: 433ea64a8b6feec47b683be92ad07e5f5e044189f1b04609176b7bdd724a0e0f
+  metadata.gz: 91e105f0932ff1ef23c5b5214e05a714ac6ce564d3a93e55225f524d495fd4e2
+  data.tar.gz: f2152ef353747b86e661f69c1d1100e6f14f2d38b0498e9cea9d1f1877ec8612
 SHA512:
-  metadata.gz: 1d180808ac5fc59950da69521a97fe989ac28dd614c88d2f299cf413af9e9ed221dedb692e5d1e4ae8793b2b0e976d16b15f3af6a5d2c0124dc99dbf6b86320c
-  data.tar.gz: 1aed0a7496aae52534cb2fe082d161682bfbc343fb2e82944f94075338829080163a0d37e84d0ed978f6283c0848c3fec282b886bca32b400fcc5590234fed8e
+  metadata.gz: 45573ff60184d0c190061a0e241c10b80233db9f0ec184317500eaea59a735f1d48ef6cbce5ea53608250305e98132e2c964b95b074e9446d32a041a8dbed095
+  data.tar.gz: f3bf20f0dcc3a5db9c770d59ef9d30e9fd305b863e6d9e698b18ac79155d30c0c9f01bf939e46bdfec3a16de611ef0816fe059655d4a31161ca4b40aa9a8efbf

data/lib/bfs/bucket/s3.rb

@@ -82,18 +82,11 @@ module BFS
         opts[:server_side_encryption] ||= @sse if @sse
         opts[:storage_class] ||= @storage_class if @storage_class
 
-        temp = BFS::TempWriter.new(path, encoding: encoding, perm: perm) do |t|
+        BFS::Writer.new(path, encoding: encoding, perm: perm) do |t|
           File.open(t, encoding: encoding) do |file|
             @client.put_object(opts.merge(body: file))
           end
-        end
-        return temp unless block
-
-        begin
-          yield temp
-        ensure
-          temp.close
-        end
+        end.perform(&block)
       end
 
       # Opens an existing file for reading

data/spec/bfs/bucket/s3_spec.rb

@@ -1,38 +1,39 @@
 require 'spec_helper'
 
-sandbox = { bucket: 'bsm-bfs-unittest' }.freeze
+bucket_name = 'bsm-bfs-unittest'
 
 RSpec.describe BFS::Bucket::S3, s3: true do
-  let(:prefix) { "x/#{SecureRandom.uuid}/" }
-
   subject do
-    described_class.new sandbox[:bucket], prefix: prefix
+    described_class.new bucket_name, prefix: prefix
   end
-  after :all do
-    bucket = described_class.new sandbox[:bucket], prefix: 'x/'
+
+  let(:prefix) { "x/#{SecureRandom.uuid}/" }
+
+  after :all do # rubocop:disable RSpec/BeforeAfterAll
+    bucket = described_class.new bucket_name, prefix: 'x/'
     bucket.ls.each {|name| bucket.rm(name) }
   end
 
   it_behaves_like 'a bucket'
 
-  it 'should resolve from URL' do
-    bucket = BFS.resolve("s3://#{sandbox[:bucket]}/?acl=private&encoding=binary")
+  it 'resolves from URL' do
+    bucket = BFS.resolve("s3://#{bucket_name}/?acl=private&encoding=binary")
     expect(bucket).to be_instance_of(described_class)
-    expect(bucket.name).to eq(sandbox[:bucket])
+    expect(bucket.name).to eq(bucket_name)
     expect(bucket.acl).to eq(:private)
     expect(bucket.encoding).to eq('binary')
     expect(bucket.instance_variable_get(:@prefix)).to be_nil
     bucket.close
 
-    bucket = BFS.resolve("s3://#{sandbox[:bucket]}/a/b/")
+    bucket = BFS.resolve("s3://#{bucket_name}/a/b/")
     expect(bucket).to be_instance_of(described_class)
-    expect(bucket.name).to eq(sandbox[:bucket])
+    expect(bucket.name).to eq(bucket_name)
     expect(bucket.instance_variable_get(:@prefix)).to eq('a/b')
     bucket.close
   end
 
-  it 'should enumerate over a large number of files' do
-    bucket = described_class.new sandbox[:bucket], prefix: 'm/'
+  it 'enumerates over a large number of files' do
+    bucket = described_class.new bucket_name, prefix: 'm/'
     expect(bucket.ls('**/*').count).to eq(2121)
     bucket.close
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bfs-s3
 version: !ruby/object:Gem::Version
-  version: 0.7.3
+  version: 0.8.1
 platform: ruby
 authors:
 - Dimitrij Denissenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-07-10 00:00:00.000000000 Z
+date: 2021-02-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk-s3
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.7.3
+        version: 0.8.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.7.3
+        version: 0.8.1
 description: https://github.com/bsm/bfs.rb
 email: dimitrij@blacksquaremedia.com
 executables: []