bfs-s3 0.7.1 → 0.7.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e49f967d04b05a36711b1c2b858347af451e6c6aa9690739c524b21249137aee
-  data.tar.gz: dfe6415867f0e4af1a8b2106123972aa12f3531729b65bf0806081076cd1b903
+  metadata.gz: c7b0a87bb99ee39fefb3d297a891b96c740793529a187a8f19a3a9f37219b1e5
+  data.tar.gz: 4d38e77a1b7f0a1f3a9591dfd4f00a038aec0c43f9dda60203049d9036df8004
 SHA512:
-  metadata.gz: f29025c22ba7bb6bb72c0a7c517da339de233964ced69e6aae02dba86f2a69a54d80a6eb7d3915808b303159c6a864fe498703583932de1368314458ef414b4e
-  data.tar.gz: 4fd3e4ebac13f4f7f21698f8e193e977319c906902d1510e6879fdae74eecc51c41cfe3f7b1d24994511d1a8beaa2484fadde2d145a5e332c80508d3749412f2
+  metadata.gz: d79da1d49939071bd04072e3120d4e4d4dd7c35c78cae2eba0010e6e7a0900052cc92970ba6e56c17bb1b6595b0a5553d3d5fb274140429fd47ca9b57ba5eb17
+  data.tar.gz: 38831316b1684be8981982309b4d85e03b2c57e6ec29ed6e245cb8ddaa042166b5a4ca0479af621e5fa56eb0cf2740d2274029d8a4311a658428074e62f699c5

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::TempWriter.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,45 +1,40 @@
 require 'spec_helper'
 
-sandbox  = { bucket: 'bsm-bfs-unittest' }.freeze
-run_spec = \
-  begin
-    s = Aws::S3::Client.new
-    s.head_bucket(bucket: sandbox[:bucket])
-    true
-  rescue StandardError => e
-    warn "WARNING: unable to run #{File.basename __FILE__}: #{e.message}"
-    false
-  end
+bucket_name = 'bsm-bfs-unittest'
 
-RSpec.describe BFS::Bucket::S3, if: run_spec do
+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/'
+    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")
+    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/'
+    bucket = described_class.new bucket_name, prefix: 'm/'
     expect(bucket.ls('**/*').count).to eq(2121)
+    bucket.close
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bfs-s3
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.6
 platform: ruby
 authors:
 - Dimitrij Denissenko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-07-02 00:00:00.000000000 Z
+date: 2020-11-10 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.1
+        version: 0.7.6
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.7.1
+        version: 0.7.6
 description: https://github.com/bsm/bfs.rb
 email: dimitrij@blacksquaremedia.com
 executables: []
@@ -67,7 +67,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: S3 bucket adapter for bfs