spec/unit/configuration/setup_spec.rb in imap-backup-1.0.9 vs spec/unit/configuration/setup_spec.rb in imap-backup-1.0.10

@@ -5,41 +5,45 @@
   include HighLineTestHelpers
 
   context '#initialize' do
     context 'without a config file' do
       it 'works' do
-        Imap::Backup::Configuration::Setup.new
+        described_class.new
       end
     end
   end
 
   context '#run' do
+    let(:account1) { {:username => '[e-mail]'} }
+    let(:account) { double('Imap::Backup::Configuration::Account', :run => nil) }
+    let(:data) { {:accounts => [account1]} }
+    let(:store) do
+      double(
+        'Imap::Backup::Configuration::Store',
+        :data => data,
+        :path => '/base/path'
+      )
+    end
+
     before :each do
-      prepare_store
+      allow(Imap::Backup::Configuration::Store).to receive(:new).and_return(store)
       @input, @output = prepare_highline
-      subject.stub(:system => nil)
+      allow(@input).to receive(:eof?).and_return(false)
+      allow(@input).to receive(:gets).and_return("q\n")
+      allow(subject).to receive(:system)
     end
 
-    def prepare_store
-      @account1 = {:username => '[e-mail]'}
-      @data     = {:accounts => [@account1]}
-      @store    = stub('Imap::Backup::Configuration::Store', :data => @data, :path => '/base/path')
-      Imap::Backup::Configuration::Store.stub!(:new).and_return(@store)
-    end
+    subject { described_class.new }
 
-    subject { Imap::Backup::Configuration::Setup.new }
+    context 'main menu' do
+      before { subject.run }
 
-    it 'should present a main menu' do
-      @input.should_receive(:eof?).and_return(false)
-      @input.should_receive(:gets).and_return("q\n")
-
-      subject.run
-
-      @output.string.should =~ /Choose an action:/
-      @output.string.should =~ /add account/
-      @output.string.should =~ /save and exit/
-      @output.string.should =~ /quit/
+      %w(add\ account save\ and\ exit quit).each do |choice|
+        it "includes #{choice}" do
+          expect(@output.string).to include(choice)
+        end
+      end
     end
 
     it 'clears the screen' do
       subject.should_receive(:system).with('clear')
 
@@ -50,60 +54,35 @@
       subject.run
 
       @output.string.should =~ /[e-mail]/
     end
 
-    it 'should edit accounts' do
-      state = :initial
-      @input.stub(:gets) do
-        case state
-        when :initial
-          state = :editing
-          "[e-mail]\n"
-        else
-          "q\n"
-        end
+    context 'adding accounts' do
+      let(:blank_account) do
+        {
+          :username => "[e-mail]",
+          :password => "",
+          :local_path => "/base/path/new_example.com",
+          :folders => []
+        }
       end
 
-      @account = stub('Imap::Backup::Configuration::Account')
-      allow(Imap::Backup::Configuration::Account).to receive(:new).with(@store, @account1, anything).and_return(@account)
-      @account.should_receive(:run).with()
+      before do
+        allow(@input).to receive(:gets).and_return("add\n", "q\n")
+        allow(Imap::Backup::Configuration::Asker).to receive(:email).with(no_args).and_return('[e-mail]')
+        allow(Imap::Backup::Configuration::Account).to receive(:new).with(store, blank_account, anything).and_return(account)
 
-      subject.run
-    end
-
-    it 'should add accounts' do
-      state = :initial
-      @input.stub(:gets) do
-        case state
-        when :initial
-          state = :editing
-          "add\n"
-        else
-          "q\n"
-        end
+        subject.run
       end
 
-      blank_account = {:username=>"[e-mail]", :password=>"", :local_path=>"/base/path/new_example.com", :folders=>[]}
-      Imap::Backup::Configuration::Asker.should_receive(:email).with().and_return('[e-mail]')
-      @account = stub('Imap::Backup::Configuration::Account')
-      allow(Imap::Backup::Configuration::Account).to receive(:new).with(@store, blank_account, anything).and_return(@account)
-      @account.should_receive(:run).once
-
-      subject.run
-
-      @data[:accounts].size.should == 2
-      @data[:accounts][1].should == {
-        :username   => "[e-mail]",
-        :password   => "",
-        :local_path => "/base/path/new_example.com",
-        :folders    => []
-      }
+      it 'adds account data' do
+        expect(data[:accounts][1]).to eq(blank_account)
+      end
     end
 
     it 'should save the configuration' do
       @input.should_receive(:gets).with().and_return("save\n")
-      @store.should_receive(:save).with()
+      store.should_receive(:save).with()
 
       subject.run
     end
 
     it 'should exit' do