From d2e1efe4a9098bc6713301061d5f923f255034f5 Mon Sep 17 00:00:00 2001 From: Shane Kilkelly Date: Mon, 27 Mar 2017 17:45:19 +0100 Subject: [PATCH] fix a daft mistake --- services/web/app/coffee/Features/Email/EmailSender.coffee | 3 ++- .../web/test/UnitTests/coffee/Email/EmailSenderTests.coffee | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/services/web/app/coffee/Features/Email/EmailSender.coffee b/services/web/app/coffee/Features/Email/EmailSender.coffee index 009d24e182..04b4d39132 100644 --- a/services/web/app/coffee/Features/Email/EmailSender.coffee +++ b/services/web/app/coffee/Features/Email/EmailSender.coffee @@ -72,6 +72,7 @@ module.exports = client.sendMail options, (err, res)-> if err? logger.err err:err, "error sending message" + err = new Error('Cannot send email') else logger.log "Message sent to #{options.to}" - callback(new Error('Cannot send email')) + callback(err) diff --git a/services/web/test/UnitTests/coffee/Email/EmailSenderTests.coffee b/services/web/test/UnitTests/coffee/Email/EmailSenderTests.coffee index bdc2a44811..dc504907bf 100644 --- a/services/web/test/UnitTests/coffee/Email/EmailSenderTests.coffee +++ b/services/web/test/UnitTests/coffee/Email/EmailSenderTests.coffee @@ -51,7 +51,8 @@ describe "EmailSender", -> it "should set the properties on the email to send", (done)-> @sesClient.sendMail.callsArgWith(1) - @sender.sendEmail @opts, => + @sender.sendEmail @opts, (err) => + expect(err).to.not.exist args = @sesClient.sendMail.args[0][0] args.html.should.equal @opts.html args.to.should.equal @opts.to