Sfoglia il codice sorgente

fix a daft mistake

Shane Kilkelly 9 anni fa
parent
commit
d2e1efe4a9

+ 2 - 1
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)

+ 2 - 1
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