summaryrefslogtreecommitdiff
path: root/lib/chef/provider/group.rb
diff options
context:
space:
mode:
authorLamont Granquist <lamont@scriptkiddie.org>2019-07-02 15:09:07 -0700
committerLamont Granquist <lamont@scriptkiddie.org>2019-07-02 15:09:07 -0700
commit7a1a6c8ef26c787e4b6dd1602f3d158b37e81720 (patch)
tree1e390cd535b38368d091cbb33e5d419408d5ce00 /lib/chef/provider/group.rb
parent77f8739a4741e2370e40ec39345a92a6ea393a1a (diff)
downloadchef-7a1a6c8ef26c787e4b6dd1602f3d158b37e81720.tar.gz
fix Layout/EmptyLineAfterGuardClause
i like this one, gives visual priority to returns or raises that are buried in the middle of things. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
Diffstat (limited to 'lib/chef/provider/group.rb')
-rw-r--r--lib/chef/provider/group.rb4
1 files changed, 4 insertions, 0 deletions
diff --git a/lib/chef/provider/group.rb b/lib/chef/provider/group.rb
index ba84ea9468..c0396d9c6a 100644
--- a/lib/chef/provider/group.rb
+++ b/lib/chef/provider/group.rb
@@ -88,6 +88,7 @@ class Chef
missing_members = []
new_resource.members.each do |member|
next if has_current_group_member?(member)
+
validate_member!(member)
missing_members << member
end
@@ -140,6 +141,7 @@ class Chef
def action_remove
return unless @group_exists
+
converge_by("remove group #{new_resource.group_name}") do
remove_group
logger.info("#{new_resource} removed")
@@ -148,6 +150,7 @@ class Chef
def action_manage
return unless @group_exists && compare_group
+
converge_by(["manage group #{new_resource.group_name}"] + change_desc) do
manage_group
logger.info("#{new_resource} managed")
@@ -156,6 +159,7 @@ class Chef
def action_modify
return unless compare_group
+
converge_by(["modify group #{new_resource.group_name}"] + change_desc) do
manage_group
logger.info("#{new_resource} modified")