chef-cli 3.0.11 → 3.0.24

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b8a428266246dc22388ebf2714824965912bf1b17fa657bc9ffab99fef6f0a88
-  data.tar.gz: aed196c6c1cae0d61a877e6916d5514313c0b186faae75598402fcacfee0c756
+  metadata.gz: 313ac394fc05556ea7a12748f3d00256a17556e57c97be2c8cc8b875233f2607
+  data.tar.gz: 20e16fbae37e2c6bf8615bdc2d161d96d5a6f74791294f59c39bc509072f99fa
 SHA512:
-  metadata.gz: 343ec04d69d8133e65d887f5e71ab53416ba2a09013df8f530ba70cb2d2f6dcc9b60c8b9ea4c3e445e9a223175681c5d61babc9339ef381483707219d0f5cf71
-  data.tar.gz: b1fe61b9c04025de73c654d0c7bd65033a34935198ba53a68a921eb1bf1f2c1d4c7c30e55ee45eb4168e304402f3412fc30e47f8230af22d9c4f802b424a280e
+  metadata.gz: af4ff4840e87a8dba88b0f4ddc527800e6f2826f7a8e6d90da4d6c364e4d013098d707a046322948858ced67b0666ab3fc737d8c1aa9332f162ec99f8f53d9c2
+  data.tar.gz: b9b76d0a67b78d4803d8ee924986d66a7d65723384850f1baaa59063917b28bf48456ebbad0fb7ba5fc283f1eccfe75c942b2bce7556070caeac71ca1c81bd1e

data/Gemfile

@@ -13,8 +13,8 @@ group :test do
   gem "rspec", "~> 3.8"
   gem "rspec-expectations", "~> 3.8"
   gem "rspec-mocks", "~> 3.8"
-  gem "cookstyle"
-  gem "chefstyle"
+  gem "cookstyle", "6.14.7" # this forces dependabot PRs to open which triggers cookstyle CI on the chef generate command
+  gem "chefstyle", "1.2.0"
   gem "test-kitchen", "> 2.5"
   if Gem::Version.new(RUBY_VERSION) < Gem::Version.new("2.6")
     gem "chef-zero", "~> 14"

data/chef-cli.gemspec

@@ -48,6 +48,6 @@ Gem::Specification.new do |gem|
   gem.add_dependency "addressable", ">= 2.3.5", "< 2.8"
   gem.add_dependency "cookbook-omnifetch", "~> 0.5"
   gem.add_dependency "diff-lcs", ">= 1.0", "< 1.4" # 1.4 changes the output
-  gem.add_dependency "paint", ">= 1", "< 3"
+  gem.add_dependency "pastel", "~> 0.7" # used for policyfile differ
   gem.add_dependency "license-acceptance", "~> 1.0", ">= 1.0.11"
 end

data/lib/chef-cli/cli.rb

@@ -59,7 +59,7 @@ module ChefCLI
     end
 
     def run(enforce_license: false)
-      sanity_check!
+      path_check!
 
       subcommand_name, *subcommand_params = argv
 
@@ -99,42 +99,21 @@ module ChefCLI
       if omnibus_install?
         show_version_via_version_manifest
       else
-        show_version_via_shell_out
+        msg("#{ChefCLI::Dist::CLI_PRODUCT} version: #{ChefCLI::VERSION}")
       end
     end
 
     def show_version_via_version_manifest
-      msg("#{ChefCLI::Dist::PRODUCT} version: #{manifest_field("build_version")}")
-      { "#{ChefCLI::Dist::INFRA_CLIENT_PRODUCT}": "#{ChefCLI::Dist::INFRA_CLIENT_GEM}",
-        "#{ChefCLI::Dist::INSPEC_PRODUCT}": "#{ChefCLI::Dist::INSPEC_CLI}",
-        "#{ChefCLI::Dist::CLI_PRODUCT}": "#{ChefCLI::Dist::CLI_GEM}",
+      msg("#{ChefCLI::Dist::PRODUCT} version: #{component_version("build_version")}")
+
+      { "#{ChefCLI::Dist::INFRA_CLIENT_PRODUCT}": ChefCLI::Dist::INFRA_CLIENT_GEM,
+        "#{ChefCLI::Dist::INSPEC_PRODUCT}": ChefCLI::Dist::INSPEC_CLI,
+        "#{ChefCLI::Dist::CLI_PRODUCT}": ChefCLI::Dist::CLI_GEM,
+        "#{ChefCLI::Dist::HAB_PRODUCT}": ChefCLI::Dist::HAB_SOFTWARE_NAME,
         "Test Kitchen": "test-kitchen",
         "Cookstyle": "cookstyle",
-      }.each do |name, gem|
-        msg("#{name} version: #{gem_version(gem)}")
-      end
-    end
-
-    def show_version_via_shell_out
-      msg("#{ChefCLI::Dist::PRODUCT} version: #{ChefCLI::VERSION}")
-      { "#{ChefCLI::Dist::INFRA_CLIENT_PRODUCT}": "#{ChefCLI::Dist::INFRA_CLIENT_CLI}",
-        "#{ChefCLI::Dist::INSPEC_PRODUCT}": "#{ChefCLI::Dist::INSPEC_CLI}",
-        "Test Kitchen": "kitchen",
-        "Cookstyle": "cookstyle",
-      }.each do |name, cli|
-        # @todo when Ruby 2.5/2.6 support goes away this if statement can go away
-        if Gem::Version.new(Bundler::VERSION) >= Gem::Version.new("2")
-          result = Bundler.with_unbundled_env { shell_out("#{cli} --version") }
-        else
-          result = Bundler.with_clean_env { shell_out("#{cli} --version") }
-        end
-
-        if result.exitstatus != 0
-          msg("#{name} version: ERROR")
-        else
-          version = result.stdout.lines.first.scan(/(?:master\s)?[\d+\.\(\)]+\S+/).join("\s")
-          msg("#{name} version: #{version}")
-        end
+      }.each do |prod_name, component|
+        msg("#{prod_name} version: #{component_version(component)}")
       end
     end
 
@@ -170,8 +149,15 @@ module ChefCLI
       commands_map.command_specs
     end
 
+    #
+    # Is a passed parameter actually an option aka does it start with '-'
+    #
+    # @param [String] param The passed parameter to check
+    #
+    # @return [Boolean]
+    #
     def option?(param)
-      param =~ /^-/
+      param[0] == "-"
     end
 
     def instantiate_subcommand(name)
@@ -188,21 +174,23 @@ module ChefCLI
       end
     end
 
-    def gem_version(name)
+    def component_version(name)
       if gem_manifest_hash[name].is_a?(Array)
         gem_manifest_hash[name].first
+      elsif manifest_hash.key? name
+        manifest_field(name)
       else
-        "unknown"
+        manifest_hash.dig("software", name, "locked_version") || "unknown"
       end
     end
 
     def manifest_hash
-      require "json"
+      require "json" unless defined?(JSON)
       @manifest_hash ||= JSON.parse(read_version_manifest_json)
     end
 
     def gem_manifest_hash
-      require "json"
+      require "json" unless defined?(JSON)
       @gem_manifest_hash ||= JSON.parse(read_gem_version_manifest_json)
     end
 
@@ -243,7 +231,7 @@ module ChefCLI
     # catch the cases where users setup only the embedded_bin_dir in their path, or
     # when they have the embedded_bin_dir before the omnibus_bin_dir -- both of which will
     # defeat appbundler and interact very badly with our intent.
-    def sanity_check!
+    def path_check!
       # When installed outside of omnibus, trust the user to configure their PATH
       return true unless omnibus_install?
 

data/lib/chef-cli/command/env.rb

@@ -21,7 +21,7 @@ require_relative "../ui"
 require_relative "../version"
 require_relative "../dist"
 require "mixlib/shellout" unless defined?(Mixlib::ShellOut)
-require "yaml"
+require "yaml" unless defined?(YAML)
 
 module ChefCLI
   module Command

data/lib/chef-cli/command/gem.rb

@@ -20,7 +20,6 @@ require_relative "../dist"
 require "rubygems" unless defined?(Gem)
 require "rubygems/gem_runner"
 require "rubygems/exceptions"
-require "pp" unless defined?(PP)
 
 module ChefCLI
   module Command

data/lib/chef-cli/command/generator_commands.rb

@@ -16,7 +16,6 @@
 #
 
 require "mixlib/cli" unless defined?(Mixlib::CLI)
-require "rbconfig"
 require "pathname" unless defined?(Pathname)
 require_relative "base"
 require_relative "../chef_runner"
@@ -38,7 +37,7 @@ module ChefCLI
         long: "--license LICENSE",
         description: "all_rights, apachev2, mit, gplv2, gplv3 - defaults to all_rights",
         proc: Proc.new { |l|
-          l =~ /apache2/ ? "apachev2" : l
+          /apache2/.match?(l) ? "apachev2" : l
         },
         default: nil
 

data/lib/chef-cli/command/generator_commands/cookbook.rb

@@ -187,27 +187,17 @@ module ChefCLI
           @verbose
         end
 
+        #
+        # Is there a .delivery/cli.toml in the current dir or any of the parent dirs
+        #
+        # @return [Boolean]
+        #
         def have_delivery_config?
           # delivery-cli's logic is to look recursively upward for
           # .delivery/cli.toml starting from pwd:
           # https://github.com/chef/delivery-cli/blob/22cbef3987ebd0aee98405b7e161a100edc87e49/src/delivery/config/mod.rs#L225-L247
 
-          path_to_check = File.expand_path(Dir.pwd)
-          result = false
-
-          Pathname.new(path_to_check).ascend do |path|
-            if contains_delivery_cli_toml?(path)
-              result = true
-              break
-            end
-          end
-
-          result
-        end
-
-        def contains_delivery_cli_toml?(path)
-          delivery_cli_path = path.join(".delivery/cli.toml")
-          delivery_cli_path.exist?
+          Pathname.pwd.ascend.any? { |path| path.join(".delivery/cli.toml").exist? }
         end
 
         def read_and_validate_params
@@ -215,7 +205,7 @@ module ChefCLI
           @cookbook_name_or_path = arguments[0]
           if !@cookbook_name_or_path
             @params_valid = false
-          elsif /-/ =~ File.basename(@cookbook_name_or_path)
+          elsif File.basename(@cookbook_name_or_path).include?("-")
             msg("Hyphens are discouraged in cookbook names as they may cause problems with custom resources. See https://docs.chef.io/ctl_chef/#chef-generate-cookbook for more information.")
           end
 

data/lib/chef-cli/command/provision.rb

@@ -15,8 +15,6 @@
 # limitations under the License.
 #
 
-require "ostruct"
-
 require_relative "base"
 require_relative "../chef_runner"
 require_relative "../dist"

data/lib/chef-cli/command/shell_init.rb

@@ -15,7 +15,7 @@
 # limitations under the License.
 #
 
-require "erb"
+require "erb" unless defined?(ERB)
 
 require_relative "../commands_map"
 require_relative "../builtin_commands"

data/lib/chef-cli/command/update.rb

@@ -70,7 +70,7 @@ module ChefCLI
 
       option :exclude_deps,
         long:         "--exclude-deps",
-        description:  "Only update cookbooks explicitely mentioned on the command line",
+        description:  "Only update cookbooks explicitly mentioned on the command line",
         boolean:      true,
         default:      false
 

data/lib/chef-cli/cookbook_profiler/git.rb

@@ -31,7 +31,7 @@ module ChefCLI
         @unborn_branch_ref = nil
       end
 
-      # @return [Hash] Hashed used for pinning cookbook versions within a Policfile.lock
+      # @return [Hash] Hashed used for pinning cookbook versions within a Policyfile.lock
       def profile_data
         {
           "scm" => "git",

data/lib/chef-cli/dist.rb

@@ -35,5 +35,10 @@ module ChefCLI
 
     # Chef-Zero's product name
     ZERO_PRODUCT = "Chef Infra Zero".freeze
+
+    HAB_PRODUCT = "Chef Habitat".freeze
+    HAB_SOFTWARE_NAME = "habitat".freeze
+
+    HAB_CLI = "hab".freeze
   end
 end

data/lib/chef-cli/policyfile/artifactory_cookbook_source.rb

@@ -15,11 +15,11 @@
 # limitations under the License.
 #
 
-require "json"
+require "json" unless defined?(JSON)
 require_relative "../cookbook_omnifetch"
 require_relative "source_uri"
 require_relative "../exceptions"
-require "chef/http/simple"
+require "chef/http/simple" unless defined?(Chef::HTTP::Simple)
 
 module ChefCLI
   module Policyfile

data/lib/chef-cli/policyfile/attribute_merge_checker.rb

@@ -15,7 +15,7 @@
 # limitations under the License.
 #
 
-require "chef/mash"
+require "chef/mash" unless defined?(Mash)
 
 module ChefCLI
   module Policyfile

data/lib/chef-cli/policyfile/chef_server_lock_fetcher.rb

@@ -67,7 +67,7 @@ module ChefCLI
         errors.empty?
       end
 
-      # Check the options provided when craeting this class for errors
+      # Check the options provided when creating this class for errors
       #
       # @return [Array<String>] A list of errors found
       def errors

data/lib/chef-cli/policyfile/community_cookbook_source.rb

@@ -15,7 +15,7 @@
 # limitations under the License.
 #
 
-require "json"
+require "json" unless defined?(JSON)
 require_relative "../cookbook_omnifetch"
 require_relative "../exceptions"
 require "chef/http/simple"

data/lib/chef-cli/policyfile/cookbook_location_specification.rb

@@ -15,7 +15,7 @@
 # limitations under the License.
 #
 
-require "semverse"
+require "semverse" unless defined?(Semverse)
 require_relative "../cookbook_omnifetch"
 require_relative "storage_config"
 

data/lib/chef-cli/policyfile/cookbook_locks.rb

@@ -400,8 +400,8 @@ module ChefCLI
 
         dot_git = Pathname.new(".git")
         Pathname.new(cookbook_path).ascend do |parent_dir|
-          possbile_git_dir = parent_dir + dot_git
-          if possbile_git_dir.exist?
+          possible_git_dir = parent_dir + dot_git
+          if possible_git_dir.exist?
             @cookbook_in_git_repo = true
             break
           end

data/lib/chef-cli/policyfile/delivery_supermarket_source.rb

@@ -16,9 +16,7 @@
 #
 
 require "forwardable" unless defined?(Forwardable)
-
-require "semverse"
-
+require "semverse" unless defined?(Semverse)
 require_relative "community_cookbook_source"
 
 module ChefCLI
@@ -39,7 +37,7 @@ module ChefCLI
     # * Version numbers for cookbooks that pass through the Ci pipeline always
     #   increase over time (so that largest version number == newest)
     #
-    # In the future, alternative approaches may be persued to achieve the goal
+    # In the future, alternative approaches may be pursued to achieve the goal
     # of continuously integrating at the cookbook level without imposing those
     # constraints.
     #

data/lib/chef-cli/policyfile/differ.rb

@@ -17,7 +17,7 @@
 
 require "diff/lcs"
 require "diff/lcs/hunk"
-require "paint"
+require "pastel"
 require "ffi_yajl" unless defined?(FFI_Yajl)
 
 module ChefCLI
@@ -215,9 +215,15 @@ module ChefCLI
         ui.print("\n")
       end
 
+      def pastel
+        @pastel ||= Pastel.new
+      end
+
       def print_color_diff(hunk)
         hunk.to_s.each_line do |line|
-          ui.print(Paint[line, color_for_line(line)])
+          line_color = color_for_line(line)
+          line = pastel.decorate(line, line_color) unless line_color.nil?
+          ui.print(line)
         end
       end
 

data/lib/chef-cli/policyfile/dsl.rb

@@ -233,7 +233,7 @@ module ChefCLI
             message = "#{run_list_desc} has invalid cookbook name '#{cookbook}'.\nCookbook names can only contain alphanumerics, hyphens, and underscores."
 
             # Special case when there's only one colon instead of two:
-            if cookbook =~ /[^:]:[^:]/
+            if /[^:]:[^:]/.match?(cookbook)
               message << "\nDid you mean '#{item.sub(":", "::")}'?"
             end
 

data/lib/chef-cli/policyfile/local_lock_fetcher.rb

@@ -48,7 +48,7 @@ module ChefCLI
         errors.empty?
       end
 
-      # Check the options provided when craeting this class for errors
+      # Check the options provided when creating this class for errors
       #
       # @return [Array<String>] A list of errors found
       def errors

data/lib/chef-cli/policyfile/policyfile_location_specification.rb

@@ -85,7 +85,7 @@ module ChefCLI
         errors.empty?
       end
 
-      # Check the options provided when craeting this class for errors
+      # Check the options provided when creating this class for errors
       #
       # @return [Array<String>] A list of errors found
       def errors

data/lib/chef-cli/policyfile/read_cookbook_for_compat_mode_upload.rb

@@ -89,9 +89,9 @@ module ChefCLI
         @cookbook_version ||=
           begin
             cookbook_version = loader.cookbook_version
-            # TODO: dont do this for non-compat mode
+            # TODO: don't do this for non-compat mode
             cookbook_version.version = version_override
-            # TODO: dont do this either
+            # TODO: don't do this either
 
             # Fixup manifest.
             # What happens is, the 'manifest' representation of cookbook

data/lib/chef-cli/policyfile_compiler.rb

@@ -274,7 +274,7 @@ module ChefCLI
     # compared to an unmodified copy upstream. For example, the community site
     # may have a cookbook "apache2" at version "1.10.4", which the user has
     # forked on github and modified the dependencies without changing the
-    # version number. To accomodate this, the local_artifacts_graph should be
+    # version number. To accommodate this, the local_artifacts_graph should be
     # merged over the upstream's artifacts graph.
     def local_artifacts_graph
       cookbook_location_specs.inject({}) do |local_artifacts, (cookbook_name, cookbook_location_spec)|

data/lib/chef-cli/policyfile_lock.rb

@@ -1,4 +1,3 @@
-# -*- coding: UTF-8 -*-
 #
 # Copyright:: Copyright (c) 2014-2018 Chef Software Inc.
 # License:: Apache License, Version 2.0
@@ -362,7 +361,7 @@ module ChefCLI
         # doubles give 15-17 digits of precision, so we err on the safe side
         # and only use 15 digits in the string conversion. We use the `g`
         # format, which is a documented-enough "do what I mean" where floats
-        # >= 0.1 and < precsion are represented as floating point literals, and
+        # >= 0.1 and < precision are represented as floating point literals, and
         # other numbers use the exponent notation with a lowercase 'e'. Note
         # that both Ruby and Erlang document what their `g` does but have some
         # differences both subtle and non-subtle:

data/lib/chef-cli/skeletons/code_generator/templates/default/inspec_default_test.rb.erb

@@ -1,7 +1,7 @@
 # InSpec test for recipe <%= cookbook_name %>::<%= recipe_name %>
 
 # The InSpec reference, with examples and extensive documentation, can be
-# found at https://www.inspec.io/docs/reference/resources/
+# found at https://docs.chef.io/inspec/resources/
 
 unless os.windows?
   # This is an example test, replace with your own test.

data/lib/chef-cli/version.rb

@@ -16,5 +16,5 @@
 #
 
 module ChefCLI
-  VERSION = "3.0.11".freeze
+  VERSION = "3.0.24".freeze
 end

data/lib/kitchen/provisioner/chef_zero_capture.rb

@@ -1,4 +1,3 @@
-# -*- encoding: utf-8 -*-
 #
 # Author:: Marc Paradise <marc@chef.io>
 #
@@ -16,7 +15,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require "json"
+require "json" unless defined?(JSON)
 require "kitchen"
 require "kitchen/provisioner/base"
 require "kitchen/provisioner/chef_zero"
@@ -45,7 +44,7 @@ module Kitchen
     #
     # This provisioner does not do any cookbook dependency
     # resolution and will not pull in external cookbooks.  All cookbooks
-    # or cookbook artificats  + policy data as captured from the live node and are
+    # or cookbook artifacts + policy data as captured from the live node and are
     # expected to be available for chef-zero to provide to the client.
     class ChefZeroCapture < ChefZero
       # Declaring these ensure that they're available to the sandbox - it's initialized
@@ -55,7 +54,7 @@ module Kitchen
       default_config :cookbook_artifacts_path, "cookbook_artifacts"
 
       # This will load policyfile/berkshelf.  We don't want either - the client resolves all
-      # dependencies from chef-zero, exactly as preppped in the captured repository.
+      # dependencies from chef-zero, exactly as prepped in the captured repository.
       def load_needed_dependencies!; end
 
       def create_sandbox
@@ -73,7 +72,7 @@ module Kitchen
       end
 
       # Overriding the private ProviderChefZero#default_config_rb
-      # so that we can add additional configuratoin required for  chef-zeor
+      # so that we can add additional configuration required for chef-zero
       # to be able to locate our policies/, policy groups, and cookbook artifacts
       # at run-time.
       def default_config_rb

data/lib/kitchen/provisioner/policyfile_zero.rb

@@ -1,4 +1,3 @@
-# -*- encoding: utf-8 -*-
 #
 # Author:: Fletcher Nichol (<fnichol@nichol.ca>)
 #
@@ -54,7 +53,7 @@ module Kitchen
     # @author Daniel DeLeo <dan@chef.io>
     class PolicyfileZero < ChefBase
 
-      # This provsioner will forcibly set the following config options:
+      # This provisioner will forcibly set the following config options:
       # * `use_policyfile`: `true`
       # * `versioned_cookbooks`: `true`
       # * `deployment_group`: `POLICY_NAME-local`

data/spec/unit/cli_spec.rb

@@ -53,11 +53,9 @@ describe ChefCLI::CLI do
     E
   end
 
-  let(:version_message) { "#{ChefCLI::Dist::PRODUCT} version: #{ChefCLI::VERSION}\n" }
-
   def run_cli(expected_exit_code)
     expect(cli).to receive(:exit).with(expected_exit_code)
-    expect(cli).to receive(:sanity_check!)
+    expect(cli).to receive(:path_check!)
     cli.run
   end
 
@@ -66,16 +64,6 @@ describe ChefCLI::CLI do
     cli.run
   end
 
-  def run_cli_and_validate_tool_versions
-    full_version_message = version_message
-    tools.each do |name, details|
-      expect(cli).to receive(:shell_out).with("#{details["command"]} --version").and_return(mock_shell_out(0, "#{details["version_output"]}", ""))
-      full_version_message += "#{name} version: #{details["expected_version"]}\n"
-    end
-    run_cli(0)
-    expect(stdout).to eq(full_version_message)
-  end
-
   def mock_shell_out(exitstatus, stdout, stderr)
     shell_out = double("mixlib_shell_out")
     allow(shell_out).to receive(:exitstatus).and_return(exitstatus)
@@ -121,12 +109,17 @@ describe ChefCLI::CLI do
   context "given -v" do
     let(:argv) { %w{-v} }
 
-    context "#via_version_manifest" do
+    context "when installed via omnibus" do
       let(:mocked_version_manifest_json) do
         <<~E
           {
             "manifest_format": 2,
-            "build_version": "d.e.v"
+            "build_version": "d.e.v",
+            "software": {
+              "habitat": {
+                "locked_version": "1.2.3"
+              }
+            }
           }
         E
       end
@@ -160,6 +153,7 @@ describe ChefCLI::CLI do
           Chef Infra Client version: 0.0.2
           Chef InSpec version: 0.0.3
           Chef CLI version: 0.0.1
+          Chef Habitat version: 1.2.3
           Test Kitchen version: 0.0.4
           Cookstyle version: 0.0.6
         E
@@ -171,6 +165,7 @@ describe ChefCLI::CLI do
           Chef Infra Client version: unknown
           Chef InSpec version: unknown
           Chef CLI version: unknown
+          Chef Habitat version: unknown
           Test Kitchen version: unknown
           Cookstyle version: unknown
         E
@@ -195,49 +190,15 @@ describe ChefCLI::CLI do
       end
     end
 
-    context "#via_shell_out" do
-      let(:tools) do
-        {
-          "Chef Infra Client" => {
-            "command" => "chef-client",
-            "version_output" => "Chef Infra Client: 15.0.300",
-            "expected_version" => "15.0.300",
-          },
-          "Chef InSpec" => {
-            "command" => "inspec",
-            "version_output" => "4.6.2\n\nYour version of InSpec is out of date! The latest version is 4.6.4.",
-            "expected_version" => "4.6.2",
-          },
-          "Test Kitchen" => {
-            "command" => "kitchen",
-            "version_output" => "Test Kitchen version 2.2.5",
-            "expected_version" => "2.2.5",
-          },
-          "Cookstyle" => {
-            "command" => "cookstyle",
-            "version_output" => "Cookstyle 4.0.0\n  * RuboCop 0.62.0",
-            "expected_version" => "4.0.0",
-          },
-        }
-      end
-
-      it "does not print versions of tools with missing or errored tools" do
-        full_version_message = version_message
-        tools.each do |name, details|
-          if name == "inspec"
-            expect(cli).to receive(:shell_out).with("#{details["command"]} --version").and_return(mock_shell_out(1, "#{details["version_output"]}", ""))
-            full_version_message += "#{name} version: ERROR\n"
-          else
-            expect(cli).to receive(:shell_out).with("#{details["command"]} --version").and_return(mock_shell_out(0, "#{details["version_output"]}", ""))
-            full_version_message += "#{name} version: #{details["expected_version"]}\n"
-          end
-        end
-        run_cli(0)
-        expect(stdout).to eq(full_version_message)
+    context "when installed as a gem" do
+      let(:gem_version_message) { "#{ChefCLI::Dist::CLI_PRODUCT} version: #{ChefCLI::VERSION}\n" }
+      before do
+        allow(cli).to receive(:omnibus_install?).and_return false
       end
 
       it "prints the version and versions of chef-cli tools" do
-        run_cli_and_validate_tool_versions
+        run_cli(0)
+        expect(stdout).to eq(gem_version_message)
       end
     end
   end
@@ -295,7 +256,7 @@ describe ChefCLI::CLI do
     end
   end
 
-  context "sanity_check!" do
+  context "path_check!" do
 
     before do
       allow(Gem).to receive(:ruby).and_return(ruby_path)

data/spec/unit/command/clean_policy_cookbooks_spec.rb

@@ -50,9 +50,9 @@ describe ChefCLI::Command::CleanPolicyCookbooks do
 
     context "when given a path to the config" do
 
-      let(:params) { %w{ -c ~/otherstuff/config.rb } }
+      let(:params) { %w{ -c ~/other_stuff/config.rb } }
 
-      let(:config_arg) { "~/otherstuff/config.rb" }
+      let(:config_arg) { "~/other_stuff/config.rb" }
 
       before do
         expect(chef_config_loader).to receive(:load)

data/spec/unit/command/env_spec.rb

@@ -16,7 +16,7 @@
 #
 
 require "spec_helper"
-require "yaml"
+require "yaml" unless defined?(YAML)
 require "chef-cli/command/env"
 
 describe ChefCLI::Command::Env do
@@ -69,7 +69,7 @@ describe ChefCLI::Command::Env do
       it "output should be valid yaml" do
         expect { yaml }.not_to raise_error
       end
-      it "Workstation version should indicate that that we're not runnign from a WS install" do
+      it "Workstation version should indicate that that we're not running from a WS install" do
         expect(yaml).to have_key ChefCLI::Dist::PRODUCT
         expect(yaml[ChefCLI::Dist::PRODUCT]["Version"]).to eql "Not running from within Workstation"
       end

data/spec/unit/command/exec_spec.rb

@@ -59,7 +59,7 @@ describe ChefCLI::Command::Exec do
     before do
       allow(Gem).to receive(:ruby).and_return(ruby_path)
 
-      # Using a fake path separator to keep to prevent people from accidently
+      # Using a fake path separator to keep to prevent people from accidentally
       # getting things correct on their system. This enforces that, in general,
       # you should use the path separator ruby is telling you to use.
       stub_const("File::PATH_SEPARATOR", "<>")

data/spec/unit/commands_map_spec.rb

@@ -25,7 +25,7 @@ describe ChefCLI::CommandsMap do
 
   before do
     mapping.builtin("example", :Example)
-    mapping.builtin("hypenated-example", :HyphenatedExample)
+    mapping.builtin("hyphenated-example", :HyphenatedExample)
     mapping.builtin("explicit-path-example", :ExplicitPathExample, require_path: "unit/fixtures/command/explicit_path_example")
     mapping.builtin("documented-example", :DocumentedExample, desc: "I have documentation")
   end
@@ -34,16 +34,16 @@ describe ChefCLI::CommandsMap do
     expect(mapping.have_command?("example")).to be true
   end
 
-  it "infers a non-hypenated command's require path" do
+  it "infers a non-hyphenated command's require path" do
     expect(mapping.spec_for("example").require_path).to eq("chef-cli/command/example")
   end
 
   it "infers a hyphenated command's require path" do
-    expect(mapping.spec_for("hypenated-example").require_path).to eq("chef-cli/command/hypenated_example")
+    expect(mapping.spec_for("hyphenated-example").require_path).to eq("chef-cli/command/hyphenated_example")
   end
 
   it "lists the available commands" do
-    expect(mapping.command_names).to match_array(%w{example hypenated-example explicit-path-example documented-example})
+    expect(mapping.command_names).to match_array(%w{example hyphenated-example explicit-path-example documented-example})
   end
 
   it "keeps the docstring of a command" do

data/spec/unit/fixtures/cookbooks_api/update_fixtures.rb

@@ -1,6 +1,6 @@
 require "openssl"
 require "net/https"
-require "json"
+require "json" unless defined?(JSON)
 require "pp"
 require "uri"
 

data/spec/unit/kitchen/provisioner/chef_zero_capture_spec.rb

@@ -1,4 +1,3 @@
-# -*- encoding: utf-8 -*-
 #
 # Author:: Marc Paradsie <marc.paradise@gmail.com>
 #

data/spec/unit/policyfile/differ_spec.rb

@@ -243,7 +243,7 @@ describe ChefCLI::Policyfile::Differ do
 
   def output
     # ANSI codes make the tests harder to read
-    Paint.unpaint(ui.output)
+    Pastel.new.strip(ui.output)
   end
 
   subject(:differ) do

data/spec/unit/policyfile_includes_spec.rb

@@ -1,4 +1,3 @@
-# -*- coding: UTF-8 -*-
 #
 # Copyright:: Copyright (c) 2014-2018, Chef Software Inc.
 # License:: Apache License, Version 2.0

data/spec/unit/policyfile_install_with_includes_spec.rb

@@ -1,4 +1,3 @@
-# -*- coding: UTF-8 -*-
 #
 # Copyright:: Copyright (c) 2017 Chef Software Inc.
 # License:: Apache License, Version 2.0

data/spec/unit/policyfile_lock_build_spec.rb

@@ -1,4 +1,3 @@
-# -*- coding: UTF-8 -*-
 #
 # Copyright:: Copyright (c) 2014-2018 Chef Software Inc.
 # License:: Apache License, Version 2.0

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: chef-cli
 version: !ruby/object:Gem::Version
-  version: 3.0.11
+  version: 3.0.24
 platform: ruby
 authors:
 - Chef Software, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-06-29 00:00:00.000000000 Z
+date: 2020-08-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mixlib-cli
@@ -173,25 +173,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.4'
 - !ruby/object:Gem::Dependency
-  name: paint
+  name: pastel
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '1'
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '0.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '1'
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '0.7'
 - !ruby/object:Gem::Dependency
   name: license-acceptance
   requirement: !ruby/object:Gem::Requirement