feat: log all critical bounces

jest.config.js

@@ -9,7 +9,7 @@ module.exports = {
     // Needed to use @shelf/jest-mongodb preset.
     ...tsJestPreset.transform,
   },
-  collectCoverageFrom: ['./src/**/*.{ts,js}'],
+  collectCoverageFrom: ['./src/**/*.{ts,js}', '!**/__tests__/**'],
   coveragePathIgnorePatterns: ['./node_modules/', './tests'],
   coverageThreshold: {
     global: {

src/app/constants/mail.ts

@@ -14,10 +14,10 @@ export const EMAIL_HEADERS = {
 }
 
 // Types of emails we send
-export const EMAIL_TYPES = {
-  adminResponse: 'Admin (response)',
-  loginOtp: 'Login OTP',
-  verificationOtp: 'Verification OTP',
-  emailConfirmation: 'Email confirmation',
-  adminBounce: 'Admin (bounce notification)',
+export enum EmailType {
+  AdminResponse = 'Admin (response)',
+  LoginOtp = 'Login OTP',
+  VerificationOtp = 'Verification OTP',
+  EmailConfirmation = 'Email confirmation',
+  AdminBounce = 'Admin (bounce notification)',
 }

tests/unit/backend/helpers/bounce.ts → src/app/modules/bounce/__tests__/bounce-test-helpers.ts

@@ -26,6 +26,7 @@ const makeEmailNotification = (
   formId: ObjectId,
   submissionId: ObjectId,
   recipientList: string[],
+  emailType: 'Admin (response)' | 'Email confirmation',
 ): IEmailNotification => {
   return {
     notificationType,
@@ -43,7 +44,7 @@ const makeEmailNotification = (
         },
         {
           name: 'X-Formsg-Email-Type',
-          value: 'Admin (response)',
+          value: emailType,
         },
       ],
       commonHeaders: {
@@ -61,9 +62,16 @@ export const makeBounceNotification = (
   recipientList: string[] = [],
   bouncedList: string[] = [],
   bounceType: 'Transient' | 'Permanent' = 'Permanent',
+  emailType: 'Admin (response)' | 'Email confirmation' = 'Admin (response)',
 ): ISnsNotification => {
   const Message = merge(
-    makeEmailNotification('Bounce', formId, submissionId, recipientList),
+    makeEmailNotification(
+      'Bounce',
+      formId,
+      submissionId,
+      recipientList,
+      emailType,
+    ),
     {
       bounce: {
         bounceType,
@@ -83,9 +91,16 @@ export const makeDeliveryNotification = (
   submissionId: ObjectId = new ObjectId(),
   recipientList: string[] = [],
   deliveredList: string[] = [],
+  emailType: 'Admin (response)' | 'Email confirmation' = 'Admin (response)',
 ): ISnsNotification => {
   const Message = merge(
-    makeEmailNotification('Delivery', formId, submissionId, recipientList),
+    makeEmailNotification(
+      'Delivery',
+      formId,
+      submissionId,
+      recipientList,
+      emailType,
+    ),
     {
       delivery: {
         recipients: deliveredList,

tests/unit/backend/models/bounce.server.model.spec.ts → src/app/modules/bounce/__tests__/bounce.model.spec.ts

@@ -1,15 +1,15 @@
 import { ObjectId } from 'bson'
 import { omit, pick } from 'lodash'
 import mongoose from 'mongoose'
+import dbHandler from 'tests/unit/backend/helpers/jest-db'
 
-import getBounceModel from 'src/app/models/bounce.server.model'
+import getBounceModel from 'src/app/modules/bounce/bounce.model'
 
 import {
   extractBounceObject,
   makeBounceNotification,
   makeDeliveryNotification,
-} from '../helpers/bounce'
-import dbHandler from '../helpers/jest-db'
+} from './bounce-test-helpers'
 
 const Bounce = getBounceModel(mongoose)