chefspec 7.3.0 → 8.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4149e5d01cef76e9961206a3c21e026f8d1514c5411d4e736775494e7df2fd5e
-  data.tar.gz: 1c2f041d3734fb0d1b4311652e545753ae93e13ac1d5699287754a2005ab30c5
+  metadata.gz: 888dff160c4e09262e47271fc25e19e24dfa5415f9c17566d5fab2c3b07d3f38
+  data.tar.gz: a79eacde94a5abdad1ab8e4530ed95d46255323927fbfe9706873aa7d6d99545
 SHA512:
-  metadata.gz: 86b9445a05713b120154104b065300b29944c2178277c0dbaf57271baed0d675b9582272c73044101cc36625b7ff9e1d189efff6f8525dbac2aed5ba225be0fd
-  data.tar.gz: 6dc383647ef0ca7f97c3a4ecfbe68a4323949a95d03c8ee853c80000682a7eab9deebed4db231cedd7ba0765c37d18a3304e145ed012739b6a5283b927f43889
+  metadata.gz: 76b576b0e8fd51f067bda1eaca73ebb3bd5f49f0ce366ac4c59d84942b2d1437cead58afc7fd1b0c96e7051e656342e2b2ac9e1d2d3c183953ed8fefba15b79f
+  data.tar.gz: f5ba63555cde2fb189c53ceedfffd46491744605ea50f8927f85502c8ba480455a69ba397cdc27d1ace387509e95b8c5442b10b411ea50b3d18aaf075c0a5b3d

data/Rakefile

@@ -21,48 +21,51 @@ failed = []
 start_time = nil
 
 namespace :acceptance do |ns|
-  Dir.foreach("examples") do |dir|
-    next if dir == '.' or dir == '..'
-    desc "#{dir} acceptance tests"
-    task dir.to_sym do
-      start_time ||= Time.now
-      Dir.mktmpdir do |tmp|
-        FileUtils.cp_r("examples/#{dir}", tmp)
+  begin
+    Dir.foreach("examples") do |dir|
+      next if %w(. .. .DS_Store).include?(dir)
+      desc "#{dir} acceptance tests"
+      task dir.to_sym do
+        start_time ||= Time.now
+        Dir.mktmpdir do |tmp|
+          FileUtils.cp_r("examples/#{dir}", tmp)
 
-        pwd = Dir.pwd
+          pwd = Dir.pwd
 
-        Dir.chdir "#{tmp}/#{dir}" do
-          puts "rspec examples/#{dir}"
+          Dir.chdir "#{tmp}/#{dir}" do
+            puts "rspec examples/#{dir}"
 
-          #
-          # This bit of mildly awful magic below is to load each file into an in-memory
-          # RSpec runner while keeping a persistent ChefZero server alive.
-          #
-          load "#{pwd}/lib/chefspec/rspec.rb"
+            #
+            # This bit of mildly awful magic below is to load each file into an in-memory
+            # RSpec runner while keeping a persistent ChefZero server alive.
+            #
+            load "#{pwd}/lib/chefspec/rspec.rb"
 
-          RSpec.configure do |config|
-            config.color = true
-            config.run_all_when_everything_filtered = true
-            config.filter_run(:focus)
-            config.before(:suite) do
-              ChefSpec::ZeroServer.setup!
+            RSpec.configure do |config|
+              config.color = true
+              config.run_all_when_everything_filtered = true
+              config.filter_run(:focus)
+              config.before(:suite) do
+                ChefSpec::ZeroServer.setup!
+              end
+              config.after(:each) do
+                ChefSpec::ZeroServer.reset!
+              end
             end
-            config.after(:each) do
-              ChefSpec::ZeroServer.reset!
-            end
-          end
 
-          RSpec.clear_examples
-          exitstatus = RSpec::Core::Runner.run(["spec"])
-          RSpec.reset
-          failed << dir unless exitstatus == 0
+            RSpec.clear_examples
+            exitstatus = RSpec::Core::Runner.run(["spec"])
+            RSpec.reset
+            failed << dir unless exitstatus == 0
+          end
         end
       end
     end
+  rescue Errno::ENOENT # examples dir is probably missing
+    puts "The rake acceptance tests require a full git checkout of chefspec including all examples files!"
   end
 end
 
-
 task acceptance: Rake.application.tasks.select { |t| t.name.start_with?("acceptance:") } do
   puts "Acceptance tests took #{Time.now - start_time} seconds"
   raise "some tests failed: #{failed.join(', ')}" unless failed.empty?

data/chefspec.gemspec

@@ -14,18 +14,17 @@ Gem::Specification.new do |s|
                     'ChefSpec makes it easy to write examples and get fast ' \
                     'feedback on cookbook changes without the need for ' \
                     'virtual machines or cloud servers.'
-  s.homepage      = 'https://sethvargo.github.io/chefspec/'
+  s.homepage      = 'https://github.com/chefspec/chefspec'
   s.license       = 'MIT'
 
   # Packaging
-  s.files         = `git ls-files`.split($/)
-  s.executables   = s.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  s.test_files    = s.files.grep(%r{^(test|spec|features)/})
+  s.files         = %w{LICENSE Rakefile Gemfile chefspec.gemspec} + Dir.glob("{lib,templates,spec}/**/*", File::FNM_DOTMATCH).reject { |f| File.directory?(f) }
   s.require_paths = ['lib']
 
-  s.required_ruby_version = '>= 2.2'
+  s.required_ruby_version = '>= 2.4'
 
-  s.add_dependency 'chef',    '>= 12.14.89'
-  s.add_dependency 'fauxhai', '>= 4'
+  s.add_dependency 'chef',    '>= 14'
+  s.add_dependency 'chef-cli'
+  s.add_dependency 'fauxhai', '>= 6.11'
   s.add_dependency 'rspec',   '~> 3.0'
 end

data/lib/chefspec/api/core.rb

@@ -14,27 +14,33 @@ module ChefSpec
           # Only run the preload if a platform is configured via the new system
           # since otherwise it will spam warnings about the platform not being
           # set.
-          chef_runner.preload! if chefspec_platform
+          chef_runner_instance.preload! if chefspec_platform
           ex.run
         ensure
           $CHEFSPEC_MODE = old_chefspec_mode
         end
       end
 
-      # Let variables to set the platform in a scoped way. Used below by
+      # Let variables to set data in a scoped way. Used below by
       # {ClassMethods#platform}.
+      let(:chefspec_default_attributes) { chefspec_attributes(:default_attributes) }
+      let(:chefspec_normal_attributes) { chefspec_attributes(:normal_attributes) }
+      let(:chefspec_override_attributes) { chefspec_attributes(:override_attributes) }
+      let(:chefspec_automatic_attributes) { chefspec_attributes(:automatic_attributes) }
       let(:chefspec_platform) { nil }
       let(:chefspec_platform_version) { nil }
 
-      # Set up the runner object but don't actually run anything yet. This can
-      # be overridden if needed to set up very custom things.
-      let(:chef_runner) do
+      # Compute the options for the runner.
+      #
+      # @abstract
+      # @return [Hash<Symbol, Object>]
+      def chef_runner_options
         options = {
           step_into: chefspec_ancestor_gather([], :step_into) {|memo, val| memo | val },
-          default_attributes: chefspec_attributes(:default_attributes),
-          normal_attributes: chefspec_attributes(:normal_attributes),
-          override_attributes: chefspec_attributes(:override_attributes),
-          automatic_attributes: chefspec_attributes(:automatic_attributes),
+          default_attributes: chefspec_default_attributes,
+          normal_attributes: chefspec_normal_attributes,
+          override_attributes: chefspec_override_attributes,
+          automatic_attributes: chefspec_automatic_attributes,
           spec_declaration_locations: self.class.declaration_locations.last[0],
         }
         # Only specify these if set in the example so we don't override the
@@ -43,16 +49,32 @@ module ChefSpec
         options[:version] = chefspec_platform_version if chefspec_platform_version
         # Merge in any final overrides.
         options.update(chefspec_attributes(:chefspec_options).symbolize_keys)
-        # At some point this is probably going to need a flag for ServerRunner
-        # because someone will complain.
-        ChefSpec::SoloRunner.new(options)
+        options
       end
 
-      # By default, run the recipe in the base `describe` block.
-      let(:chef_run) do
-        chef_runner.converge(described_recipe)
+      # Class of runner to use.
+      #
+      # @abstract
+      # @return [Class]
+      def chef_runner_class
+        ChefSpec::SoloRunner
+      end
+
+      # Create an instance of the runner.
+      #
+      # This should only be used in cases where the `let()` cache would be a problem.
+      #
+      # @return [ChefSpec::SoloRunner]
+      def chef_runner_instance
+        chef_runner_class.new(chef_runner_options)
       end
 
+      # Set up the runner object but don't actually run anything yet.
+      let(:chef_runner) { chef_runner_instance }
+
+      # By default, run the recipe in the base `describe` block.
+      let(:chef_run) { chef_runner.converge(described_recipe) }
+
       # Helper method for some of the nestable test value methods like
       # {ClassMethods#default_attributes} and {ClassMethods#step_into}.
       #

data/lib/chefspec/api/stubs_for.rb

@@ -1,4 +1,5 @@
 require "chef/version"
+require "mixlib/shellout"
 
 module ChefSpec
   module API
@@ -86,7 +87,7 @@ module ChefSpec
       def receive_shell_out(*cmd, stdout: '', stderr: '', exitstatus: 0, **opts)
         # Ruby does not allow constructing an actual exitstatus object from Ruby code. Really.
         fake_exitstatus = double(exitstatus: exitstatus)
-        fake_cmd = Mixlib::ShellOut.new(*cmd, **opts)
+        fake_cmd = Mixlib::ShellOut.new(*cmd)
         fake_cmd.define_singleton_method(:run_command) { } # Do nothing, just in case.
         # Inject our canned data.
         fake_cmd.instance_exec do
@@ -100,7 +101,8 @@ module ChefSpec
         else
           :shell_out
         end
-        receive(shell_out_method).with(*cmd).and_return(fake_cmd)
+        with_args = cmd + (opts.empty? ? [any_args] : [hash_including(opts)])
+        receive(shell_out_method).with(*with_args).and_return(fake_cmd)
       end
 
       module ClassMethods

data/lib/chefspec/coverage.rb

@@ -90,26 +90,6 @@ module ChefSpec
       @outputs << block
     end
 
-    #
-    # Change the template for reporting of converage analysis.
-    #
-    # @param [string] path
-    #   The template file to use for the output of the report
-    #
-    # @return [true]
-    #
-    def set_template(file = 'human.erb')
-      [
-        ChefSpec.root.join('templates', 'coverage', file),
-        File.expand_path(file, Dir.pwd)
-      ].each do |temp|
-        if File.exist?(temp)
-          @template = temp
-          return
-        end
-      end
-      raise Error::TemplateNotFound.new(path: file)
-    end
     #
     # Add a resource to the resource collection. Only new resources are added
     # and only resources that match the given filter are covered (which is *

data/lib/chefspec/extensions/chef/cookbook/gem_installer.rb

@@ -25,7 +25,7 @@ Chef::Cookbook::GemInstaller.prepend(Module.new do
     gem_cmd = "gem install #{gem_name} --version '#{gem_requirements.join(', ')}'"
     gemfile_line = "gem '#{[gem_name, *gem_requirements].join('\', \'')}'"
     warn "No matching version found for '#{gem_name}' in your gem environment.\n" \
-         " - if you are using ChefDK, run the following command: \"chef #{gem_cmd}\"\n" \
+         " - if you are using Chef Workstation, run the following command: \"chef #{gem_cmd}\"\n" \
          " - if you are using bundler, append \"#{gemfile_line}\" to your Gemfile and run \"bundle install\"\n" \
          " - otherwise run: \"#{gem_cmd}\""
   end

data/lib/chefspec/policyfile.rb

@@ -1,8 +1,8 @@
 begin
-  require 'chef-dk/policyfile_services/export_repo'
-  require 'chef-dk/policyfile_services/install'
+  require 'chef-cli/policyfile_services/export_repo'
+  require 'chef-cli/policyfile_services/install'
 rescue LoadError
-  raise ChefSpec::Error::GemLoadError.new(gem: 'chef-dk', name: 'ChefDK')
+  raise ChefSpec::Error::GemLoadError.new(gem: 'chef-cli', name: 'ChefCLI')
 end
 
 module ChefSpec
@@ -22,16 +22,19 @@ module ChefSpec
     # Setup and install the necessary  dependencies in the temporary directory
     #
     def setup!
-      policyfile_path = File.join(Dir.pwd, 'Policyfile.rb')
+      policyfile_path = RSpec.configuration.policyfile_path
+      if policyfile_path.nil?
+        policyfile_path = File.join(Dir.pwd, 'Policyfile.rb')
+      end
 
-      installer = ChefDK::PolicyfileServices::Install.new(
+      installer = ChefCLI::PolicyfileServices::Install.new(
         policyfile: policyfile_path,
-        ui: ChefDK::UI.null
+        ui: ChefCLI::UI.null
       )
 
       installer.run
 
-      exporter = ChefDK::PolicyfileServices::ExportRepo.new(
+      exporter = ChefCLI::PolicyfileServices::ExportRepo.new(
         policyfile: policyfile_path,
         export_dir: @tmpdir
       )

data/lib/chefspec/rspec.rb

@@ -4,6 +4,7 @@ RSpec.configure do |config|
   end
 
   config.add_setting :berkshelf_options, default: {}
+  config.add_setting :policyfile_path
   config.add_setting :file_cache_path
   config.add_setting :cookbook_root
   config.add_setting :cookbook_path

data/lib/chefspec/version.rb

@@ -1,3 +1,3 @@
 module ChefSpec
-  VERSION = '7.3.0'
+  VERSION = '8.0.0'
 end

data/spec/spec_helper.rb

@@ -1,11 +1,6 @@
 require 'chefspec'
 require 'support/hash'
 
-
-ChefSpec::Coverage.start! do
-  set_template 'table.erb'
-end
-
 RSpec.configure do |config|
   config.expect_with(:rspec) { |c| c.syntax = :expect }
   config.filter_run(focus: true)