Merge pull request #519 from iashwash/sudo_clean

Uses sudo when running clean

lib/chef/knife/solo_clean.rb

@@ -22,7 +22,7 @@ class SoloClean < Knife
       def run
         validate!
         ui.msg "Cleaning up #{host}..."
-        run_command "rm -rf #{provisioning_path}"
+        run_command "#{sudo_command} rm -rf #{provisioning_path}"
       end
 
       def validate!
@@ -33,6 +33,10 @@ def provisioning_path
         # TODO de-duplicate this method with solo cook
         KnifeSolo::Tools.config_value(config, :provisioning_path, '~/chef-solo')
       end
+
+      def sudo_command
+        KnifeSolo::Tools.config_value(config, :sudo_command, 'sudo')
+      end
     end
   end
 end

test/solo_clean_test.rb

@@ -8,7 +8,13 @@ class SoloCleanTest < TestCase
 
   def test_removes_provision_path
     cmd = command('somehost', '--provisioning-path=/foo/bar')
-    cmd.expects(:run_command).with('rm -rf /foo/bar').returns(SuccessfulResult.new)
+    cmd.expects(:run_command).with('sudo rm -rf /foo/bar').returns(SuccessfulResult.new)
+    cmd.run
+  end
+
+  def test_removes_with_custom_sudo
+    cmd = command('somehost', '--sudo-command=/usr/some/sudo')
+    cmd.expects(:run_command).with('/usr/some/sudo rm -rf ~/chef-solo').returns(SuccessfulResult.new)
     cmd.run
   end