imap-backup 0.0.3 → 0.0.4

data/spec/unit/serializer/directory_spec.rb

@@ -7,6 +7,7 @@ describe Imap::Backup::Serializer::Directory do
   context '#initialize' do
 
     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)
 
@@ -24,6 +25,7 @@ describe Imap::Backup::Serializer::Directory 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
 
     subject { Imap::Backup::Serializer::Directory.new('/base/path', 'my_folder') }

data/spec/unit/utils_spec.rb

@@ -3,15 +3,17 @@ load File.expand_path( '../spec_helper.rb', File.dirname(__FILE__) )
 
 describe Imap::Backup::Utils do
 
-  include Imap::Backup::Utils
-
   context '#check_permissions' do
 
+    before :each do
+      File.stub!(:exist?).and_return(true)
+    end
+
     it 'should stat the file' do
       stat = stub('File::Stat', :mode => 0100)
       File.should_receive(:stat).with('foobar').and_return(stat)
 
-      check_permissions('foobar', 0345)
+      Imap::Backup::Utils.check_permissions('foobar', 0345)
     end
 
     it 'should succeed if file permissions are less than limit' do
@@ -19,7 +21,7 @@ describe Imap::Backup::Utils do
       File.stub!(:stat).and_return(stat)
 
       expect do
-        check_permissions('foobar', 0345)
+        Imap::Backup::Utils.check_permissions('foobar', 0345)
       end.to_not raise_error
     end
 
@@ -28,7 +30,7 @@ describe Imap::Backup::Utils do
       File.stub!(:stat).and_return(stat)
 
       expect do
-        check_permissions('foobar', 0345)
+        Imap::Backup::Utils.check_permissions('foobar', 0345)
       end.to_not raise_error
     end
 
@@ -37,7 +39,7 @@ describe Imap::Backup::Utils do
       File.stub!(:stat).and_return(stat)
 
       expect do
-        check_permissions('foobar', 0345)
+        Imap::Backup::Utils.check_permissions('foobar', 0345)
       end.to raise_error(RuntimeError, "Permissions on 'foobar' should be 0345, not 0777")
     end
 
@@ -48,7 +50,7 @@ describe Imap::Backup::Utils do
     it 'should do nothing if an empty path is supplied' do
       FileUtils.should_not_receive(:mkdir_p)
 
-      make_folder('aaa', '', 0222)
+      Imap::Backup::Utils.make_folder('aaa', '', 0222)
     end
 
     it 'should create the path' do
@@ -56,7 +58,7 @@ describe Imap::Backup::Utils do
 
       FileUtils.should_receive(:mkdir_p).with('/base/path/new/folder')
 
-      make_folder('/base/path', 'new/folder', 0222)
+      Imap::Backup::Utils.make_folder('/base/path', 'new/folder', 0222)
     end
 
     it 'should set permissions on the path' do
@@ -64,7 +66,7 @@ describe Imap::Backup::Utils do
 
       FileUtils.should_receive(:chmod).with(0222, '/base/path/new')
 
-      make_folder('/base/path', 'new/folder', 0222)
+      Imap::Backup::Utils.make_folder('/base/path', 'new/folder', 0222)
     end
 
   end

metadata

@@ -1,119 +1,120 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: imap-backup
-version: !ruby/object:Gem::Version 
-  hash: 25
+version: !ruby/object:Gem::Version
+  version: 0.0.4
   prerelease: 
-  segments: 
-  - 0
-  - 0
-  - 3
-  version: 0.0.3
 platform: ruby
-authors: 
+authors:
 - Joe Yates
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2012-06-12 00:00:00 +01:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id001
+date: 2012-06-20 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: rake
-  prerelease: false
-  type: :runtime
-- !ruby/object:Gem::Dependency 
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id002
-  name: json
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
   prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: highline
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-- !ruby/object:Gem::Dependency 
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id003
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
   name: pry
-  prerelease: false
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id004
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
   name: pry-doc
-  prerelease: false
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 2
-        - 3
-        - 0
-        version: 2.3.0
-  version_requirements: *id005
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
   name: rspec
-  prerelease: false
-  type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id006 !ruby/object:Gem::Requirement 
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id006
-  name: rcov
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 2.3.0
+  type: :development
   prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 2.3.0
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Backup GMail, or any other IMAP email service, to disk.
-email: 
+email:
 - joe.g.yates@gmail.com
-executables: 
+executables:
 - imap-backup
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
+files:
 - .gitignore
 - .travis.yml
 - Gemfile
@@ -125,59 +126,74 @@ files:
 - lib/imap/backup.rb
 - lib/imap/backup/account/connection.rb
 - lib/imap/backup/account/folder.rb
+- lib/imap/backup/configuration/account.rb
+- lib/imap/backup/configuration/asker.rb
+- lib/imap/backup/configuration/connection_tester.rb
+- lib/imap/backup/configuration/folder_chooser.rb
+- lib/imap/backup/configuration/list.rb
+- lib/imap/backup/configuration/setup.rb
+- lib/imap/backup/configuration/store.rb
 - lib/imap/backup/downloader.rb
 - lib/imap/backup/serializer/directory.rb
-- lib/imap/backup/settings.rb
 - lib/imap/backup/utils.rb
 - lib/imap/backup/version.rb
 - spec/gather_rspec_coverage.rb
 - spec/spec_helper.rb
 - spec/unit/account/connection_spec.rb
 - spec/unit/account/folder_spec.rb
+- spec/unit/configuration/account_spec.rb
+- spec/unit/configuration/asker_spec.rb
+- spec/unit/configuration/connection_tester_spec.rb
+- spec/unit/configuration/folder_chooser_spec.rb
+- spec/unit/configuration/list_spec.rb
+- spec/unit/configuration/setup_spec.rb
+- spec/unit/configuration/store_spec.rb
 - spec/unit/downloader_spec.rb
 - spec/unit/serializer/directory_spec.rb
-- spec/unit/settings_spec.rb
 - spec/unit/utils_spec.rb
-has_rdoc: true
 homepage: https://github.com/joeyates/imap-backup
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
       - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+      hash: 1194066517873683688
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
       - 0
-      version: "0"
+      hash: 1194066517873683688
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.6.2
+rubygems_version: 1.8.23
 signing_key: 
 specification_version: 3
 summary: Backup GMail (or other IMAP) accounts to disk
-test_files: 
+test_files:
 - spec/gather_rspec_coverage.rb
 - spec/spec_helper.rb
 - spec/unit/account/connection_spec.rb
 - spec/unit/account/folder_spec.rb
+- spec/unit/configuration/account_spec.rb
+- spec/unit/configuration/asker_spec.rb
+- spec/unit/configuration/connection_tester_spec.rb
+- spec/unit/configuration/folder_chooser_spec.rb
+- spec/unit/configuration/list_spec.rb
+- spec/unit/configuration/setup_spec.rb
+- spec/unit/configuration/store_spec.rb
 - spec/unit/downloader_spec.rb
 - spec/unit/serializer/directory_spec.rb
-- spec/unit/settings_spec.rb
 - spec/unit/utils_spec.rb
+has_rdoc: 

data/lib/imap/backup/settings.rb

@@ -1,35 +0,0 @@
-require 'json'
-
-module Imap
-  module Backup
-
-    class Settings
-
-      include Imap::Backup::Utils
-
-      attr_reader :accounts
-
-      def initialize(accounts = nil)
-        config_pathname = File.expand_path('~/.imap-backup/config.json')
-        raise "Configuration file '#{config_pathname}' not found" if ! File.exist?(config_pathname)
-        check_permissions(config_pathname, 0600)
-        @settings = JSON.parse(File.read(config_pathname), :symbolize_names => true)
-        if accounts.nil?
-          @accounts = @settings[:accounts]
-        else
-          @accounts = @settings[:accounts].select{ |account| accounts.include?(account[:username]) }
-        end
-      end
-
-      def each_connection
-        @accounts.each do |account|
-          connection = Imap::Backup::Account::Connection.new(account)
-          yield connection
-          connection.disconnect
-        end
-      end
-
-    end
-  end
-end
-

data/spec/unit/settings_spec.rb

@@ -1,109 +0,0 @@
-# encoding: utf-8
-load File.expand_path( '../spec_helper.rb', File.dirname(__FILE__) )
-
-describe Imap::Backup::Settings do
-
-  before :each do
-    @settings = {
-      :accounts => [
-        {
-          :username => 'a1@example.com'
-        },
-        {
-          :username => 'a2@example.com',
-        },
-      ]
-    }
-    File.stub!(:exist?).and_return(true)
-    stat = stub('File::Stat', :mode => 0600)
-    File.stub!(:stat).and_return(stat)
-    File.stub!(:read)
-    JSON.stub!(:parse).and_return(@settings)
-  end
-
-  context '#initialize' do
-
-    it 'should fail if the config file is missing' do
-      File.should_receive(:exist?).and_return(false)
-
-      expect do
-        Imap::Backup::Settings.new
-      end.to raise_error(RuntimeError, /not found/)
-    end
-
-    it 'should fail if the config file permissions are too lax' do
-      File.stub!(:exist?).and_return(true)
-
-      stat = stub('File::Stat', :mode => 0644)
-      File.should_receive(:stat).and_return(stat)
-
-      expect do
-        Imap::Backup::Settings.new
-      end.to raise_error(RuntimeError, /Permissions.*?should be 0600/)
-    end
-
-    it 'should load the config file' do
-      File.stub!(:exist?).and_return(true)
-
-      stat = stub('File::Stat', :mode => 0600)
-      File.stub!(:stat).and_return(stat)
-
-      configuration = 'JSON string'
-      File.should_receive(:read).with(%r{/.imap-backup/config.json}).and_return(configuration)
-      JSON.should_receive(:parse).with(configuration, :symbolize_names => true)
-
-      Imap::Backup::Settings.new
-    end
-
-    context 'with account parameter' do
-      it 'should only create requested accounts' do
-        settings = Imap::Backup::Settings.new(['a2@example.com'])
-
-        settings.accounts.should == @settings[:accounts][1..1]
-      end
-    end
-
-  end
-
-  context 'instance methods' do
-
-    before :each do
-      @connection = stub('Imap::Backup::Account::Connection', :disconnect => nil)
-    end
-
-    subject { Imap::Backup::Settings.new }
-
-    context '#each_connection' do
-
-      it 'should instantiate connections' do
-        Imap::Backup::Account::Connection.should_receive(:new).with(@settings[:accounts][0]).and_return(@connection)
-        Imap::Backup::Account::Connection.should_receive(:new).with(@settings[:accounts][1]).and_return(@connection)
-
-        subject.each_connection{}
-      end
-
-      it 'should call the block' do
-        Imap::Backup::Account::Connection.stub!(:new).and_return(@connection)
-        calls = 0
-
-        subject.each_connection do |a|
-          calls += 1
-          a.should == @connection
-        end
-        calls.should == 2
-      end
-
-      it 'should disconnect connections' do
-        Imap::Backup::Account::Connection.stub!(:new).and_return(@connection)
-
-        @connection.should_receive(:disconnect)
-
-        subject.each_connection {}
-      end
-
-    end
-
-  end
-
-end
-