spec/lib/flare_up/boot_spec.rb in flare-up-0.7 vs spec/lib/flare_up/boot_spec.rb in flare-up-0.8

@@ -17,27 +17,27 @@
 
       context 'when there is a DataSourceError' do
         let(:copy_command_error) { FlareUp::DataSourceError }
         it 'should handle the error' do
           expect(FlareUp::CLI).to receive(:bailout).with(1)
-          expect { FlareUp::Boot.boot({}) }.not_to raise_error
+          expect { FlareUp::Boot.boot }.not_to raise_error
         end
       end
 
       context 'when there is a OtherZoneBucketError' do
         let(:copy_command_error) { FlareUp::OtherZoneBucketError }
         it 'should handle the error' do
           expect(FlareUp::CLI).to receive(:bailout).with(1)
-          expect { FlareUp::Boot.boot({}) }.not_to raise_error
+          expect { FlareUp::Boot.boot }.not_to raise_error
         end
       end
 
       context 'when there is a OtherZoneBucketError' do
         let(:copy_command_error) { FlareUp::SyntaxError }
         it 'should handle the error' do
           expect(FlareUp::CLI).to receive(:bailout).with(1)
-          expect { FlareUp::Boot.boot({}) }.not_to raise_error
+          expect { FlareUp::Boot.boot }.not_to raise_error
         end
       end
 
     end
 
@@ -50,103 +50,107 @@
 
       context 'when there is a HostUnknownOrInaccessibleError' do
         let(:connection_error) { FlareUp::HostUnknownOrInaccessibleError }
         it 'should handle the error' do
           expect(FlareUp::CLI).to receive(:bailout).with(1)
-          expect { FlareUp::Boot.boot({}) }.not_to raise_error
+          expect { FlareUp::Boot.boot }.not_to raise_error
         end
       end
 
       context 'when there is a TimeoutError' do
         let(:connection_error) { FlareUp::TimeoutError }
         it 'should handle the error' do
           expect(FlareUp::CLI).to receive(:bailout).with(1)
-          expect { FlareUp::Boot.boot({}) }.not_to raise_error
+          expect { FlareUp::Boot.boot }.not_to raise_error
         end
       end
 
       context 'when there is a NoDatabaseError' do
         let(:connection_error) { FlareUp::NoDatabaseError }
         it 'should handle the error' do
           expect(FlareUp::CLI).to receive(:bailout).with(1)
-          expect { FlareUp::Boot.boot({}) }.not_to raise_error
+          expect { FlareUp::Boot.boot }.not_to raise_error
         end
       end
 
       context 'when there is a AuthenticationError' do
         let(:connection_error) { FlareUp::AuthenticationError }
         it 'should handle the error' do
           expect(FlareUp::CLI).to receive(:bailout).with(1)
-          expect { FlareUp::Boot.boot({}) }.not_to raise_error
+          expect { FlareUp::Boot.boot }.not_to raise_error
         end
       end
 
       context 'when there is a UnknownError' do
         let(:connection_error) { FlareUp::UnknownError }
         it 'should handle the error' do
           expect(FlareUp::CLI).to receive(:bailout).with(1)
-          expect { FlareUp::Boot.boot({}) }.not_to raise_error
+          expect { FlareUp::Boot.boot }.not_to raise_error
         end
       end
 
     end
 
   end
 
   describe '.create_connection' do
-    let(:options) {
-      {
-        :redshift_endpoint => 'TEST_REDSHIFT_ENDPOINT',
-        :database => 'TEST_DATABASE',
-        :redshift_username => 'TEST_REDSHIFT_USERNAME',
-        :redshift_password => 'TEST_REDSHIFT_PASSWORD'
-      }
-    }
+    before do
+      FlareUp::OptionStore.store_options(
+        {
+          :redshift_endpoint => 'TEST_REDSHIFT_ENDPOINT',
+          :database => 'TEST_DATABASE',
+          :redshift_username => 'TEST_REDSHIFT_USERNAME',
+          :redshift_password => 'TEST_REDSHIFT_PASSWORD'
+        }
+      )
+    end
 
     it 'should create a connection' do
-      connection = FlareUp::Boot.create_connection(options)
+      connection = FlareUp::Boot.send(:create_connection)
       expect(connection.host).to eq('TEST_REDSHIFT_ENDPOINT')
       expect(connection.dbname).to eq('TEST_DATABASE')
       expect(connection.user).to eq('TEST_REDSHIFT_USERNAME')
       expect(connection.password).to eq('TEST_REDSHIFT_PASSWORD')
     end
   end
 
   describe '.create_copy_command' do
 
-    let(:options) {
-      {
-        :table => 'TEST_TABLE',
-        :data_source => 'TEST_DATA_SOURCE',
-        :aws_access_key => 'TEST_ACCESS_KEY',
-        :aws_secret_key => 'TEST_SECRET_KEY'
-      }
-    }
+    before do
+      FlareUp::OptionStore.store_options(
+        {
+          :table => 'TEST_TABLE',
+          :data_source => 'TEST_DATA_SOURCE',
+          :aws_access_key => 'TEST_ACCESS_KEY',
+          :aws_secret_key => 'TEST_SECRET_KEY'
+        }
+      )
+    end
 
     it 'should create a proper copy command' do
-      command = FlareUp::Boot.create_copy_command(options)
+      command = FlareUp::Boot.send(:create_copy_command)
       expect(command.table_name).to eq('TEST_TABLE')
       expect(command.data_source).to eq('TEST_DATA_SOURCE')
       expect(command.aws_access_key_id).to eq('TEST_ACCESS_KEY')
       expect(command.aws_secret_access_key).to eq('TEST_SECRET_KEY')
     end
 
     context 'when columns are specified' do
       before do
-        options.merge!(:column_list => ['c1'])
+        FlareUp::OptionStore.store_option(:column_list, ['c1'])
       end
       it 'should create a proper copy command' do
-        command = FlareUp::Boot.create_copy_command(options)
+        command = FlareUp::Boot.send(:create_copy_command)
         expect(command.columns).to eq(['c1'])
       end
     end
 
     context 'when options are specified' do
       before do
-        options.merge!(:copy_options => '_')
+        FlareUp::OptionStore.store_option(:copy_options, '_')
       end
       it 'should create a proper copy command' do
-        command = FlareUp::Boot.create_copy_command(options)
+        command = FlareUp::Boot.send(:create_copy_command)
         expect(command.options).to eq('_')
       end
     end
 
   end
\ No newline at end of file