inspec 0.17.1 → 0.18.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 92931c330ebe81c9d87899956cec70d7c395854b
-  data.tar.gz: 93b49fae335d4c153393eca778a0973a46bebb9f
+  metadata.gz: 5fcd7959abdb3de12c11854323ad29f484f0dbda
+  data.tar.gz: 79bbc6496cc71d18e02a9b05b4e6fbd3a5043315
 SHA512:
-  metadata.gz: e78120e5ec5764f3a9a6ce35db482adf9a8304ba13dac930a181984ace5d5cb4e0b8898e9cf26f17e495a382df1acdfc7ce5f97042a7342302a6c6e5d20a2ca3
-  data.tar.gz: 37bc84c9da07415c800a24af962008098c2b2f8711ac3c1f093ec09f3482a37c4aaa8ff531a52d51f9b9a8377d3485f9b2675c44a7d1bfa29a8d740b60c84130
+  metadata.gz: 4256867dac4a3a9978b4f5dfda36aba67f9d97461d9d074e694f048d51ae22cd2e835593524e5dcbcc9d3f87a34279cbcd295f6f66d0d372939c8daf5e73688f
+  data.tar.gz: c6cc148cae574dd9dce10ad9781cb9e088fff6c52fdde9f3b40463d7166689e23923b539ba077a4d224e3611f33b0fd7c3b2799e9308590516a6231f536a78cc

data/CHANGELOG.md

@@ -1,7 +1,24 @@
 # Change Log
 
-## [0.17.1](https://github.com/chef/inspec/tree/0.17.1) (2016-03-31)
-[Full Changelog](https://github.com/chef/inspec/compare/v0.17.0...0.17.1)
+## [0.18.0](https://github.com/chef/inspec/tree/0.18.0) (2016-04-09)
+[Full Changelog](https://github.com/chef/inspec/compare/v0.17.1...0.18.0)
+
+**Implemented enhancements:**
+
+- supports keyword marks tests as skipped instead of removing them [\#620](https://github.com/chef/inspec/pull/620) ([arlimus](https://github.com/arlimus))
+- Support only\_if in controls [\#619](https://github.com/chef/inspec/pull/619) ([arlimus](https://github.com/arlimus))
+- don't remove controls with only\_if [\#618](https://github.com/chef/inspec/pull/618) ([arlimus](https://github.com/arlimus))
+
+**Merged pull requests:**
+
+- Encourage sharing of profiles [\#625](https://github.com/chef/inspec/pull/625) ([nathenharvey](https://github.com/nathenharvey))
+- add travis and appveyor badges [\#622](https://github.com/chef/inspec/pull/622) ([chris-rock](https://github.com/chris-rock))
+- remove unused profile.tar.gz [\#621](https://github.com/chef/inspec/pull/621) ([chris-rock](https://github.com/chris-rock))
+- Extended gordon\_config with more examples [\#610](https://github.com/chef/inspec/pull/610) ([alexpop](https://github.com/alexpop))
+- Create ISSUE\_TEMPLATE [\#581](https://github.com/chef/inspec/pull/581) ([srenatus](https://github.com/srenatus))
+
+## [v0.17.1](https://github.com/chef/inspec/tree/v0.17.1) (2016-03-31)
+[Full Changelog](https://github.com/chef/inspec/compare/v0.17.0...v0.17.1)
 
 **Merged pull requests:**
 

data/README.md

@@ -1,6 +1,8 @@
 # InSpec: Inspect Your Infrastructure
 
 [![Join the chat at https://gitter.im/chef/inspec](https://badges.gitter.im/chef/inspec.svg)](https://gitter.im/chef/inspec?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
+[![Build Status Master](https://travis-ci.org/chef/inspec.svg?branch=master)](https://travis-ci.org/chef/inspec)
+[![Build Status Master](https://ci.appveyor.com/api/projects/status/github/chef/inspec?branch=master&svg=true&passingText=master%20-%20Ok&pendingText=master%20-%20Pending&failingText=master%20-%20Failing)](https://ci.appveyor.com/project/Chef/inspec/branch/master)
 
 InSpec is an open-source testing framework for infrastructure with a human- and machine-readable language for specifying compliance, security and policy requirements.
 
@@ -169,6 +171,12 @@ Which will provide you with:
 
 Documentation is available: https://github.com/chef/inspec/tree/master/docs
 
+## Share your Profiles
+
+You may share your InSpec Profiles in the [Tools & Plugins section](https://supermarket.chef.io/tools-directory) of the [Chef Supermarket](https://supermarket.chef.io/).  [Sign in](https://supermarket.chef.io/sign-in) and [add the details of your profile](https://supermarket.chef.io/tools/new).
+
+You may also [browse the Supermarket for shared Compliance Profiles](https://supermarket.chef.io/tools?type=compliance_profile).
+
 ## Kudos
 
 InSpec is inspired by the wonderful [Serverspec](http://serverspec.org) project. Kudos to [mizzy](https://github.com/mizzy) and [all contributors](https://github.com/mizzy/serverspec/graphs/contributors)!

data/examples/profile/README.md

@@ -12,7 +12,7 @@ Summary
 -------
 Location: examples/profile
 Profile: profile
-Controls: 3
+Controls: 4
 Timestamp: 2016-03-24T16:20:21+00:00
 Valid: true
 
@@ -32,7 +32,7 @@ $ inspec exec examples/profile
 ..
 
 Finished in 0.0025 seconds (files took 0.12449 seconds to load)
-4 examples, 0 failures
+8 examples, 0 failures
 ```
 
 ## Execute a specific control from a profile

data/examples/profile/controls/gordon.rb

@@ -18,8 +18,20 @@ control 'gordon-1.0' do
   tag 'gordon'
   ref 'Gordon Requirements 1.0', uri: 'http://...'
 
+  # Test using the custom gordon_config Inspec resource
+  # Find the resource content here: ../libraries/
   describe gordon_config do
+    it { should exist }
     its('version') { should eq('1.0') }
-    its('size') { should <= 20 }
+    its('file_size') { should <= 20 }
+    its('comma_count') { should eq 0 }
+  end
+
+  # Test the version again to showcase variables
+  g = gordon_config
+  g_path = g.file_path
+  g_version = g.version
+  describe file(g_path) do
+    its('content') { should match g_version }
   end
 end

data/examples/profile/libraries/gordon_config.rb

@@ -11,12 +11,13 @@ class GordonConfig < Inspec.resource(1)
   example "
     describe gordon_config do
       its('version') { should eq('1.0') }
-      its('size') { should > 1 }
+      its('file_size') { should > 1 }
     end
   "
 
   # Load the configuration file on initialization
   def initialize
+    @params = {}
     @path = '/tmp/gordon/config.yaml'
     @file = inspec.file(@path)
     return skip_resource "Can't find file \"#{@path}\"" if !@file.file?
@@ -24,20 +25,29 @@ class GordonConfig < Inspec.resource(1)
     # Protect from invalid YAML content
     begin
       @params = YAML.load(@file.content)
+      # Add two extra matchers
+      @params['file_size'] = @file.size
+      @params['file_path'] = @path
+      @params['ruby'] = 'RUBY IS HERE TO HELP ME!'
     rescue Exception
       return skip_resource "#{@file}: #{$!}"
     end
-    add_some_extra_params
   end
 
-  # Extra Ruby helper method
-  def add_some_extra_params
-    @params['size'] = @file.size
-    @params['md5sum'] = @file.md5sum
+  # Example method called by 'it { should exist }'
+  # Returns true or false from the 'File.exists?' method
+  def exists?
+    return File.exists?(@path)
+  end
+
+  # Example matcher for the number of commas in the file
+  def comma_count
+    text = @file.content
+    return text.count(',')
   end
 
   # Expose all parameters
   def method_missing(name)
-    @params[name.to_s]
+    return @params[name.to_s]
   end
 end

data/lib/inspec/dsl.rb

@@ -35,7 +35,7 @@ module Inspec::DSL
   # let it run. This happens after everything
   # else is merged in.
   def self.execute_rule(r, profile_id)
-    checks = r.instance_variable_get(:@checks)
+    checks = ::Inspec::Rule.prepare_checks(r)
     fid = InspecBaseRule.full_id(r, profile_id)
     checks.each do |m, a, b|
       # check if the resource is skippable and skipped

data/lib/inspec/profile.rb

@@ -256,7 +256,7 @@ module Inspec
           impact: rule.impact,
           refs: rule.ref,
           tags: rule.tag,
-          checks: rule.instance_variable_get(:@checks),
+          checks: Inspec::Rule.checks(rule),
           code: rule.instance_variable_get(:@__code),
           source_location: rule.instance_variable_get(:@__source_location),
           group_title: rule.instance_variable_get(:@__group_title),

data/lib/inspec/profile_context.rb

@@ -102,6 +102,7 @@ module Inspec
         def initialize(backend, conf) # rubocop:disable Lint/NestedMethodDefinition, Lint/DuplicateMethods
           @backend = backend
           @conf = conf
+          @skip_profile = false
         end
 
         define_method :title do |arg|
@@ -115,20 +116,7 @@ module Inspec
         define_method :control do |*args, &block|
           id = args[0]
           opts = args[1] || {}
-
-          # Skip the control if the resource triggered a skip;
-          # However: when this is run on a mock backend, do not skip it.
-          # This is e.g. relevant for JSON generation, where we need all
-          # controls.
-          return if @skip_profile && os[:family] != 'unknown'
-
-          profile_context_owner.register_rule(rule_class.new(id, opts, &block))
-        end
-
-        alias_method :rule, :control
-
-        define_method :register_control do |control|
-          profile_context_owner.register_rule(control) unless control.nil?
+          register_control(rule_class.new(id, opts, &block))
         end
 
         define_method :describe do |*args, &block|
@@ -139,10 +127,16 @@ module Inspec
           rule = rule_class.new(id, {}) do
             res = describe(*args, &block)
           end
-          profile_context_owner.register_rule(rule, &block)
+          register_control(rule, &block)
           res
         end
 
+        define_method :register_control do |control, &block|
+          ::Inspec::Rule.set_skip_rule(control, true) if @skip_profile
+
+          profile_context_owner.register_rule(control, &block) unless control.nil?
+        end
+
         # TODO: mock method for attributes; import attribute handling
         define_method :attributes do |_name, _options|
           nil
@@ -152,13 +146,14 @@ module Inspec
           profile_context_owner.unregister_rule(id)
         end
 
-        alias_method :skip_rule, :skip_control
-
         def only_if
           return unless block_given?
-          @skip_profile = !yield
+          @skip_profile ||= !yield
         end
 
+        alias_method :rule, :control
+        alias_method :skip_rule, :skip_control
+
         private
 
         def block_location(block, alternate_caller)

data/lib/inspec/rule.rb

@@ -15,16 +15,19 @@ module Inspec
     def initialize(id, _opts, &block)
       @id = id
       @impact = nil
-      @__block = block
-      @__code = __get_block_source(&block)
-      @__source_location = __get_block_source_location(&block)
       @title = nil
       @desc = nil
       @refs = []
       @tags = {}
+
       # not changeable by the user:
-      @profile_id = nil
-      @checks = []
+      @__block = block
+      @__code = __get_block_source(&block)
+      @__source_location = __get_block_source_location(&block)
+      @__rule_id = nil
+      @__checks = []
+      @__skip_rule = nil
+
       # evaluate the given definition
       instance_eval(&block) if block_given?
     end
@@ -70,6 +73,15 @@ module Inspec
       @tags
     end
 
+    # Skip all checks if only_if is false
+    #
+    # @param [Type] &block returns true if tests are added, false otherwise
+    # @return [nil]
+    def only_if
+      return unless block_given?
+      @__skip_rule ||= !yield
+    end
+
     # Describe will add one or more tests to this control. There is 2 ways
     # of calling it:
     #
@@ -87,25 +99,57 @@ module Inspec
         dsl = self.class.ancestors[1]
         Class.new(DescribeBase) do
           include dsl
-        end.new(method(:add_check))
+        end.new(method(:__add_check))
       else
-        add_check('describe', values, block)
+        __add_check('describe', values, block)
       end
     end
 
     def expect(value, &block)
       target = Inspec::Expect.new(value, &block)
-      add_check('expect', [value], target)
+      __add_check('expect', [value], target)
       target
     end
 
+    def self.rule_id(rule)
+      rule.instance_variable_get(:@__rule_id)
+    end
+
+    def self.set_rule_id(rule, value)
+      rule.instance_variable_set(:@__rule_id, value)
+    end
+
+    def self.checks(rule)
+      rule.instance_variable_get(:@__checks)
+    end
+
+    def self.skip_status(rule)
+      rule.instance_variable_get(:@__skip_rule)
+    end
+
+    def self.set_skip_rule(rule, value)
+      rule.instance_variable_set(:@__skip_rule, value)
+    end
+
+    def self.prepare_checks(rule)
+      msg = skip_status(rule)
+      return checks(rule) unless msg
+      msg = 'Skipped control due to only_if condition.' if msg == true
+
+      # TODO: we use os as the carrier here, but should consider
+      # a separate resource to do skipping
+      resource = rule.os
+      resource.skip_resource(msg)
+      [['describe', [resource], nil]]
+    end
+
     def self.merge(dst, src)
       if src.id != dst.id
         # TODO: register an error, this case should not happen
         return
       end
-      sp = src.instance_variable_get(:@profile_id)
-      dp = dst.instance_variable_get(:@profile_id)
+      sp = rule_id(src)
+      dp = rule_id(dst)
       if sp != dp
         # TODO: register an error, this case should not happen
         return
@@ -117,10 +161,10 @@ module Inspec
       # merge indirect fields
       # checks defined in the source will completely eliminate
       # all checks that were defined in the destination
-      sc = src.instance_variable_get(:@checks)
-      unless sc.nil? || sc.empty?
-        dst.instance_variable_set(:@checks, sc)
-      end
+      sc = checks(src)
+      dst.instance_variable_set(:@__checks, sc) unless sc.empty?
+      sr = skip_status(src)
+      set_skip_rule(dst, sr) unless sr.nil?
     end
 
     # Get the full id consisting of profile id + rule id
@@ -140,11 +184,8 @@ module Inspec
           return nil
         end
       end
-      pid = rule.instance_variable_get(:@profile_id)
-      if pid.nil?
-        rule.instance_variable_set(:@profile_id, profile_id)
-        pid = profile_id
-      end
+      pid = rule_id(rule)
+      pid = set_rule_id(rule, profile_id) if pid.nil?
 
       # if we don't have a profile id, just return the rule's ID
       return rid if pid.nil? or pid.empty?
@@ -154,8 +195,8 @@ module Inspec
 
     private
 
-    def add_check(describe_or_expect, values, block)
-      @checks.push([describe_or_expect, values, block])
+    def __add_check(describe_or_expect, values, block)
+      @__checks.push([describe_or_expect, values, block])
     end
 
     # Idio(ma)tic unindent

data/lib/inspec/runner.rb

@@ -53,9 +53,8 @@ module Inspec
     end
 
     def add_profile(profile, options = {})
-      return unless options[:ignore_supports] ||
-                    profile.metadata.supports_transport?(@backend)
       @test_collector.add_profile(profile)
+      options[:metadata] = profile.metadata
 
       libs = profile.libraries.map do |k, v|
         { ref: k, content: v }
@@ -87,6 +86,19 @@ module Inspec
       tests = [tests] unless tests.is_a? Array
       tests.each { |t| add_test_to_context(t, ctx) }
 
+      # skip based on support checks in metadata
+      meta = options[:metadata]
+      if !options[:ignore_supports] && !meta.nil? &&
+         !meta.supports_transport?(@backend)
+        os_info = @backend.os[:family].to_s
+        ctx.rules.values.each do |ctrl|
+          ::Inspec::Rule.set_skip_rule(
+            ctrl,
+            "This OS/platform (#{os_info}) is not supported by this profile.",
+          )
+        end
+      end
+
       # process the resulting rules
       filter_controls(ctx.rules, options[:controls]).each do |rule_id, rule|
         register_rule(rule_id, rule)
@@ -154,7 +166,7 @@ module Inspec
 
     def register_rule(rule_id, rule)
       @rules[rule_id] = rule
-      checks = rule.instance_variable_get(:@checks)
+      checks = ::Inspec::Rule.prepare_checks(rule)
       examples = checks.map do |m, a, b|
         get_check_example(m, a, b)
       end.flatten.compact

data/lib/inspec/version.rb

@@ -3,5 +3,5 @@
 # author: Christoph Hartmann
 
 module Inspec
-  VERSION = '0.17.1'.freeze
+  VERSION = '0.18.0'.freeze
 end

data/test/functional/inspec_exec_test.rb

@@ -12,7 +12,7 @@ describe 'inspec exec' do
     out.stderr.must_equal ''
     out.exit_status.must_equal 0
     out.stdout.must_match /^Pending: /
-    out.stdout.must_include '4 examples, 0 failures, 1 pending'
+    out.stdout.must_include '5 examples, 0 failures, 1 pending'
   end
 
   it 'executes only specified controls' do
@@ -36,8 +36,8 @@ describe 'inspec exec' do
     let(:ex2) { examples.find{|x| x['id'] =~ /generated/} }
     let(:ex3) { examples.find{|x| x['id'] == 'gordon-1.0'} }
 
-    it 'must have 4 examples' do
-      json['examples'].length.must_equal 4
+    it 'must have 5 examples' do
+      json['examples'].length.must_equal 5
     end
 
     it 'id in json' do
@@ -82,8 +82,8 @@ describe 'inspec exec' do
       })
     end
 
-    it 'must have 4 examples' do
-      json['examples'].length.must_equal 4
+    it 'must have 5 examples' do
+      json['examples'].length.must_equal 5
     end
 
     it 'id in json' do
@@ -138,4 +138,26 @@ describe 'inspec exec' do
     end
   end
 
+  describe 'with a profile that is not supported on this OS/platform' do
+    let(:out) { inspec('exec ' + File.join(profile_path, 'skippy-profile-os') + ' --format fulljson') }
+    let(:json) { JSON.load(out.stdout) }
+
+    it 'exits cleanly' do
+      out.stderr.must_equal ''
+      out.exit_status.must_equal 0
+    end
+
+    it 'has one pending' do
+      json['summary']['pending_count'].must_equal 1
+      json['summary']['example_count'].must_equal 1
+    end
+
+    it 'delivers the pending message' do
+      json['examples'][0]['pending'].must_match %r{^This OS/platform \([^)]+\) is not supported by this profile\.$}
+    end
+
+    it 'never runs the actual resource' do
+      File.exist?('/tmp/inspec_test_DONT_CREATE').must_equal false
+    end
+  end
 end

data/test/unit/mock/profiles/skippy-profile-os/controls/one.rb

@@ -0,0 +1,3 @@
+describe command('touch /tmp/inspec_test_DONT_CREATE') do
+  its(:exit_status) { should eq 123 }
+end

data/test/unit/mock/profiles/skippy-profile-os/inspec.yml

@@ -0,0 +1,5 @@
+name: skippy
+title: skip-like functionality
+version: 1.0.0
+supports:
+- os-family: definitely_not_supported

data/test/unit/profile_context_test.rb

@@ -60,7 +60,7 @@ describe Inspec::ProfileContext do
   end
 
   def get_checks
-    get_rule.instance_variable_get(:@checks)
+    Inspec::Rule.prepare_checks(get_rule)
   end
 
   it 'must be able to load empty content' do
@@ -117,7 +117,71 @@ describe Inspec::ProfileContext do
       load('expect(true).to_eq true').must_raise NoMethodError
     end
 
-    it 'provides the rule keyword in the global DSL' do
+    describe 'global only_if' do
+      let(:if_true) { "only_if { true }\n" }
+      let(:if_false) { "only_if { false }\n" }
+      let(:describe) { "describe nil do its(:to_i) { should eq rand } end\n" }
+      let(:control) { "control 1 do\n#{describe}end" }
+
+      it 'provides the keyword' do
+        profile.load(if_true)
+        profile.rules.must_equal({})
+      end
+
+      it 'doesnt affect controls when positive' do
+        profile.load(if_true + 'control 1')
+        profile.rules.values[0].must_be_kind_of Inspec::Rule
+      end
+
+      it 'doesnt remove controls when negative' do
+        profile.load(if_false + 'control 1')
+        profile.rules.values[0].must_be_kind_of Inspec::Rule
+      end
+
+      it 'alters controls when positive' do
+        profile.load(if_false + control)
+        get_checks.length.must_equal 1
+        get_checks[0][1][0].resource_skipped.must_equal 'Skipped control due to only_if condition.'
+      end
+
+      it 'alters non-controls when positive' do
+        profile.load(if_false + describe)
+        get_checks.length.must_equal 1
+        get_checks[0][1][0].resource_skipped.must_equal 'Skipped control due to only_if condition.'
+      end
+
+      it 'doesnt alter controls when negative' do
+        profile.load(if_true + control)
+        get_checks.length.must_equal 1
+        get_checks[0][1][0].must_be_nil
+      end
+
+      it 'doesnt alter non-controls when negative' do
+        profile.load(if_true + describe)
+        get_checks.length.must_equal 1
+        get_checks[0][1][0].must_be_nil
+      end
+
+      it 'doesnt overwrite falsy only_ifs' do
+        profile.load(if_false + if_true + control)
+        get_checks.length.must_equal 1
+        get_checks[0][1][0].resource_skipped.must_equal 'Skipped control due to only_if condition.'
+      end
+
+      it 'doesnt overwrite falsy only_ifs' do
+        profile.load(if_true + if_false + control)
+        get_checks.length.must_equal 1
+        get_checks[0][1][0].resource_skipped.must_equal 'Skipped control due to only_if condition.'
+      end
+    end
+
+    it 'provides the control keyword in the global DSL' do
+      profile.load('control 1')
+      profile.rules.keys.must_equal [1]
+      profile.rules.values[0].must_be_kind_of Inspec::Rule
+    end
+
+    it 'provides the rule keyword in the global DSL (legacy mode)' do
       profile.load('rule 1')
       profile.rules.keys.must_equal [1]
       profile.rules.values[0].must_be_kind_of Inspec::Rule
@@ -137,7 +201,7 @@ describe Inspec::ProfileContext do
     it 'doesnt add any checks if none are provided' do
       profile.load("rule #{rule_id.inspect}")
       rule = profile.rules[rule_id]
-      rule.instance_variable_get(:@checks).must_equal([])
+      ::Inspec::Rule.prepare_checks(rule).must_equal([])
     end
 
     describe 'supports empty describe blocks' do
@@ -212,5 +276,35 @@ describe Inspec::ProfileContext do
         check[2].must_be_kind_of Proc
       end
     end
+
+    describe 'with only_if' do
+      it 'provides the only_if keyword' do
+        profile.load(format(context_format, 'only_if'))
+        get_checks.must_equal([])
+      end
+
+      it 'skips with only_if == false' do
+        profile.load(format(context_format, 'only_if { false }'))
+        get_checks.length.must_equal 1
+        get_checks[0][1][0].resource_skipped.must_equal 'Skipped control due to only_if condition.'
+      end
+
+      it 'does nothing with only_if == false' do
+        profile.load(format(context_format, 'only_if { true }'))
+        get_checks.length.must_equal 0
+      end
+
+      it 'doesnt overwrite falsy only_ifs' do
+        profile.load(format(context_format, "only_if { false }\nonly_if { true }"))
+        get_checks.length.must_equal 1
+        get_checks[0][1][0].resource_skipped.must_equal 'Skipped control due to only_if condition.'
+      end
+
+      it 'doesnt overwrite falsy only_ifs' do
+        profile.load(format(context_format, "only_if { true }\nonly_if { false }"))
+        get_checks.length.must_equal 1
+        get_checks[0][1][0].resource_skipped.must_equal 'Skipped control due to only_if condition.'
+      end
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: inspec
 version: !ruby/object:Gem::Version
-  version: 0.17.1
+  version: 0.18.0
 platform: ruby
 authors:
 - Dominik Richter
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-31 00:00:00.000000000 Z
+date: 2016-04-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: r-train
@@ -335,7 +335,6 @@ files:
 - lib/utils/filter_array.rb
 - lib/utils/find_files.rb
 - lib/utils/hash.rb
-- lib/utils/hash_map.rb
 - lib/utils/json_log.rb
 - lib/utils/modulator.rb
 - lib/utils/object_traversal.rb
@@ -518,9 +517,9 @@ files:
 - test/unit/mock/profiles/legacy-empty-metadata/metadata.rb
 - test/unit/mock/profiles/legacy-simple-metadata/metadata.rb
 - test/unit/mock/profiles/legacy-simple-metadata/test/.gitkeep
-- test/unit/mock/profiles/resource-tiny/inspec.yml
-- test/unit/mock/profiles/resource-tiny/libraries/resource.rb
 - test/unit/mock/profiles/simple-metadata/inspec.yml
+- test/unit/mock/profiles/skippy-profile-os/controls/one.rb
+- test/unit/mock/profiles/skippy-profile-os/inspec.yml
 - test/unit/plugin_test.rb
 - test/unit/plugins/resource_test.rb
 - test/unit/profile_context_test.rb
@@ -576,7 +575,6 @@ files:
 - test/unit/source_readers/inspec_test.rb
 - test/unit/utils/filter_array_test.rb
 - test/unit/utils/find_files_test.rb
-- test/unit/utils/hash_map_test.rb
 - test/unit/utils/passwd_parser_test.rb
 - test/unit/utils/simpleconfig_test.rb
 - test/unit/utils/solaris_netstat_parser.rb
@@ -600,7 +598,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.4.6
 signing_key: 
 specification_version: 4
 summary: Infrastructure and compliance testing.
@@ -780,9 +778,9 @@ test_files:
 - test/unit/mock/profiles/legacy-empty-metadata/metadata.rb
 - test/unit/mock/profiles/legacy-simple-metadata/metadata.rb
 - test/unit/mock/profiles/legacy-simple-metadata/test/.gitkeep
-- test/unit/mock/profiles/resource-tiny/inspec.yml
-- test/unit/mock/profiles/resource-tiny/libraries/resource.rb
 - test/unit/mock/profiles/simple-metadata/inspec.yml
+- test/unit/mock/profiles/skippy-profile-os/controls/one.rb
+- test/unit/mock/profiles/skippy-profile-os/inspec.yml
 - test/unit/plugin_test.rb
 - test/unit/plugins/resource_test.rb
 - test/unit/profile_context_test.rb
@@ -838,7 +836,6 @@ test_files:
 - test/unit/source_readers/inspec_test.rb
 - test/unit/utils/filter_array_test.rb
 - test/unit/utils/find_files_test.rb
-- test/unit/utils/hash_map_test.rb
 - test/unit/utils/passwd_parser_test.rb
 - test/unit/utils/simpleconfig_test.rb
 - test/unit/utils/solaris_netstat_parser.rb

data/lib/utils/hash_map.rb

@@ -1,37 +0,0 @@
-# encoding: utf-8
-# author: Dominik Richter
-# author: Christoph Hartmann
-
-class HashMap
-  class << self
-    def [](hash, *keys)
-      return hash if keys.empty? || hash.nil?
-      key = keys.shift
-      if hash.is_a?(Array)
-        map = hash.map { |i| [i, key] }
-      else
-        map = hash[key]
-      end
-      [map, *keys]
-    rescue NoMethodError => _
-      nil
-    end
-  end
-end
-
-class StringMap
-  class << self
-    def [](hash, *keys)
-      return hash if keys.empty? || hash.nil?
-      key = keys.shift
-      if hash.is_a?(Array)
-        map = hash.map { |i| [i, key] }
-      else
-        map = hash[key]
-      end
-      [map, *keys]
-    rescue NoMethodError => _
-      nil
-    end
-  end
-end

data/test/unit/mock/profiles/resource-tiny/inspec.yml

@@ -1,10 +0,0 @@
-name: complete
-title: complete example profile
-maintainer: Chef Software, Inc.
-copyright: Chef Software, Inc.
-copyright_email: support@chef.io
-license: Proprietary, All rights reserved
-summary: Testing stub
-version: 1.0.0
-supports:
-- os-family: linux

data/test/unit/mock/profiles/resource-tiny/libraries/resource.rb

@@ -1,3 +0,0 @@
-class Tiny < Inspec.resource(1)
-  name 'tiny'
-end

data/test/unit/utils/hash_map_test.rb

@@ -1,63 +0,0 @@
-# encoding: utf-8
-# author: Dominik Richter
-# author: Christoph Hartmann
-
-require 'helper'
-require 'utils/hash_map'
-
-describe HashMap do
-  describe '#get_value' do
-    it 'returns the source value if no keys are given' do
-      x = rand
-      HashMap[x].must_equal x
-    end
-
-    it 'retrieves a key from a map' do
-      x = { a: rand }
-      HashMap[x, :a].must_equal x[:a]
-    end
-
-    it 'returns nil if the key doesnt exist' do
-      x = { a: rand }
-      HashMap[x, :b].must_be :nil?
-    end
-
-    it 'can extract a nil key' do
-      x = { nil => rand }
-      HashMap[x, nil].must_equal x[nil]
-    end
-
-    it 'extracts multiple keys' do
-      y = rand
-      x = { a: { b: { c: y }}}
-      HashMap[x, :a, :b, :c].must_equal y
-    end
-
-    it 'returns nil if a key in a list doesnt exist' do
-      x = { a: { b: { c: rand }}}
-      HashMap[x, :d, :b, :c].must_be :nil?
-    end
-
-    it 'retrieves exact keys by string' do
-      x = { a: rand }
-      HashMap[x, :a].must_equal x[:a]
-      HashMap[x, 'a'].must_be :nil?
-    end
-
-    it 'retrieves exact keys by string' do
-      x = { 'a' => rand }
-      HashMap[x, 'a'].must_equal x['a']
-      HashMap[x, :a].must_be :nil?
-    end
-
-    it 'con retrieve string keys' do
-      x = { 'a' => rand }
-      HashMap.extract_keys(x, ['a'], string_keys: true).must_equal x['a']
-      HashMap.extract_keys(x, [:a], string_keys: true).must_equal x['a']
-    end
-  end
-
-  describe 'as an instance' do
-    
-  end
-end