imap-backup 0.0.5 → 1.0.0

data/spec/unit/configuration/setup_spec.rb

@@ -1,22 +1,18 @@
 # encoding: utf-8
-load File.expand_path( '../../spec_helper.rb', File.dirname(__FILE__) )
+require 'spec_helper'
 
 describe Imap::Backup::Configuration::Setup do
-
   include HighLineTestHelpers
 
   context '#initialize' do
-
     it 'should not require the config file to exist' do
-      Imap::Backup::Configuration::Store.should_receive(:new).with(false)
+      Imap::Backup::Configuration::Store.should_receive(:new)
 
       Imap::Backup::Configuration::Setup.new
     end
-
   end
 
   context '#run' do
-
     before :each do
       prepare_store
       @input, @output = prepare_highline
@@ -26,7 +22,7 @@ describe Imap::Backup::Configuration::Setup do
       @account1 = { :username => 'account@example.com' }
       @data     = { :accounts => [ @account1 ] }
       @store    = stub('Imap::Backup::Configuration::Store', :data => @data, :path => '/base/path')
-      Imap::Backup::Configuration::Store.stub!(:new).with(false).and_return(@store)
+      Imap::Backup::Configuration::Store.stub!(:new).and_return(@store)
     end
 
     subject { Imap::Backup::Configuration::Setup.new }
@@ -109,8 +105,6 @@ describe Imap::Backup::Configuration::Setup do
 
       subject.run
     end
-
   end
-
 end
 

data/spec/unit/configuration/store_spec.rb

@@ -1,8 +1,7 @@
 # encoding: utf-8
-load File.expand_path( '../../spec_helper.rb', File.dirname(__FILE__) )
+require 'spec_helper'
 
 describe Imap::Backup::Configuration::Store do
-
   before :all do
     @configuration_directory = Imap::Backup::Configuration::Store::CONFIGURATION_DIRECTORY
     Imap::Backup::Configuration::Store.instance_eval { remove_const :'CONFIGURATION_DIRECTORY' }
@@ -14,27 +13,25 @@ describe Imap::Backup::Configuration::Store do
     Imap::Backup::Configuration::Store::CONFIGURATION_DIRECTORY = @configuration_directory
   end
 
-  context '#initialize' do
+  context '.exist?' do
+    it 'checks if the file exists' do
+      File.should_receive(:exist?).with('/base/path/config.json').and_return(true)
+
+      Imap::Backup::Configuration::Store.exist?
+    end
+  end
 
+  context '#initialize' do
     before :each do
       Imap::Backup::Utils.stub!(:check_permissions => nil)
     end
 
-    it 'by default, should fail if the config file is missing' do
+    it 'should not fail if the configuration file is missing' do
       File.should_receive(:directory?).with('/base/path').and_return(true)
       File.should_receive(:exist?).with('/base/path/config.json').and_return(false)
 
       expect do
         Imap::Backup::Configuration::Store.new
-      end.to raise_error(Imap::Backup::ConfigurationNotFound, /not found/)
-    end
-
-    it 'should not fail if fail_if_missing is false' do
-      File.should_receive(:directory?).with('/base/path').and_return(true)
-      File.should_receive(:exist?).with('/base/path/config.json').and_return(false)
-
-      expect do
-        Imap::Backup::Configuration::Store.new(false)
       end.to_not raise_error
     end
 
@@ -57,11 +54,9 @@ describe Imap::Backup::Configuration::Store do
 
       Imap::Backup::Configuration::Store.new
     end
-
   end
 
   context '#save' do
-
     before :each do
       # initialize
       File.stub!(:directory?).with('/base/path').and_return(false)
@@ -78,7 +73,7 @@ describe Imap::Backup::Configuration::Store do
       FileUtils.stub!(:chmod).with(0600, '/base/path/config.json')
     end
 
-    subject { Imap::Backup::Configuration::Store.new(false) }
+    subject { Imap::Backup::Configuration::Store.new }
 
     it 'should create the config directory' do
       File.should_receive(:directory?).with('/base/path').and_return(false)
@@ -100,7 +95,6 @@ describe Imap::Backup::Configuration::Store do
     end
 
     context 'saving accounts' do
-
       before :each do
         # initialize
         File.stub!(:exist?).with('/base/path/config.json').and_return(true)
@@ -164,10 +158,7 @@ describe Imap::Backup::Configuration::Store do
 
         subject.save
       end
-
     end
-
   end
-
 end
 

data/spec/unit/downloader_spec.rb

@@ -1,10 +1,8 @@
 # encoding: utf-8
-load File.expand_path( '../spec_helper.rb', File.dirname(__FILE__) )
+require 'spec_helper'
 
 describe Imap::Backup::Downloader do
-
   context 'with account and downloader' do
-
     before :each do
       local_path = '/base/path'
       stat       = stub('File::Stat', :mode => 0700)
@@ -24,9 +22,7 @@ describe Imap::Backup::Downloader do
     subject { Imap::Backup::Downloader.new(@folder, @serializer) }
 
     context '#run' do
-
       context 'with folder' do
-
         it 'should list messages' do
           @folder.should_receive(:uids).and_return([])
 
@@ -70,16 +66,10 @@ describe Imap::Backup::Downloader do
 
               subject.run
             end
-
           end
-
         end
-
       end
-
     end
-
   end
-
 end
 

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

@@ -0,0 +1,51 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Email::Mboxrd::Message do
+  let(:from) { 'me@example.com' }
+  let(:date) { DateTime.new(2012, 12, 13, 18, 23, 45) }
+  let(:message_body) { "Foo\nBar\nFrom at the beginning of the line\n>>From quoted" }
+  subject { Email::Mboxrd::Message.new(message_body) }
+
+  context '#to_s' do
+    let(:mail) do
+      mail = stub('Mail')
+      mail.stub(:from).and_return([from])
+      mail.stub(:date).and_return(date)
+      mail
+    end
+
+    before do
+      Mail.stub(:new).with(message_body).and_return(mail)
+    end
+
+    it 'does not modify the message' do
+      message_body2 = message_body.clone
+
+      message_body.should_receive(:clone).and_return(message_body2)
+      message_body.should_not_receive(:force_encoding).with('binary')
+
+      subject.to_s
+    end
+
+    it 'parses the message' do
+      Mail.should_receive(:new).with(message_body).and_return(mail)
+
+      subject.to_s
+    end
+
+    it "adds a 'From ' line at the start" do
+      expect(subject.to_s).to start_with('From ' + from + ' ' + date.asctime + "\n")
+    end
+
+    it "replaces existing 'From ' with '>From '" do
+      expect(subject.to_s).to include("\n>From at the beginning")
+    end
+
+    it "appends > before '>+From '" do
+      expect(subject.to_s).to include("\n>>>From quoted")
+    end
+  end
+end
+

data/spec/unit/serializer/base_spec.rb

@@ -0,0 +1,19 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Imap::Backup::Serializer::Base do
+  context '#initialize' do
+    let(:stat) { stub('File::Stat', :mode => 0345) }
+
+    it 'should fail if file permissions are to lax' do
+      File.stub!(:exist?).with('/base/path').and_return(true)
+      File.should_receive(:stat).with('/base/path').and_return(stat)
+
+      expect do
+        Imap::Backup::Serializer::Base.new('/base/path', 'my_folder')
+      end.to raise_error(RuntimeError, "Permissions on '/base/path' should be 0700, not 0345")
+    end
+  end
+end
+

data/spec/unit/serializer/directory_spec.rb

@@ -1,102 +1,79 @@
 # encoding: utf-8
-
-load File.expand_path('../../spec_helper.rb', File.dirname(__FILE__))
+require 'spec_helper'
 
 describe Imap::Backup::Serializer::Directory do
+  let(:stat) { stub('File::Stat', :mode => 0700) }
+  let(:files) { ['00000123.json', '000001.json'] }
+
+  before do
+    File.stub!(:stat).with('/base/path').and_return(stat)
+    FileUtils.stub!(:mkdir_p).with('/base/path/my_folder')
+    FileUtils.stub!(:chmod).with(0700, '/base/path/my_folder')
+    File.stub!(:exist?).with('/base/path').and_return(true)
+  end
 
-  context '#initialize' do
+  subject { Imap::Backup::Serializer::Directory.new('/base/path', 'my_folder') }
 
-    it 'should fail if download path file permissions are to lax' do
-      File.stub!(:exist?).with('/base/path').and_return(true)
-      stat = stub('File::Stat', :mode => 0345)
-      File.should_receive(:stat).with('/base/path').and_return(stat)
+  context '#uids' do
+    it 'should return the backed-up uids' do
+      File.should_receive(:exist?).with('/base/path/my_folder').and_return(true)
+      Dir.should_receive(:open).with('/base/path/my_folder').and_return(files)
 
-      expect do
-        Imap::Backup::Serializer::Directory.new('/base/path', 'my_folder')
-      end.to raise_error(RuntimeError, "Permissions on '/base/path' should be 0700, not 0345")
+      subject.uids.should == [1, 123]
     end
 
-  end
-
-  context 'with object' do
+    it 'should return an empty Array if the directory does not exist' do
+      File.should_receive(:exist?).with('/base/path/my_folder').and_return(false)
 
-    before :each do
-      stat = stub('File::Stat', :mode => 0700)
-      File.stub!(:stat).with('/base/path').and_return(stat)
-      FileUtils.stub!(:mkdir_p).with('/base/path/my_folder')
-      FileUtils.stub!(:chmod).with(0700, '/base/path/my_folder')
-      File.stub!(:exist?).with('/base/path').and_return(true)
+      subject.uids.should == []
     end
+  end
 
-    subject { Imap::Backup::Serializer::Directory.new('/base/path', 'my_folder') }
-
-    context '#uids' do
-
-      it 'should return the backed-up uids' do
-        files = ['00000123.json', '000001.json']
-
-        File.should_receive(:exist?).with('/base/path/my_folder').and_return(true)
-        Dir.should_receive(:open).with('/base/path/my_folder').and_return(files)
-
-        subject.uids.should == [1, 123]
-      end
-
-      it 'should return an empty Array if the directory does not exist' do
-        File.should_receive(:exist?).with('/base/path/my_folder').and_return(false)
-
-        subject.uids.should == []
-      end
+  context '#exist?' do
+    it 'should check if the file exists' do
+      File.should_receive(:exist?).with(%r{/base/path/my_folder/0+123.json}).and_return(true)
 
+      subject.exist?(123).should be_true
     end
+  end
 
-    context '#exist?' do
-
-      it 'should check if the file exists' do
-        File.should_receive(:exist?).with(%r{/base/path/my_folder/0+123.json}).and_return(true)
+  context '#save' do
+    let(:message) do
+      {
+        'RFC822' => 'the body',
+        'other'  => 'xxx'
+      }
+    end
+    let(:file) { stub('File', :write => nil) }
 
-        subject.exist?(123).should be_true
+    before do
+      File.stub!(:exist?).with(%r{/base/path/my_folder/0+1234.json}).and_return(true)
+      FileUtils.stub!(:chmod).with(0600, /0+1234.json$/)
+      File.stub!(:open) do |&block|
+        block.call file
       end
-
     end
 
-    context '#save' do
-
-      before :each do
-        File.stub!(:exist?).with(%r{/base/path/my_folder/0+1234.json}).and_return(true)
-        FileUtils.stub!(:chmod).with(0600, /0+1234.json$/)
-        @message = {
-          'RFC822' => 'the body',
-          'other'  => 'xxx'
-        }
-        @file = stub('File', :write => nil)
-        File.stub!(:open) do |&block|
-          block.call @file
-        end
+    it 'should save messages' do
+      File.should_receive(:open) do |&block|
+        block.call file
       end
+      file.should_receive(:write).with(/the body/)
 
-      it 'should save messages' do
-        File.should_receive(:open) do |&block|
-          block.call @file
-        end
-        @file.should_receive(:write).with(/the body/)
-
-        subject.save('1234', @message)
-      end
+      subject.save('1234', message)
+    end
 
-      it 'should JSON encode messages' do
-        @message.should_receive(:to_json)
+    it 'should JSON encode messages' do
+      message.should_receive(:to_json)
 
-        subject.save('1234', @message)
-      end
+      subject.save('1234', message)
+    end
 
-      it 'should set file permissions' do
-        FileUtils.should_receive(:chmod).with(0600, /0+1234.json$/)
+    it 'should set file permissions' do
+      FileUtils.should_receive(:chmod).with(0600, /0+1234.json$/)
 
-        subject.save(1234, @message)
-      end
+      subject.save(1234, message)
     end
-
   end
-
 end
 

data/spec/unit/serializer/mbox_spec.rb

@@ -0,0 +1,113 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Imap::Backup::Serializer::Mbox do
+  let(:stat) { stub('File::Stat', :mode => 0700) }
+  let(:mbox_pathname) { '/base/path/my/folder.mbox' }
+  let(:imap_pathname) { '/base/path/my/folder.imap' }
+
+  before do
+    File.stub(:exist?).with('/base/path').and_return(true)
+    File.stub!(:stat).with('/base/path').and_return(stat)
+    Imap::Backup::Utils.stub(:make_folder)
+  end
+
+  context '#initialize' do
+    before do
+      File.stub(:exist?).with(mbox_pathname).and_return(true)
+      File.stub(:exist?).with(imap_pathname).and_return(true)
+    end
+
+    it 'creates the containing directory' do
+      Imap::Backup::Utils.should_receive(:make_folder).with('/base/path', 'my', 0700)
+
+      Imap::Backup::Serializer::Mbox.new('/base/path', 'my/folder')
+    end
+
+    context 'mbox and imap files' do
+      it 'checks if they exist' do
+        File.should_receive(:exist?).with(mbox_pathname).and_return(true)
+        File.should_receive(:exist?).with(imap_pathname).and_return(true)
+
+        Imap::Backup::Serializer::Mbox.new('/base/path', 'my/folder')
+      end
+
+      it "fails if mbox exists and imap doesn't" do
+        File.stub(:exist?).with(imap_pathname).and_return(false)
+
+        expect {
+          Imap::Backup::Serializer::Mbox.new('/base/path', 'my/folder')
+        }.to raise_error(RuntimeError, '.imap file missing')
+      end
+
+      it "fails if imap exists and mbox doesn't" do
+        File.stub(:exist?).with(mbox_pathname).and_return(false)
+
+        expect {
+          Imap::Backup::Serializer::Mbox.new('/base/path', 'my/folder')
+        }.to raise_error(RuntimeError, '.mbox file missing')
+      end
+    end
+  end
+
+  context 'instance methods' do
+    before do
+      File.stub(:exist?).with(mbox_pathname).and_return(true)
+      File.stub(:exist?).with(imap_pathname).and_return(true)
+      CSV.stub(:foreach) do |&block|
+        block.call ['1']
+        block.call ['123']
+      end
+    end
+
+    subject { Imap::Backup::Serializer::Mbox.new('/base/path', 'my/folder') }
+
+    context '#uids' do
+      it 'returns the backed-up uids' do
+        File.should_receive(:exist?).with(mbox_pathname).and_return(true)
+        File.should_receive(:exist?).with(imap_pathname).and_return(true)
+
+        expect(subject.uids).to eq(['1', '123'])
+      end
+
+      it 'returns an empty Array if the mbox does not exist' do
+        File.stub(:exist?).with(mbox_pathname).and_return(false)
+        File.stub(:exist?).with(imap_pathname).and_return(false)
+        File.should_receive(:exist?).with(mbox_pathname).and_return(false)
+        File.should_receive(:exist?).with(imap_pathname).and_return(false)
+
+        expect(subject.uids).to eq([])
+      end
+    end
+
+    context '#save' do
+      let(:mbox_formatted_message) { 'message in mbox format' }
+      let(:message_uid) { '999' }
+      let(:message) { stub('Email::Mboxrd::Message', :to_s => mbox_formatted_message) }
+      let(:mbox_file) { stub('File - mbox', :close => nil) }
+      let(:imap_file) { stub('File - imap', :close => nil) }
+
+      before do
+        Email::Mboxrd::Message.stub(:new => message)
+        File.stub(:open).with(mbox_pathname, 'ab').and_return(mbox_file)
+        File.stub(:open).with(imap_pathname, 'ab').and_return(imap_file)
+        mbox_file.stub(:write).with(mbox_formatted_message)
+        imap_file.stub(:write).with(message_uid + "\n")
+      end
+
+      it 'saves the message to the mbox' do
+        mbox_file.should_receive(:write).with(mbox_formatted_message)
+
+        subject.save(message_uid, "The\nemail\n")
+      end
+
+      it 'saves the uid to the imap file' do
+        imap_file.should_receive(:write).with(message_uid + "\n")
+
+        subject.save(message_uid, "The\nemail\n")
+      end
+    end
+  end
+end
+