simple_deploy 0.7.2 → 0.7.3

data/spec/cli/shared_spec.rb

@@ -1,6 +1,9 @@
 require 'spec_helper'
+require 'simple_deploy/cli'
 
 describe SimpleDeploy::CLI::Shared do
+  include_context 'double stubbed logger'
+
   before do
     @object = Object.new
     @object.extend SimpleDeploy::CLI::Shared
@@ -10,9 +13,7 @@ describe SimpleDeploy::CLI::Shared do
     logger_stub = stub 'logger stub', :info => true
     attributes  = [ 'test1=value1', 'test2=value2==' ]
 
-    @object.stub :logger => logger_stub
-
-    @object.parse_attributes(:logger     => logger_stub,
+    @object.parse_attributes(:logger     => @logger_stub,
                              :attributes => attributes).
            should == [ { "test1" => "value1" }, 
                        { "test2" => "value2==" } ]
@@ -20,11 +21,8 @@ describe SimpleDeploy::CLI::Shared do
 
   context "validating options " do
     it "should exit if provided options passed do not include all required" do
-      logger_stub = stub 'logger stub', :error => true
-
       provided = { :test1 => 'test1', :test2 => 'test2' }
       required = [:test1, :test2, :test3]
-      @object.stub :logger => logger_stub
 
       lambda { 
         @object.valid_options? :provided => provided,
@@ -34,13 +32,12 @@ describe SimpleDeploy::CLI::Shared do
 
     it "should exit if environment does not exist" do
       config_stub = stub 'config stub', :environments => { 'preprod' => 'data' }
-      logger_stub = stub 'logger stub', :error => true
 
       provided = { :environment => 'prod' }
       required = [:environment]
 
-      SimpleDeploy::Config.stub :new => config_stub
-      @object.stub :logger => logger_stub
+      SimpleDeploy.stub(:environments).and_return(config_stub)
+      config_stub.should_receive(:keys).and_return(['preprod'])
 
       lambda { 
         @object.valid_options? :provided => provided,
@@ -50,16 +47,16 @@ describe SimpleDeploy::CLI::Shared do
 
     it "should not exit if all options passed and environment exists" do
       config_stub = stub 'config stub', :environments => { 'prod' => 'data' }
-      logger_stub = stub 'logger stub', :error => true
 
       provided = { :environment => 'prod', :test1 => 'value1' }
       required = [:environment, :test1]
 
-      SimpleDeploy::Config.stub :new => config_stub
+      SimpleDeploy.stub(:environments).and_return(config_stub)
+      config_stub.should_receive(:keys).and_return(['prod'])
 
       @object.valid_options? :provided => provided,
                              :required => required,
-                             :logger   => logger_stub
+                             :logger   => @logger_stub
     end
   end
 
@@ -67,9 +64,9 @@ describe SimpleDeploy::CLI::Shared do
     @object.command_name.should == 'object'
   end
 
-  it "should rescue stackster exceptions and exit 1" do
-    lambda { @object.rescue_stackster_exceptions_and_exit do
-               raise Stackster::Exceptions::Base
+  it "should rescue exceptions and exit 1" do
+    lambda { @object.rescue_exceptions_and_exit do
+               raise SimpleDeploy::Exceptions::Base
              end 
            }.should raise_error SystemExit
   end

data/spec/cli/update_spec.rb

@@ -3,23 +3,19 @@ require 'spec_helper'
 require 'simple_deploy/cli'
 
 describe SimpleDeploy::CLI::Update do
+  include_context 'cli config'
+  include_context 'double stubbed logger'
+  include_context 'received stack array', 'my_stack', 'my_env', 1
+
   describe 'update' do
     before do
-      @config  = mock 'config'
-      @logger  = stub 'logger', 'info' => 'true'
-      @stack   = stub :attributes => {}
-
-      SimpleDeploy::Config.stub(:new).and_return(@config)
-      @config.should_receive(:environment).with('my_env').and_return(@config)
-      SimpleDeploy::SimpleDeployLogger.should_receive(:new).
-                                       with(:log_level => 'debug').
-                                       and_return(@logger)
+      @stack_mock1.stub(:attributes).and_return({})
     end
 
     it "should pass force true" do
       options = { :environment => 'my_env',
                   :log_level   => 'debug',
-                  :name        => ['my_stack'],
+                  :name        => ['my_stack1'],
                   :force       => true,
                   :attributes  => ['chef_repo_bucket_prefix=intu-lc'] }
 
@@ -29,14 +25,7 @@ describe SimpleDeploy::CLI::Update do
 
       Trollop.stub(:options).and_return(options)
 
-      SimpleDeploy::Stack.should_receive(:new).
-                          with(:config      => @config,
-                               :environment => 'my_env',
-                               :logger      => @logger,
-                               :name        => 'my_stack').
-                          and_return(@stack)
-
-      @stack.should_receive(:update).with(hash_including(:force => true))
+      @stack_mock1.should_receive(:update).with(hash_including(:force => true))
 
       subject.update
     end
@@ -44,7 +33,7 @@ describe SimpleDeploy::CLI::Update do
     it "should pass force false" do
       options = { :environment => 'my_env',
                   :log_level   => 'debug',
-                  :name        => ['my_stack'],
+                  :name        => ['my_stack1'],
                   :force       => false,
                   :attributes  => ['chef_repo_bucket_prefix=intu-lc'] }
 
@@ -54,14 +43,7 @@ describe SimpleDeploy::CLI::Update do
 
       Trollop.stub(:options).and_return(options)
 
-      SimpleDeploy::Stack.should_receive(:new).
-                          with(:config      => @config,
-                               :environment => 'my_env',
-                               :logger      => @logger,
-                               :name        => 'my_stack').
-                          and_return(@stack)
-
-      @stack.should_receive(:update).with(hash_including(:force => false))
+      @stack_mock1.should_receive(:update).with(hash_including(:force => false))
 
       subject.update
     end

data/spec/config_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe SimpleDeploy do
+describe SimpleDeploy::Configuration do
   let(:config_data) do
     { 'environments' => {
         'test_env' => {
@@ -14,38 +14,47 @@ describe SimpleDeploy do
       } } }
   end
 
-  describe 'new' do
+  describe 'creating a configuration' do
+    before do
+      @the_module = SimpleDeploy::Configuration
+    end
+
     it 'should accept config data as an argument' do
       YAML.should_not_receive(:load)
 
-      @config = SimpleDeploy::Config.new :config => config_data
-      @config.config.should == config_data
+      @config = @the_module.configure 'test_env', :config => config_data
+      @config.environment.should == config_data['environments']['test_env']
+      @config.notifications.should == config_data['notifications']
     end
 
     it 'should load the config from ~/.simple_deploy.yml by default' do
       File.should_receive(:open).with("#{ENV['HOME']}/.simple_deploy.yml").
                                  and_return(config_data.to_yaml)
-      @config = SimpleDeploy::Config.new
-      @config.config.should == config_data
+
+      @config = @the_module.configure 'test_env'
+      @config.environment.should == config_data['environments']['test_env']
+      @config.notifications.should == config_data['notifications']
     end
 
     it 'should load the config from SIMPLE_DEPLOY_CONFIG_FILE if supplied' do
       File.should_receive(:open).with("/my/config/file").
                                  and_return(config_data.to_yaml)
       env_mock = mock 'env'
-      SimpleDeploy::Env.stub :new => env_mock
+      @the_module.stub(:env).and_return(env_mock)
       env_mock.should_receive(:load).
                with('SIMPLE_DEPLOY_CONFIG_FILE').
                and_return "/my/config/file"
-      @config = SimpleDeploy::Config.new
-      @config.config.should == config_data
+      @config = @the_module.configure 'test_env'
+      @config.environment.should == config_data['environments']['test_env']
+      @config.notifications.should == config_data['notifications']
     end
 
   end
 
   describe "after creating a configuration" do
     before do
-      @config = SimpleDeploy::Config.new :config => config_data
+      @the_module = SimpleDeploy::Configuration
+      @config = @the_module.configure 'test_env', :config => config_data
     end
 
     it "should return the default artifacts to deploy" do
@@ -61,15 +70,26 @@ describe SimpleDeploy do
     end
 
     it "should return the environment requested" do
-      @config.environment('test_env').should == ({ 'secret_key' => 'secret', 'access_key' => 'access', 'region' => 'us-west-1' })
+      env_config = @config.environment
+      env_config['access_key'].should == 'access'
+      env_config['secret_key'].should == 'secret'
+      env_config['region'].should == 'us-west-1'
     end
 
     it "should return the notifications available" do
       @config.notifications.should == ( { 'campfire' => { 'token' => 'my_token' } } )
     end
 
+    it "should return the access_key for the environment" do
+      @config.access_key.should == 'access'
+    end
+
+    it "should return the secret_key for the environment" do
+      @config.secret_key.should == 'secret'
+    end
+
     it "should return the region for the environment" do
-      @config.region('test_env').should == 'us-west-1'
+      @config.region.should == 'us-west-1'
     end
 
     it "should return the deploy script" do
@@ -78,11 +98,24 @@ describe SimpleDeploy do
 
   end
 
+  describe 'showing raw configuration for all instances' do
+    before do
+      @the_module = SimpleDeploy::Configuration
+    end
+
+    it "should return a hash for every environment" do
+      environments = @the_module.environments :config => config_data
+      environments.keys.should == ['test_env']
+    end
+  end
+
   describe "gracefully handling yaml file errors" do
     before do
       FakeFS.activate!
       @config_file_path = "#{ENV['HOME']}/.simple_deploy.yml"
       FileUtils.mkdir_p File.dirname(@config_file_path)
+
+      @the_module = SimpleDeploy::Configuration
     end
 
     after do
@@ -92,7 +125,7 @@ describe SimpleDeploy do
 
     it "should handle a missing file gracefully" do
       expect {
-        config = SimpleDeploy::Config.new
+        config = @the_module.configure 'test_env'
       }.to raise_error(RuntimeError, "#{@config_file_path} not found")
     end
 
@@ -103,7 +136,7 @@ describe SimpleDeploy do
       end
 
       expect {
-        config = SimpleDeploy::Config.new
+        config = @the_module.configure 'test_env'
       }.to raise_error(RuntimeError, "#{@config_file_path} is corrupt")
     end
   end

data/spec/contexts/config_contexts.rb

@@ -0,0 +1,28 @@
+shared_context "cli config" do
+  before do
+    @config_mock = mock 'config mock'
+    SimpleDeploy.stub(:create_config).and_return(@config_mock)
+    SimpleDeploy.stub(:config).and_return(@config_mock)
+  end
+end
+
+shared_context "received config" do
+  before do
+    @config_mock = mock 'config mock'
+    SimpleDeploy.should_receive(:config).and_return(@config_mock)
+  end
+end
+
+shared_context "stubbed config" do
+  before do
+    @config_mock = mock 'config mock'
+    SimpleDeploy.stub(:config).and_return(@config_mock)
+  end
+end
+
+shared_context "double stubbed config" do |methods_hash|
+  before do
+    @config_stub = stub 'config stub', methods_hash
+    SimpleDeploy.stub(:config).and_return(@config_stub)
+  end
+end

data/spec/contexts/logger_contexts.rb

@@ -0,0 +1,9 @@
+shared_context "double stubbed logger" do
+  before do
+    @logger_stub = stub 'logger stub', :debug => true,
+                                       :info  => true,
+                                       :warn  => true,
+                                       :error => true
+    SimpleDeploy.stub(:logger).and_return(@logger_stub)
+  end
+end

data/spec/contexts/stack_contexts.rb

@@ -0,0 +1,40 @@
+shared_context "stubbed stack" do |name, environment, options|
+  before do
+    args = { :name => name, :environment => environment }
+    args[:use_internal_ips] = options[:internal] if options && options[:internal]
+    @stack_mock = mock 'stack mock', args
+    SimpleDeploy::Stack.stub(:new).and_return(@stack_mock)
+  end
+end
+
+shared_context "double stubbed stack" do |name, environment, options|
+  before do
+    args = { :name => name, :environment => environment }
+    args[:use_internal_ips] = options[:internal] if options && options[:internal]
+    @stack_stub = stub 'stack stub', args
+    SimpleDeploy::Stack.stub(:new).and_return(@stack_stub)
+  end
+end
+
+shared_context "clone stack pair" do |source_name, source_env, new_name, new_env|
+  before do
+    @source_stack_stub = stub 'source stack stub', :name => source_name,
+                                                   :environment => source_env
+    @new_stack_mock    = mock 'new stack mock', :name => new_name,
+                                                :environment => new_env
+    SimpleDeploy::Stack.stub(:new).and_return(@source_stack_stub, @new_stack_mock)
+  end
+end
+
+shared_context "received stack array" do |base_name, env, num_instances|
+  before do
+    1.upto(num_instances) do |n|
+      name = "#{base_name}#{n}"
+      stack_mock = mock 'stack mock', :name => name, :environment => env
+      self.instance_variable_set(:"@stack_mock#{n}", stack_mock)
+      SimpleDeploy::Stack.should_receive(:new).with(:name        => name,
+                                                    :environment => env).
+                                                    and_return(stack_mock)
+    end
+  end
+end

data/spec/entry_lister_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe SimpleDeploy::EntryLister do
+  include_context 'stubbed config'
+
+  it "should create a list of entries" do
+    @simple_db_mock = mock 'simple db'
+    SimpleDeploy::AWS::SimpleDB.should_receive(:new).and_return @simple_db_mock
+    @simple_db_mock.should_receive(:domain_exists?).
+                    with("stacks").
+                    and_return true
+    @simple_db_mock.should_receive(:select).
+                    with("select * from stacks").
+                    and_return('stack-to-find-us-west-1' => { 'attr1' => 'value1' })
+    entry_lister = SimpleDeploy::EntryLister.new
+    entry_lister.all.should == ['stack-to-find']
+  end
+
+  it "should return a blank array if the domain does not exist" do
+    @simple_db_mock = mock 'simple db'
+    SimpleDeploy::AWS::SimpleDB.should_receive(:new).and_return @simple_db_mock
+    @simple_db_mock.should_receive(:domain_exists?).
+                    with("stacks").
+                    and_return false
+    @simple_db_mock.should_receive(:select).
+                    with("select * from stacks").exactly(0).times
+    entry_lister = SimpleDeploy::EntryLister.new
+    entry_lister.all.should == []
+  end
+
+end

data/spec/entry_spec.rb

@@ -0,0 +1,86 @@
+require 'spec_helper'
+
+describe SimpleDeploy::Entry do
+  include_context 'double stubbed logger'
+
+  let(:config_data) do
+    { 'environments' => {
+        'test_env' => {
+          'secret_key' => 'the-key',
+          'access_key' => 'access',
+          'region'     => 'us-west-1'
+      } } }
+  end
+
+  before do
+    @config = SimpleDeploy.create_config 'test_env', :config => config_data
+  end
+
+  after do
+    SimpleDeploy.release_config
+  end
+
+  it "should create a new entry object" do
+    @simple_db_mock = mock 'simple db'
+    SimpleDeploy::AWS::SimpleDB.should_receive(:new).and_return @simple_db_mock
+    @simple_db_mock.should_receive(:create_domain).
+                    with("stacks").
+                    and_return true
+    entry = SimpleDeploy::Entry.new :name   => 'test-stack'
+    entry.class.should == SimpleDeploy::Entry
+  end
+  
+  it "should find the requested stack in simple db" do
+    @simple_db_mock = mock 'simple db'
+    SimpleDeploy::AWS::SimpleDB.should_receive(:new).and_return @simple_db_mock
+
+    @simple_db_mock.should_receive(:create_domain).
+                    with("stacks").
+                    and_return true
+    SimpleDeploy::Entry.find :name   => 'stack-to-find'
+  end
+
+  context "with stack object" do
+    before do
+      @simple_db_mock = mock 'simple db'
+      SimpleDeploy::AWS::SimpleDB.should_receive(:new).and_return @simple_db_mock
+      @simple_db_mock.should_receive(:create_domain).
+                      with("stacks").
+                      and_return true
+      @entry = SimpleDeploy::Entry.new :name   => 'test-stack'
+    end
+
+    it "should set the name to region-name for the stack" do
+      @entry.name.should == 'test-stack-us-west-1'
+    end
+
+    it "should set the attributes in simple db including default attributes" do
+      Timecop.travel Time.utc(2012, 10, 22, 13, 30)
+
+      @simple_db_mock.should_receive(:select).
+                      with("select * from stacks where itemName() = 'test-stack-us-west-1'").
+                      and_return('test-stack-us-west-1' => { 'key1' => ['value1'] })
+      @simple_db_mock.should_receive(:put_attributes).
+                      with("stacks", 
+                           "test-stack-us-west-1", 
+                           { "key"       => "value",
+                             "key1"      => "value1",
+                             "Name"      => "test-stack-us-west-1",
+                             "CreatedAt" => "2012-10-22 13:30:00 UTC" }, 
+                           { :replace => ["key1", "key", "Name", "CreatedAt"] } )
+      @entry.set_attributes(['key' => 'value'])
+      
+      @entry.save
+    end
+
+    it "should merge custom attributes" do
+      @simple_db_mock.should_receive(:select).
+                      with("select * from stacks where itemName() = 'test-stack-us-west-1'").
+                      and_return('test-stack' => { 'key1' => ['value1'] })
+      @entry.set_attributes(['key1' => 'value2'])
+
+      @entry.attributes.should == {'key1' => 'value2' }
+    end
+  end
+
+end