jamie 0.1.0.alpha17 → 0.1.0.alpha18

data/.travis.yml

@@ -1,3 +1,9 @@
 language: ruby
 rvm:
   - 1.9.3
+notifications:
+  irc:
+    channels:
+    - "irc.freenode.org#jamie-ci"
+    use_notice: true
+    skip_join: true

data/Gemfile

@@ -2,6 +2,12 @@ source 'https://rubygems.org'
 
 gemspec
 
+group :development do
+  gem 'rb-inotify', :require => false
+  gem 'rb-fsevent', :require => false
+  gem 'rb-fchange', :require => false
+end
+
 group :test do
   gem 'rake', '~> 0.9'
   gem 'pry'

data/Guardfile

@@ -0,0 +1,5 @@
+guard 'minitest' do
+  watch(%r|^spec/(.*)_spec\.rb|)
+  watch(%r|^lib/(.*)([^/]+)\.rb|)     { |m| "spec/#{m[1]}#{m[2]}_spec.rb" }
+  watch(%r|^spec/spec_helper\.rb|)    { "spec" }
+end

data/README.md

@@ -5,28 +5,16 @@
 
 A Chef convergence integration test harness.
 
-## Installation
+## A Note
 
-Add this line to your application's Gemfile:
+This project is currently in rapid development which means frequent releases,
+potential for massive refactorings (that could be API breaking), and minimal
+to no documentation. This will change as the project transitions to be used in
+production environments.
 
-    gem 'jamie'
+Despite the warnings above, if you are still interested, please get in touch
+via freenode/IRC (#jamie),
+Twitter ([@fnichol](https://twitter.com/fnichol)),
+or Email ([fnichol@nichol.ca](mailto:fnichol@nichol.ca)).
 
-And then execute:
-
-    $ bundle
-
-Or install it yourself as:
-
-    $ gem install jamie
-
-## Usage
-
-TODO: Write usage instructions here
-
-## Contributing
-
-1. Fork it
-2. Create your feature branch (`git checkout -b my-new-feature`)
-3. Commit your changes (`git commit -am 'Add some feature'`)
-4. Push to the branch (`git push origin my-new-feature`)
-5. Create new Pull Request
+For everyone else, watch [this space](https://github.com/jamie-ci/jamie).

data/Rakefile

@@ -1,5 +1,6 @@
 require 'bundler/gem_tasks'
 require 'cane/rake_task'
+require 'rake/testtask'
 require 'tailor/rake_task'
 
 desc "Run cane to check quality metrics"
@@ -21,7 +22,16 @@ Tailor::RakeTask.new
 
 desc "Display LOC stats"
 task :stats do
+  puts "\n## Production Code Stats"
   sh "countloc -r lib/jamie lib/jamie.rb"
+  puts "\n## Test Code Stats"
+  sh "countloc -r spec"
 end
 
-task :default => [ :cane, :tailor ]
+Rake::TestTask.new do |t|
+  t.libs.push "lib"
+  t.test_files = FileList['spec/**/*_spec.rb']
+  t.verbose = true
+end
+
+task :default => [ :test, :cane, :tailor, :stats ]

data/bin/jamie

@@ -6,6 +6,7 @@
 Signal.trap("INT") { exit 1 }
 
 $:.unshift File.join(File.dirname(__FILE__), %w{.. lib})
+require 'rubygems'
 require 'jamie/cli'
 
 Jamie::CLI.start

data/jamie.gemspec

@@ -10,21 +10,28 @@ Gem::Specification.new do |gem|
   gem.email         = ["fnichol@nichol.ca"]
   gem.description   = %q{A Chef convergence integration test harness}
   gem.summary       = gem.description
-  gem.homepage      = ""
+  gem.homepage      = "https://github.com/jamie-ci/jamie"
 
   gem.files         = `git ls-files`.split($/)
   gem.executables   = %w(jamie)
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ["lib"]
 
+  gem.required_ruby_version = ">= 1.9.1"
+
   gem.add_dependency 'thor'
   gem.add_dependency 'net-ssh'
   gem.add_dependency 'net-scp'
   gem.add_dependency 'mixlib-shellout'
 
+  gem.add_development_dependency 'minitest'
+  gem.add_development_dependency 'fakefs'
+  gem.add_development_dependency 'guard-minitest'
+
   gem.add_development_dependency 'yard'
   gem.add_development_dependency 'maruku'
   gem.add_development_dependency 'cane'
   gem.add_development_dependency 'tailor'
+  gem.add_development_dependency 'simplecov'
   gem.add_development_dependency 'countloc'
 end

data/lib/jamie/cli.rb

@@ -1,4 +1,20 @@
 # -*- encoding: utf-8 -*-
+#
+# Author:: Fletcher Nichol (<fnichol@nichol.ca>)
+#
+# Copyright (C) 2012, Fletcher Nichol
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
 
 require 'ostruct'
 require 'thor'
@@ -8,6 +24,8 @@ require 'jamie'
 module Jamie
 
   # The command line runner for Jamie.
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   class CLI < Thor
 
     include Thor::Actions
@@ -157,6 +175,8 @@ module Jamie
 
   # A project initialization generator, to help prepare a cookbook project for
   # testing with Jamie.
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   class InitGenerator < Thor
 
     include Thor::Actions
@@ -275,6 +295,8 @@ module Jamie
   end
 
   # A generator to create a new Jamie driver plugin.
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   class NewPluginGenerator < Thor
 
     include Thor::Actions
@@ -406,6 +428,8 @@ module Jamie
     end
 
     # Renders an ERB template with a hash of template variables.
+    #
+    # @author Fletcher Nichol <fnichol@nichol.ca>
     class TemplateRenderer < OpenStruct
 
       def self.render(template, data = {})

data/lib/jamie/driver/dummy.rb

@@ -23,6 +23,8 @@ module Jamie
   module Driver
 
     # Dummy driver for Jamie.
+    #
+    # @author Fletcher Nichol <fnichol@nichol.ca>
     class Dummy < Jamie::Driver::Base
 
       def create(instance, state)

data/lib/jamie/rake_tasks.rb

@@ -1,4 +1,20 @@
 # -*- encoding: utf-8 -*-
+#
+# Author:: Fletcher Nichol (<fnichol@nichol.ca>)
+#
+# Copyright (C) 2012, Fletcher Nichol
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
 
 require 'rake/tasklib'
 
@@ -7,6 +23,8 @@ require 'jamie'
 module Jamie
 
   # Jamie Rake task generator.
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   class RakeTasks < ::Rake::TaskLib
 
     # Creates Jamie Rake tasks and allows the callee to configure it.
@@ -26,7 +44,7 @@ module Jamie
       namespace "jamie" do
         config.instances.each do |instance|
           desc "Run #{instance.name} test instance"
-          task instance.name { instance.test }
+          task instance.name { instance.test(:always) }
         end
 
         desc "Run all test instances"

data/lib/jamie/thor_tasks.rb

@@ -1,4 +1,20 @@
 # -*- encoding: utf-8 -*-
+#
+# Author:: Fletcher Nichol (<fnichol@nichol.ca>)
+#
+# Copyright (C) 2012, Fletcher Nichol
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
 
 require 'thor'
 
@@ -7,6 +23,8 @@ require 'jamie'
 module Jamie
 
   # Jamie Thor task generator.
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   class ThorTasks < Thor
 
     namespace :jamie
@@ -29,7 +47,7 @@ module Jamie
       config.instances.each do |instance|
         self.class.desc instance.name, "Run #{instance.name} test instance"
         self.class.send(:define_method, instance.name.gsub(/-/, '_')) do
-          instance.test
+          instance.test(:always)
         end
       end
 

data/lib/jamie/version.rb

@@ -1,6 +1,22 @@
 # -*- encoding: utf-8 -*-
+#
+# Author:: Fletcher Nichol (<fnichol@nichol.ca>)
+#
+# Copyright (C) 2012, Fletcher Nichol
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
 
 module Jamie
 
-  VERSION = "0.1.0.alpha17"
+  VERSION = "0.1.0.alpha18"
 end

data/lib/jamie.rb

@@ -1,4 +1,20 @@
 # -*- encoding: utf-8 -*-
+#
+# Author:: Fletcher Nichol (<fnichol@nichol.ca>)
+#
+# Copyright (C) 2012, Fletcher Nichol
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
 
 require 'base64'
 require 'delegate'
@@ -29,6 +45,8 @@ module Jamie
 
   # Base configuration class for Jamie. This class exposes configuration such
   # as the location of the Jamie YAML file, instances, log_levels, etc.
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   class Config
 
     attr_writer :yaml_file
@@ -97,6 +115,8 @@ module Jamie
     # Delegate class which adds the ability to find single and multiple
     # objects by their #name in an Array. Hey, it's better than monkey-patching
     # Array, right?
+    #
+    # @author Fletcher Nichol <fnichol@nichol.ca>
     class Collection < SimpleDelegator
 
       # Returns a single object by its name, or nil if none are found.
@@ -141,6 +161,7 @@ module Jamie
 
     def new_platform(hash)
       mpc = merge_platform_config(hash)
+      mpc['driver_config'] ||= Hash.new
       mpc['driver_config']['jamie_root'] = File.dirname(yaml_file)
       mpc['driver'] = new_driver(mpc['driver_plugin'], mpc['driver_config'])
       Platform.new(mpc)
@@ -220,6 +241,8 @@ module Jamie
 
   # A Chef run_list and attribute hash that will be used in a convergence
   # integration.
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   class Suite
 
     # @return [String] logical name of this suite
@@ -261,7 +284,7 @@ module Jamie
 
     def validate_options(opts)
       %w(name run_list).each do |k|
-        raise ArgumentError, "Attribute '#{attr}' is required." if opts[k].nil?
+        raise ArgumentError, "Attribute '#{k}' is required." if opts[k].nil?
       end
     end
   end
@@ -269,6 +292,8 @@ module Jamie
   # A target operating system environment in which convergence integration
   # will take place. This may represent a specific operating system, version,
   # and machine architecture.
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   class Platform
 
     # @return [String] logical name of this platform
@@ -307,7 +332,7 @@ module Jamie
 
     def validate_options(opts)
       %w(name driver).each do |k|
-        raise ArgumentError, "Attribute '#{attr}' is required." if opts[k].nil?
+        raise ArgumentError, "Attribute '#{k}' is required." if opts[k].nil?
       end
     end
   end
@@ -315,6 +340,8 @@ module Jamie
   # An instance of a suite running on a platform. A created instance may be a
   # local virtual machine, cloud instance, container, or even a bare metal
   # server, which is determined by the platform's driver.
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   class Instance
 
     # @return [Suite] the test suite configuration
@@ -331,6 +358,8 @@ module Jamie
     # @param suite [Suite] a suite
     # @param platform [Platform] a platform
     def initialize(suite, platform)
+      validate_options(suite, platform)
+
       @suite = suite
       @platform = platform
       @jr = Jr.new(@suite.name)
@@ -440,6 +469,11 @@ module Jamie
 
     private
 
+    def validate_options(suite, platform)
+      raise ArgumentError, "Attribute 'suite' is required." if suite.nil?
+      raise ArgumentError, "Attribute 'platform' is required." if platform.nil?
+    end
+
     def transition_to(desired)
       FSM.actions(last_action, desired).each do |transition|
         send("#{transition}_action")
@@ -517,6 +551,8 @@ module Jamie
 
     # The simplest finite state machine pseudo-implementation needed to manage
     # an Instance.
+    #
+    # @author Fletcher Nichol <fnichol@nichol.ca>
     class FSM
 
       # Returns an Array of all transitions to bring an Instance from its last
@@ -555,6 +591,8 @@ module Jamie
   # Command string generator to interface with Jamie Runner (jr). The
   # commands that are generated are safe to pass to an SSH command or as an
   # unix command argument (escaped in single quotes).
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   class Jr
 
     # Constructs a new jr command generator, given a suite name.
@@ -711,6 +749,8 @@ module Jamie
 
   # Mixin that wraps a command shell out invocation, providing a #run_command
   # method.
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   module ShellOut
 
     # Wrapped exception for any interally raised shell out commands.
@@ -764,6 +804,8 @@ module Jamie
     # Base class for a driver. A driver is responsible for carrying out the
     # lifecycle activities of an instance, such as creating, converging, and
     # destroying an instance.
+    #
+    # @author Fletcher Nichol <fnichol@nichol.ca>
     class Base
 
       include ShellOut
@@ -842,6 +884,8 @@ module Jamie
     # A subclass must implement the following methods:
     # * #create(instance, state)
     # * #destroy(instance, state)
+    #
+    # @author Fletcher Nichol <fnichol@nichol.ca>
     class SSHBase < Base
 
       def create(instance, state)
@@ -973,6 +1017,8 @@ module Jamie
 
   # Uploads Chef asset files such as dna.json, data bags, and cookbooks to an
   # instance over SSH.
+  #
+  # @author Fletcher Nichol <fnichol@nichol.ca>
   class ChefDataUploader
 
     include ShellOut

data/spec/jamie_spec.rb

@@ -0,0 +1,486 @@
+# -*- encoding: utf-8 -*-
+#
+# Author:: Fletcher Nichol (<fnichol@nichol.ca>)
+#
+# Copyright (C) 2012, Fletcher Nichol
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+require 'simplecov'
+SimpleCov.adapters.define 'gem' do
+  command_name 'Specs'
+
+  add_filter '/spec/'
+  add_filter '/lib/vendor/'
+
+  add_group 'Libraries', '/lib/'
+end
+SimpleCov.start 'gem'
+
+require 'fakefs/spec_helpers'
+require 'minitest/autorun'
+
+require 'jamie'
+
+# Nasty hack to redefine IO.read in terms of File#read for fakefs
+class IO
+  def self.read(*args)
+    File.open(args[0], "rb") { |f| f.read(args[1]) }
+  end
+end
+
+describe Jamie::Config do
+  include FakeFS::SpecHelpers
+
+  let(:config) { Jamie::Config.new("/tmp/.jamie.yml") }
+
+  describe "#platforms" do
+
+    it "returns platforms loaded from a jamie.yml" do
+      stub_yaml!({'platforms' => [
+        { 'name' => 'one', 'driver_plugin' => 'dummy' },
+        { 'name' => 'two', 'driver_plugin' => 'dummy' },
+      ]})
+      config.platforms.size.must_equal 2
+      config.platforms[0].name.must_equal 'one'
+      config.platforms[1].name.must_equal 'two'
+    end
+
+    it "returns an empty Array if no platforms are given" do
+      stub_yaml!({})
+      config.platforms.must_equal []
+    end
+
+    it "returns a platform containing a driver instance" do
+      stub_yaml!({'platforms' => [
+        { 'name' => 'platform', 'driver_plugin' => 'dummy' }
+      ]})
+      config.platforms.first.driver.must_be_instance_of Jamie::Driver::Dummy
+    end
+
+    it "returns a platform with a driver initialized with jamie_root" do
+      stub_yaml!({'platforms' => [
+        { 'name' => 'platform', 'driver_plugin' => 'dummy' }
+      ]})
+      config.platforms.first.driver['jamie_root'].must_equal "/tmp"
+    end
+
+    it "returns a platform with a driver initialized with passed in config" do
+      stub_yaml!({'platforms' => [
+        { 'name' => 'platform', 'driver_plugin' => 'dummy',
+          'driver_config' => { 'foo' => 'bar' }
+        }
+      ]})
+      config.platforms.first.driver['foo'].must_equal "bar"
+    end
+  end
+
+  describe "#suites" do
+
+    it "returns suites loaded from a jamie.yml" do
+      stub_yaml!({'suites' => [
+        { 'name' => 'one', 'run_list' => [] },
+        { 'name' => 'two', 'run_list' => [] },
+      ]})
+      config.suites.size.must_equal 2
+      config.suites[0].name.must_equal 'one'
+      config.suites[1].name.must_equal 'two'
+    end
+
+    it "returns an empty Array if no suites are given" do
+      stub_yaml!({})
+      config.suites.must_equal []
+    end
+
+    it "returns a suite with nil for data_bags_path by default" do
+      stub_yaml!({'suites' => [ { 'name' => 'one', 'run_list' => [] } ]})
+      config.suites.first.data_bags_path.must_be_nil
+    end
+
+    it "retuns a suite with a common data_bags_path set" do
+      stub_yaml!({'suites' => [ { 'name' => 'one', 'run_list' => [] } ]})
+      config.test_base_path = "/tmp/base"
+      FileUtils.mkdir_p "/tmp/base/data_bags"
+      config.suites.first.data_bags_path.must_equal "/tmp/base/data_bags"
+    end
+
+    it "retuns a suite with a suite-specific data_bags_path set" do
+      stub_yaml!({'suites' => [ { 'name' => 'cool', 'run_list' => [] } ]})
+      config.test_base_path = "/tmp/base"
+      FileUtils.mkdir_p "/tmp/base/cool/data_bags"
+      config.suites.first.data_bags_path.must_equal "/tmp/base/cool/data_bags"
+    end
+
+    it "returns a suite with nil for roles_path by default" do
+      stub_yaml!({'suites' => [ { 'name' => 'one', 'run_list' => [] } ]})
+      config.suites.first.roles_path.must_be_nil
+    end
+
+    it "returns a suite with a common roles_path set" do
+      stub_yaml!({'suites' => [ { 'name' => 'one', 'run_list' => [] } ]})
+      config.test_base_path = "/tmp/base"
+      FileUtils.mkdir_p "/tmp/base/roles"
+      config.suites.first.roles_path.must_equal "/tmp/base/roles"
+    end
+
+    it "returns a suite with a suite-specific roles_path set" do
+      stub_yaml!({'suites' => [ { 'name' => 'mysuite', 'run_list' => [] } ]})
+      config.test_base_path = "/tmp/base"
+      FileUtils.mkdir_p "/tmp/base/mysuite/roles"
+      config.suites.first.roles_path.must_equal "/tmp/base/mysuite/roles"
+    end
+  end
+
+  describe "#instances" do
+
+    it "returns instances loaded from a jamie.yml" do
+      stub_yaml!({
+        'platforms' => [
+          { 'name' => 'p1', 'driver_plugin' => 'dummy' },
+          { 'name' => 'p2', 'driver_plugin' => 'dummy' },
+        ],
+        'suites' => [
+          { 'name' => 's1', 'run_list' => [] },
+          { 'name' => 's2', 'run_list' => [] },
+        ]
+      })
+      config.instances.size.must_equal 4
+      config.instances.map { |i| i.name }.must_equal %w{s1-p1 s1-p2 s2-p1 s2-p2}
+    end
+  end
+
+  describe "jamie.local.yml" do
+
+    it "merges in configuration with jamie.yml" do
+      stub_yaml!(".jamie.yml", {
+        'platforms' => [ { 'name' => 'p1', 'driver_plugin' => 'dummy' } ]
+      })
+      stub_yaml!(".jamie.local.yml", {
+        'driver_config' => { 'foo' => 'bar' }
+      })
+      config.platforms.first.driver['foo'].must_equal 'bar'
+    end
+
+    it "merges over configuration in jamie.yml" do
+      stub_yaml!(".jamie.yml", {
+        'driver_config' => { 'foo' => 'nope' },
+        'platforms' => [ { 'name' => 'p1', 'driver_plugin' => 'dummy' } ]
+      })
+      stub_yaml!(".jamie.local.yml", {
+        'driver_config' => { 'foo' => 'bar' }
+      })
+      config.platforms.first.driver['foo'].must_equal 'bar'
+    end
+  end
+
+  describe "erb filtering" do
+
+    it "evaluates jamie.yml through erb before loading" do
+      FileUtils.mkdir_p "/tmp"
+      File.open("/tmp/.jamie.yml", "wb") do |f|
+        f.write <<-YAML.gsub(/^ {10}/, '')
+          ---
+          driver_plugin: dummy
+          platforms:
+          - name: <%= "AHH".downcase + "choo" %>
+        YAML
+      end
+      config.platforms.first.name.must_equal "ahhchoo"
+    end
+
+    it "evaluates jamie.local.yml through erb before loading" do
+      stub_yaml!({
+        'platforms' => [ { 'name' => 'p1', 'driver_plugin' => 'dummy' } ]
+      })
+      FileUtils.mkdir_p "/tmp"
+      File.open("/tmp/.jamie.local.yml", "wb") do |f|
+        f.write <<-YAML.gsub(/^ {10}/, '')
+          ---
+          driver_config:
+          <% %w{noodle mushroom}.each do |kind| %>
+            <%= kind %>: soup
+          <% end %>
+        YAML
+      end
+      config.platforms.first.driver['noodle'].must_equal "soup"
+      config.platforms.first.driver['mushroom'].must_equal "soup"
+    end
+  end
+
+  describe "#log_level" do
+
+    it "returns a default log_level of info" do
+      config.log_level.must_equal :info
+    end
+
+    it "returns an overridden log_level" do
+      config.log_level = :error
+      config.log_level.must_equal :error
+    end
+  end
+
+  private
+
+  def stub_yaml!(name = ".jamie.yml", hash)
+    FileUtils.mkdir_p "/tmp"
+    File.open("/tmp/#{name}", "wb") { |f| f.write(hash.to_yaml) }
+  end
+end
+
+describe Jamie::Config::Collection do
+
+  let(:collection) do
+    Jamie::Config::Collection.new([
+      obj('one'), obj('two', 'a'), obj('two', 'b'), obj('three')
+    ])
+  end
+
+  it "transparently wraps an Array" do
+    collection.must_be_instance_of Array
+  end
+
+  describe "#get" do
+
+    it "returns a single object by its name" do
+      collection.get('three').must_equal obj('three')
+    end
+
+    it "returns the first occurance of an object by its name" do
+      collection.get('two').must_equal obj('two', 'a')
+    end
+
+    it "returns nil if an object cannot be found by its name" do
+      collection.get('nope').must_be_nil
+    end
+  end
+
+  describe "#get_all" do
+
+    it "returns a Collection of objects whose name matches the regex" do
+      result = collection.get_all(/(one|three)/)
+      result.size.must_equal 2
+      result[0].must_equal obj('one')
+      result[1].must_equal obj('three')
+      result.get_all(/one/).size.must_equal 1
+    end
+
+    it "returns an empty Collection if on matches are found" do
+      result = collection.get_all(/noppa/)
+      result.must_equal []
+      result.get("nahuh").must_be_nil
+    end
+  end
+
+  describe "#as_name" do
+
+    it "returns an Array of names as strings" do
+      collection.as_names.must_equal %w{one two two three}
+    end
+  end
+
+  private
+
+  def obj(name, extra = nil)
+    OpenStruct.new(:name => name, :extra => extra)
+  end
+end
+
+describe Jamie::Suite do
+
+  let(:opts) do ; { 'name' => 'suitezy', 'run_list' => [ 'doowah' ] } ; end
+  let(:suite) { Jamie::Suite.new(opts) }
+
+  it "raises an ArgumentError if name is missing" do
+    opts.delete('name')
+    proc { Jamie::Suite.new(opts) }.must_raise ArgumentError
+  end
+
+  it "raises an ArgumentError if run_list is missing" do
+    opts.delete('run_list')
+    proc { Jamie::Suite.new(opts) }.must_raise ArgumentError
+  end
+
+  it "returns an empty Hash given no attributes" do
+    suite.attributes.must_equal Hash.new
+  end
+
+  it "returns nil given no data_bags_path" do
+    suite.data_bags_path.must_be_nil
+  end
+
+  it "returns nil given no roles_path" do
+    suite.roles_path.must_be_nil
+  end
+
+  it "returns attributes from constructor" do
+    opts.merge!({ 'attributes' => { 'a' => 'b' }, 'data_bags_path' => 'crazy',
+                  'roles_path' => 'town' })
+    suite.name.must_equal 'suitezy'
+    suite.run_list.must_equal [ 'doowah' ]
+    suite.attributes.must_equal({ 'a' => 'b' })
+    suite.data_bags_path.must_equal 'crazy'
+    suite.roles_path.must_equal 'town'
+  end
+end
+
+describe Jamie::Platform do
+
+  let(:opts) do ; { 'name' => 'plata', 'driver' => 'imadriver' } ; end
+  let(:platform) { Jamie::Platform.new(opts) }
+
+  it "raises an ArgumentError if name is missing" do
+    opts.delete('name')
+    proc { Jamie::Platform.new(opts) }.must_raise ArgumentError
+  end
+
+  it "raises an ArgumentError if driver is missing" do
+    opts.delete('driver')
+    proc { Jamie::Platform.new(opts) }.must_raise ArgumentError
+  end
+
+  it "returns an empty Array given no run_list" do
+    platform.run_list.must_equal []
+  end
+
+  it "returns an empty Hash given no attributes" do
+    platform.attributes.must_equal Hash.new
+  end
+
+  it "returns attributes from constructor" do
+    opts.merge!({ 'run_list' => [ 'a', 'b' ], 'attributes' => { 'c' => 'd' }})
+    platform.name.must_equal 'plata'
+    platform.driver.must_equal 'imadriver'
+    platform.run_list.must_equal [ 'a', 'b' ]
+    platform.attributes.must_equal({ 'c' => 'd' })
+  end
+end
+
+describe Jamie::Instance do
+
+  let(:suite) do
+    Jamie::Suite.new({ 'name' => 'suite',
+      'run_list' => 'suite_list', 'attributes' => { 's' => 'ss' } })
+  end
+
+  let(:platform) do
+    Jamie::Platform.new({ 'name' => 'platform', 'driver' => 'driver',
+      'run_list' => 'platform_list', 'attributes' => { 'p' => 'pp' } })
+  end
+
+  let(:instance) { Jamie::Instance.new(suite, platform) }
+
+  it "raises an ArgumentError if suite is missing" do
+    proc { Jamie::Instance.new(nil, platform) }.must_raise ArgumentError
+  end
+
+  it "raises an ArgumentError if platform is missing" do
+    proc { Jamie::Instance.new(suite, nil) }.must_raise ArgumentError
+  end
+
+  it "returns suite" do
+    instance.suite.must_equal suite
+  end
+
+  it "returns platform" do
+    instance.platform.must_equal platform
+  end
+
+  it "returns an instance of Jr" do
+    instance.jr.must_be_instance_of Jamie::Jr
+  end
+
+  describe "#name" do
+
+    def combo(suite_name, platform_name)
+      Jamie::Instance.new(
+        Jamie::Suite.new({ 'name' => suite_name, 'run_list' => [] }),
+        Jamie::Platform.new({ 'name' => platform_name, 'driver' => 'd' })
+      )
+    end
+
+    it "combines the suite and platform names with a dash" do
+      combo('suite', 'platform').name.must_equal "suite-platform"
+    end
+
+    it "squashes periods" do
+      combo('suite.ness', 'platform').name.must_equal "suiteness-platform"
+      combo('suite', 'platform.s').name.must_equal "suite-platforms"
+      combo('s.s.', '.p.p').name.must_equal "ss-pp"
+    end
+
+    it "transforms underscores to dashes" do
+      combo('suite_ness', 'platform').name.must_equal "suite-ness-platform"
+      combo('suite', 'platform-s').name.must_equal "suite-platform-s"
+      combo('_s__s_', 'pp_').name.must_equal "-s--s--pp-"
+    end
+  end
+
+  describe "#run_list" do
+
+    def combo(suite_list, platform_list)
+      Jamie::Instance.new(
+        Jamie::Suite.new({ 'name' => 'suite', 'run_list' => suite_list }),
+        Jamie::Platform.new({ 'name' => 'platform', 'driver' => 'd',
+                              'run_list' => platform_list })
+      )
+    end
+
+    it "combines the platform then suite run_lists" do
+      combo(%w{s1 s2}, %w{p1 p2}).run_list.must_equal %w{p1 p2 s1 s2}
+    end
+
+    it "uses the suite run_list only when platform run_list is empty" do
+      combo(%w{sa sb}, nil).run_list.must_equal %w{sa sb}
+    end
+
+    it "returns an emtpy Array if both run_lists are empty" do
+      combo([], nil).run_list.must_equal []
+    end
+  end
+
+  describe "#attributes" do
+
+    def combo(suite_attrs, platform_attrs)
+      Jamie::Instance.new(
+        Jamie::Suite.new({ 'name' => 'suite', 'run_list' => [],
+                           'attributes' => suite_attrs }),
+        Jamie::Platform.new({ 'name' => 'platform', 'driver' => 'd',
+                              'attributes' => platform_attrs })
+      )
+    end
+
+    it "merges suite and platform hashes together" do
+      combo(
+        { 'suite' => { 's1' => 'sv1' } },
+        { 'suite' => { 'p1' => 'pv1' }, 'platform' => 'pp' }
+      ).attributes.must_equal({
+        'suite' => { 's1' => 'sv1', 'p1' => 'pv1' },
+        'platform' => 'pp'
+      })
+    end
+
+    it "merges suite values over platform values" do
+      combo(
+        { 'common' => { 'c1' => 'xxx' } },
+        { 'common' => { 'c1' => 'cv1', 'c2' => 'cv2' } },
+      ).attributes.must_equal({
+        'common' => { 'c1' => 'xxx', 'c2' => 'cv2' }
+      })
+    end
+  end
+
+  it "#dna combines attributes with the run_list" do
+    instance.dna.must_equal({ 's' => 'ss', 'p' => 'pp',
+      'run_list' => [ 'platform_list', 'suite_list' ] })
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jamie
 version: !ruby/object:Gem::Version
-  version: 0.1.0.alpha17
+  version: 0.1.0.alpha18
   prerelease: 6
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-27 00:00:00.000000000 Z
+date: 2012-12-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor
@@ -75,6 +75,54 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: fakefs
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: guard-minitest
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
@@ -139,6 +187,22 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: countloc
   requirement: !ruby/object:Gem::Requirement
@@ -167,6 +231,7 @@ files:
 - .travis.yml
 - .yardopts
 - Gemfile
+- Guardfile
 - LICENSE
 - README.md
 - Rakefile
@@ -179,6 +244,7 @@ files:
 - lib/jamie/thor_tasks.rb
 - lib/jamie/version.rb
 - lib/vendor/hash_recursive_merge.rb
+- spec/jamie_spec.rb
 - templates/plugin/driver.rb.erb
 - templates/plugin/license_apachev2.erb
 - templates/plugin/license_gplv2.erb
@@ -186,7 +252,7 @@ files:
 - templates/plugin/license_mit.erb
 - templates/plugin/license_reserved.erb
 - templates/plugin/version.rb.erb
-homepage: ''
+homepage: https://github.com/jamie-ci/jamie
 licenses: []
 post_install_message: 
 rdoc_options: []
@@ -197,10 +263,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
-      version: '0'
-      segments:
-      - 0
-      hash: -4096747563693358273
+      version: 1.9.1
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -213,5 +276,6 @@ rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: A Chef convergence integration test harness
-test_files: []
+test_files:
+- spec/jamie_spec.rb
 has_rdoc: