Always pass a symbol command to execute and test, so that the right S…

…SH user is used

lib/capistrano/tasks/puma.rake

@@ -79,7 +79,7 @@ namespace :puma do
           puma_switch_user(role) do
             with rack_env: fetch(:puma_env) do
               if test "[ -f #{fetch(:puma_pid)} ]"
-                if test "kill -0 $( cat #{fetch(:puma_pid)} )"
+                if test :kill, "-0 $( cat #{fetch(:puma_pid)} )"
                   execute :pumactl, "-S #{fetch(:puma_state)} -F #{fetch(:puma_conf)} #{command}"
                 else
                   # delete invalid pid file , process is not running.
@@ -103,7 +103,7 @@ namespace :puma do
         within current_path do
           puma_switch_user(role) do
             with rack_env: fetch(:puma_env) do
-              if test "[ -f #{fetch(:puma_pid)} ]" and test "kill -0 $( cat #{fetch(:puma_pid)} )"
+              if test "[ -f #{fetch(:puma_pid)} ]" and test :kill, "-0 $( cat #{fetch(:puma_pid)} )"
                 # NOTE pid exist but state file is nonsense, so ignore that case
                 execute :pumactl, "-S #{fetch(:puma_state)} -F #{fetch(:puma_conf)} #{command}"
               else

lib/capistrano/tasks/workers.rake

@@ -21,7 +21,7 @@ namespace :puma do
     task :more do
       on roles (fetch(:puma_role)) do |role|
         puma_switch_user(role) do
-          execute("kill -TTIN `cat  #{fetch(:puma_pid)}`")
+          execute(:kill, "-TTIN `cat  #{fetch(:puma_pid)}`")
         end
       end
     end
@@ -30,7 +30,7 @@ namespace :puma do
     task :less do
       on roles (fetch(:puma_role)) do |role|
         puma_switch_user(role) do
-          execute("kill -TTOU `cat  #{fetch(:puma_pid)}`")
+          execute(:kill, "-TTOU `cat  #{fetch(:puma_pid)}`")
         end
       end
     end