take @coderanger review comments into account

1 files changed, 1 insertions, 4 deletions

diff --git a/lib/chef/provider/git.rb b/lib/chef/provider/git.rb
index f59ab116c3..fb34b9c595 100644
--- a/lib/chef/provider/git.rb
+++ b/lib/chef/provider/git.rb
@@ -302,13 +302,10 @@ class Chef
           env["HOME"] = begin
             require "etc"
             case @new_resource.user
-            when String
-              Etc.getpwnam(@new_resource.user).dir
             when Integer
               Etc.getpwuid(@new_resource.user).dir
             else
-              Chef::Log.error("The `user` parameter of the #@new_resource resource is set to an invalid value (#{new_resource.owner.inspect})")
-              raise ArgumentError, "cannot resolve #{new_resource.owner.inspect} to uid, owner must be a string or integer"
+              Etc.getpwnam(@new_resource.user.to_s).dir
             end
             
           rescue ArgumentError # user not found