bfs-gs 0.7.3 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ba5632283f82b8a38d0f25ccc839948c8d45fcdd81da8ed4bc3ab9154f62becb
-  data.tar.gz: eaed2684e48c0cb4cad20c51b1635ae4c4c2d173d4fdbe994e7a4f59ff760f8b
+  metadata.gz: '08fac95d84875eeb3873d5aa39d8ab7ec900bb4225580c9dbe4da12702401cdc'
+  data.tar.gz: 043fad275f4abc414612801862411daeb3a33dd3e391d87a868ec701d5f9dbed
 SHA512:
-  metadata.gz: f06d8e618fc56f661c965875d5597fd1a0c692c30f23e3e67d81c0889b9e37d309a211f88c51146f674a09b858be5f060a813ea95bcfb8379ca532191caa25dd
-  data.tar.gz: aa7dced6719a04c98817e4466b514598614b2319f4fdf597dcf650bc1760c2548a3258651dc2e74f3ee2ef54d7d7ce652745e1ce308d219da4032c336abd0597
+  metadata.gz: 34b884c86445d6df45e098ceac85218efa76adea349e3d39f86a99eb7f92401be939b60fab6bdcc83e95dcaaf44a3da48e08c555fadda819806200d9f9dbdffa
+  data.tar.gz: 6fbdecadf4bb512bdf2bf6bbf224eea34646c7f915c50a8fc7f4ba9e2308c5401a4b8ed2811e7d7690502a5b4e8ac628b0e611d02f8558fdf953cbfb11107bc4

data/lib/bfs/bucket/gs.rb

@@ -59,18 +59,11 @@ module BFS
       def create(path, encoding: self.encoding, perm: self.perm, **opts, &block)
         opts[:metadata] = norm_meta(opts[:metadata])
         path = full_path(path)
-        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|
             @bucket.create_file(file, path, **opts)
           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/gs_spec.rb

@@ -6,37 +6,38 @@ module Google::Auth::CredentialsLoader
   module_function :warn_if_cloud_sdk_credentials # rubocop:disable Style/AccessModifierDeclarations
 end
 
-sandbox = { project: 'bogus', bucket: 'bsm-bfs-unittest' }.freeze
+bucket_name = 'bsm-bfs-unittest'
 
 RSpec.describe BFS::Bucket::GS, gs: true do
-  let(:prefix) { "x/#{SecureRandom.uuid}/" }
-
   subject do
-    described_class.new sandbox[:bucket], project_id: sandbox[:project], prefix: prefix
+    described_class.new bucket_name, prefix: prefix
   end
-  after :all do
-    bucket = described_class.new sandbox[:bucket], project_id: sandbox[:project], 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("gs://#{sandbox[:bucket]}/?project_id=#{sandbox[:project]}")
+  it 'resolves from URL' do
+    bucket = BFS.resolve("gs://#{bucket_name}")
     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 be_nil
     bucket.close
 
-    bucket = BFS.resolve("gs://#{sandbox[:bucket]}/a/b/")
+    bucket = BFS.resolve("gs://#{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], project_id: sandbox[:project], 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-gs
 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: bfs
@@ -16,14 +16,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
 - !ruby/object:Gem::Dependency
   name: google-cloud-storage
   requirement: !ruby/object:Gem::Requirement