knife-inspect 0.7.1 → 0.8.0

checksums.yaml

@@ -1,7 +1,15 @@
 ---
-SHA1:
-  metadata.gz: 608b6f567c14d1acc4050055b55135c26ba012a7
-  data.tar.gz: 4387aaaf508725bf67d3e93cb05ebd1e9a3bfe02
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    YTcyNzJmYmE2Nzg0ZWRjN2QyMTA4YzA0ODJiZTNiYjg5ODE0NGNiNA==
+  data.tar.gz: !binary |-
+    NDNkMTNlNWM5ODc3OGZhNmE0ZDI3ODliZTRmZGIzMGNiYTRkNjc0ZA==
 SHA512:
-  metadata.gz: b3ee1759814c36e62c058754a9424c8f6716e05917988286466cf2b359f5f937feea458c566aa8bb73127fa92522287bcb5852553de1d54d57c7a8a284e4c4f5
-  data.tar.gz: 0bd7fb3ca37b2b465851ea0a65ab100469b67e7095ad1fe75605827095f39665b82bc9405c4f98b157024058806f4b7ee7ce398f3192e2b72e34370fc1547fef
+  metadata.gz: !binary |-
+    Y2I1MTdlYTViNDdiZWQwZDA3OTM1NDhiYjgyZTM1MmNmNzM4NTMyZTliZmY1
+    YTJjOTA1MmRlZWIyYTE1NjdhNDUyMmYyNmYzYTZhZGU4ODM2YjdiNDE3ZWNj
+    Yzk5MjYzM2IyZmIwODM5ZDg5MjM5NzY4MWRjMWFkYWYzZDMyMDY=
+  data.tar.gz: !binary |-
+    OWQ2NWRiYmYxMWIyMTg1YWRiMTc0MTljNzQ2NTkwYTgzZTcyZTcwMjFmNDc2
+    NzFkN2FkOWRmNDYxNTZhZjZmNDkwY2FhYjNiNzIzMmI1Y2ZhZTMyNThkNmZh
+    YTQyZjM0ZjdhNmU5MGI3OTdmZmIyOTJjMDFhMGEwNDg1Nzk3NmU=

data/.coveralls.yml

@@ -0,0 +1 @@
+service_name: travis-ci

data/HISTORY.md

@@ -1,3 +1,10 @@
+## 0.8.0 ( 2014-01-10 )
+
+* New feature: Support data bags, data bag items, environments and roles inside
+  folders.
+* Add initial specs for the checklists.
+* General cleanup: got rid of some duplicate code and dead code.
+
 ## 0.7.1 ( 2014-01-06 )
 
 * Small bug fix: `knife data bag inspect` returned the status code only for

data/MIT-LICENSE

@@ -1,4 +1,5 @@
-Copyright (C) 2012 Ben Marini
+Copyright (C) 2012-2013 Ben Marini and contributors
+Copyright (C) 2014 Greg Karékinian and contributors
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -16,4 +17,4 @@ FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.
+SOFTWARE.

data/README.md

@@ -1,5 +1,7 @@
+[![Gem Version](https://badge.fury.io/rb/knife-inspect.png)](http://badge.fury.io/rb/knife-inspect)
 [![Build Status](https://secure.travis-ci.org/bmarini/knife-inspect.png)](http://travis-ci.org/bmarini/knife-inspect)
 [![Code Climate](https://codeclimate.com/github/bmarini/knife-inspect.png)](https://codeclimate.com/github/bmarini/knife-inspect)
+[![Coverage Status](https://coveralls.io/repos/bmarini/knife-inspect/badge.png)](https://coveralls.io/r/bmarini/knife-inspect)
 
 ## Summary
 
@@ -43,6 +45,18 @@ So far it checks if...
 * your environments are in sync
 * your roles are in sync
 
+You can use it with your favorite Continuous Integration tool, it returns 0
+when everything is in sync or 1 if it's not.
+
+## Contributors
+
+(in alphabetical order)
+
+* Adam Sinnett ([@quandrum](https://github.com/quandrum))
+* Eric Saxby ([@sax](https://github.com/sax))
+* Dan Buch ([@meatballhat](@https://github.com/meatballhat))
+* Kirt Fitzpatrick ([@kirtfitzpatrick](https://github.com/kirtfitzpatrick))
+
 ## Contributing
 
 1. Fork it

data/knife-inspect.gemspec

@@ -26,6 +26,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'rake',      '~> 10.1'
   s.add_development_dependency 'rspec',     '~> 2.14'
   s.add_development_dependency 'simplecov', '~> 0.8'
+  s.add_development_dependency 'coveralls', '~> 0.7'
 
   s.add_runtime_dependency 'chef',      chef_version
   s.add_runtime_dependency 'yajl-ruby', '~> 1.1'

data/lib/chef/knife/data_bag_inspect.rb

@@ -3,7 +3,6 @@ require 'chef/knife'
 class Chef
   class Knife
     class DataBagInspect < Knife
-
       deps do
         require 'health_inspector'
       end
@@ -17,13 +16,13 @@ class Chef
           item_name = @name_args[1]
 
           validator = HealthInspector::Checklists::DataBagItems.new(self)
-          exit validator.validate_item( validator.load_item("#{bag_name}/#{item_name}") )
+          exit validator.validate_item(validator.load_item("#{bag_name}/#{item_name}"))
 
         when 1 # We are inspecting a data bag
           bag_name = @name_args[0]
 
           validator = HealthInspector::Checklists::DataBags.new(self)
-          exit validator.validate_item( validator.load_item(bag_name) )
+          exit validator.validate_item(validator.load_item(bag_name))
 
         when 0 # We are inspecting all the data bags
           exit HealthInspector::Checklists::DataBags.run(self) &&

data/lib/health_inspector.rb

@@ -3,7 +3,6 @@ require "health_inspector/version"
 require "health_inspector/color"
 require "health_inspector/context"
 require "health_inspector/pairing"
-require "health_inspector/inspector"
 require "health_inspector/checklists/base"
 require "health_inspector/checklists/cookbooks"
 require "health_inspector/checklists/data_bags"

data/lib/health_inspector/checklists/base.rb

@@ -26,16 +26,22 @@ module HealthInspector
         @context.knife.ui
       end
 
+      def server_items
+        raise NotImplementedError, "You must implement this method in a subclass"
+      end
+
+      def local_items
+        raise NotImplementedError, "You must implement this method in a subclass"
+      end
+
       def all_item_names
-        ( server_items + local_items ).uniq.sort
+        (server_items + local_items).uniq.sort
       end
 
-      # Subclasses should collect all items from the server and the local repo,
-      # and for each item pair, yield an object that contains a reference to
-      # the server item, and the local repo item. A reference can be nil if it does
-      # not exist in one of the locations.
       def each_item
-        raise NotImplementedError, "You must implement this method in a subclass"
+        all_item_names.each do |name|
+          yield load_item(name)
+        end
       end
 
       def run
@@ -79,7 +85,7 @@ module HealthInspector
         ui.msg color('bright fail', "- #{subject}")
 
         failures.each do |message|
-          if message.kind_of? Hash
+          if message.is_a? Hash
             puts color('bright yellow',"  has the following values mismatched on the server and repo\n")
             print_failures_from_hash(message)
           else
@@ -103,31 +109,31 @@ module HealthInspector
       end
 
       def print_key(key, depth)
-        ui.msg indent( color('bright yellow',"#{key} : "), depth )
+        ui.msg indent(color('bright yellow',"#{key} : "), depth)
       end
 
       def print_value_diff(value, depth)
-        print indent( color('bright fail',"server value = "), depth + 1 )
+        print indent(color('bright fail',"server value = "), depth + 1)
         print value["server"]
         print "\n"
-        print indent( color('bright fail',"local value  = "), depth + 1 )
+        print indent(color('bright fail',"local value  = "), depth + 1)
         print value["local"]
         print "\n\n"
       end
 
       def load_ruby_or_json_from_local(chef_class, folder, name)
-        path_template = "#{@context.repo_path}/#{folder}/#{name}.%s"
-        ruby_pathname = Pathname.new(path_template % "rb")
-        json_pathname = Pathname.new(path_template % "json")
-        js_pathname   = Pathname.new(path_template % "js")
+        path_template = "#{@context.repo_path}/#{folder}/**/#{name}.%s"
+        ruby_pathname = Pathname.glob(path_template % "rb")
+        json_pathname = Pathname.glob(path_template % "json")
+        js_pathname   = Pathname.glob(path_template % "js")
 
-        if ruby_pathname.exist?
+        if ! ruby_pathname.empty?
           instance = chef_class.new
-          instance.from_file(ruby_pathname.to_s)
-        elsif json_pathname.exist?
-          instance = chef_class.json_create( Yajl::Parser.parse( json_pathname.read ) )
-        elsif js_pathname.exist?
-          instance = chef_class.json_create( Yajl::Parser.parse( js_pathname.read ) )
+          instance.from_file(ruby_pathname.first.to_s)
+        elsif ! json_pathname.empty?
+          instance = chef_class.json_create(Yajl::Parser.parse(json_pathname.first.read))
+        elsif ! js_pathname.empty?
+          instance = chef_class.json_create(Yajl::Parser.parse(js_pathname.first.read))
         end
 
         instance ? instance.to_hash : nil

data/lib/health_inspector/checklists/cookbooks.rb

@@ -5,7 +5,7 @@ module HealthInspector
       include ExistenceValidations
 
       def validate_versions
-        if versions_exist? && !versions_match?
+        if versions_exist? && ! versions_match?
           errors.add "chef server has #{server} but local version is #{local}"
         end
       end
@@ -91,30 +91,22 @@ module HealthInspector
 
       title "cookbooks"
 
-      def each_item
-        all_cookbook_names = ( server_cookbooks.keys + local_cookbooks.keys ).uniq.sort
-
-        all_cookbook_names.each do |name|
-          yield load_item(name)
-        end
-      end
-
       def load_item(name)
         Cookbook.new(@context,
           :name   => name,
-          :server => server_cookbooks[name],
-          :local  => local_cookbooks[name]
+          :server => server_items[name],
+          :local  => local_items[name]
         )
       end
 
-      def server_cookbooks
+      def server_items
         @context.rest.get_rest("/cookbooks").inject({}) do |hsh, (name,version)|
           hsh[name] = Chef::Version.new(version["versions"].first["version"])
           hsh
         end
       end
 
-      def local_cookbooks
+      def local_items
         @context.cookbook_path.
           map { |path| Dir["#{path}/*"] }.
           flatten.
@@ -129,6 +121,10 @@ module HealthInspector
           end
       end
 
+      def all_item_names
+        (server_items.keys + local_items.keys).uniq.sort
+      end
+
     end
   end
 end

data/lib/health_inspector/checklists/data_bag_items.rb

@@ -10,12 +10,6 @@ module HealthInspector
     class DataBagItems < Base
       title "data bag items"
 
-      def each_item
-        all_item_names.each do |name|
-          yield load_item(name)
-        end
-      end
-
       def load_item(name)
         DataBagItem.new(@context,
           :name   => name,
@@ -32,14 +26,11 @@ module HealthInspector
         end
       end
 
+      # JSON files are data bag items, their parent folder is the data bag
       def local_items
-        entries = nil
-
-        Dir.chdir("#{@context.repo_path}/data_bags") do
-          entries = Dir["**/*.json"].map { |entry| entry.gsub('.json', '') }
+        Dir["#{@context.repo_path}/data_bags/**/*.json"].map do |e|
+          e.split('/')[-2..-1].join('/').gsub('.json', '')
         end
-
-        return entries
       end
 
       def load_item_from_server(name)
@@ -49,8 +40,16 @@ module HealthInspector
         nil
       end
 
+      # We support data bags that are inside a folder or git submodule, for
+      # example:
+      #
+      # data_bags/corp/apps/some_app.json is in the repo, but apps/some_app on
+      # the server
       def load_item_from_local(name)
-        Yajl::Parser.parse( File.read("#{@context.repo_path}/data_bags/#{name}.json") )
+        local_data_bag_item = Dir["#{@context.repo_path}/data_bags/**/#{name}.json"].first
+        return nil if local_data_bag_item.nil?
+
+        Yajl::Parser.parse(File.read(local_data_bag_item))
       rescue IOError, Errno::ENOENT
         nil
       end

data/lib/health_inspector/checklists/data_bags.rb

@@ -9,12 +9,6 @@ module HealthInspector
     class DataBags < Base
       title "data bags"
 
-      def each_item
-        all_item_names.each do |name|
-          yield load_item(name)
-        end
-      end
-
       def load_item(name)
         DataBag.new(@context,
           :name   => name,
@@ -27,10 +21,11 @@ module HealthInspector
         @server_items ||= Chef::DataBag.list.keys
       end
 
+      # JSON files are data bag items, their parent folder is the data bag
       def local_items
-        @local_items ||= Dir["#{@context.repo_path}/data_bags/*"].entries.
-          select { |e| File.directory?(e) }.
-          map { |e| File.basename(e) }
+        @local_items ||= Dir["#{@context.repo_path}/data_bags/**/*.json"].map do |e|
+          e.split('/')[-2].gsub('.json', '')
+        end
       end
     end
 

data/lib/health_inspector/checklists/environments.rb

@@ -16,12 +16,6 @@ module HealthInspector
     class Environments < Base
       title "environments"
 
-      def each_item
-        all_item_names.each do |name|
-          yield load_item(name)
-        end
-      end
-
       def load_item(name)
         Environment.new(@context,
           :name   => name,
@@ -35,8 +29,8 @@ module HealthInspector
       end
 
       def local_items
-        Dir.chdir("#{@context.repo_path}/environments") do
-          Dir["*.{rb,json,js}"].map { |e| e.gsub(/\.(rb|json|js)/,"") }
+        Dir["#{@context.repo_path}/environments/**/*.{rb,json,js}"].map do |e|
+          File.basename(e, '.*')
         end
       end
 

data/lib/health_inspector/checklists/roles.rb

@@ -11,12 +11,6 @@ module HealthInspector
     class Roles < Base
       title "roles"
 
-      def each_item
-        all_item_names.each do |name|
-          yield load_item(name)
-        end
-      end
-
       def load_item(name)
         Role.new(@context,
           :name   => name,
@@ -30,8 +24,8 @@ module HealthInspector
       end
 
       def local_items
-        Dir.chdir("#{@context.repo_path}/roles") do
-          Dir["*.{rb,json,js}"].map { |e| e.gsub(/\.(rb|json|js)/, '') }
+        Dir["#{@context.repo_path}/roles/**/*.{rb,json,js}"].map do |e|
+          File.basename(e, '.*')
         end
       end
 

data/lib/health_inspector/color.rb

@@ -25,7 +25,7 @@ module HealthInspector
         'diff removed'  => [:red]# 41
       }
 
-      @context.knife.ui.color( str, *colors[type] )
+      @context.knife.ui.color(str, *colors[type])
     end
   end
 end

data/lib/health_inspector/context.rb

@@ -9,11 +9,7 @@ module HealthInspector
     end
 
     def cookbook_path
-      Array( config.cookbook_path )
-    end
-
-    def config
-      Chef::Config
+      Array(Chef::Config.cookbook_path)
     end
 
     def rest

data/lib/health_inspector/pairing.rb

@@ -37,7 +37,6 @@ module HealthInspector
       self.methods.grep(/^validate_/).each { |meth| send(meth) }
     end
 
-
     def hash_diff(original, other)
       recursive_diff(stringify_hash_keys(original), stringify_hash_keys(other))
     end

data/lib/health_inspector/version.rb

@@ -1,3 +1,3 @@
 module HealthInspector
-  VERSION = '0.7.1'
+  VERSION = '0.8.0'
 end

data/spec/chef-repo/cookbooks/cookbook_one/metadata.rb

@@ -0,0 +1,2 @@
+name    "cookbook_one"
+version "1.0.0"

data/spec/chef-repo/cookbooks/cookbook_two/metadata.rb

@@ -0,0 +1,2 @@
+name    "cookbook_two"
+version "0.0.1"

data/spec/health_inspector/checklists/cookbooks_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe HealthInspector::Checklists::Cookbooks do
+  let :checklist do
+    described_class.new(nil)
+  end
+
+  before do
+    expect(HealthInspector::Context).to receive(:new).with(nil).
+      and_return health_inspector_context
+  end
+
+  describe '#server_items' do
+    it 'returns a list of roles from the chef server' do
+      health_inspector_context.stub_chain(:rest, :get_rest).
+        with('/cookbooks').and_return({
+        'cookbook_one' => { 'versions' => [{ 'version' => '1.0.0' }] },
+        'cookbook_two' => { 'versions' => [{ 'version' => '0.0.1' }] }
+      })
+      expect(checklist.server_items).to eq({
+        'cookbook_one' => Chef::Version.new('1.0.0'),
+        'cookbook_two' => Chef::Version.new('0.0.1')
+      })
+    end
+  end
+
+  describe '#local_items' do
+    it 'returns a list of roles from the chef repo' do
+      expect(checklist.local_items).to eq({
+        'cookbook_one' => Chef::Version.new('1.0.0'),
+        'cookbook_two' => Chef::Version.new('0.0.1')
+      })
+    end
+  end
+end

data/spec/health_inspector/checklists/data_bag_items_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+describe HealthInspector::Checklists::DataBagItems do
+  let :checklist do
+    described_class.new(nil)
+  end
+
+  before do
+    expect(HealthInspector::Context).to receive(:new).with(nil).
+      and_return health_inspector_context
+  end
+
+  describe '#server_items' do
+    it 'returns a list of data bag items from the chef server' do
+      expect(Chef::DataBag).to receive(:list).and_return({
+        'data_bag_one'         => 'url',
+        'data_bag_two'         => 'url',
+        'data_bag_from_subdir' => 'url'
+      })
+      { 'data_bag_one' => 'one',
+        'data_bag_two' => 'two',
+        'data_bag_from_subdir' => 'three' }.each do |data_bag, item|
+        expect(Chef::DataBag).to receive(:load).with(data_bag).
+          and_return({ item => 'url' })
+        end
+      expect(checklist.server_items.sort).to eq [
+        'data_bag_from_subdir/three', 'data_bag_one/one', 'data_bag_two/two'
+      ]
+    end
+  end
+
+  describe '#local_items' do
+    it 'returns a list of data bag items from the chef repo' do
+      expect(checklist.local_items.sort).to eq [
+        'data_bag_from_subdir/three', 'data_bag_one/one', 'data_bag_two/two'
+      ]
+    end
+  end
+end

data/spec/health_inspector/checklists/data_bags_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+describe HealthInspector::Checklists::DataBags do
+  let :checklist do
+    described_class.new(nil)
+  end
+
+  before do
+    expect(HealthInspector::Context).to receive(:new).with(nil).
+      and_return health_inspector_context
+  end
+
+  describe '#server_items' do
+    it 'returns a list of data bags from the chef server' do
+      expect(Chef::DataBag).to receive(:list).and_return({
+        'data_bag_one'         => 'url',
+        'data_bag_two'         => 'url',
+        'data_bag_from_subdir' => 'url'
+      })
+      expect(checklist.server_items.sort).to eq [
+        'data_bag_from_subdir', 'data_bag_one', 'data_bag_two'
+      ]
+    end
+  end
+
+  describe '#local_items' do
+    it 'returns a list of data bags from the chef repo' do
+      expect(checklist.local_items.sort).to eq [
+        'data_bag_from_subdir', 'data_bag_one', 'data_bag_two'
+      ]
+    end
+  end
+end

data/spec/health_inspector/checklists/environments_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+describe HealthInspector::Checklists::Environments do
+  let :checklist do
+    described_class.new(nil)
+  end
+
+  before do
+    expect(HealthInspector::Context).to receive(:new).with(nil).
+      and_return health_inspector_context
+  end
+
+  describe '#server_items' do
+    it 'returns a list of environments from the chef server' do
+      expect(Chef::Environment).to receive(:list).and_return({
+        'environment_one'         => 'url',
+        'environment_two'         => 'url',
+        'environment_from_subdir' => 'url'
+      })
+      expect(checklist.server_items.sort).to eq [
+        'environment_from_subdir', 'environment_one', 'environment_two'
+      ]
+    end
+  end
+
+  describe '#local_items' do
+    it 'returns a list of environments from the chef repo' do
+      expect(checklist.local_items.sort).to eq [
+        'environment_from_subdir', 'environment_one', 'environment_two'
+      ]
+    end
+  end
+end

data/spec/health_inspector/checklists/roles_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+
+describe HealthInspector::Checklists::Roles do
+  let :checklist do
+    described_class.new(nil)
+  end
+
+  before do
+    expect(HealthInspector::Context).to receive(:new).with(nil).
+      and_return health_inspector_context
+  end
+
+  describe '#server_items' do
+    it 'returns a list of roles from the chef server' do
+      expect(Chef::Role).to receive(:list).and_return({
+        'role_one'         => 'url',
+        'role_two'         => 'url',
+        'role_from_subdir' => 'url'
+      })
+      expect(checklist.server_items.sort).to eq [
+        'role_from_subdir', 'role_one', 'role_two'
+      ]
+    end
+  end
+
+  describe '#local_items' do
+    it 'returns a list of roles from the chef repo' do
+      expect(checklist.local_items.sort).to eq [
+        'role_from_subdir', 'role_one', 'role_two'
+      ]
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,5 +1,10 @@
 if RUBY_VERSION > '1.9'
   require 'simplecov'
+  require 'coveralls'
+  SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
+    SimpleCov::Formatter::HTMLFormatter,
+    Coveralls::SimpleCov::Formatter
+  ]
   SimpleCov.start do
     add_filter "/spec/"
   end
@@ -10,8 +15,15 @@ require 'bundler/setup'
 require 'health_inspector'
 
 module HealthInspector::SpecHelpers
+  def repo_path
+    @repo_path ||= File.expand_path("../chef-repo", __FILE__)
+  end
+
   def health_inspector_context
-    @health_inspector_context ||= HealthInspector::Context.new(nil)
+    @health_inspector_context ||= double(
+      :repo_path => repo_path,
+      :cookbook_path => ["#{repo_path}/cookbooks"]
+    )
   end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: knife-inspect
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.8.0
 platform: ruby
 authors:
 - Greg Karékinian
@@ -9,82 +9,96 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-06 00:00:00.000000000 Z
+date: 2014-01-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '10.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '10.1'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '2.14'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '2.14'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '0.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '0.8'
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.7'
 - !ruby/object:Gem::Dependency
   name: chef
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '10'
-    - - "<="
+    - - <=
       - !ruby/object:Gem::Version
         version: '12'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '10'
-    - - "<="
+    - - <=
       - !ruby/object:Gem::Version
         version: '12'
 - !ruby/object:Gem::Dependency
   name: yajl-ruby
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.1'
 description: knife-inspect is a knife plugin to compare the content of your Chef repository
@@ -95,8 +109,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".travis.yml"
+- .coveralls.yml
+- .gitignore
+- .travis.yml
 - Gemfile
 - HISTORY.md
 - MIT-LICENSE
@@ -117,14 +132,29 @@ files:
 - lib/health_inspector/checklists/roles.rb
 - lib/health_inspector/color.rb
 - lib/health_inspector/context.rb
-- lib/health_inspector/inspector.rb
 - lib/health_inspector/pairing.rb
 - lib/health_inspector/version.rb
-- spec/cookbook_spec.rb
-- spec/data_bag_item_spec.rb
-- spec/data_bag_spec.rb
-- spec/environment_spec.rb
-- spec/role_spec.rb
+- spec/chef-repo/cookbooks/cookbook_one/metadata.rb
+- spec/chef-repo/cookbooks/cookbook_two/metadata.rb
+- spec/chef-repo/data_bags/data_bag_one/one.json
+- spec/chef-repo/data_bags/data_bag_two/two.json
+- spec/chef-repo/data_bags/subdir/data_bag_from_subdir/three.json
+- spec/chef-repo/environments/environment_one.rb
+- spec/chef-repo/environments/environment_two.json
+- spec/chef-repo/environments/subdir/environment_from_subdir.js
+- spec/chef-repo/roles/role_one.json
+- spec/chef-repo/roles/role_two.json
+- spec/chef-repo/roles/subdir/role_from_subdir.json
+- spec/health_inspector/checklists/cookbook_spec.rb
+- spec/health_inspector/checklists/cookbooks_spec.rb
+- spec/health_inspector/checklists/data_bag_item_spec.rb
+- spec/health_inspector/checklists/data_bag_items_spec.rb
+- spec/health_inspector/checklists/data_bag_spec.rb
+- spec/health_inspector/checklists/data_bags_spec.rb
+- spec/health_inspector/checklists/environment_spec.rb
+- spec/health_inspector/checklists/environments_spec.rb
+- spec/health_inspector/checklists/role_spec.rb
+- spec/health_inspector/checklists/roles_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/bmarini/knife-inspect
 licenses:
@@ -136,24 +166,41 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.0
+rubygems_version: 2.2.1
 signing_key: 
 specification_version: 4
 summary: Inspect your chef repo as it is compared to what is on your chef server
 test_files:
-- spec/cookbook_spec.rb
-- spec/data_bag_item_spec.rb
-- spec/data_bag_spec.rb
-- spec/environment_spec.rb
-- spec/role_spec.rb
+- spec/chef-repo/cookbooks/cookbook_one/metadata.rb
+- spec/chef-repo/cookbooks/cookbook_two/metadata.rb
+- spec/chef-repo/data_bags/data_bag_one/one.json
+- spec/chef-repo/data_bags/data_bag_two/two.json
+- spec/chef-repo/data_bags/subdir/data_bag_from_subdir/three.json
+- spec/chef-repo/environments/environment_one.rb
+- spec/chef-repo/environments/environment_two.json
+- spec/chef-repo/environments/subdir/environment_from_subdir.js
+- spec/chef-repo/roles/role_one.json
+- spec/chef-repo/roles/role_two.json
+- spec/chef-repo/roles/subdir/role_from_subdir.json
+- spec/health_inspector/checklists/cookbook_spec.rb
+- spec/health_inspector/checklists/cookbooks_spec.rb
+- spec/health_inspector/checklists/data_bag_item_spec.rb
+- spec/health_inspector/checklists/data_bag_items_spec.rb
+- spec/health_inspector/checklists/data_bag_spec.rb
+- spec/health_inspector/checklists/data_bags_spec.rb
+- spec/health_inspector/checklists/environment_spec.rb
+- spec/health_inspector/checklists/environments_spec.rb
+- spec/health_inspector/checklists/role_spec.rb
+- spec/health_inspector/checklists/roles_spec.rb
 - spec/spec_helper.rb
+has_rdoc: 

data/lib/health_inspector/inspector.rb

@@ -1,20 +0,0 @@
-module HealthInspector
-  class Inspector
-    def self.inspect(checklists, options)
-      new(options).inspect( checklists )
-    end
-
-    def initialize(options)
-      @context = Context.new( options[:repopath], options[:configpath] )
-      @context.quiet_success = options[:'quiet-success']
-      @context.no_color      = options[:'no-color']
-      @context.configure
-    end
-
-    def inspect(checklists)
-      checklists.each do |checklist|
-        Checklists.const_get(checklist).run(@context) if Checklists.const_defined?(checklist)
-      end
-    end
-  end
-end