summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Duffield <tom@chef.io>2016-10-21 14:44:49 -0500
committerTom Duffield <tom@chef.io>2016-10-21 14:44:49 -0500
commitb60269229863667fe7512fff011885f8bbf722aa (patch)
treee21671edf8f60e1416697ae86d575754857507f2
parent5d0065ab5ee6a73e6b67b5a7a8350f752cf86dbf (diff)
downloadchef-b60269229863667fe7512fff011885f8bbf722aa.tar.gz
Use `all?` instead of `any?` to check for SUSE users
Using `all?` will return `false` when the array is empty. Signed-off-by: Tom Duffield <tom@chef.io>
-rw-r--r--lib/chef/provider/group/suse.rb2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/chef/provider/group/suse.rb b/lib/chef/provider/group/suse.rb
index 273485df16..71336f9311 100644
--- a/lib/chef/provider/group/suse.rb
+++ b/lib/chef/provider/group/suse.rb
@@ -41,7 +41,7 @@ class Chef
requirements.assert(:create, :manage, :modify) do |a|
a.assertion do
begin
- to_add(@new_resource.members).any? { |member| Etc.getpwnam(member) }
+ to_add(@new_resource.members).all? { |member| Etc.getpwnam(member) }
rescue
false
end