imap-backup 1.0.1 → 1.0.2

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

@@ -13,6 +13,9 @@ module Imap
         def uids
           @connection.imap.examine(@folder)
           @connection.imap.uid_search(['ALL']).sort
+        rescue Net::IMAP::NoResponseError => e
+          warn "Folder '#{@folder}' does not exist"
+          []
         end
 
         def fetch(uid)

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

@@ -10,6 +10,7 @@ module Imap
 
         def run
           loop do
+            system('clear')
             Setup.highline.choose do |menu|
               password =
                 if @account[:password] == ''

data/lib/imap/backup/configuration/folder_chooser.rb

@@ -18,6 +18,7 @@ module Imap
           end
           @folders = @connection.folders
           loop do
+            system('clear')
             Setup.highline.choose do |menu|
               menu.header = 'Add/remove folders'
               menu.index = :number

data/lib/imap/backup/configuration/setup.rb

@@ -17,6 +17,7 @@ module Imap
 
         def run
           loop do
+            system('clear')
             self.class.highline.choose do |menu|
               menu.header = 'Choose an action'
               @config.data[:accounts].each do |account|

data/lib/imap/backup/version.rb

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

data/spec/unit/account/folder_spec.rb

@@ -2,6 +2,8 @@
 require 'spec_helper'
 
 describe Imap::Backup::Account::Folder do
+  include InputOutputTestHelpers
+
   context 'with instance' do
     before :each do
       @imap    = stub('Net::IMAP')
@@ -17,6 +19,20 @@ describe Imap::Backup::Account::Folder do
 
         subject.uids.should == [123, 5678]
       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)
+
+        capturing_output do
+          expect(subject.uids).to eq([])
+        end
+      end
     end
 
     context '#fetch' do

data/spec/unit/configuration/account_spec.rb

@@ -38,11 +38,18 @@ describe Imap::Backup::Configuration::Account do
       @store = stub('Imap::Backup::Configuration::Store')
       @store.stub!(:data => @data)
       @input, @output = prepare_highline
+      subject.stub(:system => nil)
     end
 
     subject { Imap::Backup::Configuration::Account.new(@store, @account1) }
 
     context 'menu' do
+      it 'clears the screen' do
+        subject.should_receive(:system).with('clear')
+
+        subject.run
+      end
+
       it 'should show a menu' do
         subject.run
 

data/spec/unit/configuration/folder_chooser_spec.rb

@@ -7,68 +7,93 @@ describe Imap::Backup::Configuration::FolderChooser do
   include InputOutputTestHelpers
 
   context '#run' do
-    before :each do
-      empty_account = {
+    let(:connection) do
+      stub(
+        'Imap::Backup::Account::Connection'
+      )
+    end
+    let(:existing_account) do
+      {
+        :folders => [{:name => 'my_folder'}]
+      }
+    end
+    let(:empty_account) do
+      {
         :folders => []
       }
-      @connection = setup_account(empty_account, [])
-      @input, @output = prepare_highline
+    end
+    let(:remote_folders) do
+      folder1 = stub('folder', :name => 'my_folder')       # this one is already backed up
+      folder2 = stub('folder', :name => 'another_folder')
+      [folder1, folder2]
     end
 
-    def setup_account(account, remote_folders)
-      @account   = account
-      connection = stub('Imap::Backup::Account::Connection')
-      connection.stub!(:folders).and_return(remote_folders)
-      Imap::Backup::Account::Connection.stub!(:new).with(account).and_return(connection)
-      connection
+    before do
+      @input, @output = prepare_highline
     end
 
-    subject { Imap::Backup::Configuration::FolderChooser.new(@account) }
+    context 'empty account' do
+      let(:account) { empty_account }
 
-    it 'should connect to the account' do
-      Imap::Backup::Account::Connection.should_receive(:new).with(@account).and_return(@connection)
+      subject { Imap::Backup::Configuration::FolderChooser.new(account) }
 
-      subject.run
-    end
+      before do
+        connection.stub!(:folders).and_return([])
+        Imap::Backup::Account::Connection.stub!(:new).with(account).and_return(connection)
+        subject.stub(:system).with('clear')
+      end
 
-    it 'should handle connection errors' do
-      Imap::Backup::Account::Connection.should_receive(:new).with(@account).and_raise('error')
-      Imap::Backup::Configuration::Setup.highline.should_receive(:ask).with('Press a key ')
+      it 'should connect to the account' do
+        Imap::Backup::Account::Connection.should_receive(:new).with(account).and_return(connection)
 
-      capturing_output do
         subject.run
-      end.should =~ /connection failed/i
-    end
+      end
 
-    it 'should get a list of account folders' do
-      @connection.should_receive(:folders).and_return([])
+      it 'should handle connection errors' do
+        Imap::Backup::Account::Connection.should_receive(:new).with(account).and_raise('error')
+        Imap::Backup::Configuration::Setup.highline.should_receive(:ask).with('Press a key ')
 
-      subject.run
-    end
+        capturing_output do
+          subject.run
+        end.should =~ /connection failed/i
+      end
 
-    it 'should show the menu' do
-      subject.run
-      
-      @output.string.should =~ %r{Add/remove folders}
-    end
+      it 'should get a list of account folders' do
+        connection.should_receive(:folders).and_return([])
 
-    it 'should return to the account menu' do
-      @input.should_receive(:gets).and_return("return\n")
+        subject.run
+      end
+
+      it 'clears the screen' do
+        subject.should_receive(:system).with('clear')
 
-      subject.run
-      
-      @output.string.should =~ %r{return to the account menu}
+        subject.run
+      end
+
+      it 'should show the menu' do
+        subject.run
+        
+        @output.string.should =~ %r{Add/remove folders}
+      end
+
+      it 'should return to the account menu' do
+        @input.should_receive(:gets).and_return("return\n")
+
+        subject.run
+        
+        @output.string.should =~ %r{return to the account menu}
+      end
     end
 
     context 'folder listing' do
-      before :each do
-        account = {
-          :folders => [{:name => 'my_folder'}]
-        }
-        folder1 = stub('folder', :name => 'my_folder')       # this one is already backed up
-        folder2 = stub('folder', :name => 'another_folder')
-        remote_folders = [folder1, folder2]
-        @connection = setup_account(account, remote_folders)
+      let(:account) { existing_account }
+
+      subject { Imap::Backup::Configuration::FolderChooser.new(account) }
+
+      before do
+        connection.stub!(:folders).and_return(remote_folders)
+        Imap::Backup::Account::Connection.stub!(:new).with(account).and_return(connection)
+        subject.stub(:system).with('clear')
       end
 
       it 'should list folders' do
@@ -99,7 +124,7 @@ describe Imap::Backup::Configuration::FolderChooser do
         subject.run
 
         @output.string.should =~ /\d+\. \+ another_folder/
-        @account[:folders].should include( { :name => 'another_folder' } )
+        account[:folders].should include( { :name => 'another_folder' } )
       end
 
       it 'should remove folders' do
@@ -117,7 +142,7 @@ describe Imap::Backup::Configuration::FolderChooser do
         subject.run
 
         @output.string.should =~ /\d+\. \- my_folder/
-        @account[:folders].should_not include( { :name => 'my_folder' } )
+        account[:folders].should_not include( { :name => 'my_folder' } )
       end
     end
   end

data/spec/unit/configuration/setup_spec.rb

@@ -16,6 +16,7 @@ describe Imap::Backup::Configuration::Setup do
     before :each do
       prepare_store
       @input, @output = prepare_highline
+      subject.stub(:system => nil)
     end
 
     def prepare_store
@@ -39,6 +40,12 @@ describe Imap::Backup::Configuration::Setup do
       @output.string.should =~ /quit/
     end
 
+    it 'clears the screen' do
+      subject.should_receive(:system).with('clear')
+
+      subject.run
+    end
+
     it 'should list accounts' do
       subject.run
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: imap-backup
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-01-10 00:00:00.000000000 Z
+date: 2013-02-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -187,7 +187,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1867766777440441483
+      hash: -2027383479193417702
 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: 1867766777440441483
+      hash: -2027383479193417702
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23