imap-backup 1.0.12 → 1.0.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7c42e0f05b9fe1321562fd54446d99f765ad1544
-  data.tar.gz: 6e005f9c0605e371063ea1b2eec65e04ad6bd71c
+  metadata.gz: 03ba4f6a03ae0efc265699ae27662d4089867f60
+  data.tar.gz: 645ee02996bcf94bddc4d890ce6d22c248584fb3
 SHA512:
-  metadata.gz: ea5b48a3cbbf5b9513efbb0f453941bf33e96e1f6008b8d4af4414ef5b03a8502c30393c512e630186c0ba49e6d232facc93433fa86616aaab97716ac5082d8d
-  data.tar.gz: c9d9a35daf1ed227922de3e7601a30a383ea4cc5d87259a546a1a8914db48206909b6c75c60993db12ee147a50ae204fe5d9bff41ca0ffd427fa72aa86a3cc24
+  metadata.gz: bd1aea391344bc6a3d66d2748f4de0f8400363e5bfdb747d440aacbd1cc1c67b145ee3025431afaf233e6cab69ec14e84448634c0ba87caaf20f42384fa16847
+  data.tar.gz: d891fc0666d94a25ee6c9f201944b7f29eea73742e81c6ce44f8e87b44568f1a8a2214eeeef7bff6c977ee79d9be2b591e46592760f79d9690b56d2ed622d34f

data/README.md

@@ -93,7 +93,7 @@ Manually, from the command line:
 $ imap-backup
 ```
 
-Altertatively, add it to your crontab.
+Alternatively, add it to your crontab.
 
 # Result
 

data/lib/email/mboxrd/message.rb

@@ -33,7 +33,11 @@ module Email::Mboxrd
     end
 
     def asctime
-      parsed.date.asctime
+      date ? date.asctime : ''
+    end
+
+    def date
+      parsed.date
     end
   end
 end

data/lib/imap/backup/version.rb

@@ -3,6 +3,6 @@ module Imap; end
 module Imap::Backup
   MAJOR    = 1
   MINOR    = 0
-  REVISION = 12
+  REVISION = 13
   VERSION  = [MAJOR, MINOR, REVISION].map(&:to_s).join('.')
 end

data/spec/unit/downloader_spec.rb

@@ -2,76 +2,38 @@
 require 'spec_helper'
 
 describe Imap::Backup::Downloader do
-  context 'with account and downloader' do
-    let(:local_path) { '/base/path' }
-    let(:stat) { double('File::Stat', :mode => 0700) }
-    let(:message) do
-      {
-        'RFC822' => 'the body',
-        'other'  => 'xxx'
-      }
-    end
+  describe '#run' do
+    let(:message) { 'message' }
     let(:folder) { double('Imap::Backup::Account::Folder', :fetch => message) }
-    let(:serializer) do
-      double(
-        'Imap::Backup::Serializer',
-        :prepare => nil,
-        :exist?  => true,
-        :uids    => [],
-        :save    => nil,
-      )
-    end
-
-    before { allow(File).to receive(:stat).with(local_path).and_return(stat) }
+    let(:folder_uids) { ['111', '222', '333'] }
+    let(:serializer) { double('Imap::Backup::Serializer', :save => nil) }
+    let(:serializer_uids) { ['222'] }
 
     subject { described_class.new(folder, serializer) }
 
-    context '#run' do
-      context 'with folder' do
-        it 'should list messages' do
-          allow(folder).to receive(:uids).and_return([])
+    before do
+      allow(folder).to receive(:uids).and_return(folder_uids)
+      allow(serializer).to receive(:uids).and_return(serializer_uids)
+      allow(folder).to receive(:fetch).with('333').and_return(nil)
+      subject.run
+    end
 
-          subject.run
+    context '#run' do
+      context 'fetched messages' do
+        it 'are saved' do
+          expect(serializer).to have_received(:save).with('111', message)
         end
+      end
 
-        context 'with messages' do
-          before :each do
-            allow(folder).to receive(:uids).and_return(['123', '999', '1234'])
-          end
-
-          it 'skips failed fetches' do
-            allow(folder).to receive(:fetch).with('999').and_return(nil)
-
-            subject.run
-
-            expect(serializer).to_not have_received(:save).with('999', anything)
-          end
-
-          context 'to download' do
-            before :each do
-              allow(serializer).to receive(:exist?) do |uid|
-                if uid == '123'
-                  true
-                else
-                  false
-                end
-              end
-            end
-
-            it 'requests messages' do
-              subject.run
-
-              expect(folder).to have_received(:fetch).with('999')
-              expect(folder).to have_received(:fetch).with('1234')
-            end
-
-            it 'saves messages' do
-              subject.run
+      context 'messages which are already present' do
+        specify 'are skipped' do
+          expect(serializer).to_not have_received(:save).with('222', anything)
+        end
+      end
 
-              expect(serializer).to have_received(:save).with('999', message)
-              expect(serializer).to have_received(:save).with('1234', message)
-            end
-          end
+      context 'failed fetches' do
+        specify 'are skipped' do
+          expect(serializer).to_not have_received(:save).with('333', anything)
         end
       end
     end

data/spec/unit/email/mboxrd/message_spec.rb

@@ -36,5 +36,13 @@ describe Email::Mboxrd::Message do
     it "appends > before '>+From '" do
       expect(subject.to_s).to include("\n>>>From quoted")
     end
+
+    context 'when date is missing' do
+      let(:date) { nil }
+
+      it 'does no fail' do
+        expect { subject.to_s }.to_not raise_error
+      end
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: imap-backup
 version: !ruby/object:Gem::Version
-  version: 1.0.12
+  version: 1.0.13
 platform: ruby
 authors:
 - Joe Yates
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-19 00:00:00.000000000 Z
+date: 2014-12-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake