inspec-core 4.20.6 → 4.20.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bad597d54eeb47c2f634b9c2bba2fcec8afa0d507e0b321244caee264c3b59cf
-  data.tar.gz: 966803761b3601c83b1d2474b3405b863b245396f8e4962fc950484ed87ee9ea
+  metadata.gz: ef611d3b2bb1d1c8ddca64adb1ed3eab50fcbec55a45cb49334ed2084c45e7da
+  data.tar.gz: 96664cb6183b137db4f84cd26aed2b19496475045bbb458ea9d9004e92aad3f7
 SHA512:
-  metadata.gz: 2ea62a2de16846b6d3bd1075868e070e41a14d9a8dce9a711155e7d75519eb56fa23238fa4cc3b53a97480396c4bd7a1b20a98251918268a66278c970ab0cda1
-  data.tar.gz: a420f3368a7ae4b6f264f15901848e7615bd6e7fefac559dda3c955d83b7059dd59464e685cef5e4310da2d503e61f72c254739c4a054e7b6f76c7bd5f61ef37
+  metadata.gz: f1d44fc61e4663862a0628a0fcc4c4e3538bdb35d50cee94f1130c47fed5a982297da9c559128b0dddc2a7375658252924959020f24f518202b25939357340de
+  data.tar.gz: e33e28b5863ce15a54c8f2f76dd3a458762c24fb7d365ba145ae06b6b0be14134b90da5f2610d834ae71a170e5f6b180c78a8ac8763f484b3bbcfa808eec8565

data/lib/inspec/cli.rb

@@ -375,6 +375,12 @@ class Inspec::InspecCLI < Inspec::BaseCLI
     puts "Valid schemas are #{Inspec::Schema::OutputSchema.names.join(", ")}"
   end
 
+  desc "run_context", "used to test run-context detection", hide: true
+  def run_context
+    require "inspec/utils/telemetry/run_context_probe"
+    puts Inspec::Telemetry::RunContextProbe.guess_run_context
+  end
+
   desc "version", "prints the version of this tool"
   option :format, type: :string
   def version
@@ -387,11 +393,6 @@ class Inspec::InspecCLI < Inspec::BaseCLI
   end
   map %w{-v --version} => :version
 
-  desc "nothing", "does nothing"
-  def nothing
-    puts "you did nothing"
-  end
-
   private
 
   def run_command(opts)

data/lib/inspec/config.rb

@@ -341,6 +341,7 @@ module Inspec
       # Tracked on https://github.com/inspec/inspec/issues/3667
       inspec_reporters_that_are_not_yet_plugins = %w{
         automate
+        cli
         json
         json-automate
         junit
@@ -354,12 +355,7 @@ module Inspec
         .find_activators(plugin_type: :reporter)\
         .map(&:activator_name).map(&:to_s)
 
-      # Include CLI explicitly, because it is the default reporter and will usually not be
-      # present on the command line, and thus its plugin will not appear on the list.
-      valid_types = rspec_built_in_formatters + \
-        inspec_reporters_that_are_not_yet_plugins + \
-        plugin_reporters + \
-        [ "cli" ]
+      valid_types = rspec_built_in_formatters + inspec_reporters_that_are_not_yet_plugins + plugin_reporters
 
       reporters.each do |reporter_name, reporter_config|
         raise NotImplementedError, "'#{reporter_name}' is not a valid reporter type." unless valid_types.include?(reporter_name)

data/lib/inspec/reporters.rb

@@ -1,4 +1,5 @@
 require "inspec/reporters/base"
+require "inspec/reporters/cli"
 require "inspec/reporters/json"
 require "inspec/reporters/json_automate"
 require "inspec/reporters/junit"
@@ -11,6 +12,8 @@ module Inspec::Reporters
     name, config = reporter.dup
     config[:run_data] = run_data
     case name
+    when "cli"
+      reporter = Inspec::Reporters::CLI.new(config)
     when "json"
       reporter = Inspec::Reporters::Json.new(config)
     # This reporter is only used for Chef internal. We reserve the

data/lib/{plugins/inspec-reporter-cli/lib/inspec-reporter-cli/reporter.rb → inspec/reporters/cli.rb}

@@ -1,8 +1,5 @@
-require "forwardable"
-
-module InspecPlugins::CliReporter
-  class Reporter < Inspec.plugin(2, :reporter)
-
+module Inspec::Reporters
+  class CLI < Base
     case RUBY_PLATFORM
     when /windows|mswin|msys|mingw|cygwin/
       # Most currently available Windows terminals have poor support
@@ -43,15 +40,11 @@ module InspecPlugins::CliReporter
 
     MULTI_TEST_CONTROL_SUMMARY_MAX_LEN = 60
 
-    def self.run_data_schema_constraints
-      "~> 0.0"
-    end
-
     def render
-      run_data.profiles.each do |profile|
-        if profile.status == "skipped"
-          platform = run_data.platform
-          output("Skipping profile: '#{profile.name}' on unsupported platform: '#{platform.name}/#{platform.release}'.")
+      run_data[:profiles].each do |profile|
+        if profile[:status] == "skipped"
+          platform = run_data[:platform]
+          output("Skipping profile: '#{profile[:name]}' on unsupported platform: '#{platform[:name]}/#{platform[:release]}'.")
           next
         end
         @control_count = 0
@@ -77,9 +70,9 @@ module InspecPlugins::CliReporter
     def print_profile_header(profile)
       header = {
         "Profile" => format_profile_name(profile),
-        "Version" => profile.version || "(not specified)",
+        "Version" => profile[:version] || "(not specified)",
       }
-      header["Target"] = run_data.platform.target unless run_data.platform.target.nil?
+      header["Target"] = run_data[:platform][:target] unless run_data[:platform][:target].nil?
       header["Target ID"] = @config["target_id"] unless @config["target_id"].nil?
 
       pad = header.keys.max_by(&:length).length + 1
@@ -91,8 +84,8 @@ module InspecPlugins::CliReporter
 
     def print_standard_control_results(profile)
       standard_controls_from_profile(profile).each do |control_from_profile|
-        control = ControlForCliDisplay.new(control_from_profile)
-        next if control.results.empty?
+        control = Control.new(control_from_profile)
+        next if control.results.nil?
 
         output(format_control_header(control))
         control.results.each do |result|
@@ -105,8 +98,8 @@ module InspecPlugins::CliReporter
 
     def print_anonymous_control_results(profile)
       anonymous_controls_from_profile(profile).each do |control_from_profile|
-        control = ControlForCliDisplay.new(control_from_profile)
-        next if control.results.empty?
+        control = Control.new(control_from_profile)
+        next if control.results.nil?
 
         output(format_control_header(control))
         control.results.each do |result|
@@ -117,10 +110,10 @@ module InspecPlugins::CliReporter
     end
 
     def format_profile_name(profile)
-      if profile.title.nil?
-        (profile.name || "unknown").to_s
+      if profile[:title].nil?
+        (profile[:name] || "unknown").to_s
       else
-        "#{profile.title} (#{profile.name || "unknown"})"
+        "#{profile[:title]} (#{profile[:name] || "unknown"})"
       end
     end
 
@@ -136,16 +129,16 @@ module InspecPlugins::CliReporter
     def format_result(control, result, type)
       impact = control.impact_string_for_result(result)
 
-      message = if result.status == "skipped"
-                  result.skip_message
+      message = if result[:status] == "skipped"
+                  result[:skip_message]
                 elsif type == :anonymous
-                  result.expectation_message
+                  result[:expectation_message]
                 else
-                  result.code_desc
+                  result[:code_desc]
                 end
 
       # append any failure details to the message if they exist
-      message += "\n#{result.message}" if result.message
+      message += "\n#{result[:message]}" if result[:message]
 
       format_message(
         color: impact,
@@ -177,7 +170,9 @@ module InspecPlugins::CliReporter
 
     def all_unique_controls
       @unique_controls ||= begin
-                             run_data.profiles.flat_map(&:controls).uniq
+                             run_data[:profiles].flat_map do |profile|
+                               profile[:controls]
+                             end.uniq
                            end
     end
 
@@ -187,12 +182,12 @@ module InspecPlugins::CliReporter
       passed = 0
 
       all_unique_controls.each do |control|
-        next if control.id.start_with? "(generated from "
-        next if control.results.empty?
+        next if control[:id].start_with? "(generated from "
+        next unless control[:results]
 
-        if control.results.any? { |r| r.status == "failed" }
+        if control[:results].any? { |r| r[:status] == "failed" }
           failed += 1
-        elsif control.results.any? { |r| r.status == "skipped" }
+        elsif control[:results].any? { |r| r[:status] == "skipped" }
           skipped += 1
         else
           passed += 1
@@ -216,12 +211,12 @@ module InspecPlugins::CliReporter
       passed = 0
 
       all_unique_controls.each do |control|
-        next if control.results.empty?
+        next unless control[:results]
 
-        control.results.each do |result|
-          if result.status == "failed"
+        control[:results].each do |result|
+          if result[:status] == "failed"
             failed += 1
-          elsif result.status == "skipped"
+          elsif result[:status] == "skipped"
             skipped += 1
           else
             passed += 1
@@ -278,30 +273,42 @@ module InspecPlugins::CliReporter
     end
 
     def standard_controls_from_profile(profile)
-      profile.controls.reject { |c| is_anonymous_control?(c) }
+      profile[:controls].reject { |c| is_anonymous_control?(c) }
     end
 
     def anonymous_controls_from_profile(profile)
-      profile.controls.select { |c| is_anonymous_control?(c) && !c[:results].nil? }
+      profile[:controls].select { |c| is_anonymous_control?(c) && !c[:results].nil? }
     end
 
     def is_anonymous_control?(control)
-      control.id.start_with?("(generated from ")
+      control[:id].start_with?("(generated from ")
     end
 
     def indent_lines(message, indentation)
       message.lines.map { |line| " " * indentation + line }.join
     end
 
-    class ControlForCliDisplay
+    class Control
+      attr_reader :data
 
-      extend Forwardable
+      def initialize(control_hash)
+        @data = control_hash
+      end
 
-      attr_reader :control_obj
-      def_delegators :@control_obj, :id, :impact, :results, :title
+      def id
+        data[:id]
+      end
+
+      def title
+        data[:title]
+      end
+
+      def results
+        data[:results]
+      end
 
-      def initialize(control_obj)
-        @control_obj = control_obj
+      def impact
+        data[:impact]
       end
 
       def anonymous?
@@ -310,9 +317,9 @@ module InspecPlugins::CliReporter
 
       def title_for_report
         # if this is an anonymous control, just grab the resource title from any result entry
-        return results.first.resource_title if anonymous?
+        return results.first[:resource_title] if anonymous?
 
-        title_for_report = "#{id}: #{title || results.first.resource_title}"
+        title_for_report = "#{id}: #{title || results.first[:resource_title]}"
 
         # we will not add any additional data to the title if there's only
         # zero or one test for this control.
@@ -330,9 +337,9 @@ module InspecPlugins::CliReporter
           nil
         elsif impact.nil?
           "unknown"
-        elsif results&.find { |r| r.status == "skipped" }
+        elsif results&.find { |r| r[:status] == "skipped" }
           "skipped"
-        elsif results.empty? || results.all? { |r| r.status == "passed" }
+        elsif results.nil? || results.empty? || results.all? { |r| r[:status] == "passed" }
           "passed"
         else
           "failed"
@@ -340,9 +347,9 @@ module InspecPlugins::CliReporter
       end
 
       def impact_string_for_result(result)
-        if result.status == "skipped"
+        if result[:status] == "skipped"
           "skipped"
-        elsif result.status == "passed"
+        elsif result[:status] == "passed"
           "passed"
         elsif impact.nil?
           "unknown"
@@ -352,11 +359,11 @@ module InspecPlugins::CliReporter
       end
 
       def failure_count
-        results.select { |r| r.status == "failed" }.size
+        results.select { |r| r[:status] == "failed" }.size
       end
 
       def skipped_count
-        results.select { |r| r.status == "skipped" }.size
+        results.select { |r| r[:status] == "skipped" }.size
       end
     end
   end

data/lib/inspec/utils/telemetry/run_context_probe.rb

@@ -0,0 +1,48 @@
+module Inspec
+  module Telemetry
+    # Guesses the run context of InSpec - how were we invoked?
+    # All stack values here are determined experimentally
+
+    class RunContextProbe
+      def self.guess_run_context(stack = nil)
+        stack ||= caller_locations
+        return "test-kitchen" if kitchen?(stack)
+        return "cli" if run_by_thor?(stack)
+        return "audit-cookbook" if audit_cookbook?(stack)
+
+        "unknown"
+      end
+
+      def self.run_by_thor?(stack)
+        stack_match(stack: stack, path: "thor/command", label: "run") &&
+          stack_match(stack: stack, path: "thor/invocation", label: "invoke_command")
+      end
+
+      def self.kitchen?(stack)
+        stack_match(stack: stack, path: "kitchen/instance", label: "verify_action") &&
+          stack_match(stack: stack, path: "kitchen/instance", label: "verify")
+      end
+
+      def self.audit_cookbook?(stack)
+        stack_match(stack: stack, path: "chef/handler", label: "run_report_handlers") &&
+          stack_match(stack: stack, path: "handler/audit_report", label: "report")
+      end
+
+      def self.stack_match(stack: [], label: nil, path: nil)
+        return false if stack.nil?
+
+        stack.any? do |frame|
+          if label && path
+            frame.label == label && frame.absolute_path.include?(path)
+          elsif label
+            frame.label == label
+          elsif path
+            frame.absolute_path.include?(path)
+          else
+            false
+          end
+        end
+      end
+    end
+  end
+end

data/lib/inspec/version.rb

@@ -1,3 +1,3 @@
 module Inspec
-  VERSION = "4.20.6".freeze
+  VERSION = "4.20.10".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: inspec-core
 version: !ruby/object:Gem::Version
-  version: 4.20.6
+  version: 4.20.10
 platform: ruby
 authors:
 - Chef InSpec Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-06-17 00:00:00.000000000 Z
+date: 2020-06-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: chef-telemetry
@@ -480,6 +480,7 @@ files:
 - lib/inspec/reporters.rb
 - lib/inspec/reporters/automate.rb
 - lib/inspec/reporters/base.rb
+- lib/inspec/reporters/cli.rb
 - lib/inspec/reporters/json.rb
 - lib/inspec/reporters/json_automate.rb
 - lib/inspec/reporters/junit.rb
@@ -665,6 +666,7 @@ files:
 - lib/inspec/utils/telemetry/collector.rb
 - lib/inspec/utils/telemetry/data_series.rb
 - lib/inspec/utils/telemetry/global_methods.rb
+- lib/inspec/utils/telemetry/run_context_probe.rb
 - lib/inspec/version.rb
 - lib/matchers/matchers.rb
 - lib/plugins/README.md
@@ -722,10 +724,6 @@ files:
 - lib/plugins/inspec-plugin-manager-cli/lib/inspec-plugin-manager-cli.rb
 - lib/plugins/inspec-plugin-manager-cli/lib/inspec-plugin-manager-cli/cli_command.rb
 - lib/plugins/inspec-plugin-manager-cli/lib/inspec-plugin-manager-cli/plugin.rb
-- lib/plugins/inspec-reporter-cli/README.md
-- lib/plugins/inspec-reporter-cli/lib/inspec-reporter-cli.rb
-- lib/plugins/inspec-reporter-cli/lib/inspec-reporter-cli/reporter.rb
-- lib/plugins/inspec-reporter-cli/lib/inspec-reporter-cli/version.rb
 - lib/plugins/inspec-reporter-json-min/README.md
 - lib/plugins/inspec-reporter-json-min/lib/inspec-reporter-json-min.rb
 - lib/plugins/inspec-reporter-json-min/lib/inspec-reporter-json-min/reporter.rb

data/lib/plugins/inspec-reporter-cli/README.md

@@ -1,12 +0,0 @@
-# cli reporter
-
-This is the implementation of the "cli" reporter. It is the default reporter used if you do not specify a reporter.
-
-## To Install This Plugin
-
-This plugin is included with inspec. There is no need to install it separately.
-
-## What This Plugin Does
-
-This reporter generates a summary of output at the end of the run to STDOUT by default. By default it uses color and terminal graphics if possible, depending on the capabilities of the host platform.
-

data/lib/plugins/inspec-reporter-cli/lib/inspec-reporter-cli.rb

@@ -1,13 +0,0 @@
-require_relative "inspec-reporter-cli/version"
-
-module InspecPlugins
-  module CliReporter
-    class Plugin < ::Inspec.plugin(2)
-      plugin_name :'inspec-reporter-cli'
-      reporter :cli do
-        require_relative "inspec-reporter-cli/reporter"
-        InspecPlugins::CliReporter::Reporter
-      end
-    end
-  end
-end

data/lib/plugins/inspec-reporter-cli/lib/inspec-reporter-cli/version.rb

@@ -1,5 +0,0 @@
-module InspecPlugins
-  module CliReporter
-    VERSION = "0.1.0".freeze
-  end
-end