imap-backup 1.0.2 → 1.0.3

data/lib/imap/backup/account/folder.rb

@@ -23,6 +23,9 @@ module Imap
           message = @connection.imap.uid_fetch([uid.to_i], REQUESTED_ATTRIBUTES)[0][1]
           message['RFC822'].force_encoding('utf-8') if RUBY_VERSION > '1.9'
           message
+        rescue Net::IMAP::NoResponseError => e
+          warn "Folder '#{@folder}' does not exist"
+          nil
         end
       end
     end

data/lib/imap/backup/downloader.rb

@@ -13,6 +13,7 @@ module Imap
         uids = @folder.uids - @serializer.uids
         uids.each do |uid|
           message = @folder.fetch(uid)
+          next if message.nil?
           @serializer.save(uid, message)
         end
       end

data/lib/imap/backup/version.rb

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

data/spec/unit/account/folder_spec.rb

@@ -4,6 +4,10 @@ require 'spec_helper'
 describe Imap::Backup::Account::Folder do
   include InputOutputTestHelpers
 
+  let(:missing_mailbox_data) { stub('Data', :text => 'Unknown Mailbox: my_folder') }
+  let(:missing_mailbox_response) { stub('Response', :data => missing_mailbox_data) }
+  let(:missing_mailbox_error) { Net::IMAP::NoResponseError.new(missing_mailbox_response) }
+
   context 'with instance' do
     before :each do
       @imap    = stub('Net::IMAP')
@@ -21,13 +25,10 @@ describe Imap::Backup::Account::Folder do
       end
 
       it 'returns an empty array for missing mailboxes' do
-        data     = stub('Data', :text => 'Unknown Mailbox: my_folder')
-        response = stub('Response', :data => data)
-        error    = Net::IMAP::NoResponseError.new(response)
         @imap.
           should_receive(:examine).
           with('my_folder').
-          and_raise(error)
+          and_raise(missing_mailbox_error)
 
         capturing_output do
           expect(subject.uids).to eq([])
@@ -53,6 +54,17 @@ describe Imap::Backup::Account::Folder do
         subject.fetch(123)
       end
 
+      it "returns nil if the mailbox doesn't exist" do
+        @imap.
+          should_receive(:examine).
+          with('my_folder').
+          and_raise(missing_mailbox_error)
+
+        capturing_output do
+          expect(subject.fetch(123)).to be_nil
+        end
+      end
+
       if RUBY_VERSION > '1.9'
         it 'should set the encoding on the message' do
           @imap.stub!(:examine => nil, :uid_fetch => [[nil, @message]])

data/spec/unit/downloader_spec.rb

@@ -42,6 +42,13 @@ describe Imap::Backup::Downloader do
             subject.run
           end
 
+          it 'skips failed fetches' do
+            @folder.should_receive(:fetch).with('999').and_return(nil)
+            @serializer.should_not_receive(:save).with('999', anything)
+
+            subject.run
+          end
+
           context 'to download' do
             before :each do
               @serializer.stub!(:exist?) do |uid|

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: imap-backup
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.0.3
   prerelease: 
 platform: ruby
 authors:
@@ -187,7 +187,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -2027383479193417702
+      hash: 1167713560722086376
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -196,7 +196,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -2027383479193417702
+      hash: 1167713560722086376
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23