imap-backup 4.0.5 → 4.1.2

data/spec/unit/imap/backup/{configuration/setup_spec.rb → setup_spec.rb}

@@ -1,23 +1,46 @@
-describe Imap::Backup::Configuration::Setup do
+describe Imap::Backup::Setup do
   include HighLineTestHelpers
 
   subject { described_class.new }
 
-  let(:normal) { {username: "account@example.com"} }
-  let(:accounts) { [normal] }
+  let(:normal_account) do
+    instance_double(
+      Imap::Backup::Account,
+      username: "account@example.com",
+      marked_for_deletion?: false,
+      modified?: false
+    )
+  end
+  let(:modified_account) do
+    instance_double(
+      Imap::Backup::Account,
+      username: "modified@example.com",
+      marked_for_deletion?: false,
+      modified?: true
+    )
+  end
+  let(:deleted_account) do
+    instance_double(
+      Imap::Backup::Account,
+      username: "delete@example.com",
+      marked_for_deletion?: true,
+      modified?: false
+    )
+  end
+  let(:accounts) { [normal_account] }
   let(:store) do
     instance_double(
-      Imap::Backup::Configuration::Store,
+      Imap::Backup::Configuration,
       "accounts": accounts,
       "path": "/base/path",
       "save": nil,
       "debug?": debug,
       "debug=": nil,
-      "modified?": modified
+      "modified?": config_modified
     )
   end
   let(:debug) { false }
-  let(:modified) { false }
+  let(:config_modified) { false }
   let!(:highline_streams) { prepare_highline }
   let(:input) { highline_streams[0] }
   let(:output) { highline_streams[1] }
@@ -33,39 +56,48 @@ describe Imap::Backup::Configuration::Setup do
 
   describe "#run" do
     before do
-      allow(Imap::Backup::Configuration::Store).to receive(:new) { store }
-      allow(Imap::Backup).to receive(:setup_logging)
+      allow(Imap::Backup::Configuration).to receive(:new) { store }
+      allow(Imap::Backup::Logger).to receive(:setup_logging)
       allow(input).to receive(:eof?) { false }
-      allow(input).to receive(:gets) { "exit\n" }
+      allow(input).to receive(:gets) { "q\n" }
       allow(Kernel).to receive(:system)
     end
 
     describe "main menu" do
-      before { subject.run }
+      context "when changes have not been made" do
+        before { subject.run }
 
-      ["add account", "save and exit", "exit without saving"].each do |choice|
-        it "includes #{choice}" do
-          expect(output.string).to include(choice)
+        ["add account", "quit"].each do |choice|
+          it "includes #{choice}" do
+            expect(output.string).to include(choice)
+          end
         end
       end
-    end
 
-    it "clears the screen" do
-      expect(Kernel).to receive(:system).with("clear")
+      context "when changes have been made" do
+        let(:config_modified) { true }
 
-      subject.run
+        before do
+          allow(input).to receive(:gets) { "exit\n" }
+          subject.run
+        end
+
+        ["save and exit", "exit without saving"].each do |choice|
+          it "includes '#{choice}'" do
+            expect(output.string).to include(choice)
+          end
+        end
+      end
     end
 
-    it "updates logging status" do
-      expect(Imap::Backup).to receive(:setup_logging)
+    it "clears the screen" do
+      expect(Kernel).to receive(:system).with("clear")
 
       subject.run
     end
 
     describe "listing" do
-      let(:accounts) { [normal, modified, deleted] }
-      let(:modified) { {username: "modified@example.com", modified: true} }
-      let(:deleted) { {username: "deleted@example.com", delete: true} }
+      let(:accounts) { [normal_account, modified_account, deleted_account] }
 
       before { subject.run }
 
@@ -90,15 +122,16 @@ describe Imap::Backup::Configuration::Setup do
 
     context "when editing accounts" do
       let(:account) do
-        instance_double(Imap::Backup::Configuration::Account, run: nil)
+        instance_double(Imap::Backup::Setup::Account, run: nil)
       end
+      let(:config_modified) { true }
 
       before do
         allow(input).to receive(:gets).and_return("1\n", "exit\n")
-        allow(Imap::Backup::Configuration::Asker).to receive(:email).
+        allow(Imap::Backup::Setup::Asker).to receive(:email).
           with(no_args) { "new@example.com" }
-        allow(Imap::Backup::Configuration::Account).to receive(:new).
-          with(store, normal, anything) { account }
+        allow(Imap::Backup::Setup::Account).to receive(:new).
+          with(store, normal_account, anything) { account }
       end
 
       it "edits the account" do
@@ -118,35 +151,36 @@ describe Imap::Backup::Configuration::Setup do
         }
       end
       let(:account) do
-        instance_double(Imap::Backup::Configuration::Account, run: nil)
+        instance_double(Imap::Backup::Setup::Account, run: nil)
       end
+      let(:config_modified) { true }
       let(:added_email) { "new@example.com" }
       let(:local_path) { "/base/path/new_example.com" }
 
       before do
         allow(input).to receive(:gets).and_return("add\n", "exit\n")
-        allow(Imap::Backup::Configuration::Asker).to receive(:email).
+        allow(Imap::Backup::Setup::Asker).to receive(:email).
           with(no_args) { added_email }
-        allow(Imap::Backup::Configuration::Account).to receive(:new).
+        allow(Imap::Backup::Setup::Account).to receive(:new).
           with(store, anything, anything) { account }
 
         subject.run
       end
 
       it "sets username" do
-        expect(accounts[1][:username]).to eq(added_email)
+        expect(accounts[1].username).to eq(added_email)
       end
 
       it "sets blank password" do
-        expect(accounts[1][:password]).to eq("")
+        expect(accounts[1].password).to eq("")
       end
 
       it "sets local_path" do
-        expect(accounts[1][:local_path]).to eq(local_path)
+        expect(accounts[1].local_path).to eq(local_path)
       end
 
       it "sets folders" do
-        expect(accounts[1][:folders]).to eq([])
+        expect(accounts[1].folders).to eq([])
       end
 
       context "when the account is a GMail account" do
@@ -154,17 +188,19 @@ describe Imap::Backup::Configuration::Setup do
         let(:local_path) { "/base/path/new_gmail.com" }
 
         it "sets the server" do
-          expect(accounts[1][:server]).to eq(gmail_imap_server)
+          expect(accounts[1].server).to eq(gmail_imap_server)
         end
       end
 
       it "doesn't flag the unedited account as modified" do
-        expect(accounts[1][:modified]).to be_nil
+        expect(accounts[1].modified?).to be_falsey
       end
     end
 
     describe "logging" do
       context "when debug logging is disabled" do
+        let(:config_modified) { true }
+
         before do
           allow(input).to receive(:gets).and_return("start\n", "exit\n")
         end
@@ -183,7 +219,7 @@ describe Imap::Backup::Configuration::Setup do
           end
 
           it "updates logging status" do
-            expect(Imap::Backup).to receive(:setup_logging).twice
+            expect(Imap::Backup::Logger).to receive(:setup_logging)
 
             subject.run
           end
@@ -192,6 +228,7 @@ describe Imap::Backup::Configuration::Setup do
 
       context "when debug logging is enabled" do
         let(:debug) { true }
+        let(:config_modified) { true }
 
         before do
           allow(input).to receive(:gets).and_return("stop\n", "exit\n")
@@ -215,7 +252,7 @@ describe Imap::Backup::Configuration::Setup do
           end
 
           it "updates logging status" do
-            expect(Imap::Backup).to receive(:setup_logging).twice
+            expect(Imap::Backup::Logger).to receive(:setup_logging)
 
             subject.run
           end
@@ -224,6 +261,8 @@ describe Imap::Backup::Configuration::Setup do
     end
 
     context "when 'save' is selected" do
+      let(:config_modified) { true }
+
       before do
         allow(input).to receive(:gets) { "save\n" }
       end
@@ -241,6 +280,8 @@ describe Imap::Backup::Configuration::Setup do
     end
 
     context "when 'exit without saving' is selected" do
+      let(:config_modified) { true }
+
       before do
         allow(input).to receive(:gets) { "exit\n" }
       end
@@ -250,22 +291,10 @@ describe Imap::Backup::Configuration::Setup do
         subject.run
       end
 
-      context "when the configuration is modified" do
-        let(:modified) { true }
+      it "doesn't save the configuration" do
+        expect(store).to_not receive(:save)
 
-        it "doesn't save the configuration" do
-          expect(store).to_not receive(:save)
-
-          subject.run
-        end
-      end
-
-      context "when the configuration isn't modified" do
-        it "doesn't save the configuration" do
-          expect(store).to_not receive(:save)
-
-          subject.run
-        end
+        subject.run
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: imap-backup
 version: !ruby/object:Gem::Version
-  version: 4.0.5
+  version: 4.1.2
 platform: ruby
 authors:
 - Joe Yates
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-12-22 00:00:00.000000000 Z
+date: 2022-01-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: highline
@@ -191,6 +191,7 @@ files:
 - lib/imap/backup/account/connection.rb
 - lib/imap/backup/account/folder.rb
 - lib/imap/backup/cli.rb
+- lib/imap/backup/cli/accounts.rb
 - lib/imap/backup/cli/backup.rb
 - lib/imap/backup/cli/folders.rb
 - lib/imap/backup/cli/helpers.rb
@@ -201,18 +202,20 @@ files:
 - lib/imap/backup/cli/utils.rb
 - lib/imap/backup/client/apple_mail.rb
 - lib/imap/backup/client/default.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/configuration.rb
 - lib/imap/backup/downloader.rb
+- lib/imap/backup/logger.rb
+- lib/imap/backup/sanitizer.rb
 - lib/imap/backup/serializer.rb
 - lib/imap/backup/serializer/mbox.rb
 - lib/imap/backup/serializer/mbox_enumerator.rb
 - lib/imap/backup/serializer/mbox_store.rb
+- lib/imap/backup/setup.rb
+- lib/imap/backup/setup/account.rb
+- lib/imap/backup/setup/asker.rb
+- lib/imap/backup/setup/connection_tester.rb
+- lib/imap/backup/setup/folder_chooser.rb
+- lib/imap/backup/setup/helpers.rb
 - lib/imap/backup/thunderbird/mailbox_exporter.rb
 - lib/imap/backup/uploader.rb
 - lib/imap/backup/utils.rb
@@ -228,6 +231,7 @@ files:
 - spec/features/backup_spec.rb
 - spec/features/helper.rb
 - spec/features/restore_spec.rb
+- spec/features/status_spec.rb
 - spec/features/support/backup_directory.rb
 - spec/features/support/email_server.rb
 - spec/features/support/shared/connection_context.rb
@@ -237,7 +241,6 @@ files:
 - spec/spec_helper.rb
 - spec/support/fixtures.rb
 - spec/support/higline_test_helpers.rb
-- spec/support/shared_examples/account_flagging.rb
 - spec/support/silence_logging.rb
 - spec/unit/email/mboxrd/message_spec.rb
 - spec/unit/email/provider/apple_mail_spec.rb
@@ -247,23 +250,23 @@ files:
 - spec/unit/email/provider_spec.rb
 - spec/unit/imap/backup/account/connection_spec.rb
 - spec/unit/imap/backup/account/folder_spec.rb
+- spec/unit/imap/backup/cli/accounts_spec.rb
 - spec/unit/imap/backup/cli/local_spec.rb
 - spec/unit/imap/backup/cli/utils_spec.rb
 - spec/unit/imap/backup/client/default_spec.rb
-- spec/unit/imap/backup/configuration/account_spec.rb
-- spec/unit/imap/backup/configuration/asker_spec.rb
-- spec/unit/imap/backup/configuration/connection_tester_spec.rb
-- spec/unit/imap/backup/configuration/folder_chooser_spec.rb
-- spec/unit/imap/backup/configuration/list_spec.rb
-- spec/unit/imap/backup/configuration/setup_spec.rb
-- spec/unit/imap/backup/configuration/store_spec.rb
+- spec/unit/imap/backup/configuration_spec.rb
 - spec/unit/imap/backup/downloader_spec.rb
+- spec/unit/imap/backup/logger_spec.rb
 - spec/unit/imap/backup/serializer/mbox_enumerator_spec.rb
 - spec/unit/imap/backup/serializer/mbox_spec.rb
 - spec/unit/imap/backup/serializer/mbox_store_spec.rb
+- spec/unit/imap/backup/setup/account_spec.rb
+- spec/unit/imap/backup/setup/asker_spec.rb
+- spec/unit/imap/backup/setup/connection_tester_spec.rb
+- spec/unit/imap/backup/setup/folder_chooser_spec.rb
+- spec/unit/imap/backup/setup_spec.rb
 - spec/unit/imap/backup/uploader_spec.rb
 - spec/unit/imap/backup/utils_spec.rb
-- spec/unit/imap/backup_spec.rb
 homepage: https://github.com/joeyates/imap-backup
 licenses:
 - MIT
@@ -283,47 +286,47 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Backup GMail (or other IMAP) accounts to disk
 test_files:
-- spec/spec_helper.rb
-- spec/fixtures/connection.yml
+- spec/features/helper.rb
+- spec/features/support/backup_directory.rb
+- spec/features/support/shared/message_fixtures.rb
+- spec/features/support/shared/connection_context.rb
+- spec/features/support/email_server.rb
+- spec/features/restore_spec.rb
+- spec/features/status_spec.rb
+- spec/features/backup_spec.rb
+- spec/support/higline_test_helpers.rb
+- spec/support/fixtures.rb
+- spec/support/silence_logging.rb
+- spec/gather_rspec_coverage.rb
 - spec/unit/email/provider/gmail_spec.rb
-- spec/unit/email/provider/apple_mail_spec.rb
 - spec/unit/email/provider/fastmail_spec.rb
 - spec/unit/email/provider/base_spec.rb
-- spec/unit/email/provider_spec.rb
+- spec/unit/email/provider/apple_mail_spec.rb
 - spec/unit/email/mboxrd/message_spec.rb
-- spec/unit/imap/backup_spec.rb
+- spec/unit/email/provider_spec.rb
 - spec/unit/imap/backup/cli/utils_spec.rb
+- spec/unit/imap/backup/cli/accounts_spec.rb
 - spec/unit/imap/backup/cli/local_spec.rb
+- spec/unit/imap/backup/setup/account_spec.rb
+- spec/unit/imap/backup/setup/connection_tester_spec.rb
+- spec/unit/imap/backup/setup/folder_chooser_spec.rb
+- spec/unit/imap/backup/setup/asker_spec.rb
+- spec/unit/imap/backup/uploader_spec.rb
 - spec/unit/imap/backup/utils_spec.rb
-- spec/unit/imap/backup/client/default_spec.rb
+- spec/unit/imap/backup/configuration_spec.rb
+- spec/unit/imap/backup/setup_spec.rb
+- spec/unit/imap/backup/logger_spec.rb
+- spec/unit/imap/backup/serializer/mbox_store_spec.rb
+- spec/unit/imap/backup/serializer/mbox_enumerator_spec.rb
+- spec/unit/imap/backup/serializer/mbox_spec.rb
 - spec/unit/imap/backup/downloader_spec.rb
-- spec/unit/imap/backup/configuration/store_spec.rb
-- spec/unit/imap/backup/configuration/connection_tester_spec.rb
-- spec/unit/imap/backup/configuration/asker_spec.rb
-- spec/unit/imap/backup/configuration/setup_spec.rb
-- spec/unit/imap/backup/configuration/folder_chooser_spec.rb
-- spec/unit/imap/backup/configuration/account_spec.rb
-- spec/unit/imap/backup/configuration/list_spec.rb
 - spec/unit/imap/backup/account/folder_spec.rb
 - spec/unit/imap/backup/account/connection_spec.rb
-- spec/unit/imap/backup/serializer/mbox_spec.rb
-- spec/unit/imap/backup/serializer/mbox_store_spec.rb
-- spec/unit/imap/backup/serializer/mbox_enumerator_spec.rb
-- spec/unit/imap/backup/uploader_spec.rb
-- spec/support/fixtures.rb
-- spec/support/shared_examples/account_flagging.rb
-- spec/support/higline_test_helpers.rb
-- spec/support/silence_logging.rb
-- spec/gather_rspec_coverage.rb
-- spec/features/backup_spec.rb
-- spec/features/helper.rb
-- spec/features/support/backup_directory.rb
-- spec/features/support/shared/message_fixtures.rb
-- spec/features/support/shared/connection_context.rb
-- spec/features/support/email_server.rb
-- spec/features/restore_spec.rb
+- spec/unit/imap/backup/client/default_spec.rb
+- spec/spec_helper.rb
+- spec/fixtures/connection.yml

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

@@ -1,14 +0,0 @@
-module Imap::Backup
-  module Configuration; end
-
-  module Configuration::ConnectionTester
-    def self.test(account)
-      Account::Connection.new(account).client
-      "Connection successful"
-    rescue Net::IMAP::NoResponseError
-      "No response"
-    rescue StandardError => e
-      "Unexpected error: #{e}"
-    end
-  end
-end

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

@@ -1,53 +0,0 @@
-module Imap::Backup
-  module Configuration; end
-
-  class Configuration::List
-    attr_reader :required_accounts
-
-    def initialize(required_accounts = [])
-      @required_accounts = required_accounts
-    end
-
-    def setup_logging
-      return if !config_exists?
-
-      Imap::Backup.setup_logging config
-      Net::IMAP.debug = config.debug?
-    end
-
-    def each_connection
-      accounts.each do |account|
-        connection = Account::Connection.new(account)
-        yield connection
-        connection.disconnect
-      end
-    end
-
-    def accounts
-      @accounts ||=
-        if required_accounts.empty?
-          config.accounts
-        else
-          config.accounts.select do |account|
-            required_accounts.include?(account[:username])
-          end
-        end
-    end
-
-    private
-
-    def config
-      return @config if @config
-
-      if !config_exists?
-        path = Configuration::Store.default_pathname
-        raise ConfigurationNotFound, "Configuration file '#{path}' not found"
-      end
-      @config = Configuration::Store.new
-    end
-
-    def config_exists?
-      Configuration::Store.exist?
-    end
-  end
-end

data/spec/support/shared_examples/account_flagging.rb

@@ -1,23 +0,0 @@
-shared_examples "it flags the account as modified" do
-  it "flags that the account has changed" do
-    expect(account[:modified]).to be_truthy
-  end
-end
-
-shared_examples "it doesn't flag the account as modified" do
-  it "does not flag that the account has changed" do
-    expect(account[:modified]).to be_falsey
-  end
-end
-
-shared_examples "it flags the account to be deleted" do
-  it "flags that the account is to be deleted" do
-    expect(account[:delete]).to be_truthy
-  end
-end
-
-shared_examples "it doesn't flag the account to be deleted" do
-  it "does not flags that the account is to be deleted" do
-    expect(account[:delete]).to be_falsey
-  end
-end

data/spec/unit/imap/backup/configuration/list_spec.rb

@@ -1,89 +0,0 @@
-describe Imap::Backup::Configuration::List do
-  subject { described_class.new }
-
-  let(:accounts) do
-    [
-      {username: "a1@example.com"},
-      {username: "a2@example.com"}
-    ]
-  end
-  let(:store) do
-    instance_double(Imap::Backup::Configuration::Store, accounts: accounts)
-  end
-  let(:exists) { true }
-  let(:connection1) do
-    instance_double(Imap::Backup::Account::Connection, disconnect: nil)
-  end
-  let(:connection2) do
-    instance_double(Imap::Backup::Account::Connection, disconnect: nil)
-  end
-
-  before do
-    allow(Imap::Backup::Configuration::Store).to receive(:new) { store }
-    allow(Imap::Backup::Configuration::Store).
-      to receive(:exist?) { exists }
-    allow(Imap::Backup::Account::Connection).
-      to receive(:new).with(accounts[0]) { connection1 }
-    allow(Imap::Backup::Account::Connection).
-      to receive(:new).with(accounts[1]) { connection2 }
-  end
-
-  describe "#setup_logging" do
-    let(:config_exists) { true }
-
-    before do
-      allow(Imap::Backup::Configuration::Store).
-        to receive(:exist?) { config_exists }
-      allow(Imap::Backup).to receive(:setup_logging)
-      allow(store).to receive(:debug?)
-    end
-
-    it "sets global logging level" do
-      expect(Imap::Backup).to receive(:setup_logging).with(store)
-
-      subject.setup_logging
-    end
-
-    context "without a config" do
-      let(:config_exists) { false }
-
-      it "does nothing" do
-        expect(Imap::Backup).to_not receive(:setup_logging).with(store)
-
-        subject.setup_logging
-      end
-    end
-  end
-
-  describe "#each_connection" do
-    specify "calls the block with each account's connection" do
-      connections = []
-
-      subject.each_connection { |a| connections << a }
-
-      expect(connections).to eq([connection1, connection2])
-    end
-
-    context "with account parameter" do
-      subject { described_class.new(["a2@example.com"]) }
-
-      it "only creates requested accounts" do
-        connections = []
-
-        subject.each_connection { |a| connections << a }
-
-        expect(connections).to eq([connection2])
-      end
-    end
-
-    context "when the configuration file is missing" do
-      let(:exists) { false }
-
-      it "fails" do
-        expect do
-          subject.each_connection {}
-        end.to raise_error(Imap::Backup::ConfigurationNotFound, /not found/)
-      end
-    end
-  end
-end

data/spec/unit/imap/backup_spec.rb

@@ -1,28 +0,0 @@
-require "ostruct"
-require "imap/backup"
-
-describe Imap::Backup do
-  describe ".setup_logging" do
-    let!(:previous) { described_class.logger.level }
-
-    before { described_class.setup_logging(config) }
-
-    after { described_class.logger.level = previous }
-
-    context "when config.debug?" do
-      let(:config) { OpenStruct.new(debug?: true) }
-
-      it "sets logger level to debug" do
-        expect(described_class.logger.level).to eq(::Logger::Severity::DEBUG)
-      end
-    end
-
-    context "when not config.debug?" do
-      let(:config) { OpenStruct.new(debug?: false) }
-
-      it "sets logger level to debug" do
-        expect(described_class.logger.level).to eq(::Logger::Severity::ERROR)
-      end
-    end
-  end
-end