imap-backup 2.1.1 → 2.2.0

data/spec/unit/imap/backup/uploader_spec.rb

@@ -2,7 +2,9 @@ describe Imap::Backup::Uploader do
   subject { described_class.new(folder, serializer) }
 
   let(:folder) do
-    instance_double(Imap::Backup::Account::Folder, uids: [2, 3], append: 99)
+    instance_double(
+      Imap::Backup::Account::Folder, uids: [2, 3], append: 99, name: "foo"
+    )
   end
   let(:serializer) do
     instance_double(
@@ -11,27 +13,38 @@ describe Imap::Backup::Uploader do
       update_uid: nil
     )
   end
+  let(:missing_message) do
+    instance_double(Email::Mboxrd::Message, supplied_body: "missing message")
+  end
+  let(:existing_message) do
+    instance_double(Email::Mboxrd::Message, supplied_body: "existing message")
+  end
 
   describe "#run" do
     before do
-      allow(serializer).to receive(:load).with(1) { "missing message" }
-      allow(serializer).to receive(:load).with(2) { "existing message" }
-      subject.run
+      allow(serializer).to receive(:load).with(1) { missing_message }
+      allow(serializer).to receive(:load).with(2) { existing_message }
     end
 
     context "with messages that are missing" do
       it "restores them" do
-        expect(folder).to have_received(:append).with("missing message")
+        expect(folder).to receive(:append).with(missing_message)
+
+        subject.run
       end
 
       it "updates the local message id" do
-        expect(serializer).to have_received(:update_uid).with(1, 99)
+        expect(serializer).to receive(:update_uid).with(1, 99)
+
+        subject.run
       end
     end
 
     context "with messages that are present on server" do
       it "does nothing" do
-        expect(folder).to_not have_received(:append).with("existing message")
+        expect(folder).to_not receive(:append).with(existing_message)
+
+        subject.run
       end
     end
   end

data/spec/unit/imap/backup/utils_spec.rb

@@ -5,8 +5,8 @@ describe Imap::Backup::Utils do
   let(:exists) { true }
 
   before do
-    allow(File).to receive(:stat).and_return(stat)
-    allow(File).to receive(:exist?).with(filename).and_return(exists)
+    allow(File).to receive(:stat) { stat }
+    allow(File).to receive(:exist?).with(filename) { exists }
   end
 
   describe ".check_permissions" do
@@ -72,22 +72,21 @@ describe Imap::Backup::Utils do
     end
 
     it "does nothing if an empty path is supplied" do
-      described_class.make_folder("aaa", "", 0o222)
+      expect(FileUtils).to_not receive(:mkdir_p)
 
-      expect(FileUtils).to_not have_received(:mkdir_p)
+      described_class.make_folder("aaa", "", 0o222)
     end
 
     it "creates the path" do
-      described_class.make_folder("/base/path", "new/folder", 0o222)
+      expect(FileUtils).to receive(:mkdir_p).with("/base/path/new/folder")
 
-      expect(FileUtils).to have_received(:mkdir_p).with("/base/path/new/folder")
+      described_class.make_folder("/base/path", "new/folder", 0o222)
     end
 
     it "sets permissions on the path" do
-      described_class.make_folder("/base/path/new", "folder", 0o222)
+      expect(FileUtils).to receive(:chmod).with(0o222, "/base/path/new/folder")
 
-      expect(FileUtils).
-        to have_received(:chmod).with(0o222, "/base/path/new/folder")
+      described_class.make_folder("/base/path/new", "folder", 0o222)
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: imap-backup
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.2.0
 platform: ruby
 authors:
 - Joe Yates
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-06-12 00:00:00.000000000 Z
+date: 2019-09-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: highline