Issue 95: Interactor for Volunteers#knight

app/controllers/volunteers_controller.rb

@@ -222,8 +222,7 @@ def knight
     end
 
     volunteer = Volunteer.find(params[:volunteer_id])
-    volunteer.admin = !volunteer.admin
-    if volunteer.save
+    if ToggleSuperAdmin.call(volunteer: volunteer).success?
       flash[:notice] = "#{volunteer.name} Updated to Admin: #{volunteer.admin}"
     else
       flash[:error] = "#{volunteer.errors.full_messages}"

app/interactors/toggle_super_admin.rb

@@ -0,0 +1,12 @@
+class ToggleSuperAdmin
+  include Interactor
+
+  delegate :volunteer,
+           :fail!,
+           to: :context
+
+  def call
+    volunteer.admin = !volunteer.admin
+    fail! unless volunteer.save
+  end
+end

spec/interactors/toggle_super_admin_spec.rb

@@ -0,0 +1,42 @@
+require 'rails_helper'
+
+RSpec.describe ToggleSuperAdmin do
+  describe '::call' do
+    let(:volunteer) do
+      create(:volunteer)
+    end
+
+    subject do
+      described_class.call(
+          volunteer: volunteer
+      )
+    end
+
+    it 'fails if it cannot save the volunteer' do
+      expect(volunteer).to receive(:save).and_return false
+      expect(subject.success?).to eq(false)
+    end
+
+    context 'is admin' do
+      let(:volunteer) do
+        create(:volunteer, admin: true)
+      end
+
+      it 'unsets the admin flag' do
+        expect(subject.success?).to eq(true)
+        expect(volunteer.reload.admin).to eq(false)
+      end
+    end
+
+    context 'is not admin' do
+      let(:volunteer) do
+        create(:volunteer, admin: false)
+      end
+
+      it 'sets the admin flag' do
+        expect(subject.success?).to eq(true)
+        expect(volunteer.reload.admin).to eq(true)
+      end
+    end
+  end
+end