chefspec 9.1.0 → 9.3.1

data/lib/chefspec/matchers.rb

@@ -1,13 +1,13 @@
 module ChefSpec
   module Matchers
-    require_relative 'matchers/do_nothing_matcher'
-    require_relative 'matchers/include_any_recipe_matcher'
-    require_relative 'matchers/include_recipe_matcher'
-    require_relative 'matchers/link_to_matcher'
-    require_relative 'matchers/notifications_matcher'
-    require_relative 'matchers/render_file_matcher'
-    require_relative 'matchers/resource_matcher'
-    require_relative 'matchers/state_attrs_matcher'
-    require_relative 'matchers/subscribes_matcher'
+    require_relative "matchers/do_nothing_matcher"
+    require_relative "matchers/include_any_recipe_matcher"
+    require_relative "matchers/include_recipe_matcher"
+    require_relative "matchers/link_to_matcher"
+    require_relative "matchers/notifications_matcher"
+    require_relative "matchers/render_file_matcher"
+    require_relative "matchers/resource_matcher"
+    require_relative "matchers/state_attrs_matcher"
+    require_relative "matchers/subscribes_matcher"
   end
 end

data/lib/chefspec/mixins/normalize.rb

@@ -16,7 +16,7 @@ module ChefSpec
       else
         name = thing
       end
-      name.to_s.gsub('-', '_').to_sym
+      name.to_s.gsub("-", "_").to_sym
     end
   end
 end

data/lib/chefspec/policyfile.rb

@@ -1,8 +1,8 @@
 begin
-  require 'chef-cli/policyfile_services/export_repo'
-  require 'chef-cli/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-cli', name: 'ChefCLI')
+  raise ChefSpec::Error::GemLoadError.new(gem: "chef-cli", name: "ChefCLI")
 end
 
 module ChefSpec
@@ -24,7 +24,7 @@ module ChefSpec
     def setup!
       policyfile_path = RSpec.configuration.policyfile_path
       if policyfile_path.nil?
-        policyfile_path = File.join(Dir.pwd, 'Policyfile.rb')
+        policyfile_path = File.join(Dir.pwd, "Policyfile.rb")
       end
 
       installer = ChefCLI::PolicyfileServices::Install.new(
@@ -44,8 +44,8 @@ module ChefSpec
 
       ::RSpec.configure do |config|
         config.cookbook_path = [
-          File.join(@tmpdir, 'cookbooks'),
-          File.join(@tmpdir, 'cookbook_artifacts')
+          File.join(@tmpdir, "cookbooks"),
+          File.join(@tmpdir, "cookbook_artifacts"),
         ]
       end
     end

data/lib/chefspec/renderer.rb

@@ -1,6 +1,6 @@
 begin
-  require 'chef/mixin/template'
-  require 'chef/provider/template_finder'
+  require "chef/mixin/template"
+  require "chef/provider/template_finder"
 rescue LoadError
 end
 
@@ -67,8 +67,8 @@ module ChefSpec
       if Chef::Mixin::Template.const_defined?(:TemplateContext) # Chef 11+
         template_context = Chef::Mixin::Template::TemplateContext.new([])
         template_context.update({
-          :node => chef_run.node,
-          :template_finder => template_finder(chef_run, cookbook_name),
+          node: chef_run.node,
+          template_finder: template_finder(chef_run, cookbook_name),
         }.merge(template.variables))
         if template.respond_to?(:helper_modules) # Chef 11.4+
           template_context._extend_modules(template.helper_modules)

data/lib/chefspec/rspec.rb

@@ -1,5 +1,5 @@
 RSpec.configure do |config|
-  unless ENV['CHEFSPEC_NO_INCLUDE']
+  unless ENV["CHEFSPEC_NO_INCLUDE"]
     config.include(ChefSpec::API)
   end
 

data/lib/chefspec/server.rb

@@ -1,4 +1,4 @@
 # @todo Remove in v5.0.0
-require_relative 'deprecations'
+require_relative "deprecations"
 deprecated "require 'chefspec/server' is no longer required and " \
   "will be removed in the next major release."

data/lib/chefspec/server_methods.rb

@@ -88,11 +88,11 @@ module ChefSpec
       EOH
     end
 
-    entity :client,      Chef::Client, 'clients'
-    entity :data_bag,    Chef::DataBag, 'data'
-    entity :environment, Chef::Environment, 'environments'
-    entity :node,        Chef::Node, 'nodes'
-    entity :role,        Chef::Role, 'roles'
+    entity :client,      Chef::Client, "clients"
+    entity :data_bag,    Chef::DataBag, "data"
+    entity :environment, Chef::Environment, "environments"
+    entity :node,        Chef::Node, "nodes"
+    entity :role,        Chef::Role, "roles"
 
     #
     # Create a new data_bag on the Chef Server. This overrides the method
@@ -104,7 +104,7 @@ module ChefSpec
     #   the data to load into the data bag
     #
     def create_data_bag(name, data = {})
-      load_data(name, 'data', data)
+      load_data(name, "data", data)
     end
 
     #
@@ -138,7 +138,7 @@ module ChefSpec
         data = JSON.fast_generate(data)
       end
 
-      load_data(name, 'nodes', data)
+      load_data(name, "nodes", data)
     end
     alias_method :update_node, :create_node
 
@@ -161,7 +161,7 @@ module ChefSpec
     # Get the path to an item in the data store.
     #
     def get(*args)
-      args.unshift('organizations', 'chef')
+      args.unshift("organizations", "chef")
 
       if args.size == 3
         server.data_store.list(args)

data/lib/chefspec/server_runner.rb

@@ -1,10 +1,10 @@
-require 'chef/cookbook_loader'
-require 'chef/cookbook_uploader'
+require "chef/cookbook_loader"
+require "chef/cookbook_uploader"
 
-require_relative 'zero_server'
-require_relative 'file_cache_path_proxy'
-require_relative 'server_methods'
-require_relative 'solo_runner'
+require_relative "zero_server"
+require_relative "file_cache_path_proxy"
+require_relative "server_methods"
+require_relative "solo_runner"
 
 module ChefSpec
   class ServerRunner < SoloRunner
@@ -54,8 +54,8 @@ module ChefSpec
     #
     def client_key
       tmp = Dir.mktmpdir
-      path = File.join(tmp, 'client.pem')
-      File.open(path, 'wb') { |f| f.write(ChefZero::PRIVATE_KEY) }
+      path = File.join(tmp, "client.pem")
+      File.open(path, "wb") { |f| f.write(ChefZero::PRIVATE_KEY) }
       at_exit { FileUtils.rm_rf(tmp) }
       path
     end
@@ -64,8 +64,8 @@ module ChefSpec
     def apply_chef_config!
       super
       Chef::Config[:client_key]       = client_key
-      Chef::Config[:client_name]      = 'chefspec'
-      Chef::Config[:node_name]        = 'chefspec'
+      Chef::Config[:client_name]      = "chefspec"
+      Chef::Config[:node_name]        = "chefspec"
       Chef::Config[:solo]             = false
       Chef::Config[:solo_legacy_mode] = false
 

data/lib/chefspec/solo_runner.rb

@@ -1,9 +1,9 @@
-require 'fauxhai'
-require 'chef/client'
-require 'chef/cookbook/metadata'
-require 'chef/mash'
-require 'chef/providers'
-require 'chef/resources'
+require "fauxhai"
+require "chef/client"
+require "chef/cookbook/metadata"
+require "chef/mash"
+require "chef/providers"
+require "chef/resources"
 
 module ChefSpec
   class SoloRunner
@@ -123,6 +123,7 @@ module ChefSpec
       if converge_val.is_a?(Exception)
         raise converge_val
       end
+
       self
     end
 
@@ -136,7 +137,7 @@ module ChefSpec
     #
     def converge_block(&block)
       converge do
-        recipe = Chef::Recipe.new(cookbook_name, '_test', run_context)
+        recipe = Chef::Recipe.new(cookbook_name, "_test", run_context)
         recipe.instance_exec(&block)
       end
     end
@@ -149,7 +150,8 @@ module ChefSpec
     #
     def preload!
       # Flag to disable preloading for situations where it doesn't make sense.
-      return if ENV['CHEFSPEC_NO_PRELOAD']
+      return if ENV["CHEFSPEC_NO_PRELOAD"]
+
       begin
         old_preload = $CHEFSPEC_PRELOAD
         $CHEFSPEC_PRELOAD = true
@@ -206,7 +208,7 @@ module ChefSpec
     #
     def find_resource(type, name, action = nil)
       resource_collection.all_resources.reverse_each.find do |resource|
-        resource.declared_type == type.to_sym && (name === resource.identity || name === resource.name) && (action.nil? || resource.performed_action?(action))
+        resource.declared_type == type.to_sym && (name === resource.name || name === resource.identity) && (action.nil? || resource.performed_action?(action))
       end
     end
 
@@ -292,7 +294,8 @@ module ChefSpec
     # Respond to custom matchers defined by the user.
     #
     def method_missing(m, *args, &block)
-      if block = ChefSpec.matchers[resource_name(m.to_sym)]
+      block = ChefSpec.matchers[resource_name(m.to_sym)]
+      if block
         instance_exec(args.first, &block)
       else
         super
@@ -358,11 +361,11 @@ module ChefSpec
     # @return [String]
     #
     def calling_cookbook_root(options, kaller)
-      calling_spec = options[:spec_declaration_locations] || kaller.find { |line| line =~ /\/spec/ }
+      calling_spec = options[:spec_declaration_locations] || kaller.find { |line| line =~ %r{/spec} }
       raise Error::CookbookPathNotFound if calling_spec.nil?
 
       bits = calling_spec.split(/:[0-9]/, 2).first.split(File::SEPARATOR)
-      spec_dir = bits.index('spec') || 0
+      spec_dir = bits.index("spec") || 0
 
       File.join(bits.slice(0, spec_dir))
     end
@@ -378,7 +381,7 @@ module ChefSpec
     # @return [String]
     #
     def calling_cookbook_path(options, kaller)
-      File.expand_path(File.join(calling_cookbook_root(options, kaller), '..'))
+      File.expand_path(File.join(calling_cookbook_root(options, kaller), ".."))
     end
 
     #
@@ -388,7 +391,7 @@ module ChefSpec
     #
     def default_role_path
       Pathname.new(Dir.pwd).ascend do |path|
-        possible = File.join(path, 'roles')
+        possible = File.join(path, "roles")
         return possible if File.exist?(possible)
       end
 
@@ -402,7 +405,7 @@ module ChefSpec
     #
     def default_environment_path
       Pathname.new(Dir.pwd).ascend do |path|
-        possible = File.join(path, 'environments')
+        possible = File.join(path, "environments")
         return possible if File.exist?(possible)
       end
 
@@ -459,7 +462,7 @@ module ChefSpec
     # @return [Chef::Cookbook::Metadata]
     #
     def cookbook
-      @cookbook ||= Chef::Cookbook::Metadata.new.tap {|m| m.from_file("#{options[:cookbook_root]}/metadata.rb") }
+      @cookbook ||= Chef::Cookbook::Metadata.new.tap { |m| m.from_file("#{options[:cookbook_root]}/metadata.rb") }
     end
 
     #
@@ -470,12 +473,11 @@ module ChefSpec
     def cookbook_name
       # Try to figure out the name of this cookbook, pretending this block
       # is in the name context as the cookbook under test.
-      begin
-        cookbook.name
-      rescue IOError
-        # Old cookbook, has no metadata, use the folder name I guess.
-        File.basename(options[:cookbook_root])
-      end
+
+      cookbook.name
+    rescue IOError
+      # Old cookbook, has no metadata, use the folder name I guess.
+      File.basename(options[:cookbook_root])
     end
 
     #
@@ -489,8 +491,8 @@ module ChefSpec
 
       Chef::Config.reset!
       Chef::Config.formatters.clear
-      Chef::Config.add_formatter('chefspec')
-      Chef::Config[:cache_type]      = 'Memory'
+      Chef::Config.add_formatter("chefspec")
+      Chef::Config[:cache_type]      = "Memory"
       Chef::Config[:client_key]      = nil
       Chef::Config[:client_name]     = nil
       Chef::Config[:node_name]       = nil
@@ -506,7 +508,7 @@ module ChefSpec
       Chef::Config[:force_logger]    = true
       Chef::Config[:solo]            = true
       Chef::Config[:solo_legacy_mode] = true
-      Chef::Config[:use_policyfile]  = false
+      Chef::Config[:use_policyfile] = false
       Chef::Config[:environment_path] = @options[:environment_path]
     end
 

data/lib/chefspec/stubs/command_registry.rb

@@ -1,4 +1,4 @@
-require_relative 'registry'
+require_relative "registry"
 
 module ChefSpec
   module Stubs

data/lib/chefspec/stubs/command_stub.rb

@@ -1,4 +1,4 @@
-require_relative 'stub'
+require_relative "stub"
 
 module ChefSpec
   module Stubs

data/lib/chefspec/stubs/data_bag_item_registry.rb

@@ -1,4 +1,4 @@
-require_relative 'registry'
+require_relative "registry"
 
 module ChefSpec
   module Stubs

data/lib/chefspec/stubs/data_bag_item_stub.rb

@@ -1,4 +1,4 @@
-require_relative 'stub'
+require_relative "stub"
 
 module ChefSpec
   module Stubs

data/lib/chefspec/stubs/data_bag_registry.rb

@@ -1,4 +1,4 @@
-require_relative 'registry'
+require_relative "registry"
 
 module ChefSpec
   module Stubs

data/lib/chefspec/stubs/data_bag_stub.rb

@@ -1,4 +1,4 @@
-require_relative 'stub'
+require_relative "stub"
 
 module ChefSpec
   module Stubs

data/lib/chefspec/stubs/registry.rb

@@ -25,7 +25,7 @@ module ChefSpec
       end
 
       def stub_for(*args)
-        raise ArgumentError, '#stub_for is an abstract function'
+        raise ArgumentError, "#stub_for is an abstract function"
       end
     end
   end

data/lib/chefspec/stubs/search_registry.rb

@@ -1,9 +1,9 @@
-require_relative 'registry'
+require_relative "registry"
 
 module ChefSpec
   module Stubs
     class SearchRegistry < Registry
-      def stub_for(type, query = '*:*')
+      def stub_for(type, query = "*:*")
         @stubs.find do |stub|
           stub.type.to_s == type.to_s && stub.query === query
         end

data/lib/chefspec/stubs/search_stub.rb

@@ -1,4 +1,4 @@
-require_relative 'stub'
+require_relative "stub"
 
 module ChefSpec
   module Stubs
@@ -7,7 +7,7 @@ module ChefSpec
       attr_reader :query
       attr_reader :type
 
-      def initialize(type, query = '*:*', &block)
+      def initialize(type, query = "*:*", &block)
         @type  = type.to_s
         @query = query
         @block = block

data/lib/chefspec/util.rb

@@ -14,10 +14,10 @@ module ChefSpec
     def underscore(string)
       string
         .to_s
-        .gsub(/::/, '/')
-        .gsub(/([A-Z]+)([A-Z][a-z])/,'\1_\2')
-        .gsub(/([a-z\d])([A-Z])/,'\1_\2')
-        .tr('-', '_')
+        .gsub(/::/, "/")
+        .gsub(/([A-Z]+)([A-Z][a-z])/, '\1_\2')
+        .gsub(/([a-z\d])([A-Z])/, '\1_\2')
+        .tr("-", "_")
         .downcase
     end
 
@@ -32,8 +32,8 @@ module ChefSpec
     def camelize(string)
       string
         .to_s
-        .split('_')
-        .map { |e| e.capitalize }
+        .split("_")
+        .map(&:capitalize)
         .join
     end
 
@@ -49,7 +49,7 @@ module ChefSpec
       length = options[:length] || 30
 
       if string.length > length
-        string[0..length-3] + '...'
+        string[0..length - 3] + "..."
       else
         string
       end

data/lib/chefspec/version.rb

@@ -1,3 +1,3 @@
 module ChefSpec
-  VERSION = '9.1.0'
+  VERSION = "9.3.1".freeze
 end