Finished refactoring converge and updated tests

so that converge and converge! are no longer a method in the provider and why_run, respectively. There is instead set_updated_status in provider. Everything is now converged in why_run's add_action, so neither of the converge methods should ever need to be called. Updated tests to reflect this. Removed all calls to converge, and replaced with set_updated_status only for tests that are looking at the update status of a provider, where the converge behavior has been mocked out.

1 files changed, 0 insertions, 11 deletions

diff --git a/lib/chef/mixin/why_run.rb b/lib/chef/mixin/why_run.rb
index 888d8e2fef..788e2fe2bc 100644
--- a/lib/chef/mixin/why_run.rb
+++ b/lib/chef/mixin/why_run.rb
@@ -58,17 +58,6 @@ class Chef
         def empty?
           @actions.empty?
         end
-
-        # Iterate over the actions, and either print the action's message, or
-        # run its code block, depending on whether why_run mode is active.
-        def converge!
-          # @actions.each do |descriptions, block|
-          #   if !Chef::Config[:why_run]
-          #     block.call
-          #   end
-          #   events.resource_update_applied(@resource, @action, descriptions)
-          # end
-        end
       end
 
       # == ResourceRequirements