fix(auth): Send email when changing recovery phone

Because:

* No email was sent when changing recovery phone

This commit:

* Correct the mailer function name

Closes #FXA-11822

Author: vpomerleau

packages/fxa-auth-server/lib/routes/recovery-phone.ts

@@ -531,7 +531,6 @@ class RecoveryPhoneHandler {
   }
 
   async changePhoneNumber(request: AuthRequest) {
-    // need to check first that there is an existing phone number
     const { uid } = request.auth.credentials as SessionTokenAuthCredential;
 
     const { code } = request.payload as unknown as {
@@ -597,11 +596,17 @@ class RecoveryPhoneHandler {
 
     const { phoneNumber, nationalFormat } =
       await this.recoveryPhoneService.hasConfirmed(uid);
+
+    recordSecurityEvent('account.recovery_phone_replace_complete', {
+      db: this.db,
+      request,
+    });
+
     const { acceptLanguage, geo, ua } = request.app;
     const account = await this.db.account(uid);
 
     try {
-      await this.mailer.postChangeRecoveryPhoneEmail(account.emails, account, {
+      await this.mailer.sendPostChangeRecoveryPhoneEmail(account.emails, account, {
         acceptLanguage,
         timeZone: geo.timeZone,
         uaBrowser: ua.browser,
@@ -611,11 +616,6 @@ class RecoveryPhoneHandler {
         uaDeviceType: ua.deviceType,
         uid,
       });
-
-      recordSecurityEvent('account.recovery_phone_replace_complete', {
-        db: this.db,
-        request,
-      });
     } catch (error) {
       // log error, but don't throw
       // user should be allowed to proceed if email or security event fails

packages/fxa-auth-server/test/local/routes/recovery-phone.js

@@ -694,7 +694,7 @@ describe('/recovery_phone', () => {
     });
 
     it('does not reject if email does not send', async () => {
-      mockMailer.postChangeRecoveryPhoneEmail = sinon.fake.returns(
+      mockMailer.sendPostChangeRecoveryPhoneEmail = sinon.fake.returns(
         Promise.reject(new Error('BOOM'))
       );
 
@@ -706,7 +706,7 @@ describe('/recovery_phone', () => {
       });
 
       assert.isDefined(resp);
-      assert.calledOnce(mockMailer.postChangeRecoveryPhoneEmail);
+      assert.calledOnce(mockMailer.sendPostChangeRecoveryPhoneEmail);
       assert.deepEqual(resp, {
         status: 'success',
         phoneNumber,

packages/fxa-auth-server/test/mocks.js

@@ -184,7 +184,7 @@ const MAILER_METHOD_NAMES = [
   'sendPostRemoveRecoveryPhoneEmail',
   'sendPostSigninRecoveryPhoneEmail',
   'sendPostSigninRecoveryCodeEmail',
-  'postChangeRecoveryPhoneEmail',
+  'sendPostChangeRecoveryPhoneEmail',
 ];
 
 const METRICS_CONTEXT_METHOD_NAMES = [