diff options
author | Bryan McLellan <btm@loftninjas.org> | 2020-04-06 14:25:52 -0400 |
---|---|---|
committer | Bryan McLellan <btm@loftninjas.org> | 2020-04-06 19:18:14 -0400 |
commit | fadde52142b0a7e4c7d24bd6a697246a81bc3051 (patch) | |
tree | cbcd967243a4916b265bbba42fbc39402e584da2 /spec | |
parent | 7696937f41e3476547e68b2e0ea20641eb10a0f2 (diff) | |
download | chef-fadde52142b0a7e4c7d24bd6a697246a81bc3051.tar.gz |
Really skip the reboot pending func test if a reboot is pending
This looks like it never worked but we happened to never run into it until we added a desktop platform to the pipeline.
Signed-off-by: Bryan McLellan <btm@loftninjas.org>
Diffstat (limited to 'spec')
-rw-r--r-- | spec/functional/dsl/reboot_pending_spec.rb | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/spec/functional/dsl/reboot_pending_spec.rb b/spec/functional/dsl/reboot_pending_spec.rb index 89701802b3..09f587a1bc 100644 --- a/spec/functional/dsl/reboot_pending_spec.rb +++ b/spec/functional/dsl/reboot_pending_spec.rb @@ -17,10 +17,13 @@ # require "chef/dsl/reboot_pending" +require "chef/dsl/registry_helper" require "chef/win32/registry" require "spec_helper" describe Chef::DSL::RebootPending, :windows_only do + include Chef::DSL::RegistryHelper + def run_ohai node.consume_external_attrs(OHAI_SYSTEM.data, {}) end @@ -36,10 +39,6 @@ describe Chef::DSL::RebootPending, :windows_only do let(:reg_key) { nil } let(:original_set) { false } - before(:all) { @any_flag = {} } - - after { @any_flag[reg_key] = original_set } - describe 'HKLM\SYSTEM\CurrentControlSet\Control\Session Manager\PendingFileRenameOperations' do let(:reg_key) { 'HKLM\SYSTEM\CurrentControlSet\Control\Session Manager' } let(:original_set) { registry.value_exists?(reg_key, { name: "PendingFileRenameOperations" }) } @@ -79,7 +78,9 @@ describe Chef::DSL::RebootPending, :windows_only do describe "when there is nothing to indicate a reboot is pending" do it "should return false" do - skip "reboot pending" if @any_flag.any? { |_, v| v == true } + skip "reboot pending" if registry_value_exists?('HKLM\SYSTEM\CurrentControlSet\Control\Session Manager', { name: "PendingFileRenameOperations" }) || + registry_key_exists?('HKLM\SOFTWARE\Microsoft\Windows\CurrentVersion\WindowsUpdate\Auto Update\RebootRequired') || + registry_key_exists?('HKLM\SOFTWARE\Microsoft\Windows\CurrentVersion\Component Based Servicing\RebootPending') expect(recipe.reboot_pending?).to be_falsey end end |