refactor(invitations): extract send_email_with_logging helper method

Author: mroderick

app/models/concerns/workshop_invitation_manager_concerns.rb

@@ -117,14 +117,7 @@ def invite_coaches_to_virtual_workshop(workshop, logger = nil)
         next unless invitation
 
         count += 1
-        if logger
-          begin
-            VirtualWorkshopInvitationMailer.invite_coach(workshop, coach, invitation).deliver_now
-            logger.log_success(coach, invitation)
-          rescue StandardError => e
-            logger.log_failure(coach, invitation, e)
-          end
-        else
+        send_email_with_logging(logger, coach, invitation) do
           VirtualWorkshopInvitationMailer.invite_coach(workshop, coach, invitation).deliver_now
         end
       end
@@ -138,14 +131,7 @@ def invite_coaches_to_workshop(workshop, logger = nil)
         next unless invitation
 
         count += 1
-        if logger
-          begin
-            WorkshopInvitationMailer.invite_coach(workshop, coach, invitation).deliver_now
-            logger.log_success(coach, invitation)
-          rescue StandardError => e
-            logger.log_failure(coach, invitation, e)
-          end
-        else
+        send_email_with_logging(logger, coach, invitation) do
           WorkshopInvitationMailer.invite_coach(workshop, coach, invitation).deliver_now
         end
       end
@@ -159,14 +145,7 @@ def invite_students_to_virtual_workshop(workshop, logger = nil)
         next unless invitation
 
         count += 1
-        if logger
-          begin
-            VirtualWorkshopInvitationMailer.invite_student(workshop, student, invitation).deliver_now
-            logger.log_success(student, invitation)
-          rescue StandardError => e
-            logger.log_failure(student, invitation, e)
-          end
-        else
+        send_email_with_logging(logger, student, invitation) do
           VirtualWorkshopInvitationMailer.invite_student(workshop, student, invitation).deliver_now
         end
       end
@@ -180,20 +159,26 @@ def invite_students_to_workshop(workshop, logger = nil)
         next unless invitation
 
         count += 1
-        if logger
-          begin
-            WorkshopInvitationMailer.invite_student(workshop, student, invitation).deliver_now
-            logger.log_success(student, invitation)
-          rescue StandardError => e
-            logger.log_failure(student, invitation, e)
-          end
-        else
+        send_email_with_logging(logger, student, invitation) do
           WorkshopInvitationMailer.invite_student(workshop, student, invitation).deliver_now
         end
       end
       count
     end
 
+    def send_email_with_logging(logger, member, invitation)
+      if logger
+        begin
+          yield
+          logger.log_success(member, invitation)
+        rescue StandardError => e
+          logger.log_failure(member, invitation, e)
+        end
+      else
+        yield
+      end
+    end
+
     def retrieve_and_notify_waitlisted(workshop, role:)
       WaitingList.by_workshop(workshop).where_role(role).each do |waiting_list|
         WorkshopInvitationMailer.notify_waiting_list(waiting_list.invitation).deliver_now