stack_master 0.2.0 → 0.3.0

data/spec/stack_master/commands/validate_spec.rb

@@ -18,7 +18,7 @@ RSpec.describe StackMaster::Commands::Validate do
   describe "#perform" do
     context "can find stack" do
       it "calls the validator to validate the stack definition" do
-        expect(StackMaster::Validator).to receive(:valid?).with(stack_definition)
+        expect(StackMaster::Validator).to receive(:valid?).with(stack_definition, config)
         validate.perform
       end
     end

data/spec/stack_master/paged_response_accumulator_spec.rb

@@ -0,0 +1,39 @@
+RSpec.describe StackMaster::PagedResponseAccumulator do
+  let(:cf) { Aws::CloudFormation::Client.new }
+  subject(:accumulator) { described_class.new(cf, :describe_stack_events, { stack_name: 'blah' }, :stack_events) }
+
+  context 'with one page' do
+    let(:page_one_events) { [
+      { event_id: '1', stack_id: '1', stack_name: 'blah', timestamp: Time.now},
+      { event_id: '2', stack_id: '1', stack_name: 'blah', timestamp: Time.now}
+    ] }
+
+    before do
+      cf.stub_responses(:describe_stack_events, { stack_events: page_one_events, next_token: nil })
+    end
+
+    it 'returns the first page' do
+      events = accumulator.call
+      expect(events.stack_events.count).to eq 2
+    end
+  end
+
+  context 'with two pages' do
+    let(:page_one_events) { [
+      { event_id: '1', stack_id: '1', stack_name: 'blah', timestamp: Time.now},
+      { event_id: '2', stack_id: '1', stack_name: 'blah', timestamp: Time.now}
+    ] }
+    let(:page_two_events) { [
+      { event_id: '3', stack_id: '1', stack_name: 'blah', timestamp: Time.now}
+    ] }
+
+    before do
+      cf.stub_responses(:describe_stack_events, { stack_events: page_one_events, next_token: 'blah' }, { stack_events: page_two_events } )
+    end
+
+    it 'returns all the stack events combined' do
+      events = accumulator.call
+      expect(events.stack_events.count).to eq 3
+    end
+  end
+end

data/spec/stack_master/stack_events/fetcher_spec.rb

@@ -1,46 +1,25 @@
 RSpec.describe StackMaster::StackEvents::Fetcher do
   let(:cf) { Aws::CloudFormation::Client.new }
+  let(:stack_name) { 'blah' }
 
   before do
     allow(Aws::CloudFormation::Client).to receive(:new).and_return(cf)
     allow(StackMaster::StackEvents::Streamer).to receive(:stream)
+    allow(StackMaster::PagedResponseAccumulator).to receive(:call).with(StackMaster.cloud_formation_driver, :describe_stack_events, { stack_name: stack_name }, :stack_events).and_return(OpenStruct.new(stack_events: events))
   end
 
   context 'with 2 stack events' do
     let(:events) { [
-      { event_id: '1', stack_id: '1', stack_name: 'blah', timestamp: Time.now},
-      { event_id: '2', stack_id: '1', stack_name: 'blah', timestamp: Time.now}
+      OpenStruct.new(event_id: '1', stack_id: '1', stack_name: 'blah', timestamp: Time.now),
+      OpenStruct.new(event_id: '2', stack_id: '1', stack_name: 'blah', timestamp: Time.now)
     ] }
 
-    before do
-      cf.stub_responses(:describe_stack_events, stack_events: events)
-    end
-
     it 'returns stack events' do
-      events = StackMaster::StackEvents::Fetcher.fetch('blah', 'us-east-1')
+      events = StackMaster::StackEvents::Fetcher.fetch(stack_name, 'us-east-1')
       expect(events.count).to eq 2
     end
   end
 
-  context 'when more than one page is present' do
-    let(:page_one_events) { [
-      { event_id: '1', stack_id: '1', stack_name: 'blah', timestamp: Time.now},
-      { event_id: '2', stack_id: '1', stack_name: 'blah', timestamp: Time.now}
-    ] }
-    let(:page_two_events) { [
-      { event_id: '3', stack_id: '1', stack_name: 'blah', timestamp: Time.now}
-    ] }
-
-    before do
-      cf.stub_responses(:describe_stack_events, { stack_events: page_one_events, next_token: 'blah' }, { stack_events: page_two_events } )
-    end
-
-    it 'returns all the stack events combined' do
-      events = StackMaster::StackEvents::Fetcher.fetch('blah', 'us-east-1')
-      expect(events.count).to eq 3
-    end
-  end
-
   context 'filtering with a from timestamp' do
     let(:two_pm) { Time.parse('2015-10-27 14:00') }
     let(:three_pm) { Time.parse('2015-10-27 15:00') }
@@ -48,17 +27,13 @@ RSpec.describe StackMaster::StackEvents::Fetcher do
 
     let(:events) {
       [
-        { event_id: '1', stack_id: '1', stack_name: 'blah', timestamp: two_pm },
-        { event_id: '2', stack_id: '1', stack_name: 'blah', timestamp: four_pm },
+        OpenStruct.new(event_id: '1', stack_id: '1', stack_name: 'blah', timestamp: two_pm),
+        OpenStruct.new(event_id: '2', stack_id: '1', stack_name: 'blah', timestamp: four_pm),
       ]
     }
 
-    before do
-      cf.stub_responses(:describe_stack_events, stack_events: events)
-    end
-
     it 'only returns events after the timestamp' do
-      events = StackMaster::StackEvents::Fetcher.fetch('blah', 'us-east-1', from: three_pm)
+      events = StackMaster::StackEvents::Fetcher.fetch(stack_name, 'us-east-1', from: three_pm)
       expect(events.map(&:event_id)).to eq ['2']
     end
   end

data/spec/stack_master/stack_spec.rb

@@ -82,7 +82,7 @@ RSpec.describe StackMaster::Stack do
     before do
       allow(StackMaster::ParameterLoader).to receive(:load).and_return(parameter_hash)
       allow(StackMaster::ParameterResolver).to receive(:resolve).and_return(resolved_parameters)
-      allow(StackMaster::TemplateCompiler).to receive(:compile).with(stack_definition.template_file_path).and_return(template_body)
+      allow(StackMaster::TemplateCompiler).to receive(:compile).with(config, stack_definition.template_file_path).and_return(template_body)
       allow(File).to receive(:read).with(stack_definition.stack_policy_file_path).and_return(stack_policy_body)
     end
 
@@ -154,4 +154,22 @@ RSpec.describe StackMaster::Stack do
       expect(little_stack.too_big?).to be_falsey
     end
   end
+
+  describe '#missing_parameters?' do
+    subject { stack.missing_parameters? }
+
+    let(:stack) { StackMaster::Stack.new(parameters: parameters, template_body: '{}') }
+
+    context 'when a parameter has a nil value' do
+      let(:parameters) { { 'my_param' => nil } }
+
+      it { should eq true }
+    end
+
+    context 'when no parameers have a nil value' do
+      let(:parameters) { { 'my_param' => '1' } }
+
+      it { should eq false }
+    end
+  end
 end

data/spec/stack_master/template_compiler_spec.rb

@@ -1,47 +1,20 @@
 RSpec.describe StackMaster::TemplateCompiler do
-  describe ".compile" do
-    def compile
-      StackMaster::TemplateCompiler.compile(template_file_path)
-    end
-
-    context 'json template' do
-      let(:template_file_path) { '/base_dir/templates/template.json' }
-
-      context "small json template" do
-        before do
-          allow(File).to receive(:read).with(template_file_path).and_return('{ }')
-        end
-
-        it "reads from the template file path" do
-          expect(compile).to eq('{ }')
-        end
-      end
-
-      context 'extra big json template' do
-        before do
-          allow(File).to receive(:read).with(template_file_path).and_return("{ #{' ' * 60000} }")
-        end
+  describe '.compile' do
+    let(:config) { double(template_compilers: { fab: :test_template_compiler }) }
+    let(:template_file_path) { '/base_dir/templates/template.fab' }
 
-        it "reads from the template file path" do
-          expect(compile).to eq('{}')
-        end
-      end
+    class TestTemplateCompiler
+      def self.compile(template_file_path); end
     end
 
-    context 'sparkleformation template' do
-      let(:template_file_path) { '/base_dir/templates/template.rb' }
-
-      before do
-        allow(SparkleFormation).to receive(:compile).with(template_file_path).and_return({})
-      end
-
-      it 'compiles with sparkleformation' do
-        expect(compile).to eq("{\n}")
-      end
+    context 'when a template compiler is registered for the given file type' do
+      before {
+        StackMaster::TemplateCompiler.register(:test_template_compiler, TestTemplateCompiler)
+      }
 
-      it 'sets the appropriate sparkle_path' do
-        compile
-        expect(SparkleFormation.sparkle_path).to eq File.dirname(template_file_path)
+      it 'compiles the template using the relevant template compiler' do
+        expect(TestTemplateCompiler).to receive(:compile).with(template_file_path)
+        StackMaster::TemplateCompiler.compile(config, template_file_path)
       end
     end
   end

data/spec/stack_master/template_compilers/json_spec.rb

@@ -0,0 +1,29 @@
+RSpec.describe StackMaster::TemplateCompilers::Json do
+  describe '.compile' do
+    def compile
+      described_class.compile(template_file_path)
+    end
+
+    let(:template_file_path) { '/base_dir/templates/template.json' }
+
+    context "small json template" do
+      before do
+        allow(File).to receive(:read).with(template_file_path).and_return('{ }')
+      end
+
+      it "reads from the template file path" do
+        expect(compile).to eq('{ }')
+      end
+    end
+
+    context 'extra big json template' do
+      before do
+        allow(File).to receive(:read).with(template_file_path).and_return("{ #{' ' * 60000} }")
+      end
+
+      it "reads from the template file path" do
+        expect(compile).to eq('{}')
+      end
+    end
+  end
+end

data/spec/stack_master/template_compilers/sparkle_formation_spec.rb

@@ -0,0 +1,22 @@
+RSpec.describe StackMaster::TemplateCompilers::SparkleFormation do
+  describe '.compile' do
+    def compile
+      described_class.compile(template_file_path)
+    end
+
+    before do
+      allow(SparkleFormation).to receive(:compile).with(template_file_path).and_return({})
+    end
+
+    let(:template_file_path) { '/base_dir/templates/template.rb' }
+
+    it 'compiles with sparkleformation' do
+      expect(compile).to eq("{\n}")
+    end
+
+    it 'sets the appropriate sparkle_path' do
+      compile
+      expect(SparkleFormation.sparkle_path).to eq File.dirname(template_file_path)
+    end
+  end
+end

data/spec/stack_master/test_driver/cloud_formation_spec.rb

@@ -3,22 +3,49 @@ require 'stack_master/test_driver/cloud_formation'
 RSpec.describe StackMaster::TestDriver::CloudFormation do
   subject(:test_cf_driver) { described_class.new }
 
-  it 'adds and gets stacks' do
-    test_cf_driver.add_stack(stack_id: "1", stack_name: 'stack-1')
-    test_cf_driver.add_stack(stack_id: "2", stack_name: 'stack-2')
-    expect(test_cf_driver.describe_stacks.stacks.map(&:stack_id)).to eq(["1", "2"])
+  context 'stacks' do
+    it 'creates and describes stacks' do
+      test_cf_driver.create_stack(stack_id: "1", stack_name: 'stack-1')
+      test_cf_driver.create_stack(stack_id: "2", stack_name: 'stack-2')
+      expect(test_cf_driver.describe_stacks.stacks.map(&:stack_id)).to eq(["1", "2"])
 
+    end
+
+    it 'adds and gets stack events' do
+      test_cf_driver.add_stack_event(stack_name: 'stack-1', resource_status: "UPDATE_COMPLETE")
+      test_cf_driver.add_stack_event(stack_name: 'stack-1', resource_status: "UPDATE_COMPLETE")
+      test_cf_driver.add_stack_event(stack_name: 'stack-2')
+      expect(test_cf_driver.describe_stack_events(stack_name: 'stack-1').stack_events.map(&:stack_name)).to eq ['stack-1', 'stack-1']
+    end
+
+    it 'sets and gets templates' do
+      test_cf_driver.set_template('stack-1', 'blah')
+      expect(test_cf_driver.get_template(stack_name: 'stack-1').template_body).to eq 'blah'
+    end
   end
 
-  it 'adds and gets stack events' do
-    test_cf_driver.add_stack_event(stack_name: 'stack-1', resource_status: "UPDATE_COMPLETE")
-    test_cf_driver.add_stack_event(stack_name: 'stack-1', resource_status: "UPDATE_COMPLETE")
-    test_cf_driver.add_stack_event(stack_name: 'stack-2')
-    expect(test_cf_driver.describe_stack_events(stack_name: 'stack-1').stack_events.map(&:stack_name)).to eq ['stack-1', 'stack-1']
+  context 'change sets' do
+    it 'creates and describes change sets' do
+      change_set_id = test_cf_driver.create_change_set(
+        stack_name: 'stack-1',
+        change_set_name: 'change-set-1',
+        template_body: '{}',
+        parameters: [{ paramater_key: 'param_1', parameter_value: 'value_1'}]
+      ).id
+      change_set = test_cf_driver.describe_change_set(change_set_name: change_set_id)
+      expect(change_set.change_set_id).to eq change_set_id
+      expect(change_set.change_set_name).to eq 'change-set-1'
+      change_set = test_cf_driver.describe_change_set(change_set_name: 'change-set-1')
+      expect(change_set.change_set_id).to eq change_set_id
+      expect(change_set.change_set_name).to eq 'change-set-1'
+    end
   end
 
-  it 'sets and gets templates' do
-    test_cf_driver.set_template('stack-1', 'blah')
-    expect(test_cf_driver.get_template(stack_name: 'stack-1').template_body).to eq 'blah'
+  it 'deletes change sets' do
+    change_set_id = test_cf_driver.create_change_set(stack_name: '1', change_set_name: '2').id
+    test_cf_driver.delete_change_set(change_set_name: change_set_id)
+    expect {
+      test_cf_driver.describe_change_set(change_set_name: change_set_id)
+    }.to raise_error(KeyError)
   end
 end

data/spec/stack_master/validator_spec.rb

@@ -1,6 +1,7 @@
 RSpec.describe StackMaster::Validator do
 
-  subject(:validator) { described_class.new(stack_definition) }
+  subject(:validator) { described_class.new(stack_definition, config) }
+  let(:config) { StackMaster::Config.new({'stacks' => {}}, '/base_dir') }
   let(:stack_definition) do
     StackMaster::StackDefinition.new(
       region: 'us-east-1',

data/spec/stack_master_spec.rb

@@ -0,0 +1,81 @@
+RSpec.describe StackMaster do
+  describe '.debug' do
+    let(:message) { "Doing some stuff" }
+
+    context 'when debugging' do
+      before { allow(StackMaster).to receive(:debug?).and_return(true) }
+
+      it 'outputs the message to STDERR' do
+        expect { StackMaster.debug(message) }.to output(/\[DEBUG\] #{message}/).to_stderr
+      end
+    end
+
+    context 'when not debugging' do
+      before { allow(StackMaster).to receive(:debug?).and_return(false) }
+
+      it "doesn't output the message to STDERR" do
+        expect { StackMaster.debug(message) }.to output("").to_stderr
+      end
+    end
+  end
+
+  describe '.debug?' do
+    subject { StackMaster.debug? }
+
+    context "when debug! isn't called" do
+      it { should eq false }
+    end
+
+    context 'when debug! is called' do
+      before { StackMaster.debug! }
+
+      it { should eq true }
+
+      after { StackMaster.instance_variable_set('@debug', false) }
+    end
+  end
+
+  describe '.interactive?' do
+    subject { StackMaster.interactive? }
+
+    context "when non_interactive! isn't called" do
+      it { should eq true }
+    end
+
+    context 'when non_interactive! is called' do
+      before { StackMaster.non_interactive! }
+
+      it { should eq false }
+
+      after { StackMaster.instance_variable_set('@non_interactive', false) }
+    end
+  end
+
+  describe '.non_interactive?' do
+    subject { StackMaster.non_interactive? }
+
+    context "when non_interactive! isn't called" do
+      it { should eq false }
+    end
+
+    context 'when non_interactive! is called' do
+      before { StackMaster.non_interactive! }
+
+      it { should eq true }
+
+      after { StackMaster.instance_variable_set('@non_interactive', false) }
+    end
+  end
+
+  describe '.non_interactive_answer' do
+    it 'defaults to y' do
+      expect(StackMaster.non_interactive_answer).to eq 'y'
+    end
+
+    it 'can be overridden' do
+      StackMaster.non_interactive_answer = 'n'
+      expect(StackMaster.non_interactive_answer).to eq 'n'
+      StackMaster.instance_variable_set('@non_interactive_answer', 'y')
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stack_master
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Steve Hodgkiss
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-09 00:00:00.000000000 Z
+date: 2016-03-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -302,6 +302,7 @@ files:
 - features/validate.feature
 - lib/stack_master.rb
 - lib/stack_master/aws_driver/cloud_formation.rb
+- lib/stack_master/change_set.rb
 - lib/stack_master/cli.rb
 - lib/stack_master/command.rb
 - lib/stack_master/commands/apply.rb
@@ -317,6 +318,7 @@ files:
 - lib/stack_master/commands/validate.rb
 - lib/stack_master/config.rb
 - lib/stack_master/ctrl_c.rb
+- lib/stack_master/paged_response_accumulator.rb
 - lib/stack_master/parameter_loader.rb
 - lib/stack_master/parameter_resolver.rb
 - lib/stack_master/parameter_resolvers/latest_ami_by_tags.rb
@@ -337,6 +339,8 @@ files:
 - lib/stack_master/stack_states.rb
 - lib/stack_master/stack_status.rb
 - lib/stack_master/template_compiler.rb
+- lib/stack_master/template_compilers/json.rb
+- lib/stack_master/template_compilers/sparkle_formation.rb
 - lib/stack_master/test_driver/cloud_formation.rb
 - lib/stack_master/testing.rb
 - lib/stack_master/utils.rb
@@ -350,6 +354,7 @@ files:
 - spec/fixtures/stack_master.yml
 - spec/fixtures/templates/myapp_vpc.json
 - spec/spec_helper.rb
+- spec/stack_master/change_set_spec.rb
 - spec/stack_master/command_spec.rb
 - spec/stack_master/commands/apply_spec.rb
 - spec/stack_master/commands/delete_spec.rb
@@ -357,6 +362,7 @@ files:
 - spec/stack_master/commands/status_spec.rb
 - spec/stack_master/commands/validate_spec.rb
 - spec/stack_master/config_spec.rb
+- spec/stack_master/paged_response_accumulator_spec.rb
 - spec/stack_master/parameter_loader_spec.rb
 - spec/stack_master/parameter_resolver_spec.rb
 - spec/stack_master/parameter_resolvers/latest_ami_by_tag_spec.rb
@@ -376,9 +382,12 @@ files:
 - spec/stack_master/stack_events/streamer_spec.rb
 - spec/stack_master/stack_spec.rb
 - spec/stack_master/template_compiler_spec.rb
+- spec/stack_master/template_compilers/json_spec.rb
+- spec/stack_master/template_compilers/sparkle_formation_spec.rb
 - spec/stack_master/test_driver/cloud_formation_spec.rb
 - spec/stack_master/utils_spec.rb
 - spec/stack_master/validator_spec.rb
+- spec/stack_master_spec.rb
 - stack_master.gemspec
 - stacktemplates/parameter_region.yml
 - stacktemplates/parameter_stack_name.yml
@@ -404,7 +413,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5.1
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: StackMaster is a sure-footed way of creating, updating and keeping track
@@ -427,6 +436,7 @@ test_files:
 - spec/fixtures/stack_master.yml
 - spec/fixtures/templates/myapp_vpc.json
 - spec/spec_helper.rb
+- spec/stack_master/change_set_spec.rb
 - spec/stack_master/command_spec.rb
 - spec/stack_master/commands/apply_spec.rb
 - spec/stack_master/commands/delete_spec.rb
@@ -434,6 +444,7 @@ test_files:
 - spec/stack_master/commands/status_spec.rb
 - spec/stack_master/commands/validate_spec.rb
 - spec/stack_master/config_spec.rb
+- spec/stack_master/paged_response_accumulator_spec.rb
 - spec/stack_master/parameter_loader_spec.rb
 - spec/stack_master/parameter_resolver_spec.rb
 - spec/stack_master/parameter_resolvers/latest_ami_by_tag_spec.rb
@@ -453,6 +464,9 @@ test_files:
 - spec/stack_master/stack_events/streamer_spec.rb
 - spec/stack_master/stack_spec.rb
 - spec/stack_master/template_compiler_spec.rb
+- spec/stack_master/template_compilers/json_spec.rb
+- spec/stack_master/template_compilers/sparkle_formation_spec.rb
 - spec/stack_master/test_driver/cloud_formation_spec.rb
 - spec/stack_master/utils_spec.rb
 - spec/stack_master/validator_spec.rb
+- spec/stack_master_spec.rb