flare-up 0.7 → 0.8

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YTIzMzhkOTUyZjYwNzUyMzYyYTUxYTEwZGJlZmY1ODdmNWJkYjUzMw==
+    NzFmZDVjOWIyZmE4MDcxODhhM2E1YjhmNmE3YmNiODA2NWZhMjhkMg==
   data.tar.gz: !binary |-
-    NDJiMGI4OWQ2ZjMyNWQ2MDQ4NmRjZWY4MjVkNTllY2I1NWJkYmVkOA==
+    Y2ZmYTExMTRhMWUxYTdlNDQ4NjA5ZmRmODJiMjQxMDdmNjQ2ZGQyNg==
 SHA512:
   metadata.gz: !binary |-
-    YTdiYjFkMmFmODI0MTNkYzgyZjAyY2UyNGNkMzM0ZDljMDgxNjNjMjJiNjQx
-    ZmM5ZTZiOGI0NDMyZjZhM2VmZGUwMGJkYTg5M2EwM2EzOTFiMTY5ZDBkYjU2
-    ZjAxODg1YTFiZTIxNDk4NWVjZmQ3YzJhZjAwNjk5YWM4ZjUxZmY=
+    MGY5YjZjNjI1ZWExZTRhODMyNWU2MzgzMGUyZTQ4NjJjNTk5MjYxNTliNzcw
+    N2M0ZDY0NWZlMmZjZWZiYzc2NTE3NjA3OTRjNTAxNDQ1OGYyMzc3MDFlOWMw
+    NzhlMjlmNDBkMjQ3ZGQ2ZjZhMGFhOTE1NmYyMDZjMjg0NDI5NjY=
   data.tar.gz: !binary |-
-    MTFkMTlkOTliMGM0YTMxZDExZjRhNmJlMTA5NGMwODFmM2I3MzM0ZTQ5YTYw
-    ZTc4N2VlYjQxZTRkZmEwMTdlYjdjM2E5MjZjNTA0YTczNzVkY2Y1MDVlNDEx
-    M2VhNDcyZjI1MzU4NmVlNTIzMjZlMTE0OWU0MzViMzdmMWQzNTI=
+    MDFjZTI1MDU0OGE2OTcyYmMwOGM1OWIyYWJiYWJmNjFmMGVlYWM4ZWYxMjM2
+    YzIxMmEzMmZmYWY0OTg5YTJlMDRjYjJiMWM3NDExMTUxMGM1OGMxNGE2ZmRi
+    ZDhkNmM1ZmJjOTYyYmIxZDE3NmNkM2ZiMDFmOWMzZDU5MjJkNWU=

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    flare-up (0.7)
+    flare-up (0.8)
       pg (~> 0.17)
       thor (~> 0.19)
 

data/lib/flare_up.rb

@@ -1,8 +1,10 @@
 require 'pg'
 require 'thor'
+require 'time'
 
 require 'flare_up/version'
 
+require 'flare_up/option_store'
 require 'flare_up/env_wrap'
 require 'flare_up/emitter'
 

data/lib/flare_up/boot.rb

@@ -3,13 +3,13 @@ module FlareUp
   class Boot
 
     # TODO: This control flow is untested and too procedural
-    def self.boot(options)
-      conn = create_connection(options)
-      copy = create_copy_command(options)
+    def self.boot
+      conn = create_connection
+      copy = create_copy_command
 
       begin
         trap('SIGINT') do
-          Emitter.warn('CTRL-C received; cancelling COPY command...')
+          Emitter.warn("\nCTRL-C received; cancelling COPY command...")
           error_message = conn.cancel_current_command
           if error_message
             Emitter.error("Error cancelling COPY: #{error_message}")
@@ -31,28 +31,30 @@ module FlareUp
 
     end
 
-    def self.create_connection(options)
+    def self.create_connection
       FlareUp::Connection.new(
-        options[:redshift_endpoint],
-        options[:database],
-        options[:redshift_username],
-        options[:redshift_password]
+        OptionStore.get(:redshift_endpoint),
+        OptionStore.get(:database),
+        OptionStore.get(:redshift_username),
+        OptionStore.get(:redshift_password)
       )
     end
+    private_class_method :create_connection
 
-    def self.create_copy_command(options)
+    def self.create_copy_command
       copy = FlareUp::CopyCommand.new(
-        options[:table],
-        options[:data_source],
-        options[:aws_access_key],
-        options[:aws_secret_key]
+        OptionStore.get(:table),
+        OptionStore.get(:data_source),
+        OptionStore.get(:aws_access_key),
+        OptionStore.get(:aws_secret_key)
       )
-      copy.columns = options[:column_list] if options[:column_list]
-      copy.options = options[:copy_options] if options[:copy_options]
+      copy.columns = OptionStore.get(:column_list) if OptionStore.get(:column_list)
+      copy.options = OptionStore.get(:copy_options) if OptionStore.get(:copy_options)
       copy
     end
+    private_class_method :create_copy_command
 
-    # TODO: How can we test this?
+    # TODO: Backfill tests
     def self.handle_load_errors(stl_load_errors)
       return if stl_load_errors.empty?
       Emitter.error("There was an error processing the COPY command.  Displaying the last (#{stl_load_errors.length}) errors.")
@@ -61,6 +63,7 @@ module FlareUp
       end
       CLI.bailout(1)
     end
+    private_class_method :handle_load_errors
 
   end
 

data/lib/flare_up/cli.rb

@@ -37,8 +37,9 @@ https://github.com/sharethrough/flare-up/blob/v#{FlareUp::VERSION}/README.md
         CLI.bailout(1)
       end
 
-      Emitter.store_options(boot_options)
-      Boot.boot(boot_options)
+      OptionStore.store_options(boot_options)
+
+      Boot.boot
     end
 
     def self.env_validator(options, option_name, env_variable_name)
@@ -47,9 +48,8 @@ https://github.com/sharethrough/flare-up/blob/v#{FlareUp::VERSION}/README.md
       raise ArgumentError, "One of either the --#{option_name} option or the ENV['#{env_variable_name}'] must be set"
     end
 
-    # TODO: Extract
     def self.bailout(exit_code)
-      exit(1)
+      exit(exit_code)
     end
 
   end

data/lib/flare_up/emitter.rb

@@ -9,39 +9,40 @@ module FlareUp
       :redshift_password
     ]
 
-    # TODO: How do we test this?
     def self.error(message)
-      $stderr.puts sanitize("\x1b[31m#{message}") unless ENV['TESTING']
+      err("\x1b[31m#{message}")
     end
 
-    # TODO: How do we test this?
     def self.success(message)
-      $stdout.puts sanitize("\x1b[32m#{message}") unless ENV['TESTING']
+      out("\x1b[32m#{message}")
     end
 
-    # TODO: How do we test this?
     def self.warn(message)
-      $stdout.puts sanitize("\x1b[33m#{message}") unless ENV['TESTING']
+      err("\x1b[33m#{message}")
     end
 
-    # TODO: How do we test this?
     def self.info(message)
-      $stdout.puts sanitize(message) unless ENV['TESTING']
+      out(message)
     end
 
-    # TODO: How do we test this?
+    def self.out(message)
+      $stderr.puts(sanitize(message)) unless ENV['TESTING']
+    end
+    private_class_method :out
+
+    def self.err(message)
+      $stdout.puts(sanitize(message)) unless ENV['TESTING']
+    end
+    private_class_method :err
+
     def self.sanitize(message)
       RISKY_OPTIONS.each do |risky_option|
-        message.gsub!(@BOOT_OPTIONS[risky_option], 'REDACTED') if @BOOT_OPTIONS[risky_option]
+        message.gsub!(OptionStore.get(risky_option), 'REDACTED') if OptionStore.get(risky_option)
       end
-      message.gsub!(/\e\[(\d+)(;\d+)*m/, '') unless @BOOT_OPTIONS[:colorize_output]
+      message.gsub!(/\e\[(\d+)(;\d+)*m/, '') unless OptionStore.get(:colorize_output)
       message
     end
-
-    # TODO: How do we test this?
-    def self.store_options(options)
-      @BOOT_OPTIONS = options
-    end
+    private_class_method :sanitize
 
   end
 

data/lib/flare_up/option_store.rb

@@ -0,0 +1,32 @@
+module FlareUp
+
+  class OptionStore
+
+    def self.store_option(name, value)
+      storage[name] = value
+    end
+
+    def self.store_options(options)
+      storage.merge!(options)
+    end
+
+    def self.get(name)
+      storage[name]
+    end
+
+    def self.get_options
+      storage
+    end
+
+    def self.clear
+      storage.clear
+    end
+
+    def self.storage
+      @storage ||= {}
+    end
+    private_class_method :storage
+
+  end
+
+end

data/lib/flare_up/version.rb

@@ -1,3 +1,3 @@
 module FlareUp
-  VERSION = '0.7'
+  VERSION = '0.8'
 end

data/spec/lib/flare_up/boot_spec.rb

@@ -19,7 +19,7 @@ describe FlareUp::Boot 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
 
@@ -27,7 +27,7 @@ describe FlareUp::Boot 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
 
@@ -35,7 +35,7 @@ describe FlareUp::Boot 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
 
@@ -52,7 +52,7 @@ describe FlareUp::Boot 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
 
@@ -60,7 +60,7 @@ describe FlareUp::Boot 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
 
@@ -68,7 +68,7 @@ describe FlareUp::Boot 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
 
@@ -76,7 +76,7 @@ describe FlareUp::Boot 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
 
@@ -84,7 +84,7 @@ describe FlareUp::Boot 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
 
@@ -93,17 +93,19 @@ describe FlareUp::Boot do
   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')
@@ -113,17 +115,19 @@ describe FlareUp::Boot do
 
   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')
@@ -132,20 +136,20 @@ describe FlareUp::Boot do
 
     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

data/spec/lib/flare_up/cli_spec.rb

@@ -25,29 +25,29 @@ describe FlareUp::CLI do
 
   describe '#copy' do
 
-    it 'should pass the options on the emitter' do
-      expect(FlareUp::Emitter).to receive(:store_options).with(required_options)
+    it 'should boot' do
+      expect(FlareUp::Boot).to receive(:boot)
       FlareUp::CLI.start(required_arguments)
     end
 
     context 'when no options are specified' do
       it 'should boot with the proper options' do
-        expect(FlareUp::Boot).to receive(:boot).with(required_options)
         FlareUp::CLI.start(required_arguments)
+        expect(FlareUp::OptionStore.get_options).to eq(required_options)
       end
     end
 
     context 'when column ordering is specified' do
       it 'should boot with the proper options' do
-        expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:column_list => %w(c1 c2 c3)))
         FlareUp::CLI.start(required_arguments + %w(--column_list c1 c2 c3))
+        expect(FlareUp::OptionStore.get(:column_list)).to eq(%w(c1 c2 c3))
       end
     end
 
     context 'when COPY options are specified' do
       it 'should boot with the proper options' do
-        expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:copy_options => 'TEST_COPY_OPTIONS WITH A SPACE'))
         FlareUp::CLI.start(required_arguments + ['--copy_options', 'TEST_COPY_OPTIONS WITH A SPACE'])
+        expect(FlareUp::OptionStore.get(:copy_options)).to eq('TEST_COPY_OPTIONS WITH A SPACE')
       end
     end
 
@@ -55,15 +55,15 @@ describe FlareUp::CLI do
 
       context 'when it is not specified' do
         it 'should boot with the proper options' do
-          expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:colorize_output => true))
           FlareUp::CLI.start(required_arguments)
+          expect(FlareUp::OptionStore.get(:colorize_output)).to eq(true)
         end
       end
 
       context 'when it is specified' do
         it 'should boot with the proper options' do
-          expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:colorize_output => false))
           FlareUp::CLI.start(required_arguments + ['--colorize_output=false'])
+          expect(FlareUp::OptionStore.get(:colorize_output)).to eq(false)
         end
       end
 
@@ -74,23 +74,21 @@ describe FlareUp::CLI do
       describe 'username' do
         context 'when it is specified on the CLI' do
           it 'should boot with the proper options' do
-            expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:redshift_username => 'CLI_VALUE'))
             FlareUp::CLI.start(required_arguments + %w(--redshift_username CLI_VALUE))
+            expect(FlareUp::OptionStore.get(:redshift_username)).to eq('CLI_VALUE')
           end
         end
         context 'when it is not specified on the CLI' do
           context 'when it is available via ENV' do
             it 'should boot with the key from the environment' do
-              expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:redshift_username => 'TEST_REDSHIFT_USERNAME'))
               FlareUp::CLI.start(required_arguments)
+              expect(FlareUp::OptionStore.get(:redshift_username)).to eq('TEST_REDSHIFT_USERNAME')
             end
           end
           context 'when it is not available via ENV' do
-            before do
+            it 'should be an error' do
               allow(FlareUp::ENVWrap).to receive(:get).with('REDSHIFT_USERNAME').and_return(nil)
               expect(FlareUp::CLI).to receive(:bailout).with(1)
-            end
-            it 'should be an error' do
               FlareUp::CLI.start(required_arguments)
             end
           end
@@ -100,23 +98,23 @@ describe FlareUp::CLI do
       describe 'password' do
         context 'when it is specified on the CLI' do
           it 'should boot with the proper options' do
-            expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:redshift_password => 'CLI_VALUE'))
             FlareUp::CLI.start(required_arguments + %w(--redshift_password CLI_VALUE))
+            expect(FlareUp::OptionStore.get(:redshift_password)).to eq('CLI_VALUE')
           end
         end
         context 'when it is not specified on the CLI' do
           context 'when it is available via ENV' do
             it 'should boot with the key from the environment' do
-              expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:redshift_password => 'TEST_REDSHIFT_PASSWORD'))
               FlareUp::CLI.start(required_arguments)
+              expect(FlareUp::OptionStore.get(:redshift_password)).to eq('TEST_REDSHIFT_PASSWORD')
             end
           end
           context 'when it is not available via ENV' do
             before do
-              allow(FlareUp::ENVWrap).to receive(:get).with('REDSHIFT_PASSWORD').and_return(nil)
-              expect(FlareUp::CLI).to receive(:bailout).with(1)
             end
             it 'should be an error' do
+              allow(FlareUp::ENVWrap).to receive(:get).with('REDSHIFT_PASSWORD').and_return(nil)
+              expect(FlareUp::CLI).to receive(:bailout).with(1)
               FlareUp::CLI.start(required_arguments)
             end
           end
@@ -130,15 +128,15 @@ describe FlareUp::CLI do
       describe 'access key' do
         context 'when an AWS access key is specified' do
           it 'should boot with the proper options' do
-            expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:aws_access_key => 'CLI_KEY'))
             FlareUp::CLI.start(required_arguments + %w(--aws_access_key CLI_KEY))
+            expect(FlareUp::OptionStore.get(:aws_access_key)).to eq('CLI_KEY')
           end
         end
         context 'when an AWS access key is not specified' do
           context 'when the key is available via ENV' do
             it 'should boot with the key from the environment' do
-              expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:aws_access_key => 'TEST_AWS_ACCESS_KEY'))
               FlareUp::CLI.start(required_arguments)
+              expect(FlareUp::OptionStore.get(:aws_access_key)).to eq('TEST_AWS_ACCESS_KEY')
             end
           end
           context 'when the key is not available via ENV' do
@@ -156,15 +154,15 @@ describe FlareUp::CLI do
       describe 'secret key' do
         context 'when an AWS secret key is specified' do
           it 'should boot with the proper options' do
-            expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:aws_secret_key => 'CLI_KEY'))
             FlareUp::CLI.start(required_arguments + %w(--aws_secret_key CLI_KEY))
+            expect(FlareUp::OptionStore.get(:aws_secret_key)).to eq('CLI_KEY')
           end
         end
         context 'when an AWS secret key is not specified' do
           context 'when the key is available via ENV' do
             it 'should boot with the key from the environment' do
-              expect(FlareUp::Boot).to receive(:boot).with(required_options.merge(:aws_secret_key => 'TEST_AWS_SECRET_KEY'))
               FlareUp::CLI.start(required_arguments)
+              expect(FlareUp::OptionStore.get(:aws_secret_key)).to eq('TEST_AWS_SECRET_KEY')
             end
           end
           context 'when the key is not available via ENV' do

data/spec/lib/flare_up/emitter_spec.rb

@@ -1,22 +1,55 @@
 describe FlareUp::Emitter do
 
+  describe '.error' do
+    it 'should emit' do
+      expect(FlareUp::Emitter).to receive(:err).with("\x1b[31mTEST_MESSAGE")
+      FlareUp::Emitter.error('TEST_MESSAGE')
+    end
+  end
+
+  describe '.success' do
+    it 'should emit' do
+      expect(FlareUp::Emitter).to receive(:out).with("\x1b[32mTEST_MESSAGE")
+      FlareUp::Emitter.success('TEST_MESSAGE')
+    end
+  end
+
+  describe '.warn' do
+    it 'should emit' do
+      expect(FlareUp::Emitter).to receive(:err).with("\x1b[33mTEST_MESSAGE")
+      FlareUp::Emitter.warn('TEST_MESSAGE')
+    end
+  end
+
+  describe '.info' do
+    it 'should emit' do
+      expect(FlareUp::Emitter).to receive(:out).with('TEST_MESSAGE')
+      FlareUp::Emitter.info('TEST_MESSAGE')
+    end
+  end
+
   describe '.sanitize' do
 
     context 'when colorize output is disabled' do
       before do
-        FlareUp::Emitter.store_options({:colorize_output => false})
+        FlareUp::OptionStore.store_option(:colorize_output, false)
       end
       it 'should remove color codes' do
-        expect(FlareUp::Emitter.sanitize("\x1b[31mHello, World")).to eq('Hello, World')
+        expect(FlareUp::Emitter.send(:sanitize, "\x1b[31mHello, World")).to eq('Hello, World')
       end
     end
 
     context 'when a risky option is being output' do
       before do
-        FlareUp::Emitter.store_options({:aws_access_key => 'foo'})
+        FlareUp::OptionStore.store_options({
+            :aws_access_key => 'access',
+            :aws_secret_key => 'secret',
+            :redshift_username => 'user',
+            :redshift_password => 'pass'
+          })
       end
       it 'should hide it' do
-        expect(FlareUp::Emitter.sanitize('Hellofoo')).to eq('HelloREDACTED')
+        expect(FlareUp::Emitter.send(:sanitize, 'accesssecretuserpass')).to eq('REDACTEDREDACTEDREDACTEDREDACTED')
       end
     end
 

data/spec/lib/flare_up/option_store_spec.rb

@@ -0,0 +1,34 @@
+describe FlareUp::OptionStore do
+
+  describe '.store_option' do
+    it 'should store the specified option' do
+      FlareUp::OptionStore.store_option('name', 'value')
+      expect(FlareUp::OptionStore.get('name')).to eq('value')
+    end
+  end
+
+  describe '.store_options' do
+    it 'should store all the options' do
+      FlareUp::OptionStore.store_options(:o1 => 'v1', :o2 => 'v2')
+      expect(FlareUp::OptionStore.get(:o1)).to eq('v1')
+      expect(FlareUp::OptionStore.get(:o2)).to eq('v2')
+    end
+  end
+
+  describe '.get_options' do
+    let(:options) {{:o1 => 'v1', :o2 => 'v2'}}
+    it 'should return all options' do
+      FlareUp::OptionStore.store_options(options)
+      expect(FlareUp::OptionStore.get_options).to eq(options)
+    end
+  end
+
+  describe '.clear' do
+    it 'should remove all options' do
+      FlareUp::OptionStore.store_option('name', 'value')
+      FlareUp::OptionStore.clear
+      expect(FlareUp::OptionStore.get('name')).to eq(nil)
+    end
+  end
+
+end

data/spec/spec_helper.rb

@@ -1,5 +1,12 @@
 require 'flare_up'
-
 require 'rspec/its'
 
-ENV['TESTING'] = 'TESTING'
+ENV['TESTING'] = 'TESTING'
+
+RSpec.configure do |config|
+
+  config.before(:each) do
+    FlareUp::OptionStore.clear
+  end
+
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flare-up
 version: !ruby/object:Gem::Version
-  version: '0.7'
+  version: '0.8'
 platform: ruby
 authors:
 - Robert Slifka
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-25 00:00:00.000000000 Z
+date: 2014-11-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pg
@@ -106,6 +106,7 @@ files:
 - lib/flare_up/copy_command.rb
 - lib/flare_up/emitter.rb
 - lib/flare_up/env_wrap.rb
+- lib/flare_up/option_store.rb
 - lib/flare_up/stl_load_error.rb
 - lib/flare_up/stl_load_error_fetcher.rb
 - lib/flare_up/version.rb
@@ -119,6 +120,7 @@ files:
 - spec/lib/flare_up/connection_spec.rb
 - spec/lib/flare_up/copy_command_spec.rb
 - spec/lib/flare_up/emitter_spec.rb
+- spec/lib/flare_up/option_store_spec.rb
 - spec/lib/flare_up/stl_load_error_fetcher_spec.rb
 - spec/lib/flare_up/stl_load_error_spec.rb
 - spec/spec_helper.rb
@@ -151,6 +153,7 @@ test_files:
 - spec/lib/flare_up/connection_spec.rb
 - spec/lib/flare_up/copy_command_spec.rb
 - spec/lib/flare_up/emitter_spec.rb
+- spec/lib/flare_up/option_store_spec.rb
 - spec/lib/flare_up/stl_load_error_fetcher_spec.rb
 - spec/lib/flare_up/stl_load_error_spec.rb
 - spec/spec_helper.rb