refactor email service to log events and added specs

app/services/emails/base_service.rb

@@ -5,5 +5,10 @@ module Emails
       @user = user
       @email = opts[:email]
     end
+
+    def log_audit_event(options = {})
+      AuditEventService.new(@current_user, @user, options)
+          .for_email(@email).security_event
+    end
   end
 end

app/services/emails/create_service.rb

@@ -2,6 +2,8 @@ module Emails
   class CreateService < ::Emails::BaseService
     def execute
       @user.emails.create(email: @email)
+
+      log_audit_event(action: :create)
     end
   end
 end

app/services/emails/destroy_service.rb

@@ -2,6 +2,8 @@ module Emails
   class DestroyService < ::Emails::BaseService
     def execute
       Email.find_by_email!(@email).destroy && update_secondary_emails!
+
+      log_audit_event(action: :destroy)
     end
 
     private

spec/services/emails/create_service_spec.rb

@@ -17,5 +17,9 @@ describe Emails::CreateService do
 
       expect(user.emails).to eq(Email.where(opts))
     end
+
+    it 'registers a security event' do
+      expect { service.execute }.to change { SecurityEvent.count }.by(1)
+    end
   end
 end

spec/services/emails/destroy_service_spec.rb

@@ -10,5 +10,9 @@ describe Emails::DestroyService do
     it 'removes an email' do
       expect { service.execute }.to change { user.emails.count }.by(-1)
     end
+
+    it 'registers a security event' do
+      expect { service.execute }.to change { SecurityEvent.count }.by(1)
+    end
   end
 end