stackster 0.1.3 → 0.2.0

data/README.md

@@ -28,6 +28,10 @@ stackster create -n STACK_NAME -a ATTRIBUTES -t TEMPLATE_PATH
 stackster update -n STACK_NAME -a ATTRIBUTES
 stackster show -n STACK_NAME
 stackster destroy -n STACK_NAME
+
+Attributes are specified as '=' seperated key value pairs.  Multiple can be specified.  For example:
+
+stackster create -n test-stack -t ~/my-template.json -a arg1=val1 arg2=vol2
 ```
 
 For more information, run stackster -h.

data/lib/stackster/cli.rb

@@ -14,9 +14,12 @@ stackster update -n STACK_NAME -a ATTRIBUTES
 stackster show -n STACK_NAME
 stackster destroy -n STACK_NAME
 
+Attributes are specified as '=' seperated key value pairs.  Multiple can be specified.  For example:
+
+stackster create -n test-stack -t ~/my-template.json -a arg1=val1 arg2=vol2
 EOS
         opt :help, "Display Help"
-        opt :attributes, "CSV list of = seperated attributes to set", :type => :string
+        opt :attributes, "CSV list of = seperated attributes to set", :type => :strings
         opt :name, "Stack name to manage", :type => :string
         opt :template, "Path to the template file", :type => :string
       end
@@ -29,8 +32,6 @@ EOS
                        :config => nil
       end
 
-      read_attributes
-
       case @cmd
       when 'create'
         @s.create :attributes => attributes,
@@ -45,12 +46,14 @@ EOS
       when 'show'
         puts @s.display.to_yaml
       when 'list'
-        puts Stack.list(:config => nil)
+        puts StackLister.new.all
       else
         puts "Unkown command '#{@cmd}'"
       end
     end
 
+    private
+
     def self.attributes
       attrs = []
       read_attributes.each do |attribs|
@@ -61,7 +64,7 @@ EOS
     end
 
     def self.read_attributes
-      @opts[:attributes].nil? ? [] :  @opts[:attributes].split(',')
+      @opts[:attributes].nil? ? [] : @opts[:attributes]
     end
 
   end

data/lib/stackster/config.rb

@@ -1,25 +1,34 @@
+require 'logger'
+
 module Stackster
   class Config
 
     def initialize(args = {})
-      @config = args[:config]
-    end
-
-    def load_config_file
-      config_file = "#{ENV['HOME']}/.stackster.yml"
-      @config ? @config : (YAML::load File.open( config_file ))
+      @config = load_config_file args[:config]
+      @logger = args[:logger] ||= StacksterLogger.new
     end
 
     def access_key
-      load_config_file['access_key']
+      @config['access_key']
     end
 
     def secret_key
-      load_config_file['secret_key']
+      @config['secret_key']
     end
 
     def region
-      load_config_file['region']
+      @config['region']
+    end
+
+    def logger
+      @logger
+    end
+
+    private
+
+    def load_config_file(config = nil)
+      config_file = "#{ENV['HOME']}/.stackster.yml"
+      config ? config : (YAML::load File.open( config_file ))
     end
 
   end

data/lib/stackster/entry/entry_lister.rb

@@ -4,16 +4,21 @@ module Stackster
     def initialize(args)
       @domain = 'stacks'
       @config = args[:config]
-      @sdb_connect = AWS::SimpleDB.new :config => args[:config]
     end
 
     def all
-      entries = @sdb_connect.select "select * from #{@domain}"
+      entries = sdb_connect.select "select * from #{@domain}"
       entries.keys.map do |name|
         remove_region_from_entry(name)
       end
     end
 
+    private
+
+    def sdb_connect
+      @sdb_connect ||= AWS::SimpleDB.new :config => @config
+    end
+
     def remove_region_from_entry(name)
       name.gsub(/-[a-z]{2}-[a-z]*-[0-9]{1,2}$/, '')
     end

data/lib/stackster/entry.rb

@@ -7,53 +7,54 @@ module Stackster
     def initialize(args)
       @domain = 'stacks'
       @config = args[:config]
-      self.name = "#{args[:name]}-#{@config.region}"
+      self.name = region_specific_name args[:name]
       get_attributes
     end
 
     def self.find(args)
-      e = Entry.new :name => args[:name],
-                    :config => args[:config]
-      e.get_attributes
-      e
+      entry = Entry.new :name   => args[:name],
+                        :config => args[:config]
+      entry
     end
 
-    def create_domain
-      sdb_connect.create_domain @domain
+    def set_attributes(a)
+      a.each { |attribute| set_attribute(attribute) }
+    end
+
+    def save
+      create_domain
+      attributes.each_pair do |k,v|
+        sdb_connect.put_attributes('stacks', name, { k => v }, { :replace => k })
+      end
+    end
+
+    def delete_attributes
+      sdb_connect.delete('stacks', name)
     end
 
+    private
+
     def get_attributes
       u = {}
       attrs = sdb_connect.select("select * from stacks where itemName() = '#{name}'")
       if attrs[name]
-        attrs[name].each_pair do |k, v|
-          u[k] = v.first
-        end
+        attrs[name].each_pair { |k, v| u[k] = v.first }
       end
       self.attributes = u
     end
 
-    def save
-      attributes.each_pair do |k,v|
-        sdb_connect.put_attributes('stacks', name, { k => v }, { :replace => k })
-      end
+    def region_specific_name(name)
+      "#{name}-#{@config.region}"
     end
 
-    def set_attributes(a)
-      a.each { |attribute| set_attribute(attribute) }
+    def create_domain
+      sdb_connect.create_domain @domain
     end
 
     def set_attribute(attribute)
-      create_domain
       self.attributes = attributes.merge(attribute)
     end
 
-    def delete_attributes
-      sdb_connect.delete('stacks', name)
-    end
-
-    private
-
     def sdb_connect
       @sdb_connect ||= AWS::SimpleDB.new :config => @config
     end

data/lib/stackster/instance/instance_reader.rb

@@ -2,18 +2,20 @@ module Stackster
   class InstanceReader
 
     def initialize(args)
-      @ec2 = AWS::EC2.new :config => args[:config]
+      @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|
         tag_set = instance['instancesSet'].first['tagSet']
-        if tag_set['aws:cloudformation:stack-name']
-          if tag_set['aws:cloudformation:stack-name'] == stack_name
-            h << instance if instance_running? instance
+        instance_stack_name = tag_set['aws:cloudformation:stack-name']
+        if instance_stack_name && instance_stack_name == stack_name
+          if instance_running? instance
+            h << instance
           end
         end
       end
@@ -22,6 +24,10 @@ module Stackster
 
     private
 
+    def ec2
+      @ec2 ||= AWS::EC2.new :config => @config
+    end
+
     def get_instance_state(instance_hash)
       instance_hash['instancesSet'].first['instanceState']['name']
     end
@@ -31,7 +37,7 @@ module Stackster
     end
 
     def describe_instances
-      @ec2.describe_instances
+      ec2.describe_instances
     end
 
   end

data/lib/stackster/logger.rb

@@ -0,0 +1,21 @@
+module Stackster
+  class StacksterLogger
+
+    def initialize(args = {})
+      @logger = args[:logger] ||= Logger.new(STDOUT)
+
+      unless args[:logger]
+        @logger.datetime_format = "%Y-%m-%d %H:%M:%S"
+        @logger.formatter = proc do |severity, datetime, progname, msg|
+            "#{datetime}: #{msg}\n"
+        end
+      end
+
+      @logger
+    end
+
+    def info(msg)
+      @logger.info msg
+    end
+  end
+end

data/lib/stackster/stack/stack_creater.rb

@@ -8,7 +8,18 @@ module Stackster
       @entry = args[:entry]
       @name = args[:name]
       @template = read_template_from_file args[:template_file]
-      @cf = AWS::CloudFormation.new :config => @config
+    end
+
+    def create
+      cloud_formation.create :name => @name,
+                             :parameters => read_parameters_from_entry,
+                             :template => @template
+    end
+
+    private
+
+    def cloud_formation
+      @cf ||= AWS::CloudFormation.new :config => @config
     end
 
     def read_template_from_file(template_file)
@@ -23,17 +34,12 @@ module Stackster
 
     def read_parameters_from_entry
       h = {}
+      attributes = @entry.attributes
       read_parameters_from_template.each do |p|
-        h[p] = @entry.attributes[p] if @entry.attributes[p]
+        h[p] = attributes[p] if attributes[p]
       end
       h
     end
 
-    def create
-      @cf.create :name => @name,
-                 :parameters => read_parameters_from_entry,
-                 :template => @template
-    end
-
   end
 end

data/lib/stackster/stack/stack_destroyer.rb

@@ -4,11 +4,16 @@ module Stackster
     def initialize(args)
       @config = args[:config]
       @name = args[:name]
-      @cf = AWS::CloudFormation.new :config => @config
     end
 
     def destroy
-      @cf.destroy @name
+      cloud_formation.destroy @name
+    end
+
+    private
+
+    def cloud_formation
+      @cf ||= AWS::CloudFormation.new :config => @config
     end
 
   end

data/lib/stackster/stack/stack_formater.rb

@@ -13,38 +13,9 @@ module Stackster
         'outputs'         => stack_reader.outputs,
         'events'          => stack_reader.events,
         'resources'       => stack_reader.resources,
-        'instances'       => instances_private_ip_addresses
       }
     end
 
-    def instances_private_ip_addresses
-      stack_reader.instances.map do |i| 
-        i['instancesSet'].first['privateIpAddress'] if i['instancesSet'].first['privateIpAddress']
-      end
-    end
-
-    def attribute_names
-      attributes.keys
-    end
-
-    def attributes
-      stack_reader.attributes
-    end
-
-    def resources_summary
-      stack_reader.resources.map do |i| 
-        { 'LogicalResourceId' => i['LogicalResourceId'],
-          'PhysicalResourceId' => i['PhysicalResourceId'] }
-      end
-    end
-
-    def events_summary
-      stack_reader.events.map do |i| 
-        { 'ResourceStatus' => i['ResourceStatus'],
-          'LogicalResourceId' => i['LogicalResourceId'] }
-      end
-    end
-
     private
 
     def stack_reader

data/lib/stackster/stack/stack_lister.rb

@@ -1,13 +1,18 @@
 module Stackster
   class StackLister
 
-    def initialize(args={})
-      @config = args[:config]
+    def initialize(args = {})
+      @config = args[:config] ||= Config.new
     end
 
     def all
-      e = EntryLister.new :config => @config
-      e.all
+      entry_lister.all
+    end
+
+    private
+
+    def entry_lister
+      @entry_lister ||= EntryLister.new :config => @config
     end
 
   end

data/lib/stackster/stack/stack_reader.rb

@@ -4,38 +4,53 @@ module Stackster
     def initialize(args)
       @name = args[:name]
       @config = args[:config]
-      @cf = AWS::CloudFormation.new :config => @config
-      @entry = Entry.find :name => @name,
-                          :config => @config
     end
 
     def attributes
-      @entry.attributes
+      entry.attributes
     end
 
     def outputs
-      @cf.stack_outputs @name
+      cloud_formation.stack_outputs @name
     end
 
     def status
-      @cf.stack_status @name
+      cloud_formation.stack_status @name
     end
 
     def events
-      @cf.stack_events @name
+      cloud_formation.stack_events @name
     end
 
     def resources
-      @cf.stack_resources @name
+      cloud_formation.stack_resources @name
     end
 
     def template
-      @cf.template @name
+      cloud_formation.template @name
+    end
+
+    def parameters
+      json = JSON.parse template
+      json['Parameters'].nil? ? [] : json['Parameters'].keys
     end
 
     def instances
-      ir = InstanceReader.new :config => @config
-      ir.list_stack_instances @name
+      instance_reader.list_stack_instances @name
+    end
+
+    private
+
+    def entry
+      @entry ||= Entry.find :name => @name, :config => @config
+    end
+
+    def cloud_formation
+      @cloud_formation ||= AWS::CloudFormation.new :config => @config
+    end
+
+    def instance_reader
+      @instance_reader ||= InstanceReader.new :config => @config
     end
 
   end

data/lib/stackster/stack/stack_updater.rb

@@ -8,26 +8,44 @@ module Stackster
       @entry = args[:entry]
       @name = args[:name]
       @template_body = args[:template_body]
-      @cf = AWS::CloudFormation.new :config => @config
     end
 
+    def update_stack_if_parameters_changed(attributes)
+      parameter_updated?(attributes) ? update : false
+    end
+
+    private
+
     def update
-      @cf.update :name => @name,
-                 :parameters => read_parameters_from_entry,
-                 :template => @template_body
+      cloud_formation.update :name => @name,
+                             :parameters => read_parameters_from_entry_attributes,
+                             :template => @template_body
+    end
+
+    def parameter_updated?(attributes)
+      (template_parameters - updated_parameters(attributes)) != template_parameters
+    end
+
+    def template_parameters
+      json = JSON.parse @template_body
+      json['Parameters'].nil? ? [] : json['Parameters'].keys
+    end
+
+    def updated_parameters attributes
+      (attributes.map { |s| s.keys }).flatten
     end
 
-    def read_parameters_from_entry
+    def read_parameters_from_entry_attributes
       h = {}
-      read_parameters_from_template.each do |p|
-        h[p] = @entry.attributes[p] if @entry.attributes[p]
+      entry_attributes = @entry.attributes
+      template_parameters.each do |p|
+        h[p] = entry_attributes[p] if entry_attributes[p]
       end
       h
     end
 
-    def read_parameters_from_template
-      t = JSON.parse @template_body
-      t['Paramaters'].nil? ? t['Parameters'].keys : []
+    def cloud_formation
+      @cloud_formation ||= AWS::CloudFormation.new :config => @config
     end
 
   end

data/lib/stackster/stack.rb

@@ -9,52 +9,45 @@ module Stackster
   class Stack
 
     def initialize(args)
-      @config = Config.new :config => args[:config]
       @name = args[:name]
-      @entry = Entry.new :name => @name,
+      @config = Config.new :config => args[:config],
+                           :logger => args[:logger]
+      @entry = Entry.new :name   => @name,
                          :config => @config
     end
 
-    def self.list(args)
-      @config = Config.new :config => args[:config]
-      stack_lister.all
-    end
-
+    # ToDo - maybe in stack creater
+    # Rescure stack creation errors and verify entry is not saved
     def create(args)
       @template_file = args[:template]
       @entry.set_attributes args[:attributes]
-      @entry.save
       stack_creater.create
+      @entry.save
     end
 
+    # To Do - maybe in stack udpater
+    # Rescure stack creation errors and verify entry is not saved
     def update(args)
       @template_body = template
       @entry.set_attributes args[:attributes]
+      stack_updater.update_stack_if_parameters_changed args[:attributes]
       @entry.save
-      update_stack_if_parameters_changed args[:attributes]
-    end
-
-    def update_stack_if_parameters_changed(updated_parameters)
-      current_parameters = JSON.parse(template)['Parameters'].keys
-      updated_parameters = (updated_parameters.map { |s| s.keys }).flatten
-
-      if (current_parameters - updated_parameters) != current_parameters
-        stack_updater.update
-      end
     end
 
+    # To do - maybe in stack destroyer
+    # Rescue stack creation exception and only delete on success
     def destroy
       stack_destroyer.destroy
       @entry.delete_attributes
     end
-    
-    def attributes
-      stack_formater.attributes
-    end
 
     def display
       stack_formater.display
     end
+    
+    def attributes
+      stack_reader.attributes
+    end
 
     def show
       stack_reader.show
@@ -76,14 +69,6 @@ module Stackster
       stack_reader.instances
     end
 
-    def instances_public_ip_addresses
-      stack_formater.instances_public_ip_addresses
-    end
-
-    def instances_private_ip_addresses
-      stack_formater.instances_private_ip_addresses
-    end
-
     def resources
       stack_reader.resources
     end
@@ -92,6 +77,10 @@ module Stackster
       stack_reader.template
     end
 
+    def parameters
+      stack_reader.parameters
+    end
+
     private
 
     def stack_creater
@@ -123,9 +112,5 @@ module Stackster
                                             :config => @config
     end
 
-    def self.stack_lister
-      @stack_lister ||= StackLister.new :config => @config
-    end
-
   end
 end

data/lib/stackster/version.rb

@@ -1,3 +1,3 @@
 module Stackster
-  VERSION = "0.1.3"
+  VERSION = "0.2.0"
 end

data/lib/stackster.rb

@@ -2,5 +2,6 @@ require "stackster/aws"
 require "stackster/entry"
 require "stackster/stack"
 require "stackster/instance"
+require "stackster/logger"
 require "stackster/config"
 require "stackster/version"

data/spec/aws/cloud_formation_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+describe Stackster do
+
+  ['create', 'update', 'destroy', 'describe_stack', 'stack_resources', 
+   'stack_events', 'stack_status', 'stack_outputs', 'template'].each do |t|
+    it "should test #{t}"
+  end
+
+end

data/spec/aws/ec2_spec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+describe Stackster do
+
+  it "should test describe instances"
+
+end

data/spec/aws/simpledb_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+describe Stackster do
+
+  ['domains', 'create_domain', 'put_attributes', 
+    'select', 'delete' ].each do |t|
+    it "should test #{t}"
+  end
+
+end

data/spec/config_spec.rb

@@ -1,11 +1,38 @@
 require 'spec_helper'
 
-describe Config do
+describe Stackster do
 
-  it "should create a new config object" do
+  before do
+    @config = { 'access_key' => 'the-key', 
+                'secret_key' => 'secret',
+                'region'     => 'us-west-1' }
+  end
+
+  it "should create a new config object from the hash passed as config" do
+    config = Stackster::Config.new :config => @config
+    config.access_key.should == @config['access_key']
+    config.secret_key.should == @config['secret_key']
+    config.region.should == @config['region']
+  end
+
+  it "should create a new config object and read from ~/.stackster.yml" do
+    File.should_receive(:open).with("#{ENV['HOME']}/.stackster.yml").
+                               and_return(@config.to_yaml)
     config = Stackster::Config.new
-    config.class.should == Stackster::Config
+    config.access_key.should == @config['access_key']
+    config.secret_key.should == @config['secret_key']
+    config.region.should == @config['region']
   end
 
-end
+  it "should create a new logger when one is not specified" do
+    config = Stackster::Config.new :config => @config
+    config.logger.class.should == Stackster::StacksterLogger
+  end
+
+  it "should accept a logger passed as :logger" do
+    config = Stackster::Config.new :config => @config,
+                                   :logger => 'the-logger'
+    config.logger.should == 'the-logger'
+  end
 
+end

data/spec/entry/entry_lister_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+
+describe Stackster do
+
+  it "should create a list of entries" do
+    @simple_db_mock = mock 'simple db'
+    config = Stackster::Config.new
+    Stackster::AWS::SimpleDB.should_receive(:new).and_return @simple_db_mock
+    @simple_db_mock.should_receive(:select).
+                    with("select * from stacks").
+                    and_return('stack-to-find-us-west-1' => { 'attr1' => 'value1' })
+    entry_lister = Stackster::EntryLister.new :config => config
+    entry_lister.all.should == ['stack-to-find']
+  end
+
+end

data/spec/entry_spec.rb

@@ -0,0 +1,61 @@
+require 'spec_helper'
+
+describe Stackster do
+
+  it "should create a new entry object" do
+    @simple_db_mock = mock 'simple db'
+    config = Stackster::Config.new
+    Stackster::AWS::SimpleDB.should_receive(:new).and_return @simple_db_mock
+    @simple_db_mock.should_receive(:select).
+                    with("select * from stacks where itemName() = 'test-stack-us-west-1'").
+                    and_return('stack-to-find' => { 'attr1' => 'value1' })
+    entry = Stackster::Entry.new :config => config,
+                                 :name   => 'test-stack'
+    entry.class.should == Stackster::Entry
+  end
+  
+  it "should find the requested stack in simple db" do
+    @simple_db_mock = mock 'simple db'
+    Stackster::AWS::SimpleDB.should_receive(:new).and_return @simple_db_mock
+
+    @simple_db_mock.should_receive(:select).
+                    with("select * from stacks where itemName() = 'stack-to-find-us-west-1'").
+                    and_return('stack-to-find' => { 'attr1' => 'value1' })
+    Stackster::Entry.find :name   => 'stack-to-find',
+                          :config => Stackster::Config.new
+  end
+
+  context "with stack object" do
+    before do
+      @config = Stackster::Config.new
+      @simple_db_mock = mock 'simple db'
+      Stackster::AWS::SimpleDB.should_receive(:new).and_return @simple_db_mock
+      @simple_db_mock.should_receive(:select).
+                      with("select * from stacks where itemName() = 'test-stack-us-west-1'").
+                      and_return('test-stack' => { 'attr1' => 'value1' })
+      @entry = Stackster::Entry.new :config => @config,
+                                    :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" do
+      @simple_db_mock.should_receive(:create_domain).with('stacks')
+      @simple_db_mock.should_receive(:put_attributes).
+                      with("stacks", "test-stack-us-west-1", {"key"=>"value"}, {:replace=>"key"})
+      @entry.attributes = {"key"=>"value"}
+      
+      @entry.save
+    end
+
+    it "should set the attributes" do
+      @entry.attributes = {'key1' => 'value1' }
+      @entry.set_attributes(['key2' => 'value2'])
+      @entry.attributes.should == {'key1' => 'value1', 'key2' => 'value2' }
+    end
+
+  end
+
+end

data/spec/instance/instance_reader_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+
+describe Stackster do
+
+
+  describe "with a single running instance" do
+    before do
+      @instances = [
+                     { 'instancesSet' => 
+                       [
+                         { 'tagSet' => 
+                           { 'aws:cloudformation:stack-name' => 'my-stack' },
+                           'instanceState' =>
+                           { 'name' => 'running' }
+                         }
+                       ]
+                     },
+                     { 'instancesSet' =>
+                       [
+                         { 'tagSet' =>
+                           { 'aws:cloudformation:stack-name' => 'my-stack' },
+                           'instanceState' =>
+                           { 'name' => 'terminated' }
+                         }
+                       ]
+                     },
+                     { 'instancesSet' =>
+                       [
+                         { 'tagSet' =>
+                           { 'aws:cloudformation:stack-name' => 'dead-stack' },
+                           'instanceState' =>
+                           { 'name' => 'terminated' }
+                         }
+                       ]
+                     }
+                   ]
+    end
+
+    it "should list running instance within the given stack" do
+      @ec2_mock = mock 'simple ec2'
+      config = Stackster::Config.new
+      Stackster::AWS::EC2.should_receive(:new).
+                          with(:config => config).
+                          and_return @ec2_mock
+      instance_reader = Stackster::InstanceReader.new :config => config
+      @ec2_mock.should_receive(:describe_instances).
+                and_return(@instances)
+      instance_reader.list_stack_instances('my-stack').count.
+                      should == 1
+    end
+
+    it "should not include instances which are in different stacks" do
+      @ec2_mock = mock 'simple ec2'
+      config = Stackster::Config.new
+      Stackster::AWS::EC2.should_receive(:new).
+                          with(:config => config).
+                          and_return @ec2_mock
+      instance_reader = Stackster::InstanceReader.new :config => config
+      @ec2_mock.should_receive(:describe_instances).
+                and_return(@instances)
+      instance_reader.list_stack_instances('another-stack').count.
+                      should == 0
+    end
+
+    it "should not include instances which are not running" do
+      @ec2_mock = mock 'simple ec2'
+      config = Stackster::Config.new
+      Stackster::AWS::EC2.should_receive(:new).
+                          with(:config => config).
+                          and_return @ec2_mock
+      instance_reader = Stackster::InstanceReader.new :config => config
+      @ec2_mock.should_receive(:describe_instances).
+                and_return(@instances)
+      instance_reader.list_stack_instances('dead-stack').count.
+                      should == 0
+    end
+  end
+end

data/spec/logger_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+
+describe Stackster do
+
+  it "should create a new logger object from the hash passed as :logger" do
+    logger_mock = mock 'logger'
+    logger_mock.should_receive(:info).with 'a message'
+    logger = Stackster::StacksterLogger.new :logger => logger_mock
+    logger.info 'a message'
+  end
+
+  it "should create a new logger object when one is not passed" do
+    logger_mock = mock 'logger'
+    Logger.should_receive(:new).with(STDOUT).and_return logger_mock
+    logger_mock.should_receive(:info).with 'a message'
+    logger = Stackster::StacksterLogger.new
+    logger.info 'a message'
+  end
+
+end

data/spec/stack/stack_creater_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+require 'json'
+
+describe Stackster do
+
+  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
+    config_mock = mock 'config mock'
+    entry_mock = mock 'entry mock'
+    file_mock = mock 'file mock'
+    cloud_formation_mock = mock 'cloud formation mock'
+    Stackster::AWS::CloudFormation.should_receive(:new).
+                                   with(:config => config_mock).
+                                   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_creater = Stackster::StackCreater.new :name          => 'test-stack',
+                                                :template_file => 'path_to_file',
+                                                :entry         => entry_mock,
+                                                :config        => config_mock
+
+    stack_creater.create
+  end
+
+end

data/spec/stack/stack_destroyer_spec.rb

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

data/spec/stack/stack_formater_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+describe Stackster do
+
+  ['attributes', 'attribute_names', 'instances_private_ip_addresses',
+   'display', 'resources', 'template'].each do |t|
+    it "should test #{t}"
+  end
+
+end

data/spec/stack/stack_lister_spec.rb

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

data/spec/stack/stack_reader_spec.rb

@@ -0,0 +1,10 @@
+require 'spec_helper'
+
+describe Stackster do
+
+  ['attributes', 'outputs', 'status', 'events', 'resources', 
+   'instances', 'resources', 'template'].each do |t|
+    it "should test #{t}"
+  end
+
+end

data/spec/stack/stack_updater_spec.rb

@@ -0,0 +1,58 @@
+require 'spec_helper'
+require 'json'
+
+describe Stackster do
+
+  before do
+    @template_body = '{ "Parameters": 
+                        { 
+                          "param1" : 
+                            {
+                              "Description" : "param-1"
+                            },
+                          "param2" : 
+                            {
+                              "Description" : "param-2"
+                            }
+                        }
+                      }'
+  end
+
+  it "should update the stack when parameters change" do
+    attributes = { "param1" => "value1", "param3" => "value3" }
+    config_mock = mock 'config mock'
+    entry_mock = mock 'entry mock'
+    cloud_formation_mock = mock 'cloud formation mock'
+    Stackster::AWS::CloudFormation.should_receive(:new).
+                                   with(:config => config_mock).
+                                   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
+    stack_updater = Stackster::StackUpdater.new :name          => 'test-stack',
+                                                :template_body => @template_body,
+                                                :entry         => entry_mock,
+                                                :config        => config_mock
+
+    stack_updater.update_stack_if_parameters_changed( [ { 'param1' => 'new-value' } ] ).
+                  should == true
+  end
+
+  it "should not update the stack when parameters don't change" do
+    attributes = { "param3" => "value3" }
+    config_mock = mock 'config mock'
+    entry_mock = mock 'entry mock'
+    Stackster::AWS::CloudFormation.should_receive(:new).exactly(0).times
+    stack_updater = Stackster::StackUpdater.new :name          => 'test-stack',
+                                                :template_body => @template_body,
+                                                :entry         => entry_mock,
+                                                :config        => config_mock
+
+    stack_updater.update_stack_if_parameters_changed( [ { 'another-param' => 'new-value' } ] ).
+                  should == false
+  end
+
+end

data/spec/stack_spec.rb

@@ -0,0 +1,93 @@
+require 'spec_helper'
+
+describe Stackster do
+
+  it "create a new stack object" do
+    config_mock = mock 'config mock'
+    Stackster::Config.should_receive(:new).and_return config_mock
+    Stackster::Entry.should_receive(:new).with :name => 'test-stack',
+                                               :config => config_mock
+    stack = Stackster::Stack.new :name => 'test-stack'
+    stack.class.should == Stackster::Stack
+  end
+
+  describe "mocking a stack object" do
+    before do
+      @config_mock = mock 'config mock'
+      Stackster::Config.should_receive(:new).and_return @config_mock
+      @entry_mock = mock 'entry mock'
+      Stackster::Entry.should_receive(:new).with(:name => 'test-stack',
+                                                 :config => @config_mock).
+                                            and_return @entry_mock
+      @stack = Stackster::Stack.new :name => 'test-stack',
+                                    :config => 'my-config',
+                                    :logger => 'my-logger'
+    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).with({ 'test-attr' => 'test-value' })
+        @entry_mock.should_receive(:save)
+        Stackster::StackCreater.should_receive(:new).with(:name          => 'test-stack',
+                                                          :entry         => @entry_mock,
+                                                          :template_file => 'template_file',
+                                                          :config        => @config_mock).
+                                          and_return stack_creater_mock
+        stack_creater_mock.should_receive(:create)
+
+        @stack.create :template   => 'template_file',
+                      :attributes => { 'test-attr' => 'test-value' }
+      end
+
+      it "should not create a stack entry if the create fails"
+
+    end
+
+    context "when updating a stack" do
+
+      it "should should update an existing stack" do
+        stack_updater_mock = mock 'stack updater'
+        stack_reader_mock = mock 'stack reader'
+        @entry_mock.should_receive(:set_attributes).
+                    with({ 'update' => 'test-attrs' })
+        @entry_mock.should_receive(:save)
+        stack_reader_mock.should_receive(:template).and_return('template-body-json')
+
+        Stackster::StackReader.should_receive(:new).with(:name   => 'test-stack',
+                                                         :config => @config_mock).
+                                                    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).
+                                                     and_return stack_updater_mock
+        stack_updater_mock.should_receive(:update_stack_if_parameters_changed)
+        @stack.update :attributes => { 'update' => 'test-attrs' }
+      end
+
+      it "should not update a stack entry if the create fails"
+    end
+
+    context "when destroying a stack" do
+      it "should destroy a stack" do
+        stack_destroyer_mock = mock 'stack destroyer'  
+        Stackster::StackDestroyer.should_receive(:new).with(:name   => 'test-stack',
+                                                            :config => @config_mock).
+                                                       and_return stack_destroyer_mock
+        stack_destroyer_mock.should_receive(:destroy)
+        @entry_mock.should_receive(:delete_attributes)
+        @stack.destroy
+      end
+    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}"
+    end
+
+  end
+
+end

data/stackster.gemspec

@@ -22,5 +22,6 @@ Gem::Specification.new do |s|
   s.add_development_dependency "rspec"
 
   s.add_runtime_dependency "fog"
+  s.add_runtime_dependency 'logger'
   s.add_runtime_dependency "trollop"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stackster
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.2.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-06-22 00:00:00.000000000 Z
+date: 2012-06-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70157233988800 !ruby/object:Gem::Requirement
+  requirement: &70115631981100 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70157233988800
+  version_requirements: *70115631981100
 - !ruby/object:Gem::Dependency
   name: fog
-  requirement: &70157233987100 !ruby/object:Gem::Requirement
+  requirement: &70115631980080 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,21 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70157233987100
+  version_requirements: *70115631980080
+- !ruby/object:Gem::Dependency
+  name: logger
+  requirement: &70115631979060 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: *70115631979060
 - !ruby/object:Gem::Dependency
   name: trollop
-  requirement: &70157233986600 !ruby/object:Gem::Requirement
+  requirement: &70115631978360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,7 +54,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70157233986600
+  version_requirements: *70115631978360
 description: Thats what I do
 email:
 - brett@weav.net
@@ -69,6 +80,7 @@ files:
 - lib/stackster/entry/entry_lister.rb
 - lib/stackster/instance.rb
 - lib/stackster/instance/instance_reader.rb
+- lib/stackster/logger.rb
 - lib/stackster/stack.rb
 - lib/stackster/stack/stack_creater.rb
 - lib/stackster/stack/stack_destroyer.rb
@@ -78,8 +90,22 @@ files:
 - lib/stackster/stack/stack_updater.rb
 - lib/stackster/version.rb
 - script/ci_setup
+- spec/aws/cloud_formation_spec.rb
+- spec/aws/ec2_spec.rb
+- spec/aws/simpledb_spec.rb
 - spec/config_spec.rb
+- spec/entry/entry_lister_spec.rb
+- spec/entry_spec.rb
+- spec/instance/instance_reader_spec.rb
+- spec/logger_spec.rb
 - spec/spec_helper.rb
+- spec/stack/stack_creater_spec.rb
+- spec/stack/stack_destroyer_spec.rb
+- spec/stack/stack_formater_spec.rb
+- spec/stack/stack_lister_spec.rb
+- spec/stack/stack_reader_spec.rb
+- spec/stack/stack_updater_spec.rb
+- spec/stack_spec.rb
 - stackster.gemspec
 homepage: ''
 licenses: []
@@ -95,7 +121,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1350487739952735544
+      hash: -3503830738136148484
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -104,7 +130,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1350487739952735544
+      hash: -3503830738136148484
 requirements: []
 rubyforge_project: stackster
 rubygems_version: 1.8.16
@@ -112,5 +138,19 @@ signing_key:
 specification_version: 3
 summary: I make deployments easier
 test_files:
+- spec/aws/cloud_formation_spec.rb
+- spec/aws/ec2_spec.rb
+- spec/aws/simpledb_spec.rb
 - spec/config_spec.rb
+- spec/entry/entry_lister_spec.rb
+- spec/entry_spec.rb
+- spec/instance/instance_reader_spec.rb
+- spec/logger_spec.rb
 - spec/spec_helper.rb
+- spec/stack/stack_creater_spec.rb
+- spec/stack/stack_destroyer_spec.rb
+- spec/stack/stack_formater_spec.rb
+- spec/stack/stack_lister_spec.rb
+- spec/stack/stack_reader_spec.rb
+- spec/stack/stack_updater_spec.rb
+- spec/stack_spec.rb