test that splitting a string on commas DTRTtm/split_group_members

2 files changed, 7 insertions, 2 deletions

diff --git a/lib/chef/resource/group.rb b/lib/chef/resource/group.rb
index f7874e7ccd..aee4727baf 100644
--- a/lib/chef/resource/group.rb
+++ b/lib/chef/resource/group.rb
@@ -55,7 +55,7 @@ class Chef
       end
 
       def members(arg = nil)
-        converted_members = arg.is_a?(String) ? arg.split(',') : arg
+        converted_members = arg.is_a?(String) ? arg.split(",") : arg
         set_or_return(
           :members,
           converted_members,
@@ -66,7 +66,7 @@ class Chef
       alias_method :users, :members
 
       def excluded_members(arg = nil)
-        converted_members = arg.is_a?(String) ? arg.split(',') : arg
+        converted_members = arg.is_a?(String) ? arg.split(",") : arg
         set_or_return(
           :excluded_members,
           converted_members,
diff --git a/spec/unit/resource/group_spec.rb b/spec/unit/resource/group_spec.rb
index 2bb4b57e26..9d9b5c1111 100644
--- a/spec/unit/resource/group_spec.rb
+++ b/spec/unit/resource/group_spec.rb
@@ -110,6 +110,11 @@ describe Chef::Resource::Group, "members" do
       expect(@resource.send(method)).to eql(["aj"])
     end
 
+    it "(#{method}) should split a string on commas" do
+      @resource.send(method, "aj,adam")
+      expect(@resource.send(method)).to eql( %w{aj adam} )
+    end
+
     it "(#{method}) should allow an array" do
       @resource.send(method, %w{aj adam})
       expect(@resource.send(method)).to eql( %w{aj adam} )