Merge pull request #2698 from mozilla/pb/fix-notification-format

#2698 r=shane-tomlinson
mozilla · Oct 29, 2018 · dbde76a · dbde76a
2 parents 4d4e843 + ec3ff7b
commit dbde76a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 22 deletions.
diff --git a/lib/email/notifications.js b/lib/email/notifications.js
@@ -22,10 +22,10 @@ module.exports = (log, error) => {
 
         let addresses = [], eventType = 'bounced', isDeletionCandidate = false
         if (message.bounce) {
-          addresses = mapBounceComplaintRecipients(message.bounce.bouncedRecipients)
+          addresses = message.bounce.bouncedRecipients
           isDeletionCandidate = true
         } else if (message.complaint) {
-          addresses = mapBounceComplaintRecipients(message.complaint.complainedRecipients)
+          addresses = message.complaint.complainedRecipients
           isDeletionCandidate = true
         } else if (message.delivery) {
           addresses = message.delivery.recipients
@@ -57,7 +57,3 @@ module.exports = (log, error) => {
     })
   }
 }
-
-function mapBounceComplaintRecipients (recipients) {
-  return recipients.map(recipient => recipient.emailAddress)
-}
diff --git a/test/local/email/notifications.js b/test/local/email/notifications.js
@@ -71,7 +71,7 @@ describe('lib/email/notifications:', () => {
           }
         },
         bounce: {
-          bouncedRecipients: [ { emailAddress: 'wibble@example.com' } ]
+          bouncedRecipients: [ 'wibble@example.com' ]
         }
       })
     })
@@ -136,10 +136,7 @@ describe('lib/email/notifications:', () => {
           }
         },
         complaint: {
-          complainedRecipients: [
-            { emailAddress: 'foo@example.com' },
-            { emailAddress: 'pmbooth@gmail.com' }
-          ]
+          complainedRecipients: [ 'foo@example.com', 'pmbooth@gmail.com' ]
         }
       })
     })
@@ -234,10 +231,7 @@ describe('lib/email/notifications:', () => {
           }
         },
         bounce: {
-          bouncedRecipients: [
-            { emailAddress: 'wibble@example.com' },
-            { emailAddress: 'blee@example.com' }
-          ]
+          bouncedRecipients: [ 'wibble@example.com', 'blee@example.com' ]
         }
       })
     })
@@ -309,9 +303,7 @@ describe('lib/email/notifications:', () => {
           }
         },
         complaint: {
-          complainedRecipients: [
-            { emailAddress: 'foo@example.com' }
-          ]
+          complainedRecipients: [ 'foo@example.com' ]
         }
       })
     })
@@ -351,9 +343,7 @@ describe('lib/email/notifications:', () => {
           }
         },
         bounce: {
-          bouncedRecipients: [
-            { emailAddress: 'wibble@example.com' }
-          ]
+          bouncedRecipients: [ 'wibble@example.com' ]
         }
       })
     })
@@ -444,7 +434,7 @@ describe('lib/email/notifications:', () => {
         del,
         mail: {},
         bounce: {
-          bouncedRecipients: [ { emailAddress: 'wibble@example.com' } ]
+          bouncedRecipients: [ 'wibble@example.com' ]
         }
       })
     })