saviour 0.4.9 → 0.4.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 52d3e1c76ad21adf351834a7d0dd101582e9a835
-  data.tar.gz: 2ac25be01722c1432e8f75408be8a448b0505761
+  metadata.gz: d8f60221415cb3f9496b915d56999f2404784dba
+  data.tar.gz: 9de5a6f7f39a3df2c15e72486915a60cd1cf7231
 SHA512:
-  metadata.gz: ace50b4dfc8bfc04ab889fcbf92955d0815eb9f48fa97900ca3f9227d80deabba9f8e74e79f1ee4258f288058268d34faeab0f0563e3ece363c868e53d637195
-  data.tar.gz: b7b9b71f3150a8142a84ddcf9979ba3be6cea056692d42f580e2496db2525103368ceb158e9fadba150292103573998e82488ba0ca29af654efabfaed3273da9
+  metadata.gz: c2995d015c6a1f6b66278a85a953230c76ceba7cddc4e5fc5ba8cd76bbfd4dd53f760023e735fa06723f738da4bbd18a28e32d10bc73dab9ae53e380b41b48ec
+  data.tar.gz: ab58fec40cdaac44bc69bf529336c7e07065e924fd49b15eab39268d129bfaa791c553c34614d090adbf76a7d72c8d2bdcf9d939b759b8f77b8a6de74132fbd0

data/lib/saviour/local_storage.rb

@@ -91,7 +91,7 @@ module Saviour
       basedir = ::File.dirname(path)
       return if basedir == "."
 
-      while basedir != "/" && Dir.entries(real_path(basedir)) - [".", ".."] ==[]
+      while basedir != "/" && basedir != "." && Dir.entries(real_path(basedir)) - [".", ".."] == []
         Dir.rmdir(real_path(basedir))
         basedir = ::File.dirname(basedir)
       end

data/lib/saviour/version.rb

@@ -1,3 +1,3 @@
 module Saviour
-  VERSION = "0.4.9"
+  VERSION = "0.4.10"
 end

data/spec/models/local_storage_spec.rb

@@ -87,19 +87,27 @@ describe Saviour::LocalStorage do
     end
 
     it "does not leave an empty dir behind" do
-      with_test_file("camaloon.jpg") do |file, _|
-        final_path = File.join(@tmpdir, "some/folder/dest.jpg")
+      final_path = File.join(@tmpdir, "some/folder/dest.jpg")
+      subject.write "Some contents", "some/folder/dest.jpg"
+      expect(File.file?(final_path)).to be_truthy
 
-        FileUtils.mkdir_p(File.dirname(final_path))
-        FileUtils.cp file.path, final_path
-        expect(File.file?(final_path)).to be_truthy
+      subject.delete("/some/folder/dest.jpg")
 
-        subject.delete("/some/folder/dest.jpg")
+      expect(File.file?(final_path)).to be_falsey
+      expect(File.directory?(File.join(@tmpdir, "some/folder"))).to be_falsey
+      expect(File.directory?(File.join(@tmpdir, "some"))).to be_falsey
+    end
 
-        expect(File.file?(final_path)).to be_falsey
-        expect(File.directory?(File.join(@tmpdir, "some/folder"))).to be_falsey
-        expect(File.directory?(File.join(@tmpdir, "some"))).to be_falsey
-      end
+    it "does not leave an empty dir behind with relative paths" do
+      final_path = File.join @tmpdir, "some/folder/dest.jpg"
+      subject.write "Some contents", "some/folder/dest.jpg"
+      expect(File.file?(final_path)).to be_truthy
+
+      subject.delete("some/folder/dest.jpg")
+
+      expect(File.file?(final_path)).to be_falsey
+      expect(File.directory?(File.join(@tmpdir, "some/folder"))).to be_falsey
+      expect(File.directory?(File.join(@tmpdir, "some"))).to be_falsey
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: saviour
 version: !ruby/object:Gem::Version
-  version: 0.4.9
+  version: 0.4.10
 platform: ruby
 authors:
 - Roger Campos
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-01-16 00:00:00.000000000 Z
+date: 2018-01-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord