bfs-ftp 0.7.3 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b481cc58f871c2d32a31c6b27a2d7663fabb5e780823aaf8a59766ca84ce92a7
-  data.tar.gz: d38ae55120b1b1ba286b0754a8cd549652a52d7dcec15938e2f174beb7a1995a
+  metadata.gz: 6f743896fc3db215bd1d8219ac90456e26495f6329629801a9b0fa55552bcab0
+  data.tar.gz: d1103b798c992e32966b7f4bcf083ca3c41b7a125faa9a10ec6aea7cf9681d22
 SHA512:
-  metadata.gz: f367b5df775d462366cc2af0017d5684cac8c59f2e291ca535050e6bbe5807aa22993f4ebab6b3a7189cd311d3644f56791c36f210d5f4a9d83111f80c93ee02
-  data.tar.gz: d4f8b6949f088c003e97d0de38e69e574096ddc650588e8cadee5782addba3c5c99a95a7f27b078cb92c935ccc954c3deedb5eaf32de3452a1aa26c88c05ac80
+  metadata.gz: 92f53a5a7c038278dc3f0ed6c6a2dfb1544945ac1caa0929e165859c1e6f3f8bbba454bf6d91180e02099a24d189a8923709e38ea0bf6579073e2671df999fff
+  data.tar.gz: cd75cb8f1db2c67c668bdb68116d69d664a4aa83c6d9e083cdef0ba3d094b280bc04ddfd8513a88cfa72ea90098253f1a43ab445c63a716540dc91c098f1275a

data/lib/bfs/bucket/ftp.rb

@@ -52,17 +52,10 @@ module BFS
       # Creates a new file and opens it for writing
       def create(path, encoding: self.encoding, perm: self.perm, **_opts, &block)
         path = norm_path(path)
-        temp = BFS::TempWriter.new(path, encoding: encoding, perm: perm) do |t|
+        BFS::Writer.new(path, encoding: encoding, perm: perm) do |t|
           mkdir_p File.dirname(path)
           @client.put(t, path)
-        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/ftp_spec.rb

@@ -1,16 +1,18 @@
 require 'spec_helper'
 
-sandbox = { host: '127.0.0.1', port: 7021, username: 'ftpuser', password: 'ftppass' }.freeze
-
 RSpec.describe BFS::Bucket::FTP, ftp: true do
-  subject { described_class.new sandbox[:host], **sandbox.merge(prefix: SecureRandom.uuid) }
-  after   { subject.close }
+  subject { described_class.new hostname, **conn_opts }
+
+  let(:hostname) { '127.0.0.1' }
+  let(:conn_opts) { { port: 7021, username: 'ftpuser', password: 'ftppass', prefix: SecureRandom.uuid } }
+
+  after { subject.close }
 
   it_behaves_like 'a bucket',
                   content_type: false,
                   metadata: false
 
-  it 'should resolve from URL' do
+  it 'resolves from URL' do
     bucket = BFS.resolve('ftp://ftpuser:ftppass@127.0.0.1:7021')
     expect(bucket).to be_instance_of(described_class)
     expect(bucket.instance_variable_get(:@client).pwd).to eq('/ftp/ftpuser')

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bfs-ftp
 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: net-ftp-list
   requirement: !ruby/object:Gem::Requirement