Integrate rm_policy_group service with undo stack

lib/chef-dk/policyfile/undo_record.rb

@@ -120,10 +120,6 @@ def for_serialization
         }
       end
 
-      def commit!
-        raise NotImplementedError, "TODO"
-      end
-
       private
 
       def reset!

lib/chef-dk/policyfile_services/rm_policy_group.rb

@@ -17,6 +17,7 @@
 
 require 'chef-dk/service_exceptions'
 require 'chef-dk/authenticated_http'
+require 'chef-dk/policyfile/undo_stack'
 require 'chef-dk/policyfile/undo_record'
 
 module ChefDK
@@ -35,12 +36,16 @@ class RmPolicyGroup
       # @api private
       attr_reader :undo_record
 
+      # @api private
+      attr_reader :undo_stack
+
       def initialize(config: nil, ui: nil, policy_group: nil)
         @chef_config = config
         @ui = ui
         @policy_group = policy_group
 
         @undo_record = Policyfile::UndoRecord.new
+        @undo_stack = Policyfile::UndoStack.new
       end
 
       def run
@@ -59,7 +64,7 @@ def run
         http_client.delete("/policy_groups/#{policy_group}")
         undo_record.add_policy_group(policy_group)
         ui.err("Removed policy group '#{policy_group}'.")
-        undo_record.commit!
+        undo_stack.push(undo_record)
       rescue => e
         raise DeletePolicyGroupError.new("Failed to delete policy group '#{policy_group}'", e)
       end

spec/unit/policyfile_services/rm_policy_group_spec.rb

@@ -66,6 +66,11 @@
     rm_policy_group_service.undo_record
   end
 
+  let(:undo_stack) do
+    rm_policy_group_service.undo_stack
+  end
+
+
   it "configures an HTTP client" do
     expect(ChefDK::AuthenticatedHTTP).to receive(:new).with("https://localhost:10443",
                                                        signing_key_filename: "/path/to/client/key.pem",
@@ -145,7 +150,7 @@
       allow(rm_policy_group_service).to receive(:http_client).and_return(http_client)
       expect(http_client).to receive(:get).with("/policy_groups").and_return(empty_policy_groups)
       expect(http_client).to receive(:delete).with("/policy_groups/preprod")
-      expect(undo_record).to receive(:commit!)
+      expect(undo_stack).to receive(:push).with(undo_record)
     end
 
     it "removes the group" do
@@ -198,7 +203,7 @@
         and_return(policy_db_9)
 
       expect(http_client).to receive(:delete).with("/policy_groups/preprod")
-      expect(undo_record).to receive(:commit!)
+      expect(undo_stack).to receive(:push).with(undo_record)
     end
 
     it "removes the group" do