imap-backup 6.0.0.rc2 → 6.1.0

data/spec/unit/imap/backup/setup/account_spec.rb

@@ -1,461 +0,0 @@
-describe Imap::Backup::Setup::Account do
-  ACCOUNT = "account".freeze
-  GMAIL_IMAP_SERVER = "imap.gmail.com".freeze
-  HIGHLINE = "highline".freeze
-  CONFIG = "config".freeze
-
-  subject { described_class.new(config, account, highline) }
-
-  let(:account) do
-    instance_double(
-      Imap::Backup::Account,
-      username: existing_email,
-      password: existing_password,
-      local_path: "/backup/path",
-      folders: [{name: "my_folder"}],
-      multi_fetch_size: multi_fetch_size,
-      server: current_server,
-      connection_options: connection_options,
-      modified?: false
-    )
-  end
-  let(:account1) do
-    instance_double(
-      Imap::Backup::Account,
-      username: other_email,
-      local_path: other_existing_path
-    )
-  end
-  let(:accounts) { [account, account1] }
-  let(:existing_email) { "user@example.com" }
-  let(:new_email) { "foo@example.com" }
-  let(:existing_password) { "password" }
-  let(:other_email) { "other@example.com" }
-  let(:other_existing_path) { "/other/existing/path" }
-  let(:multi_fetch_size) { 1 }
-  let(:current_server) { "imap.example.com" }
-  let(:connection_options) { nil }
-
-  let(:highline) { HIGHLINE }
-  let(:config) { CONFIG }
-
-  describe "#initialize" do
-    [:config, :account, :highline].each do |param|
-      it "expects #{param}" do
-        expect(subject.send(param)).to eq(send(param))
-      end
-    end
-  end
-
-  describe "#run" do
-    let(:highline_menu_class) do
-      Class.new do
-        attr_reader :choices
-        attr_accessor :header
-
-        def initialize
-          @choices = {}
-        end
-
-        def choice(name, &block)
-          choices[name] = block
-        end
-
-        def hidden(name, &block)
-          choices[name] = block
-        end
-      end
-    end
-
-    let(:highline) { instance_double(HighLine) }
-    let(:menu) { highline_menu_class.new }
-    let(:config) do
-      instance_double(Imap::Backup::Configuration, accounts: accounts)
-    end
-
-    before do
-      allow(Kernel).to receive(:system)
-      allow(Kernel).to receive(:puts)
-      allow(highline).to receive(:choose) do |&block|
-        block.call(menu)
-        throw :done
-      end
-    end
-
-    describe "preparation" do
-      it "clears the screen" do
-        expect(Kernel).to receive(:system).with("clear")
-
-        subject.run
-      end
-
-      describe "menu" do
-        it "shows the menu" do
-          expect(highline).to receive(:choose)
-
-          subject.run
-        end
-      end
-    end
-
-    describe "menu" do
-      [
-        "modify email",
-        "modify password",
-        "modify backup path",
-        "choose backup folders",
-        "modify multi-fetch size (number of emails to fetch at a time)",
-        "modify server",
-        "modify connection options",
-        "test connection",
-        "delete",
-        "(q) return to main menu",
-        "quit" # TODO: quit is hidden
-      ].each do |item|
-        before { subject.run }
-
-        it "has a '#{item}' item" do
-          expect(menu.choices).to include(item)
-        end
-      end
-    end
-
-    describe "account details" do
-      [
-        ["email", /email\s+user@example.com/],
-        ["password", /password\s+x+/],
-        ["path", %r(path\s+/backup/path)],
-        ["folders", /folders\s+my_folder/],
-        ["server", /server\s+imap.example.com/]
-      ].each do |attribute, value|
-        before { subject.run }
-
-        it "shows the #{attribute}" do
-          expect(menu.header).to match(value)
-        end
-      end
-
-      context "with no password" do
-        let(:existing_password) { "" }
-
-        before { subject.run }
-
-        it "indicates that a password is not set" do
-          expect(menu.header).to match(/^password\s+\(unset\)/)
-        end
-      end
-
-      context "with multi_fetch_size" do
-        let(:multi_fetch_size) { 4 }
-
-        it "shows the size" do
-          expect(menu.header).to match(/^multi-fetch\s+4/)
-        end
-      end
-
-      context "with connection_options" do
-        let(:connection_options) { {some: "option"} }
-
-        it "shows the options" do
-          expect(menu.header).to match(/^connection options\s+'{"some":"option"}'/)
-        end
-      end
-    end
-
-    describe "choosing 'modify email'" do
-      before do
-        allow(account).to receive(:"username=")
-        allow(account).to receive(:"server=")
-        allow(Imap::Backup::Setup::Asker).
-          to receive(:email) { new_email }
-        subject.run
-        menu.choices["modify email"].call
-      end
-
-      context "when the server is blank" do
-        [
-          ["GMail", "foo@gmail.com", GMAIL_IMAP_SERVER],
-          ["Fastmail", "bar@fastmail.fm", "imap.fastmail.com"],
-          ["Fastmail", "bar@fastmail.com", "imap.fastmail.com"]
-        ].each do |service, email, expected|
-          context service do
-            let(:new_email) { email }
-
-            context "with nil" do
-              let(:current_server) { nil }
-
-              it "sets a default server" do
-                expect(account).to have_received(:"server=").with(expected)
-              end
-            end
-
-            context "with an empty string" do
-              let(:current_server) { "" }
-
-              it "sets a default server" do
-                expect(account).to have_received(:"server=").with(expected)
-              end
-            end
-          end
-        end
-
-        context "when the domain is unrecognized" do
-          let(:current_server) { nil }
-          let(:provider) do
-            instance_double(Email::Provider, provider: :default)
-          end
-
-          before do
-            allow(Email::Provider).to receive(:for_address) { provider }
-          end
-
-          it "does not set a default server" do
-            expect(account).to_not have_received(:"server=")
-          end
-        end
-      end
-
-      context "when the email is new" do
-        it "modifies the email address" do
-          expect(account).to have_received(:"username=").with(new_email)
-        end
-      end
-
-      context "when the email already exists" do
-        let(:new_email) { other_email }
-
-        it "indicates the error" do
-          expect(Kernel).to have_received(:puts).
-            with("There is already an account set up with that email address")
-        end
-
-        it "doesn't set the email" do
-          expect(account.username).to eq(existing_email)
-        end
-      end
-    end
-
-    describe "choosing 'modify password'" do
-      let(:new_password) { "new_password" }
-
-      before do
-        allow(account).to receive(:"password=")
-        allow(Imap::Backup::Setup::Asker).
-          to receive(:password) { new_password }
-        subject.run
-        menu.choices["modify password"].call
-      end
-
-      context "when the user enters a password" do
-        it "updates the password" do
-          expect(account).to have_received(:"password=").with(new_password)
-        end
-      end
-
-      context "when the user cancels" do
-        let(:new_password) { nil }
-
-        it "does nothing" do
-          expect(account.password).to eq(existing_password)
-        end
-      end
-    end
-
-    describe "choosing 'modify backup path'" do
-      let(:new_backup_path) { "/new/path" }
-
-      before do
-        allow(account).to receive(:"local_path=")
-        @validator = nil
-        allow(
-          Imap::Backup::Setup::Asker
-        ).to receive(:backup_path) do |_path, validator|
-          @validator = validator
-          new_backup_path
-        end
-        subject.run
-        menu.choices["modify backup path"].call
-      end
-
-      it "updates the path" do
-        expect(account).to have_received(:"local_path=").with(new_backup_path)
-      end
-
-      context "when the path is not used by other backups" do
-        it "is accepts it" do
-          # rubocop:disable RSpec/InstanceVariable
-          expect(@validator.call("/unknown/path")).to be_truthy
-          # rubocop:enable RSpec/InstanceVariable
-        end
-      end
-
-      context "when the path is used by other backups" do
-        it "fails validation" do
-          # rubocop:disable RSpec/InstanceVariable
-          expect(@validator.call(other_existing_path)).to be_falsey
-          # rubocop:enable RSpec/InstanceVariable
-        end
-      end
-    end
-
-    describe "choosing 'choose backup folders'" do
-      let(:chooser) do
-        instance_double(Imap::Backup::Setup::FolderChooser, run: nil)
-      end
-
-      before do
-        allow(Imap::Backup::Setup::FolderChooser).
-          to receive(:new) { chooser }
-        subject.run
-        menu.choices["choose backup folders"].call
-      end
-
-      it "edits folders" do
-        expect(chooser).to have_received(:run)
-      end
-    end
-
-    describe "choosing 'modify multi-fetch size'" do
-      let(:supplied) { "10" }
-
-      before do
-        allow(account).to receive(:multi_fetch_size=)
-        allow(highline).to receive(:ask).with("size: ") { supplied }
-
-        subject.run
-        menu.choices[
-          "modify multi-fetch size (number of emails to fetch at a time)"
-        ].call
-      end
-
-      it "sets the multi-fetch size" do
-        expect(account).to have_received(:multi_fetch_size=).with(10)
-      end
-
-      context "when the supplied value is not a number" do
-        let(:supplied) { "wrong!" }
-
-        it "does nothing" do
-          expect(account).to_not have_received(:multi_fetch_size=)
-        end
-      end
-
-      context "when the supplied value is not a positive number" do
-        let(:supplied) { "0" }
-
-        it "does nothing" do
-          expect(account).to_not have_received(:multi_fetch_size=)
-        end
-      end
-    end
-
-    describe "choosing 'modify server'" do
-      let(:server) { "server" }
-
-      before do
-        allow(account).to receive(:"server=")
-        allow(highline).to receive(:ask).with("server: ") { server }
-
-        subject.run
-
-        menu.choices["modify server"].call
-      end
-
-      it "updates the server" do
-        expect(account).to have_received(:"server=").with(server)
-      end
-    end
-
-    describe "choosing 'modify connection options'" do
-      context "when the JSON is well formed" do
-        let(:json) { "{}" }
-
-        before do
-          allow(highline).to receive(:ask).with("connections options (as JSON): ") { json }
-          allow(account).to receive(:"connection_options=")
-
-          subject.run
-
-          menu.choices["modify connection options"].call
-        end
-
-        it "updates the connection options" do
-          expect(account).to have_received(:"connection_options=").with(json)
-        end
-      end
-
-      context "when the JSON is malformed" do
-        before do
-          allow(highline).to receive(:ask).with("connections options (as JSON): ") { "xx" }
-          allow(account).to receive(:"connection_options=").and_raise(JSON::ParserError)
-          allow(highline).to receive(:ask).with("Press a key ")
-
-          subject.run
-
-          menu.choices["modify connection options"].call
-        end
-
-        it "does not fail" do
-          subject.run
-        end
-
-        it "reports the problem" do
-          expect(Kernel).to have_received(:puts).
-            with(/Malformed/)
-        end
-      end
-    end
-
-    describe "choosing 'test connection'" do
-      let(:connection_tester) do
-        instance_double(
-          Imap::Backup::Setup::ConnectionTester,
-          test: "All fine"
-        )
-      end
-
-      before do
-        allow(Imap::Backup::Setup::ConnectionTester).
-          to receive(:new) { connection_tester }
-        allow(highline).to receive(:ask)
-        subject.run
-        menu.choices["test connection"].call
-      end
-
-      it "tests the connection" do
-        expect(connection_tester).to have_received(:test)
-      end
-    end
-
-    describe "choosing 'delete'" do
-      let(:confirmed) { true }
-
-      before do
-        allow(account).to receive(:mark_for_deletion)
-        allow(highline).to receive(:agree) { confirmed }
-        subject.run
-        catch :done do
-          menu.choices["delete"].call
-        end
-      end
-
-      it "asks for confirmation" do
-        expect(highline).to have_received(:agree)
-      end
-
-      context "when the user confirms deletion" do
-        it "flags the account to be deleted" do
-          expect(account).to have_received(:mark_for_deletion)
-        end
-      end
-
-      context "without confirmation" do
-        let(:confirmed) { false }
-
-        it "doesn't flag the account to be deleted" do
-          expect(account).to_not have_received(:mark_for_deletion)
-        end
-      end
-    end
-  end
-end

data/spec/unit/imap/backup/setup/asker_spec.rb

@@ -1,137 +0,0 @@
-module Imap::Backup
-  describe Setup::Asker do
-    subject { described_class.new(highline) }
-
-    let(:highline) { double }
-    let(:query) do
-      instance_double(
-        HighLine::Question,
-        "default=": nil,
-        "readline=": nil,
-        "validate=": nil,
-        "responses": {},
-        "echo=": nil
-      )
-    end
-    let(:answer) { "foo" }
-
-    before do
-      allow(Setup).to receive(:highline) { highline }
-      allow(highline).to receive(:ask) do |&b|
-        b.call query
-        answer
-      end
-    end
-
-    [
-      [:email, [], "email address"],
-      [:password, [], "password"],
-      [:backup_path, %w(x y), "backup directory"]
-    ].each do |method, params, prompt|
-      context ".#{method}" do
-        it "asks for input" do
-          expect(highline).to receive(:ask).with("#{prompt}: ")
-
-          described_class.send(method, *params)
-        end
-
-        it "returns the answer" do
-          expect(described_class.send(method, *params)).to eq(answer)
-        end
-      end
-    end
-
-    describe "#initialize" do
-      it "requires 1 parameter" do
-        expect do
-          described_class.new
-        end.to raise_error(ArgumentError, /wrong number/)
-      end
-
-      it "expects a higline" do
-        expect(subject.highline).to eq(highline)
-      end
-    end
-
-    describe "#email" do
-      let(:email) { "email@example.com" }
-      let(:answer) { email }
-
-      it "asks for an email" do
-        expect(highline).to receive(:ask).with(/email/)
-
-        subject.email
-      end
-
-      it "returns the address" do
-        expect(subject.email).to eq(email)
-      end
-    end
-
-    describe "#password" do
-      let(:password1) { "password" }
-      let(:password2) { "password" }
-      let(:answers) { [true, false] }
-
-      before do
-        i = 0
-        allow(highline).to receive(:ask).with("password: ") { password1 }
-        allow(highline).to receive(:ask).with("repeat password: ") { password2 }
-        allow(highline).to receive(:agree) do
-          answer = answers[i]
-          i += 1
-          answer
-        end
-      end
-
-      it "asks for a password" do
-        expect(highline).to receive(:ask).with("password: ")
-
-        subject.password
-      end
-
-      it "asks for confirmation" do
-        expect(highline).to receive(:ask).with("repeat password: ")
-
-        subject.password
-      end
-
-      it "returns the password" do
-        expect(subject.password).to eq(password1)
-      end
-
-      context "with different answers" do
-        let(:password2) { "secret" }
-
-        it "asks to continue" do
-          expect(highline).to receive(:agree).
-            at_least(:once).with(/Continue\?/)
-
-          subject.password
-        end
-      end
-    end
-
-    describe "#backup_path" do
-      let(:path) { "/path" }
-      let(:answer) { path }
-
-      before do
-        allow(highline).to receive(:ask) do |&b|
-          b.call query
-          path
-        end
-      end
-
-      it "asks for a directory" do
-        expect(highline).to receive(:ask).with(/directory/)
-
-        subject.backup_path("", //)
-      end
-
-      it "returns the path" do
-        expect(subject.backup_path("", //)).to eq(path)
-      end
-    end
-  end
-end

data/spec/unit/imap/backup/setup/connection_tester_spec.rb

@@ -1,51 +0,0 @@
-describe Imap::Backup::Setup::ConnectionTester do
-  describe "#test" do
-    subject { described_class.new("foo") }
-
-    let(:connection) do
-      instance_double(Imap::Backup::Account::Connection, client: nil)
-    end
-
-    before do
-      allow(Imap::Backup::Account::Connection).to receive(:new) { connection }
-    end
-
-    it "tries to connect" do
-      expect(connection).to receive(:client)
-
-      subject.test
-    end
-
-    describe "success" do
-      it "returns success" do
-        expect(subject.test).to match(/successful/)
-      end
-    end
-
-    describe "failure" do
-      before do
-        allow(connection).to receive(:client).and_raise(error)
-      end
-
-      context "with no connection" do
-        let(:error) do
-          data = OpenStruct.new(text: "bar")
-          response = OpenStruct.new(data: data)
-          Net::IMAP::NoResponseError.new(response)
-        end
-
-        it "returns error" do
-          expect(subject.test).to match(/no response/i)
-        end
-      end
-
-      context "when caused by other errors" do
-        let(:error) { "Error" }
-
-        it "returns error" do
-          expect(subject.test).to match(/unexpected error/i)
-        end
-      end
-    end
-  end
-end