bfs-scp 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b576e021897096431c5d8b7a5b25d854057a4c2410f3ef149c098678a3db3598
-  data.tar.gz: 8129ba1132e4282c346491ea0a8a3eec40cc7a38fa3ec3e27b354f0fce44f8de
+  metadata.gz: 6262201fae915737a6ed368d730f812a0e179ce6a8b21f722f4aacd93fe47ceb
+  data.tar.gz: 2c1225b186236d9b112ec3ceb787aa19cb4cf0b28ca2dcb43742b0f2c3ef1e2e
 SHA512:
-  metadata.gz: 5c679c83e17e8f06a8ccf06269755f007b7e72fc2ea12d2a9c4e99e36b252f688336bbf9a3495ae181594452b3d00abb15e66217accd8e4c320505191b6395c9
-  data.tar.gz: d84fe1852fd409e2fbf60b1eac8b48237bf0b3fae3db98eac1529e7922278fbf7b9674bd2ffbbfe716b26230dfe3ab4d8c40654fae17fcb8d0b84247d144a563
+  metadata.gz: 1cdc7db7d5bc18d93733ceaf15a114b832c234cfbf1099387072425716d182f1c063c877cae63948667b6c8ee26b776b58653d07153993243cfb54f85121e33d
+  data.tar.gz: b7a0dad1fa8b08604ab4ec6fb844160ea12e05345aee867884b4d5a85f65f96713979f9d807029d706ba0d6f6f828b03bd635526d11c68542ab190995f34fb5a

data/lib/bfs/bucket/scp.rb

@@ -41,17 +41,17 @@ module BFS
 
         if @prefix # rubocop:disable Style/GuardClause
           @prefix = norm_path(@prefix) + '/'
-          mkdir_p(@prefix)
+          mkdir_p abs_path(@prefix)
         end
       end
 
       # Lists the contents of a bucket using a glob pattern
       def ls(pattern='**/*', _opts={})
-        prefix = @prefix || '.'
+        prefix = @prefix ? abs_path(@prefix) : '/'
         Enumerator.new do |y|
           sh! 'find', prefix, '-type', 'f' do |out|
             out.each_line do |line|
-              path = trim_prefix(line.strip)
+              path = trim_prefix(norm_path(line.strip))
               y << path if File.fnmatch?(pattern, path, File::FNM_PATHNAME)
             end
           end
@@ -141,6 +141,15 @@ module BFS
 
       private
 
+      def abs_path(path)
+        path = "/#{path}" unless path.start_with?('~/', './')
+        path
+      end
+
+      def full_path(*)
+        abs_path(super)
+      end
+
       def mkdir_p(path)
         sh! 'mkdir', '-p', path
       end

data/spec/bfs/bucket/scp_spec.rb

@@ -13,15 +13,36 @@ run_spec = \
   end
 
 RSpec.describe BFS::Bucket::SCP, if: run_spec do
-  subject { described_class.new sandbox[:host], sandbox[:opts].merge(prefix: SecureRandom.uuid) }
-  after   { subject.close }
+  context 'absolute' do
+    subject { described_class.new sandbox[:host], sandbox[:opts].merge(prefix: SecureRandom.uuid) }
+    after   { subject.close }
 
-  it_behaves_like 'a bucket',
-    content_type: false,
-    metadata: false
+    it_behaves_like 'a bucket', content_type: false, metadata: false
+  end
+
+  context 'relative' do
+    subject { described_class.new sandbox[:host], sandbox[:opts].merge(prefix: "~/#{SecureRandom.uuid}") }
+    after   { subject.close }
+
+    it_behaves_like 'a bucket', content_type: false, metadata: false
+  end
 
   it 'should resolve from URL' do
     bucket = BFS.resolve('scp://root:root@127.0.0.1:7022')
     expect(bucket).to be_instance_of(described_class)
   end
+
+  it 'should handle absolute and relative paths' do
+    abs = BFS::Blob.new("scp://root:root@127.0.0.1:7022/#{SecureRandom.uuid}/file.txt")
+    abs.create {|w| w.write 'absolute' }
+
+    rel = BFS::Blob.new("scp://root:root@127.0.0.1:7022/~/#{SecureRandom.uuid}/file.txt")
+    rel.create {|w| w.write 'relative' }
+
+    expect(abs.read).to eq('absolute')
+    expect(rel.read).to eq('relative')
+
+    abs.close
+    rel.close
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bfs-scp
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Dimitrij Denissenko
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: 0.5.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: 0.5.1
 - !ruby/object:Gem::Dependency
   name: net-scp
   requirement: !ruby/object:Gem::Requirement