diff options
author | Pete Higgins <pete@peterhiggins.org> | 2020-07-13 16:35:06 -0700 |
---|---|---|
committer | Pete Higgins <pete@peterhiggins.org> | 2020-07-14 11:10:31 -0700 |
commit | dab3d44e36ba9dc9907085feb95275f3fd1952eb (patch) | |
tree | c3b03e419ce39d4f1b0853e4b922083d43150949 /spec/functional | |
parent | ebfbce99c8c1a4caaa3fb3df98d3b49625883bf4 (diff) | |
download | chef-dab3d44e36ba9dc9907085feb95275f3fd1952eb.tar.gz |
Don't allow expectations of raising an error without an error type.
Signed-off-by: Pete Higgins <pete@peterhiggins.org>
Diffstat (limited to 'spec/functional')
-rw-r--r-- | spec/functional/resource/group_spec.rb | 8 | ||||
-rw-r--r-- | spec/functional/resource/remote_file_spec.rb | 8 |
2 files changed, 7 insertions, 9 deletions
diff --git a/spec/functional/resource/group_spec.rb b/spec/functional/resource/group_spec.rb index 8fca689e68..7e5b0725c1 100644 --- a/spec/functional/resource/group_spec.rb +++ b/spec/functional/resource/group_spec.rb @@ -159,8 +159,10 @@ describe Chef::Resource::Group, :requires_root_or_running_windows do # excluded_members can only be used when append is set. It is ignored otherwise. let(:excluded_members) { [] } + let(:expected_error_class) { windows? ? ArgumentError : Mixlib::ShellOut::ShellCommandFailed } + it "should raise an error" do - expect { group_resource.run_action(tested_action) }.to raise_error + expect { group_resource.run_action(tested_action) }.to raise_error(expected_error_class) end end @@ -169,8 +171,10 @@ describe Chef::Resource::Group, :requires_root_or_running_windows do group_resource.append(true) end + let(:expected_error_class) { windows? ? Chef::Exceptions::Win32APIError : Mixlib::ShellOut::ShellCommandFailed } + it "should raise an error" do - expect { group_resource.run_action(tested_action) }.to raise_error + expect { group_resource.run_action(tested_action) }.to raise_error(expected_error_class) end end end diff --git a/spec/functional/resource/remote_file_spec.rb b/spec/functional/resource/remote_file_spec.rb index f4a3d5346d..09e4fdccb4 100644 --- a/spec/functional/resource/remote_file_spec.rb +++ b/spec/functional/resource/remote_file_spec.rb @@ -409,13 +409,7 @@ describe Chef::Resource::RemoteFile do it "should not create the file" do # This can legitimately raise either Errno::EADDRNOTAVAIL or Errno::ECONNREFUSED # in different Ruby versions. - old_value = RSpec::Expectations.configuration.on_potential_false_positives - RSpec::Expectations.configuration.on_potential_false_positives = :nothing - begin - expect { resource.run_action(:create) }.to raise_error - ensure - RSpec::Expectations.configuration.on_potential_false_positives = old_value - end + expect { resource.run_action(:create) }.to raise_error(SystemCallError) expect(File).not_to exist(path) end |