stackster 0.2.9 → 0.3.0

data/.gitignore

@@ -3,3 +3,5 @@
 Gemfile.lock
 pkg/*
 .rvmrc
+coverage
+*.swp

data/CHANGELOG

@@ -1,3 +1,8 @@
+## v0.3.0
+
+* Now uses consistent reads
+* Increased the testing coverage
+
 ## v0.2.9
 
 * Bug fix in private method.

data/Rakefile

@@ -1 +1,8 @@
 require "bundler/gem_tasks"
+require 'rspec/core/rake_task'
+
+desc 'Run specs'
+
+RSpec::Core::RakeTask.new do |t|
+      t.rspec_opts = %w(--color)
+end

data/lib/stackster/aws/simpledb.rb

@@ -28,7 +28,7 @@ module Stackster
       end
 
       def select(query)
-        @connect.select(query).body['Items']
+        @connect.select(query, 'ConsistentRead' => true).body['Items']
       end
 
       def delete(domain, key)

data/lib/stackster/entry.rb

@@ -24,7 +24,7 @@ module Stackster
     end
 
     def save
-      set_name_attribute
+      set_default_attributes
 
       attributes.each_pair do |key,value|
         @logger.debug "Setting attribute #{key}=#{value}"
@@ -45,8 +45,9 @@ module Stackster
 
     private
 
-    def set_name_attribute
+    def set_default_attributes
       attributes.merge!('Name' => name)
+      attributes.merge!('CreatedAt' => Time.now.utc)
     end
 
     def get_attributes

data/lib/stackster/instance/instance_reader.rb

@@ -5,9 +5,6 @@ module Stackster
       @config = args[:config]
     end
 
-    # Data structure is return deffernelty for class
-    # ToDo - Add support for classic
-    # Currently only supports VPC
     def list_stack_instances(stack_name)
       h = []
       describe_instances.each do |instance|

data/lib/stackster/stack.rb

@@ -50,10 +50,6 @@ module Stackster
       stack_reader.attributes
     end
 
-    def show
-      stack_reader.show
-    end
-
     def outputs
       stack_reader.outputs
     end

data/lib/stackster/version.rb

@@ -1,3 +1,3 @@
 module Stackster
-  VERSION = "0.2.9"
+  VERSION = "0.3.0"
 end

data/script/ci_setup

@@ -2,13 +2,13 @@
 
 # Source RVM profile and set ruby / gemset
 . /etc/profile
-rvm use "1.9.3-p125@stackster" --create
+
+# Exit with error if any command returns non zero
+set -e
 
 # Bundle gems
+rvm use "1.9.3-p125@stackster" --create
 bundle
 
-# Create blank deployment config
-touch $HOME/.stackster.yml
-
 # Run spec tests
-rspec spec
+rake spec

data/spec/aws/cloud_formation/error_spec.rb

@@ -1,9 +1,39 @@
 require 'spec_helper'
 
 describe Stackster do
+  before do
+    #@logger_stub = stub 'logger stub', :info => 'true', :warn => 'true'
+    @logger_mock = mock 'Logger'
+    @config_stub = stub 'Config', :logger => @logger_mock, :access_key => 'key', :secret_key => 'XXX', :region => 'us-west1'
 
-  ['error'].each do |t|
-    it "should test #{t}"
+    @exception_stub1 = stub 'Excon::Response'
+    @exception_stub1.stub(:response).and_return(@exception_stub1)
+    @exception_stub1.stub(:body).and_return('<opt><Error><Message>No updates are to be performed.</Message></Error></opt>')
+
+    @exception_stub2 = stub 'Excon::Response'
+    @exception_stub2.stub(:response).and_return(@exception_stub2)
+    @exception_stub2.stub(:body).and_return('<opt><Error><Message>Oops.</Message></Error></opt>')
   end
 
+  describe 'process' do
+    it 'should process no update messages' do
+      @logger_mock.should_receive(:info).with('No updates are to be performed.')
+
+      error = Stackster::AWS::CloudFormation::Error.new :exception => @exception_stub1, :config => @config_stub
+      error.process
+    end
+
+    it 'should process general messages' do
+      @logger_mock.should_receive(:error).with('Oops.')
+
+      error = Stackster::AWS::CloudFormation::Error.new :exception => @exception_stub2, :config => @config_stub
+
+      begin
+        error.process
+        fail 'should have exited'
+      rescue SystemExit => e
+        e.status.should == 1
+      end
+    end
+  end
 end

data/spec/aws/cloud_formation_spec.rb

@@ -1,10 +1,174 @@
 require 'spec_helper'
 
 describe Stackster do
+  before do
+    @logger_stub = stub 'logger stub', :info => 'true', :warn => 'true'
+    @config_stub = stub 'Config', :logger => @logger_stub, :access_key => 'key', :secret_key => 'XXX', :region => 'us-west1'
+    @error_stub = stub 'Error', :process => 'Processed Error'
+    @response_stub = stub 'Excon::Response', :body => { 
+        'Stacks' => [{'StackStatus' => 'green', 'Outputs' => [{'key' => 'value'}]}],
+        'StackResources' => [{'StackName' => 'my_stack'}],
+        'StackEvents' => ['event1', 'event2'],
+        'TemplateBody' => '{EIP: "string"}'
+    }
 
-  ['create', 'update', 'destroy', 'describe_stack', 'stack_resources', 
-   'stack_events', 'stack_status', 'stack_outputs', 'template'].each do |t|
-    it "should test #{t}"
+    @cf_mock = mock 'CloudFormation'
+    Fog::AWS::CloudFormation.stub(:new).and_return(@cf_mock)
+
+    @args = {
+      :parameters => { 'parameter1' => 'my_param' },
+      :name => 'my_stack',
+      :template => 'my_template'
+    }
+
+    @exception = Exception.new('Failed')
+
+    @cf = Stackster::AWS::CloudFormation.new(:config => @config_stub)
+  end
+
+  describe "create" do
+    it "should create the stack on Cloud Formation" do
+      @cf_mock.should_receive(:create_stack).with('my_stack',
+                                                  { 'Capabilities' => ['CAPABILITY_IAM'],
+                                                    'TemplateBody' => 'my_template',
+                                                    'Parameters' => { 'parameter1' => 'my_param' }
+                                                  })
+
+      @cf.create(@args)
+    end
+
+    it "should trap exceptions" do
+      @cf_mock.should_receive(:create_stack).with('my_stack',
+                                                  { 'Capabilities' => ['CAPABILITY_IAM'],
+                                                    'TemplateBody' => 'my_template',
+                                                    'Parameters' => { 'parameter1' => 'my_param' }
+                                                  }).and_raise(@exception)
+      Stackster::AWS::CloudFormation::Error.should_receive(:new).
+        with(:config => @config_stub, :exception => @exception).and_return(@error_stub)
+
+      @cf.create(@args).should == 'Processed Error'
+    end
   end
 
+  describe "update" do
+    it "should update the stack on Cloud Formation" do
+      @cf_mock.should_receive(:update_stack).with('my_stack',
+                                                  { 'Capabilities' => ['CAPABILITY_IAM'],
+                                                    'TemplateBody' => 'my_template',
+                                                    'Parameters' => { 'parameter1' => 'my_param' }
+                                                  })
+
+      @cf.update(@args)
+    end
+
+    it "should trap exceptions" do
+      @cf_mock.should_receive(:update_stack).with('my_stack',
+                                                  { 'Capabilities' => ['CAPABILITY_IAM'],
+                                                    'TemplateBody' => 'my_template',
+                                                    'Parameters' => { 'parameter1' => 'my_param' }
+                                                  }).and_raise(@exception)
+      Stackster::AWS::CloudFormation::Error.should_receive(:new).
+        with(:config => @config_stub, :exception => @exception).and_return(@error_stub)
+
+      @cf.update(@args).should == 'Processed Error'
+    end
+  end
+
+
+  describe 'destroy' do
+    it "should delete the stack on Cloud Formation" do
+      @cf_mock.should_receive(:delete_stack).with('my_stack')
+
+      @cf.destroy('my_stack')
+    end
+
+    it "should trap exceptions" do
+      @cf_mock.should_receive(:delete_stack).with('my_stack').and_raise(@exception)
+      Stackster::AWS::CloudFormation::Error.should_receive(:new).
+        with(:config => @config_stub, :exception => @exception).and_return(@error_stub)
+
+      @cf.destroy('my_stack').should == 'Processed Error'
+    end
+  end
+
+
+  describe 'describe_stack' do
+    it "should return the Cloud Formation description of the stack" do
+      @cf_mock.should_receive(:describe_stacks).with('StackName' => 'my_stack').and_return(@response_stub)
+
+      @cf.describe_stack('my_stack').should == [{'StackStatus' => 'green', 'Outputs' => [{'key' => 'value'}]}]
+    end
+
+    it "should trap exceptions" do
+      @cf_mock.should_receive(:describe_stacks).with('StackName' => 'my_stack').and_raise(@exception)
+      Stackster::AWS::CloudFormation::Error.should_receive(:new).
+        with(:config => @config_stub, :exception => @exception).and_return(@error_stub)
+
+      @cf.describe_stack('my_stack').should == 'Processed Error'
+    end
+  end
+
+  describe "stack_resources" do
+    it "should return the Cloud Formation description of the stack resources" do
+      @cf_mock.should_receive(:describe_stack_resources).with('StackName' => 'my_stack').and_return(@response_stub)
+
+      @cf.stack_resources('my_stack').should == [{'StackName' => 'my_stack'}]
+    end
+
+    it "should trap exceptions" do
+      @cf_mock.should_receive(:describe_stack_resources).with('StackName' => 'my_stack').and_raise(@exception)
+      Stackster::AWS::CloudFormation::Error.should_receive(:new).
+        with(:config => @config_stub, :exception => @exception).and_return(@error_stub)
+
+      @cf.stack_resources('my_stack').should == 'Processed Error'
+    end
+  end
+
+  describe "stack_events" do
+    it "should return the Cloud Formation description of the stack events" do
+      @cf_mock.should_receive(:describe_stack_events).with('my_stack').and_return(@response_stub)
+
+      @cf.stack_events('my_stack', 2).should == ['event1', 'event2']
+    end
+
+    it "should trap exceptions" do
+      @cf_mock.should_receive(:describe_stack_events).with('my_stack').and_raise(@exception)
+      Stackster::AWS::CloudFormation::Error.should_receive(:new).
+        with(:config => @config_stub, :exception => @exception).and_return(@error_stub)
+
+      @cf.stack_events('my_stack', 2).should == 'Processed Error'
+    end
+  end
+
+  describe "stack_status" do
+    it "should return the Cloud Formation status of the stack" do
+      @cf_mock.should_receive(:describe_stacks).with('StackName' => 'my_stack').and_return(@response_stub)
+
+      @cf.stack_status('my_stack').should == 'green'
+    end
+  end
+
+  describe "stack_outputs" do
+    it "should return the Cloud Formation outputs for the stack" do
+      @cf_mock.should_receive(:describe_stacks).with('StackName' => 'my_stack').and_return(@response_stub)
+
+      @cf.stack_outputs('my_stack').should == [{'key' => 'value'}]
+    end
+  end
+
+  describe "template" do
+    it "should return the Cloud Formation template for the stack" do
+      @cf_mock.should_receive(:get_template).with('my_stack').and_return(@response_stub)
+
+      @cf.template('my_stack').should == '{EIP: "string"}'
+    end
+
+    it "should trap exceptions" do
+      @cf_mock.should_receive(:get_template).with('my_stack').and_raise(@exception)
+      Stackster::AWS::CloudFormation::Error.should_receive(:new).
+        with(:config => @config_stub, :exception => @exception).and_return(@error_stub)
+
+      @cf.template('my_stack').should == 'Processed Error'
+    end
+  end
 end

data/spec/aws/ec2_spec.rb

@@ -1,7 +1,23 @@
 require 'spec_helper'
 
 describe Stackster do
+  before do
+    @config_stub = stub 'Config', :logger => @logger_stub, :access_key => 'key', :secret_key => 'XXX', :region => 'us-west1'
+    @response_stub = stub 'Excon::Response', :body => { 
+        'reservationSet' => [{'instanceSet' => [{'ipAddress' => '192.168.1.1'}]}]
+    }
 
-  it "should test describe instances"
+    @cf_mock = mock 'CloudFormation'
+    Fog::Compute::AWS.stub(:new).and_return(@cf_mock)
 
+    @ec2 = Stackster::AWS::EC2.new(:config => @config_stub)
+  end
+
+  describe "describe_instances" do
+    it "should return  the Cloud Formation description of the instances" do
+      @cf_mock.should_receive(:describe_instances).and_return(@response_stub)
+
+      @ec2.describe_instances.should == [{'instanceSet' => [{'ipAddress' => '192.168.1.1'}]}]
+    end
+  end
 end

data/spec/aws/simpledb_spec.rb

@@ -1,10 +1,72 @@
 require 'spec_helper'
 
 describe Stackster do
+  before do
+    @config_stub = stub 'Config', :logger => @logger_stub, :access_key => 'key', :secret_key => 'XXX', :region => 'us-west1'
+    @response_stub = stub 'Excon::Response', :body => { 
+      'RequestId' => 'rid',
+      'Domains' => ['domain1', 'domain2'],
+      'Items' => { 'item1' => { 'key' => ['value'] } }
+    }
 
-  ['domains', 'create_domain', 'put_attributes', 
-    'select', 'delete' ].each do |t|
-    it "should test #{t}"
+    @db_mock = mock 'SimpleDB'
+    Fog::AWS::SimpleDB.stub(:new).and_return(@db_mock)
+    @db_mock.stub(:list_domains).and_return(@response_stub)
+
+    @db = Stackster::AWS::SimpleDB.new(:config => @config_stub)
+  end
+
+  describe 'domains' do
+    it 'should return a list of domains' do
+      @db.domains.should == ['domain1', 'domain2']
+    end
+  end
+
+  describe 'domain_exists?' do
+    it 'should return true for existing domains' do
+      @db.domain_exists?('domain1').should be_true
+    end
+
+    it 'should return false for non-existent domains' do
+      @db.domain_exists?('baddomain1').should_not be_true
+    end
   end
 
+  describe 'create_domain' do
+    it 'should create a new domain' do
+      @db_mock.should_receive(:create_domain).with('newdomain').and_return(@response_stub)
+
+      @db.create_domain('newdomain').body['RequestId'].should == 'rid'
+    end
+
+    it 'should not create a duplicate domain' do
+      @db_mock.should_not_receive(:create_domain)
+
+      @db.create_domain('domain1').should be_nil
+    end
+  end
+
+  describe 'put_attributes' do
+    it 'should update the specified domain' do
+      @db_mock.should_receive(:put_attributes).with('domain1', 'item1', { 'key' => 'value' }, {}).and_return(@response_stub)
+
+      @db.put_attributes('domain1', 'item1', { 'key' => 'value' }, {}).body['RequestId'].should == 'rid'
+    end
+  end
+
+  describe 'select' do
+    it 'should return query items' do
+      @db_mock.should_receive(:select).with('item1', { "ConsistentRead" => true } ).and_return(@response_stub)
+
+      @db.select('item1').should == { 'item1' => { 'key' => ['value'] } }
+    end
+  end
+
+  describe 'delete' do
+    it 'should delete the attributes identified by domain and key' do
+      @db_mock.should_receive(:delete_attributes).with('domain1', 'item1').and_return(@response_stub)
+
+      @db.delete('domain1', 'item1').body['RequestId'].should == 'rid'
+    end
+  end
 end

data/spec/entry_spec.rb

@@ -57,14 +57,12 @@ describe Stackster do
       @entry.name.should == 'test-stack-us-west-1'
     end
 
-    it "should set the Name attribute to stack name before saving"
-
-    it "should set the attributes in simple db" do
+    it "should set the attributes in simple db including default attributes" do
       @simple_db_mock.should_receive(:put_attributes).
                       with("stacks", 
                            "test-stack-us-west-1", 
-                           { "key" => "value", "Name" => "test-stack-us-west-1" }, 
-                           { :replace => ["key", "Name"] } )
+                           { "key" => "value", "Name" => "test-stack-us-west-1", "CreatedAt" => anything }, 
+                           { :replace => ["key", "Name", "CreatedAt"] } )
       @entry.attributes = {"key"=>"value"}
       
       @entry.save

data/spec/spec_helper.rb

@@ -1,6 +1,11 @@
 require 'rubygems'
 require 'bundler/setup'
 
+require 'simplecov'
+SimpleCov.start do
+  add_filter "/spec/"
+end
+
 require 'stackster'
 
 RSpec.configure do |config|

data/spec/stack/stack_formater_spec.rb

@@ -1,10 +1,30 @@
 require 'spec_helper'
 
 describe Stackster do
+  before do
+    @logger_stub = stub 'logger stub', :info => 'true', :warn => 'true'
+    @config_stub = stub 'Config', :logger => @logger_stub, :access_key => 'key', :secret_key => 'XXX', :region => 'us-west1'
 
-  ['attributes', 'attribute_names', 'instances_private_ip_addresses',
-   'display', 'resources', 'template'].each do |t|
-    it "should test #{t}"
+    @stack_reader_mock = mock 'StackReader'
+    Stackster::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_formater = Stackster::StackFormater.new(:name => 'my_stack', :config => @config_stub)
   end
 
+  describe 'display' do
+    it 'should return formatted information for the stack' do
+      @stack_formater.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_reader_spec.rb

@@ -1,10 +1,75 @@
 require 'spec_helper'
 
 describe Stackster do
+  before do
+    @logger_stub = stub 'logger stub', :info => 'true', :warn => 'true'
+    @config_stub = stub 'Config', :logger => @logger_stub, :access_key => 'key', :secret_key => 'XXX', :region => 'us-west1'
 
-  ['attributes', 'outputs', 'status', 'events', 'resources', 
-   'instances', 'resources', 'template'].each do |t|
-    it "should test #{t}"
+    @entry_mock = mock 'Entry'
+    @entry_mock.stub(:attributes).and_return(:chef_repo_bucket_prefix => 'chef_repo_bp')
+    Stackster::Entry.stub(:new).and_return(@entry_mock)
+
+    @cf_mock = mock 'CloudFormation'
+    Stackster::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'
+    Stackster::InstanceReader.stub(:new).and_return(@instance_reader_mock)
+    @instance_reader_mock.stub(:list_stack_instances).and_return(['instance1', 'instance2'])
+
+    @stack_reader = Stackster::StackReader.new(:name => 'my_stack', :config => @config_stub)
+  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_spec.rb

@@ -26,11 +26,12 @@ describe Stackster do
 
   describe "mocking a stack object" do
     before do
-      @config_mock = mock 'config mock'
-      Stackster::Config.should_receive(:new).and_return @config_mock
+      @logger_stub = stub 'logger stub', :info => 'true', :warn => 'true', :error => 'true'
+      @config_stub = stub 'Config', :logger => @logger_stub, :access_key => 'key', :secret_key => 'XXX', :region => 'us-west-1'
+      Stackster::Config.should_receive(:new).and_return @config_stub
       @entry_mock = mock 'entry mock'
       Stackster::Entry.should_receive(:new).with(:name => 'test-stack',
-                                                 :config => @config_mock).
+                                                 :config => @config_stub).
                                             and_return @entry_mock
       @stack = Stackster::Stack.new :name => 'test-stack',
                                     :config => 'my-config',
@@ -38,7 +39,6 @@ describe Stackster do
     end
 
     context "when creating a stack" do
-
       it "should create a new stack" do
         stack_creater_mock = mock 'stack creater'
         @entry_mock.should_receive(:set_attributes).
@@ -48,7 +48,7 @@ describe Stackster do
                                 with(:name          => 'test-stack',
                                      :entry         => @entry_mock,
                                      :template_file => 'template_file',
-                                     :config        => @config_mock).
+                                     :config        => @config_stub).
                                 and_return stack_creater_mock
         stack_creater_mock.should_receive(:create)
 
@@ -56,13 +56,22 @@ describe Stackster do
                       :attributes => { 'test-attr' => 'test-value' }
       end
 
-      it "should not create a stack entry if the create fails"
-
+      it "should not create a stack entry if the create fails" do
+        @entry_mock.should_receive(:set_attributes).
+                    with({ 'test-attr' => 'test-value' })
+        @entry_mock.should_not_receive(:save)
+
+        begin
+          @stack.create :template   => 'template_file',
+                        :attributes => { 'test-attr' => 'test-value' }
+          fail "should have raised Errno::ENOENT"
+        rescue Errno::ENOENT
+        end
+      end
     end
 
     context "when updating a stack" do
-
-      it "should should update an existing stack" do
+      it "should update an existing stack" do
         stack_updater_mock = mock 'stack updater'
         stack_reader_mock = mock 'stack reader'
         @entry_mock.should_receive(:set_attributes).
@@ -72,20 +81,47 @@ describe Stackster do
 
         Stackster::StackReader.should_receive(:new).
                                with(:name   => 'test-stack',
-                                    :config => @config_mock).
+                                    :config => @config_stub).
                                and_return stack_reader_mock
         Stackster::StackUpdater.should_receive(:new).
                                 with(:name          => 'test-stack',
                                      :entry         => @entry_mock,
                                      :template_body => 'template-body-json',
-                                     :config        => @config_mock).
+                                     :config        => @config_stub).
                                 and_return stack_updater_mock
         stack_updater_mock.should_receive(:update_stack_if_parameters_changed).
                            and_return true
         @stack.update :attributes => { 'update' => 'test-attrs' }
       end
 
-      it "should not update a stack entry if the create fails"
+      it "should not update a stack entry if the update fails" do
+        @entry_mock.should_receive(:set_attributes).
+                    with({ 'update' => 'test-attrs' })
+        @entry_mock.should_not_receive(:save)
+
+        stack_reader_mock = mock 'stack reader'
+        stack_reader_mock.should_receive(:template).and_return('template-body-json')
+        Stackster::StackReader.should_receive(:new).
+                               with(:name   => 'test-stack',
+                                    :config => @config_stub).
+                               and_return stack_reader_mock
+
+        stack_updater_mock = mock 'stack updater'
+        Stackster::StackUpdater.should_receive(:new).
+                                with(:name          => 'test-stack',
+                                     :entry         => @entry_mock,
+                                     :template_body => 'template-body-json',
+                                     :config        => @config_stub).
+                                and_return stack_updater_mock
+        stack_updater_mock.should_receive(:update_stack_if_parameters_changed).
+          and_raise('test-stack has not reached a stable state')
+
+        begin
+          @stack.update :attributes => { 'update' => 'test-attrs' }
+          fail 'should have raised RuntimeException'
+        rescue RuntimeError
+        end
+      end
     end
 
     context "when destroying a stack" do
@@ -93,20 +129,97 @@ describe Stackster do
         stack_destroyer_mock = mock 'stack destroyer'  
         Stackster::StackDestroyer.should_receive(:new).
                                   with(:name   => 'test-stack',
-                                       :config => @config_mock).
+                                       :config => @config_stub).
                                   and_return stack_destroyer_mock
         stack_destroyer_mock.should_receive(:destroy)
         @entry_mock.should_receive(:delete_attributes)
         @stack.destroy
       end
     end
+  end
+
+  describe 'wrapper methods' do
+    before do
+      @logger_stub = stub 'logger stub', :info => 'true', :warn => 'true', :error => 'true'
+      @config_stub = stub 'Config', :logger => @logger_stub, :access_key => 'key', :secret_key => 'XXX', :region => 'us-west-1'
+      @entry_stub = stub 'Entry'
+      Stackster::Config.should_receive(:new).and_return(@config_stub)
+      Stackster::Entry.should_receive(:new).and_return(@entry_stub)
+
+      @stack = Stackster::Stack.new :name => 'test-stack',
+                                    :config => @config_stub,
+                                    :logger => @logger_stub
+    end
 
-    ['attributes', 'display', 'show', 'outputs', 
-     'status', 'events', 'instances', 'instances_public_ip_addresses',
-     'instances_private_ip_addresses', 'resources', 'template'].each do |t|
-      it "should test #{t}"
+    context "formater" do
+      before do
+        @stack_formater_mock = mock 'StackFormater'
+        Stackster::StackFormater.should_receive(:new).and_return(@stack_formater_mock)
+        @stack_formater_mock.stub(:display).and_return({
+          'attributes' => { :chef_repo_bucket_prefix => 'chef_repo_bp' },
+          'status' => 'green',
+          'outputs' => [{'key' => 'value'}],
+          'events' => ['event1', 'event2', 'event3'],
+          'resources' => [{'StackName' => 'test-stack'}]
+        })
+      end
+
+      it 'should display formatted stack attributes' do
+        @stack.display.should == {
+          'attributes' => { :chef_repo_bucket_prefix => 'chef_repo_bp' },
+          'status' => 'green',
+          'outputs' => [{'key' => 'value'}],
+          'events' => ['event1', 'event2', 'event3'],
+          'resources' => [{'StackName' => 'test-stack'}]
+        }
+      end
     end
 
-  end
+    context 'reader' do
+      before do
+        @stack_reader_mock = mock 'StackReader'
+        Stackster::StackReader.should_receive(: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_reader_mock.stub(:template).and_return('{"Parameters": {"EIP": "string"}}')
+        @stack_reader_mock.stub(:parameters).and_return(['EIP'])
+        @stack_reader_mock.stub(:instances).and_return(['instance1', 'instance2'])
+      end
+
+      it 'should return the stack attributes' do
+        @stack.attributes.should == { :chef_repo_bucket_prefix => 'chef_repo_bp' }
+      end
+
+      it 'should return the stack outputs' do
+        @stack.outputs.should == [{'key' => 'value'}]
+      end
 
+      it 'should return the stack status' do
+        @stack.status.should == 'green'
+      end
+
+      it 'should return the stack events' do
+        @stack.events(3).should == ['event1', 'event2', 'event3']
+      end
+
+      it 'should return the stack resources' do
+        @stack.resources.should == [{'StackName' => 'my_stack'}]
+      end
+
+      it 'should return the stack template' do
+        @stack.template.should == '{"Parameters": {"EIP": "string"}}'
+      end
+
+      it 'should return the stack parameters' do
+        @stack.parameters.should == ['EIP']
+      end
+
+      it 'should return the stack instances' do
+        @stack.instances.should == ['instance1', 'instance2']
+      end
+    end
+  end
 end

data/stackster.gemspec

@@ -20,6 +20,7 @@ Gem::Specification.new do |s|
 
   # specify any dependencies here; for example:
   s.add_development_dependency "rspec"
+  s.add_development_dependency "simplecov"
 
   s.add_runtime_dependency "fog"
   s.add_runtime_dependency 'logger'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stackster
 version: !ruby/object:Gem::Version
-  version: 0.2.9
+  version: 0.3.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-07-23 00:00:00.000000000 Z
+date: 2012-10-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70160791318120 !ruby/object:Gem::Requirement
+  requirement: &2160251380 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,21 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70160791318120
+  version_requirements: *2160251380
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: &2160250100 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *2160250100
 - !ruby/object:Gem::Dependency
   name: fog
-  requirement: &70160791317340 !ruby/object:Gem::Requirement
+  requirement: &2160248940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70160791317340
+  version_requirements: *2160248940
 - !ruby/object:Gem::Dependency
   name: logger
-  requirement: &70160791315460 !ruby/object:Gem::Requirement
+  requirement: &2160247880 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +54,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70160791315460
+  version_requirements: *2160247880
 - !ruby/object:Gem::Dependency
   name: trollop
-  requirement: &70160791312720 !ruby/object:Gem::Requirement
+  requirement: &2160246820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +65,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70160791312720
+  version_requirements: *2160246820
 - !ruby/object:Gem::Dependency
   name: xml-simple
-  requirement: &70160791309680 !ruby/object:Gem::Requirement
+  requirement: &2160244280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,7 +76,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70160791309680
+  version_requirements: *2160244280
 description: Thats what I do
 email:
 - brett@weav.net
@@ -135,21 +146,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 107402892190402761
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 107402892190402761
 requirements: []
 rubyforge_project: stackster
-rubygems_version: 1.8.16
+rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
 summary: I make deployments easier