chefspec 8.0.1 → 9.3.0

data/spec/unit/expect_exception_spec.rb

@@ -1,30 +1,30 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe ChefSpec::ExpectException do
-  context 'when there have been no `raise_error` matchers' do
+  context "when there have been no `raise_error` matchers" do
     subject { described_class.new(Exception) }
 
-    it 'does not match' do
+    it "does not match" do
       allow(RSpec::Matchers::BuiltIn::RaiseError).to receive(:last_run).and_return(nil)
       expect(subject.expected?).to be_falsy
     end
   end
 
-  context 'when the last error does not match the expected type' do
+  context "when the last error does not match the expected type" do
     subject { described_class.new(RuntimeError) }
 
-    it 'does not match' do
-      last_error = double('last error', last_error_for_chefspec: ArgumentError)
+    it "does not match" do
+      last_error = double("last error", last_error_for_chefspec: ArgumentError)
       allow(RSpec::Matchers::BuiltIn::RaiseError).to receive(:last_run).and_return(last_error)
       expect(subject.expected?).to be_falsy
     end
   end
 
-  context 'when the last error matches the expected type' do
+  context "when the last error matches the expected type" do
     subject { described_class.new(RuntimeError) }
 
-    it 'does not match' do
-      last_error = double('last error', last_error_for_chefspec: RuntimeError)
+    it "does not match" do
+      last_error = double("last error", last_error_for_chefspec: RuntimeError)
       allow(RSpec::Matchers::BuiltIn::RaiseError).to receive(:last_run).and_return(last_error)
       expect(subject.expected?).to be_truthy
     end

data/spec/unit/macros_spec.rb

@@ -1,10 +1,10 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe ChefSpec::API::Stubs do
-  describe '#stub_command' do
-    let(:command_stub) { double('command') }
+  describe "#stub_command" do
+    let(:command_stub) { double("command") }
 
-    it 'adds the command to the command registry' do
+    it "adds the command to the command registry" do
       allow(ChefSpec::Stubs::CommandStub).to receive(:new).and_return(command_stub)
       stub_command('echo "hello"')
 
@@ -12,21 +12,21 @@ describe ChefSpec::API::Stubs do
     end
   end
 
-  describe '#stub_search' do
-    let(:search_stub) { double('search') }
+  describe "#stub_search" do
+    let(:search_stub) { double("search") }
 
-    it 'adds the query to the search registry' do
+    it "adds the query to the search registry" do
       allow(ChefSpec::Stubs::SearchStub).to receive(:new).and_return(search_stub)
-      stub_search(:node, '*:*')
+      stub_search(:node, "*:*")
 
       expect(ChefSpec::Stubs::SearchRegistry.stubs).to include(search_stub)
     end
   end
 
-  describe '#stub_data_bag' do
-    let(:data_bag_stub) { double('data_bag') }
+  describe "#stub_data_bag" do
+    let(:data_bag_stub) { double("data_bag") }
 
-    it 'adds the query to the data_bag registry' do
+    it "adds the query to the data_bag registry" do
       allow(ChefSpec::Stubs::DataBagStub).to receive(:new).and_return(data_bag_stub)
       stub_data_bag(:users)
 
@@ -34,84 +34,84 @@ describe ChefSpec::API::Stubs do
     end
   end
 
-  describe '#stub_data_bag_item' do
-    let(:data_bag_item_stub) { double('data_bag_item') }
+  describe "#stub_data_bag_item" do
+    let(:data_bag_item_stub) { double("data_bag_item") }
 
-    it 'adds the query to the data_bag_item registry' do
+    it "adds the query to the data_bag_item registry" do
       allow(ChefSpec::Stubs::DataBagItemStub).to receive(:new).and_return(data_bag_item_stub)
-      stub_data_bag_item(:users, 'id')
+      stub_data_bag_item(:users, "id")
 
       expect(ChefSpec::Stubs::DataBagItemRegistry.stubs).to include(data_bag_item_stub)
     end
   end
 
-  describe '#stub_node' do
-    it 'returns a Chef::Node' do
+  describe "#stub_node" do
+    it "returns a Chef::Node" do
       expect(stub_node).to be_a(Chef::Node)
     end
 
-    it 'defaults the node name to `node.example`' do
+    it "defaults the node name to `node.example`" do
       node = stub_node
-      expect(node.name).to eq('node.example')
+      expect(node.name).to eq("node.example")
     end
 
-    it 'sets the node name when given' do
-      node = stub_node('example.com')
-      expect(node.name).to eq('example.com')
+    it "sets the node name when given" do
+      node = stub_node("example.com")
+      expect(node.name).to eq("example.com")
     end
 
-    it 'sets the automatic attributes' do
+    it "sets the automatic attributes" do
       node = stub_node
       expect(node.automatic).to eq(Fauxhai.mock.data)
     end
 
-    it 'sets the automatic attributes with ohai overrides' do
-      node = stub_node('node.example', ohai: { ipaddress: '1.2.3.4' })
-      expect(node['ipaddress']).to eq('1.2.3.4')
+    it "sets the automatic attributes with ohai overrides" do
+      node = stub_node("node.example", ohai: { ipaddress: "1.2.3.4" })
+      expect(node["ipaddress"]).to eq("1.2.3.4")
     end
 
-    it 'sets the automatic attributes for a specific platform and version' do
-      node = stub_node('node.example', platform: 'ubuntu', version: '18.04')
-      expect(node.automatic).to eq(Fauxhai.mock(platform: 'ubuntu', version: '18.04').data)
+    it "sets the automatic attributes for a specific platform and version" do
+      node = stub_node("node.example", platform: "ubuntu", version: "18.04")
+      expect(node.automatic).to eq(Fauxhai.mock(platform: "ubuntu", version: "18.04").data)
     end
 
-    it 'sets the automatic attributes from a JSON data path' do
-      allow(File).to receive(:exist?).with('/path/to/json').and_return(true)
-      allow(File).to receive(:read).with('/path/to/json').and_return('{ "ipaddress": "1.2.3.4" }')
-      node = stub_node('node.example', path: '/path/to/json')
-      expect(node['ipaddress']).to eq('1.2.3.4')
+    it "sets the automatic attributes from a JSON data path" do
+      allow(File).to receive(:exist?).with("/path/to/json").and_return(true)
+      allow(File).to receive(:read).with("/path/to/json").and_return('{ "ipaddress": "1.2.3.4" }')
+      node = stub_node("node.example", path: "/path/to/json")
+      expect(node["ipaddress"]).to eq("1.2.3.4")
     end
 
-    it 'yields a block' do
+    it "yields a block" do
       expect { |block| stub_node(&block) }.to yield_with_args(Chef::Node)
     end
   end
 end
 
-describe 'nginx::source' do
-  describe '#described_cookbook' do
-    describe 'nginx::source' do
-      it 'returns the name of the cookbook' do
-        expect(described_cookbook).to eq('nginx')
+describe "nginx::source" do
+  describe "#described_cookbook" do
+    describe "nginx::source" do
+      it "returns the name of the cookbook" do
+        expect(described_cookbook).to eq("nginx")
       end
 
-      context 'in a nested context' do
-        it 'still returns the name of the cookbook' do
-          expect(described_cookbook).to eq('nginx')
+      context "in a nested context" do
+        it "still returns the name of the cookbook" do
+          expect(described_cookbook).to eq("nginx")
         end
       end
     end
   end
 
-  describe '#described_recipe' do
-    describe 'nginx::source' do
-      it 'returns the cookbook::recipe' do
-        expect(described_recipe).to eq('nginx::source')
+  describe "#described_recipe" do
+    describe "nginx::source" do
+      it "returns the cookbook::recipe" do
+        expect(described_recipe).to eq("nginx::source")
       end
 
-      context 'in a nested context' do
-        it 'still retrns the cookbook::recipe' do
-          expect(described_recipe).to eq('nginx::source')
+      context "in a nested context" do
+        it "still retrns the cookbook::recipe" do
+          expect(described_recipe).to eq("nginx::source")
         end
       end
     end

data/spec/unit/matchers/do_nothing_matcher.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe ChefSpec::Matchers::DoNothingMatcher do
   pending

data/spec/unit/matchers/include_any_recipe_matcher_spec.rb

@@ -0,0 +1,52 @@
+require "spec_helper"
+
+describe ChefSpec::Matchers::IncludeAnyRecipeMatcher do
+  let(:node) { double("Chef::Node", run_list: run_list) }
+  let(:run_list) { double("Chef::RunList", run_list_items: run_list_items) }
+  let(:run_list_items) { [run_list_item_one] }
+  let(:run_list_item_one) { double("Chef::RunList::RunListItem", name: "one") }
+  let(:run_list_item_two) { double("Chef::RunList::RunListItem", name: "two") }
+  let(:loaded_recipes) { %w{one} }
+  let(:chef_run) { double("chef run", run_context: { loaded_recipes: loaded_recipes, node: node }) }
+
+  subject { described_class.new }
+
+  describe "#failure_message" do
+    it "has the right value" do
+      subject.matches?(chef_run)
+      expect(subject.failure_message).to eq("expected to include any recipe")
+    end
+  end
+
+  describe "#failure_message_when_negated" do
+    it "has the right value" do
+      subject.matches?(chef_run)
+      expect(subject.failure_message_when_negated).to eq("expected not to include any recipes")
+    end
+  end
+
+  describe "#description" do
+    it "has the right value" do
+      subject.matches?(chef_run)
+      expect(subject.description).to eq("include any recipe")
+    end
+  end
+
+  describe "#matches?" do
+    context "when 0 recipes are included" do
+      let(:loaded_recipes) { %w{one} }
+
+      it "returns false" do
+        expect(subject.matches?(chef_run)).to be false
+      end
+    end
+
+    context "when at least one recipe is included" do
+      let(:loaded_recipes) { %w{one two} }
+
+      it "returns true" do
+        expect(subject.matches?(chef_run)).to be true
+      end
+    end
+  end
+end

data/spec/unit/matchers/include_recipe_matcher_spec.rb

@@ -1,38 +1,38 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe ChefSpec::Matchers::IncludeRecipeMatcher do
-  let(:chef_run) { double('chef run', run_context: { loaded_recipes: %w(one two three) }) }
-  subject { described_class.new('one::default') }
+  let(:chef_run) { double("chef run", run_context: { loaded_recipes: %w{one two three} }) }
+  subject { described_class.new("one::default") }
 
-  describe '#failure_message' do
-    it 'has the right value' do
+  describe "#failure_message" do
+    it "has the right value" do
       subject.matches?(chef_run)
       expect(subject.failure_message)
-        .to eq(%q(expected ["one::default", "two::default", "three::default"] to include "one::default"))
+        .to eq(%q{expected ["one::default", "two::default", "three::default"] to include "one::default"})
     end
   end
 
-  describe '#failure_message_when_negated' do
-    it 'has the right value' do
+  describe "#failure_message_when_negated" do
+    it "has the right value" do
       subject.matches?(chef_run)
       expect(subject.failure_message_when_negated)
-        .to eq(%q(expected "one::default" to not be included))
+        .to eq(%q{expected "one::default" to not be included})
     end
   end
 
-  describe '#description' do
-    it 'has the right value' do
+  describe "#description" do
+    it "has the right value" do
       subject.matches?(chef_run)
-      expect(subject.description).to eq(%q(include recipe "one::default"))
+      expect(subject.description).to eq(%q{include recipe "one::default"})
     end
   end
 
-  it 'matches when the recipe is included' do
+  it "matches when the recipe is included" do
     expect(subject.matches?(chef_run)).to be_truthy
   end
 
-  it 'does not match when the recipe is not included' do
-    failure = described_class.new('nope')
+  it "does not match when the recipe is not included" do
+    failure = described_class.new("nope")
     expect(failure.matches?(chef_run)).to be_falsy
   end
 end

data/spec/unit/matchers/link_to_matcher_spec.rb

@@ -1,8 +1,8 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe ChefSpec::Matchers::LinkToMatcher do
-  let(:from) { '/var/www' }
-  let(:to)   { '/var/html' }
+  let(:from) { "/var/www" }
+  let(:to)   { "/var/html" }
   let(:link) do
     Chef::Resource::Link.new(from).tap do |link|
       link.to(to)
@@ -11,44 +11,44 @@ describe ChefSpec::Matchers::LinkToMatcher do
   end
   subject { described_class.new(to) }
 
-  describe '#failure_message' do
-    it 'has the right value' do
+  describe "#failure_message" do
+    it "has the right value" do
       subject.matches?(link)
       expect(subject.failure_message)
-        .to eq(%Q(expected "link[#{from}]" to link to "#{to}" but was "#{to}"))
+        .to eq(%Q{expected "link[#{from}]" to link to "#{to}" but was "#{to}"})
     end
   end
 
-  describe '#failure_message_when_negated' do
-    it 'has the right value' do
+  describe "#failure_message_when_negated" do
+    it "has the right value" do
       subject.matches?(link)
       expect(subject.failure_message_when_negated)
-        .to eq(%Q(expected "link[#{from}]" to not link to "#{to}"))
+        .to eq(%Q{expected "link[#{from}]" to not link to "#{to}"})
     end
   end
 
-  describe '#description' do
-    it 'has the right value' do
+  describe "#description" do
+    it "has the right value" do
       subject.matches?(link)
-      expect(subject.description).to eq(%Q(link to "#{to}"))
+      expect(subject.description).to eq(%Q{link to "#{to}"})
     end
   end
 
-  context 'when the link is correct' do
-    it 'matches' do
+  context "when the link is correct" do
+    it "matches" do
       expect(subject.matches?(link)).to be_truthy
     end
 
-    it 'adds the link to the coverage report' do
+    it "adds the link to the coverage report" do
       expect(ChefSpec::Coverage).to receive(:cover!).with(link)
       subject.matches?(link)
     end
   end
 
-  context 'when the link is not correct' do
-    subject { described_class.new('/nope/bad/path/bro') }
+  context "when the link is not correct" do
+    subject { described_class.new("/nope/bad/path/bro") }
 
-    it 'does not match' do
+    it "does not match" do
       expect(subject.matches?(link)).to be_falsy
     end
   end

data/spec/unit/matchers/notifications_matcher_spec.rb

@@ -1,40 +1,39 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe ChefSpec::Matchers::NotificationsMatcher do
-  subject { described_class.new('execute[install]') }
+  subject { described_class.new("execute[install]") }
   let(:package) do
-    double('package',
-      name: 'package',
-      to_s: 'package[foo]',
+    double("package",
+      name: "package",
+      to_s: "package[foo]",
       is_a?: true,
       performed_action?: true,
       immediate_notifications: [],
       delayed_notifications: [],
-      before_notifications: []
-    )
+      before_notifications: [])
   end
 
-  describe '#failure_message' do
-    it 'has the right value' do
+  describe "#failure_message" do
+    it "has the right value" do
       subject.matches?(package)
       expect(subject.failure_message)
-        .to include %|expected "package[foo]" to notify "execute[install]", but did not.|
+        .to include %{expected "package[foo]" to notify "execute[install]", but did not.}
     end
   end
 
-  describe '#failure_message_when_negated' do
-    it 'has the right value' do
+  describe "#failure_message_when_negated" do
+    it "has the right value" do
       subject.matches?(package)
       expect(subject.failure_message_when_negated)
-        .to eq %|expected "package[foo]" to not notify "execute[install]", but it did.|
+        .to eq %{expected "package[foo]" to not notify "execute[install]", but it did.}
     end
   end
 
-  describe '#description' do
-    it 'has the right value' do
+  describe "#description" do
+    it "has the right value" do
       subject.matches?(package)
       expect(subject.description)
-        .to eq %|notify "execute[install]"|
+        .to eq %{notify "execute[install]"}
     end
   end
 end