chefspec 7.2.1 → 7.3.0

data/examples/spec_step_into/recipes/default.rb

@@ -0,0 +1,3 @@
+spec_step_into_one 'one'
+
+spec_step_into_two 'two'

data/examples/spec_step_into/resources/one.rb

@@ -0,0 +1,3 @@
+action :run do
+  log 'one'
+end

data/examples/spec_step_into/resources/two.rb

@@ -0,0 +1,3 @@
+action :run do
+  log 'two'
+end

data/examples/spec_step_into/spec/default_spec.rb

@@ -0,0 +1,50 @@
+describe 'spec_step_into' do
+  platform 'ubuntu', '18.04'
+
+  context 'with no step_into' do
+    it { is_expected.to_not write_log('one') }
+    it { is_expected.to_not write_log('two') }
+  end
+
+  context 'with step_into one' do
+    step_into :spec_step_into_one
+    it { is_expected.to write_log('one') }
+    it { is_expected.to_not write_log('two') }
+
+    context 'with nested step_into one' do
+      step_into :spec_step_into_one
+      it { is_expected.to write_log('one') }
+      it { is_expected.to_not write_log('two') }
+    end
+
+    context 'with nested step_into two' do
+      step_into :spec_step_into_two
+      it { is_expected.to write_log('one') }
+      it { is_expected.to write_log('two') }
+    end
+  end
+
+  context 'with step_into two' do
+    step_into :spec_step_into_two
+    it { is_expected.to_not write_log('one') }
+    it { is_expected.to write_log('two') }
+  end
+
+  context 'with step_into both' do
+    step_into :spec_step_into_one, :spec_step_into_two
+    it { is_expected.to write_log('one') }
+    it { is_expected.to write_log('two') }
+  end
+
+  context 'with step_into as a string' do
+    step_into 'spec_step_into_one'
+    it { is_expected.to write_log('one') }
+    it { is_expected.to_not write_log('two') }
+  end
+
+  context 'with step_into as an array' do
+    step_into [:spec_step_into_one]
+    it { is_expected.to write_log('one') }
+    it { is_expected.to_not write_log('two') }
+  end
+end

data/examples/state_attrs/spec/default_spec.rb

@@ -3,7 +3,7 @@ require 'chefspec'
 ChefSpec.define_matcher(:state_attrs_lwrp)
 
 describe 'state_attrs::default' do
-  let(:chef_run) { ChefSpec::ServerRunner.new(platform: 'ubuntu', version: '16.04').converge(described_recipe) }
+  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '18.04').converge(described_recipe) }
   let(:lwrp) { chef_run.state_attrs_lwrp('name') }
 
   it 'has the correct state attributes' do

data/examples/step_into/spec/default_spec.rb

@@ -2,7 +2,7 @@ require 'chefspec'
 
 describe 'step_into::default' do
   context 'without :step_into' do
-    let(:chef_run) { ChefSpec::ServerRunner.new(platform: 'ubuntu', version: '16.04').converge(described_recipe) }
+    let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '18.04').converge(described_recipe) }
 
     it 'does not execute the LWRPs action' do
       expect(chef_run).to_not write_log('message')
@@ -15,7 +15,7 @@ describe 'step_into::default' do
 
   context 'with :step_into' do
     let(:chef_run) do
-      ChefSpec::ServerRunner.new(platform: 'ubuntu', version: '16.04', step_into: %w(step_into_lwrp provides_this))
+      ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '18.04', step_into: %w(step_into_lwrp provides_this))
                             .converge(described_recipe)
     end
 

data/examples/stub_command/spec/default_spec.rb

@@ -1,7 +1,7 @@
 require 'chefspec'
 
 describe 'stub_command::default' do
-  let(:chef_run) { ChefSpec::ServerRunner.new(platform: 'ubuntu', version: '16.04').converge(described_recipe) }
+  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '18.04').converge(described_recipe) }
 
   context 'when the command is not stubbed' do
     it 'raises an exception' do

data/examples/stub_data_bag/spec/default_spec.rb

@@ -1,7 +1,7 @@
 require 'chefspec'
 
 describe 'stub_data_bag::default' do
-  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '16.04').converge(described_recipe) }
+  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '18.04').converge(described_recipe) }
 
   context 'when the data_bag is not stubbed' do
     it 'raises an exception' do

data/examples/stub_data_bag_item/spec/default_spec.rb

@@ -1,7 +1,7 @@
 require 'chefspec'
 
 describe 'stub_data_bag_item::default' do
-  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '16.04').converge(described_recipe) } 
+  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '18.04').converge(described_recipe) } 
 
   context 'when the data_bag_item is not stubbed' do
     it 'raises an exception' do

data/examples/stub_node/spec/default_spec.rb

@@ -2,7 +2,7 @@ require 'chefspec'
 
 describe 'stub_node::default' do
   let(:my_node) do
-    stub_node('example.com', platform: 'ubuntu', version: '14.04') do |node|
+    stub_node('example.com', platform: 'ubuntu', version: '18.04') do |node|
       node.normal['foo']['bar'] = 'zip'
     end
   end

data/examples/stub_search/spec/block_spec.rb

@@ -1,7 +1,7 @@
 require 'chefspec'
 
 describe 'stub_search::block' do
-  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '16.04').converge(described_recipe) }
+  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '18.04').converge(described_recipe) }
 
   context 'when the search is not stubbed' do
     it 'raises an exception' do

data/examples/stub_search/spec/default_spec.rb

@@ -1,7 +1,7 @@
 require 'chefspec'
 
 describe 'stub_search::default' do
-  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '16.04').converge(described_recipe) }
+  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '18.04').converge(described_recipe) }
 
   context 'when the search is not stubbed' do
     it 'raises an exception' do

data/examples/stubs_for/resources/default.rb

@@ -0,0 +1,20 @@
+resource_name :stubs_for_test
+
+property :cmd
+property :value
+property :run_resource, [true, false], default: false
+property :run_load, [true, false], default: false
+property :run_provider, [true, false], default: false
+
+def foo
+  shell_out(cmd).stdout
+end
+
+load_current_value do |new_resource|
+  value shell_out(new_resource.cmd).stdout if new_resource.run_load
+end
+
+action :run do
+  new_resource.foo if new_resource.run_resource
+  shell_out!(new_resource.cmd) if new_resource.run_provider
+end

data/examples/stubs_for/resources/old.rb

@@ -0,0 +1,26 @@
+resource_name :stubs_for_old
+
+property :cmd
+property :value
+property :run_resource, [true, false], default: false
+property :run_load, [true, false], default: false
+property :run_provider, [true, false], default: false
+
+def foo
+  shell_out(cmd).stdout
+end
+
+action_class do
+  def load_current_resource
+    @current_resource = new_resource.class.new(new_resource.name)
+    # current_resource.shell_out is weird but mostly doing it for completeness and parity checking with load_curent_value-style.
+    @current_resource.value @current_resource.shell_out(new_resource.cmd).stdout if new_resource.run_load
+    @current_resource
+  end
+end
+
+
+action :run do
+  new_resource.foo if new_resource.run_resource
+  shell_out!(new_resource.cmd) if new_resource.run_provider
+end

data/examples/stubs_for/spec/default_spec.rb

@@ -0,0 +1,240 @@
+describe 'stubs_for' do
+  platform 'ubuntu'
+  step_into :stubs_for_test, :stubs_for_old
+  default_attributes['test'] = {run_load: false, run_resource: false, run_provider: false}
+  recipe do
+    stubs_for_test 'test' do
+      cmd 'this_is_not_a_cmd'
+      run_load node['test']['run_load']
+      run_resource node['test']['run_resource']
+      run_provider node['test']['run_provider']
+    end
+  end
+
+  context 'with no stubs' do
+    context 'running nothing' do
+      it { expect { subject }.to_not raise_error }
+    end
+
+    context 'running load' do
+      default_attributes['test']['run_load'] = true
+      it { expect { subject }.to raise_error ChefSpec::Error::ShellOutNotStubbed }
+    end
+
+    context 'running resource' do
+      default_attributes['test']['run_resource'] = true
+      it { expect { subject }.to raise_error ChefSpec::Error::ShellOutNotStubbed }
+    end
+
+    context 'running provider' do
+      default_attributes['test']['run_provider'] = true
+      it { expect { subject }.to raise_error ChefSpec::Error::ShellOutNotStubbed }
+    end
+
+    context 'running all three' do
+      default_attributes['test'] = {run_load: true, run_resource: true, run_provider: true}
+      it { expect { subject }.to raise_error ChefSpec::Error::ShellOutNotStubbed }
+    end
+  end
+
+  context 'with a single inline resource stub' do
+    context 'running load' do
+      default_attributes['test']['run_load'] = true
+
+      it do
+        stubs_for_resource('stubs_for_test[test]') do |res|
+          allow(res).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf')
+        end
+        subject
+      end
+
+      it do
+        stubs_for_current_resource('stubs_for_test[test]') do |res|
+          allow(res).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf')
+        end
+        subject
+      end
+
+      it do
+        stubs_for_resource('stubs_for_test[test]', current_resource: false) do |res|
+          allow(res).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf')
+        end
+        expect { subject }.to raise_error ChefSpec::Error::ShellOutNotStubbed
+      end
+
+      context 'with old-style load_current_resource' do
+        recipe do
+          stubs_for_old 'test' do
+            cmd 'this_is_not_a_cmd'
+            run_load true
+          end
+        end
+
+        it do
+          stubs_for_current_resource('stubs_for_old[test]') do |res|
+            allow(res).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf')
+          end
+          subject
+        end
+      end
+    end
+
+    context 'running resource' do
+      default_attributes['test']['run_resource'] = true
+
+      it do
+        stubs_for_resource('stubs_for_test[test]') do |res|
+          allow(res).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf')
+        end
+        is_expected.to run_stubs_for_test('test').with(foo: 'asdf')
+      end
+    end
+
+    context 'running both' do
+      default_attributes['test']['run_load'] = true
+      default_attributes['test']['run_resource'] = true
+
+      it do
+        stubs_for_resource('stubs_for_test[test]') do |res|
+          # Use expect instead of allow for this test.
+          expect(res).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf').at_least(:once)
+        end
+        is_expected.to run_stubs_for_test('test').with(foo: 'asdf')
+      end
+    end
+  end
+
+  context 'with a single inline provider stub' do
+    default_attributes['test']['run_provider'] = true
+
+    it do
+      stubs_for_provider('stubs_for_test[test]') do |prov|
+        allow(prov).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf')
+      end
+      subject
+    end
+  end
+
+  context 'with a single group-level resource stub' do
+    context 'running load' do
+      default_attributes['test']['run_load'] = true
+
+      context 'with normal usage' do
+        stubs_for_resource('stubs_for_test[test]') do |res|
+          allow(res).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf')
+        end
+
+        it { subject }
+      end
+
+      context 'with stubs_for_current_resource' do
+        stubs_for_current_resource('stubs_for_test[test]') do |res|
+          allow(res).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf')
+        end
+
+        it { subject }
+      end
+
+      context 'with current_resource: false' do
+        stubs_for_resource('stubs_for_test[test]', current_resource: false) do |res|
+          allow(res).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf')
+        end
+        it { expect { subject }.to raise_error ChefSpec::Error::ShellOutNotStubbed }
+      end
+    end
+
+    context 'running resource' do
+      default_attributes['test']['run_resource'] = true
+
+      stubs_for_resource('stubs_for_test[test]') do |res|
+        allow(res).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf')
+      end
+
+      it { is_expected.to run_stubs_for_test('test').with(foo: 'asdf') }
+    end
+
+    context 'running both' do
+      default_attributes['test']['run_load'] = true
+      default_attributes['test']['run_resource'] = true
+
+      stubs_for_resource('stubs_for_test[test]') do |res|
+        # Use expect instead of allow for this test.
+        expect(res).to receive_shell_out('this_is_not_a_cmd', stdout: 'asdf').at_least(:once)
+      end
+
+      it { is_expected.to run_stubs_for_test('test').with(foo: 'asdf') }
+    end
+  end
+
+  context 'with multiple nested stubs' do
+    recipe do
+      stubs_for_test 'test1' do
+        cmd 'this_is_not_a_cmd1'
+        run_resource true
+      end
+      stubs_for_test 'test2' do
+        cmd 'this_is_not_a_cmd2'
+        run_resource true
+      end
+      stubs_for_test 'test3' do
+        cmd 'this_is_not_a_cmd2'
+        run_resource true
+      end
+      stubs_for_old 'test4' do
+        cmd 'this_is_not_a_cmd3'
+        run_resource true
+      end
+    end
+
+    stubs_for_resource('stubs_for_test[test1]') do |res|
+      allow(res).to receive_shell_out('this_is_not_a_cmd1', stdout: 'one')
+    end
+    stubs_for_resource('stubs_for_test') do |res|
+      allow(res).to receive_shell_out('this_is_not_a_cmd2', stdout: 'two')
+    end
+    stubs_for_resource do |res|
+      allow(res).to receive_shell_out('this_is_not_a_cmd3', stdout: 'three')
+    end
+
+    it { is_expected.to run_stubs_for_test('test1').with(foo: 'one') }
+    it { is_expected.to run_stubs_for_test('test2').with(foo: 'two') }
+    it { is_expected.to run_stubs_for_test('test3').with(foo: 'two') }
+    it { is_expected.to run_stubs_for_old('test4').with(foo: 'three') }
+
+    context 'nested a' do
+      stubs_for_resource('stubs_for_test[test1]') do |res|
+        allow(res).to receive_shell_out('this_is_not_a_cmd1', stdout: 'four')
+      end
+
+      it { is_expected.to run_stubs_for_test('test1').with(foo: 'four') }
+      it { is_expected.to run_stubs_for_test('test2').with(foo: 'two') }
+      it { is_expected.to run_stubs_for_test('test3').with(foo: 'two') }
+      it { is_expected.to run_stubs_for_old('test4').with(foo: 'three') }
+
+      context 'nested b' do
+        stubs_for_resource do |res|
+          allow(res).to receive_shell_out('this_is_not_a_cmd3', stdout: 'five')
+        end
+
+        it { is_expected.to run_stubs_for_test('test1').with(foo: 'four') }
+        it { is_expected.to run_stubs_for_test('test2').with(foo: 'two') }
+        it { is_expected.to run_stubs_for_test('test3').with(foo: 'two') }
+        it { is_expected.to run_stubs_for_old('test4').with(foo: 'five') }
+      end
+    end
+
+    context 'nested c' do
+      stubs_for_resource('stubs_for_test') do |res|
+        allow(res).to receive_shell_out('this_is_not_a_cmd2', stdout: 'six')
+      end
+      stubs_for_resource do |res|
+        allow(res).to receive_shell_out('this_is_not_a_cmd3', stdout: 'seven')
+      end
+
+      it { is_expected.to run_stubs_for_test('test1').with(foo: 'one') }
+      it { is_expected.to run_stubs_for_test('test2').with(foo: 'six') }
+      it { is_expected.to run_stubs_for_test('test3').with(foo: 'six') }
+      it { is_expected.to run_stubs_for_old('test4').with(foo: 'seven') }
+    end
+  end
+end

data/examples/subscribes/spec/before_spec.rb

@@ -1,7 +1,7 @@
 require 'chefspec'
 
 describe 'subscribes::before' do
-  let(:chef_run) { ChefSpec::ServerRunner.new(platform: 'ubuntu', version: '16.04').converge(described_recipe) }
+  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'ubuntu', version: '18.04').converge(described_recipe) }
   let(:service) { chef_run.service('receiving_resource') }
 
   it 'subscribes to the template creation' do