simple_deploy 0.7.2 → 0.7.3

data/spec/stack/output_mapper_spec.rb

@@ -1,9 +1,10 @@
 require 'spec_helper'
 
 describe SimpleDeploy::Stack::OutputMapper do
+  include_context 'double stubbed logger'
+
   before do
     @config_mock = mock 'config'
-    @logger_stub = stub 'logger', :debug => true, :info => true
 
     stack1_outputs = [ { 'OutputKey' => 'Test1', 'OutputValue' => 'val1' },
                        { 'OutputKey' => 'Nother', 'OutputValue' => 'another' } ]
@@ -23,8 +24,7 @@ describe SimpleDeploy::Stack::OutputMapper do
     @template_stub = stub 'template', :parameters => ["Test1", "Test2", "Tests"]
 
     @mapper = SimpleDeploy::Stack::OutputMapper.new :config      => @config_mock,
-                                                    :environment => 'default',
-                                                    :logger      => @logger_stub
+                                                    :environment => 'default'
   end
 
   context "when provided stacks" do
@@ -37,8 +37,6 @@ describe SimpleDeploy::Stack::OutputMapper do
     it "should return the outputs which match parameters" do
       SimpleDeploy::Stack.should_receive(:new).
                           with(:environment => 'default',
-                               :config      => @config_mock,
-                               :logger      => @logger_stub,
                                :name        => 'stack1').
                           and_return @stack1_stub
       @mapper.should_receive(:sleep)
@@ -50,8 +48,6 @@ describe SimpleDeploy::Stack::OutputMapper do
     it "should return the outputs which match pluralized parameters" do
       SimpleDeploy::Stack.should_receive(:new).
                           with(:environment => 'default',
-                               :config      => @config_mock,
-                               :logger      => @logger_stub,
                                :name        => 'stack4').
                           and_return @stack4_stub
       @mapper.should_receive(:sleep)
@@ -63,14 +59,10 @@ describe SimpleDeploy::Stack::OutputMapper do
     it "should return the outputs which match parameters from multiple stacks" do
       SimpleDeploy::Stack.should_receive(:new).
                           with(:environment => 'default',
-                               :config      => @config_mock,
-                               :logger      => @logger_stub,
                                :name        => 'stack1').
                           and_return @stack1_stub
       SimpleDeploy::Stack.should_receive(:new).
                           with(:environment => 'default',
-                               :config      => @config_mock,
-                               :logger      => @logger_stub,
                                :name        => 'stack2').
                           and_return @stack2_stub
       @mapper.should_receive(:sleep).exactly(3).times
@@ -82,14 +74,10 @@ describe SimpleDeploy::Stack::OutputMapper do
     it "should concatenate multiple outputs of same name into CSV" do
       SimpleDeploy::Stack.should_receive(:new).
                           with(:environment => 'default',
-                               :config      => @config_mock,
-                               :logger      => @logger_stub,
                                :name        => 'stack1').
                           and_return @stack1_stub
       SimpleDeploy::Stack.should_receive(:new).
                           with(:environment => 'default',
-                               :config      => @config_mock,
-                               :logger      => @logger_stub,
                                :name        => 'stack3').
                           and_return @stack3_stub
       @mapper.should_receive(:sleep).exactly(3).times

data/spec/stack/ssh_spec.rb

@@ -1,24 +1,21 @@
 require 'spec_helper'
 
 describe SimpleDeploy::Stack::SSH do
+  include_context 'stubbed config'
+  include_context 'double stubbed logger'
+  include_context 'double stubbed stack', :name        => 'my_stack',
+                                          :environment => 'my_env'
+
   before do
-    @stack_mock = mock 'stack'
     @task_mock = mock 'task'
-    @config_mock = mock 'config'
-    @logger_stub = stub 'logger', :debug => true,
-                        :info  => true,
-                        :error => true
-    @config_mock.stub :logger => @logger_stub
-    @config_mock.should_receive(:region).
-        with('test-env').
-        and_return 'test-us-west-1'
-    @stack_mock.stub :attributes => { :ssh_gateway => false }
-    @options = { :config      => @config_mock,
-                 :instances   => ['1.2.3.4', '4.3.2.1'],
+    @config_mock.should_receive(:region).and_return 'test-us-west-1'
+
+    @stack_stub.stub(:attributes).and_return({ :ssh_gateway => false })
+    @options = { :instances   => ['1.2.3.4', '4.3.2.1'],
                  :environment => 'test-env',
                  :ssh_user    => 'user',
                  :ssh_key     => 'key',
-                 :stack       => @stack_mock,
+                 :stack       => @stack_stub,
                  :name        => 'test-stack' }
     @task_logger_mock = mock 'task_logger'
     @ssh_options = Hash.new
@@ -26,6 +23,10 @@ describe SimpleDeploy::Stack::SSH do
                     :variables => @ssh_options
   end
 
+  after do
+    SimpleDeploy.release_config
+  end
+
   context "when unsuccessful" do
     it "should return false when no running instances running" do
       @ssh = SimpleDeploy::Stack::SSH.new @options.merge({ :instances   => [] })

data/spec/stack/{stack_attribute_formater_spec.rb → stack_attribute_formatter_spec.rb}

@@ -1,26 +1,31 @@
 require 'spec_helper'
 
-describe SimpleDeploy do
+describe SimpleDeploy::StackAttributeFormatter do
+  include_context 'stubbed config'
+  include_context 'double stubbed logger'
+
   before do
-    @logger_mock = mock 'logger mock', :info => 'true'
-    @config_mock = mock 'config mock', :logger => @logger_mock, :region => 'us-west-1'
+    @config_mock.stub(:region).and_return('us-west-1')
     @config_mock.stub(:artifact_cloud_formation_url).and_return('ChefRepoURL')
     @config_mock.stub(:artifacts).and_return(['chef_repo', 'cookbooks', 'app'])
   end
 
+  after do
+    SimpleDeploy.release_config
+  end
+
   context "when chef_repo unencrypted" do
     before do
-      options = { :config      => @config_mock,
-                  :environment => 'preprod',
+      options = { :environment => 'preprod',
                   :main_attributes => {
                     'chef_repo_bucket_prefix' => 'test-prefix',
                     'chef_repo_domain'        => 'test-domain' }
                 }
-      @formater = SimpleDeploy::StackAttributeFormater.new options
+      @formatter = SimpleDeploy::StackAttributeFormatter.new options
     end
 
     it 'should return updated attributes including the un encrypted cloud formation url' do
-      updates = @formater.updated_attributes([ { 'chef_repo' => 'test123' } ])
+      updates = @formatter.updated_attributes([ { 'chef_repo' => 'test123' } ])
       updates.should == [{ 'chef_repo' => 'test123' }, 
                          { 'ChefRepoURL' => 's3://test-prefix-us-west-1/test-domain/test123.tar.gz' }]
     end
@@ -28,18 +33,17 @@ describe SimpleDeploy do
 
   context "when main_attributes set chef_repo encrypted" do
     before do
-      options = { :config      => @config_mock,
-                  :environment => 'preprod',
+      options = { :environment => 'preprod',
                   :main_attributes => {
                     'chef_repo_bucket_prefix' => 'test-prefix',
                     'chef_repo_encrypted'     => 'true',
                     'chef_repo_domain'        => 'test-domain' }
                 }
-      @formater = SimpleDeploy::StackAttributeFormater.new options
+      @formatter = SimpleDeploy::StackAttributeFormatter.new options
     end
 
     it 'should return updated attributes including the encrypted cloud formation url ' do
-      updates = @formater.updated_attributes([ { 'chef_repo' => 'test123' } ])
+      updates = @formatter.updated_attributes([ { 'chef_repo' => 'test123' } ])
       updates.should == [{ 'chef_repo' => 'test123' }, 
                          { 'ChefRepoURL' => 's3://test-prefix-us-west-1/test-domain/test123.tar.gz.gpg' }]
     end
@@ -47,18 +51,17 @@ describe SimpleDeploy do
 
   context "when provided attributes set chef_repo encrypted" do
     before do
-      options = { :config      => @config_mock,
-                  :environment => 'preprod',
+      options = { :environment => 'preprod',
                   :main_attributes => {
                     'chef_repo_bucket_prefix' => 'test-prefix',
                     'chef_repo_domain'        => 'test-domain' }
                 }
-      @formater = SimpleDeploy::StackAttributeFormater.new options
+      @formatter = SimpleDeploy::StackAttributeFormatter.new options
     end
 
     it 'should return updated attributes including the encrypted cloud formation url ' do
-      updates = @formater.updated_attributes([ { 'chef_repo' => 'test123' }, 
-                                               { 'chef_repo_encrypted' => 'true' } ])
+      updates = @formatter.updated_attributes([ { 'chef_repo' => 'test123' }, 
+                                                { 'chef_repo_encrypted' => 'true' } ])
       updates.should == [{ 'chef_repo' => 'test123' },
                          { 'chef_repo_encrypted' => 'true' },
                          { 'ChefRepoURL' => 's3://test-prefix-us-west-1/test-domain/test123.tar.gz.gpg' }]

data/spec/stack/stack_creator_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+require 'json'
+
+describe SimpleDeploy::StackCreator do
+  include_context 'stubbed config'
+  include_context 'double stubbed logger'
+
+  before do
+    @attributes = { "param1" => "value1", "param3" => "value3" }
+    @template_json = '{ "Parameters": 
+                        { 
+                          "param1" : 
+                            {
+                              "Description" : "param-1"
+                            },
+                          "param2" : 
+                            {
+                              "Description" : "param-2"
+                            }
+                        }
+                      }'
+  end
+
+  it "should map the attributes to a template's parameters and create a stack " do
+    entry_mock = mock 'entry mock'
+    file_mock = mock 'file mock'
+    cloud_formation_mock = mock 'cloud formation mock'
+
+    SimpleDeploy::AWS::CloudFormation.should_receive(:new).
+                                      and_return cloud_formation_mock
+    File.should_receive(:open).with('path_to_file').
+                               and_return file_mock
+    file_mock.should_receive(:read).and_return @template_json
+    entry_mock.should_receive(:attributes).and_return @attributes
+    cloud_formation_mock.should_receive(:create).
+                         with(:name      => 'test-stack', 
+                              :parameters => { 'param1' => 'value1' },
+                              :template   => @template_json)
+    stack_creator = SimpleDeploy::StackCreator.new :name          => 'test-stack',
+                                                   :template_file => 'path_to_file',
+                                                   :entry         => entry_mock
+
+    stack_creator.create
+  end
+
+end

data/spec/stack/stack_destroyer_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe SimpleDeploy::StackDestroyer do
+  include_context 'stubbed config'
+  include_context 'double stubbed logger'
+
+  it "should destroy the stack" do
+    cloud_formation_mock = mock 'cloud formation mock'
+
+    SimpleDeploy::AWS::CloudFormation.should_receive(:new).
+                                      and_return cloud_formation_mock
+    cloud_formation_mock.should_receive(:destroy).with 'test-stack'
+                         
+    stack_destroyer = SimpleDeploy::StackDestroyer.new :name   => 'test-stack'
+    stack_destroyer.destroy
+  end
+
+end

data/spec/stack/stack_formatter_spec.rb

@@ -0,0 +1,37 @@
+require 'spec_helper'
+
+describe ::SimpleDeploy::StackFormatter do
+  include_context 'double stubbed config', :access_key => 'key',
+                                           :secret_key => 'XXX',
+                                           :region     => 'us-west-1'
+
+  before do
+    @logger_stub = stub 'logger stub', :info => 'true', :warn => 'true'
+
+    @stack_reader_mock = mock 'StackReader'
+    SimpleDeploy::StackReader.stub(:new).and_return(@stack_reader_mock)
+    @stack_reader_mock.stub(:attributes).and_return(:chef_repo_bucket_prefix => 'chef_repo_bp')
+    @stack_reader_mock.stub(:outputs).and_return([{'key' => 'value'}])
+    @stack_reader_mock.stub(:status).and_return('green')
+    @stack_reader_mock.stub(:events).and_return(['event1', 'event2', 'event3'])
+    @stack_reader_mock.stub(:resources).and_return([{'StackName' => 'my_stack'}])
+
+    @stack_formatter = SimpleDeploy::StackFormatter.new(:name => 'my_stack')
+  end
+
+  after do
+    SimpleDeploy.release_config
+  end
+
+  describe 'display' do
+    it 'should return formatted information for the stack' do
+      @stack_formatter.display.should == {
+        'attributes' => { :chef_repo_bucket_prefix => 'chef_repo_bp' },
+        'status' => 'green',
+        'outputs' => [{'key' => 'value'}],
+        'events' => ['event1', 'event2', 'event3'],
+        'resources' => [{'StackName' => 'my_stack'}]
+      }
+    end
+  end
+end

data/spec/stack/stack_lister_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe SimpleDeploy::StackLister do
+  include_context 'stubbed config'
+
+  it "should list the stack entries" do
+    entry_lister_mock = mock 'entry lister mock'
+
+    SimpleDeploy::EntryLister.should_receive(:new).
+                              and_return entry_lister_mock
+    entry_lister_mock.should_receive(:all)
+                         
+    stack_lister = SimpleDeploy::StackLister.new
+    stack_lister.all
+  end
+
+end

data/spec/stack/stack_reader_spec.rb

@@ -0,0 +1,81 @@
+require 'spec_helper'
+
+describe SimpleDeploy::StackReader do
+  include_context 'double stubbed config', :access_key => 'key',
+                                           :secret_key => 'XXX',
+                                           :region     => 'us-west-1'
+
+  before do
+    @logger_stub = stub 'logger stub', :info => 'true', :warn => 'true'
+
+    @entry_mock = mock 'Entry'
+    @entry_mock.stub(:attributes).and_return(:chef_repo_bucket_prefix => 'chef_repo_bp')
+    SimpleDeploy::Entry.stub(:new).and_return(@entry_mock)
+
+    @cf_mock = mock 'CloudFormation'
+    SimpleDeploy::AWS::CloudFormation.stub(:new).and_return(@cf_mock)
+    @cf_mock.stub(:stack_outputs).and_return([{'key' => 'value'}])
+    @cf_mock.stub(:stack_status).and_return('green')
+    @cf_mock.stub(:stack_events).and_return(['event1', 'event2'])
+    @cf_mock.stub(:stack_resources).and_return([{'StackName' => 'my_stack'}])
+    @cf_mock.stub(:template).and_return('{"Parameters": {"EIP": "string"}}')
+
+    @instance_reader_mock = mock 'InstanceReader'
+    SimpleDeploy::AWS::InstanceReader.stub(:new).and_return(@instance_reader_mock)
+    @instance_reader_mock.stub(:list_stack_instances).and_return(['instance1', 'instance2'])
+
+    @stack_reader = SimpleDeploy::StackReader.new(:name => 'my_stack', :logger => @logger_stub)
+  end
+
+  after do
+    SimpleDeploy.release_config
+  end
+
+  describe 'attributes' do
+    it 'should return the stack attributes' do
+      @stack_reader.attributes.should == { :chef_repo_bucket_prefix => 'chef_repo_bp' }
+    end
+  end
+
+  describe 'outputs' do
+    it 'should return the stack outputs' do
+      @stack_reader.outputs.should == [{'key' => 'value'}]
+    end
+  end
+
+  describe 'status' do
+    it 'should return the stack status' do
+      @stack_reader.status.should == 'green'
+    end
+  end
+
+  describe 'events' do
+    it 'should return the stack events' do
+      @stack_reader.events(2).should == ['event1', 'event2']
+    end
+  end
+
+  describe 'resources' do
+    it 'should return the stack resources' do
+      @stack_reader.resources.should == [{'StackName' => 'my_stack'}]
+    end
+  end
+
+  describe 'template' do
+    it 'should return the stack template' do
+      @stack_reader.template.should == '{"Parameters": {"EIP": "string"}}'
+    end
+  end
+
+  describe 'parameters' do
+    it 'should return the stack parameters' do
+      @stack_reader.parameters.should == ['EIP']
+    end
+  end
+
+  describe 'instances' do
+    it 'should return the stack instances' do
+      @stack_reader.instances.should == ['instance1', 'instance2']
+    end
+  end
+end

data/spec/stack/stack_updater_spec.rb

@@ -0,0 +1,79 @@
+require 'spec_helper'
+require 'json'
+
+describe SimpleDeploy::StackUpdater do
+  include_context 'stubbed config'
+  include_context 'double stubbed logger'
+
+  before do
+    @template_body = '{ "Parameters": 
+                        { 
+                          "param1" : 
+                            {
+                              "Description" : "param-1"
+                            },
+                          "param2" : 
+                            {
+                              "Description" : "param-2"
+                            }
+                        }
+                      }'
+  end
+
+  it "should update the stack when parameters change and stack is stable" do
+    attributes = { "param1" => "value1", "param3" => "value3" }
+    entry_mock = mock 'entry mock'
+    status_mock = mock 'status mock'
+    cloud_formation_mock = mock 'cloud formation mock'
+    SimpleDeploy::AWS::CloudFormation.should_receive(:new).
+                                      and_return cloud_formation_mock
+    entry_mock.should_receive(:attributes).and_return attributes
+    cloud_formation_mock.should_receive(:update).
+                         with(:name      => 'test-stack', 
+                              :parameters => { 'param1' => 'value1' },
+                              :template   => @template_body).
+                         and_return true
+    SimpleDeploy::Status.should_receive(:new).
+                      with(:name   => 'test-stack').
+                      and_return status_mock
+    status_mock.should_receive(:wait_for_stable).and_return true
+    stack_updater = SimpleDeploy::StackUpdater.new :name          => 'test-stack',
+                                                   :template_body => @template_body,
+                                                   :entry         => entry_mock
+
+    stack_updater.update_stack_if_parameters_changed( [ { 'param1' => 'new-value' } ] ).
+                  should == true
+  end
+
+  it "should raise an error when parameters change and stack is not stable" do
+    attributes = { "param1" => "value1", "param3" => "value3" }
+    entry_mock = mock 'entry mock'
+    status_mock = mock 'status mock'
+    cloud_formation_mock = mock 'cloud formation mock'
+    SimpleDeploy::AWS::CloudFormation.should_receive(:new).
+                                      exactly(0).times
+    SimpleDeploy::Status.should_receive(:new).
+                         with(:name   => 'test-stack').
+                         and_return status_mock
+    status_mock.should_receive(:wait_for_stable).and_return false
+    stack_updater = SimpleDeploy::StackUpdater.new :name          => 'test-stack',
+                                                   :template_body => @template_body,
+                                                   :entry         => entry_mock
+
+    lambda {stack_updater.update_stack_if_parameters_changed( [ { 'param1' => 'new-value' } ] ) }.
+                  should raise_error
+  end
+
+  it "should not update the stack when parameters don't change" do
+    attributes = { "param3" => "value3" }
+    entry_mock = mock 'entry mock'
+    SimpleDeploy::AWS::CloudFormation.should_receive(:new).exactly(0).times
+    stack_updater = SimpleDeploy::StackUpdater.new :name          => 'test-stack',
+                                                   :template_body => @template_body,
+                                                   :entry         => entry_mock
+
+    stack_updater.update_stack_if_parameters_changed( [ { 'another-param' => 'new-value' } ] ).
+                  should == false
+  end
+
+end