Yield on PeopleManagerController#create

Author: daniel-illi

app/controllers/people_managers_controller.rb

@@ -21,10 +21,13 @@ def new
 
   def create
     assign_attributes
-    if entry.save
-      redirect_to redirect_to_path
-    else
-      render :new
+    ActiveRecord::Base.transaction do
+      if entry.save
+        yield entry if block_given?
+        redirect_to redirect_to_path
+      else
+        render :new
+      end
     end
   end
 

spec/controllers/people_managers_controller_spec.rb

@@ -49,4 +49,25 @@ def params
       expect { entry.reload }.not_to raise_error
     end
   end
+
+  context '#create' do
+    let(:params) do
+      attr = described_class.assoc == :people_managers ? :manager_id : :managed_id
+      { person_id: people(:bottom_leader).id, people_manager: { attr => people(:bottom_member).id } }
+    end
+
+    it 'yields' do
+      expect_any_instance_of(PeopleManager).to receive(:call_on_yielded)
+
+      expect { post :create, params: params }.to change { PeopleManager.count }.by(1)
+    end
+
+    it 'does not create entry if yielded block raises error' do
+      expect_any_instance_of(PeopleManager).to receive(:call_on_yielded).and_raise('baaad stuff')
+
+      expect { post :create, params: params }.
+        to raise_error('baaad stuff').
+        and not_change { PeopleManager.count }
+    end
+  end
 end

spec/controllers/person/manageds_controller_spec.rb

@@ -9,5 +9,6 @@
 require_relative '../people_managers_controller_spec'
 
 describe Person::ManagedsController do
+  it_behaves_like 'people_managers#create'
   it_behaves_like 'people_managers#destroy'
 end

spec/controllers/person/managers_controller_spec.rb

@@ -9,5 +9,6 @@
 require_relative '../people_managers_controller_spec'
 
 describe Person::ManagersController do
+  it_behaves_like 'people_managers#create'
   it_behaves_like 'people_managers#destroy'
 end