bcoe / nodemailer-mock-transport

mock-mailer for putting tests around services that use node-mailer
ISC License
16 stars 4 forks source link

Add ability to trigger arbritrary error-on-send #61

Open JulesAU opened 6 years ago

JulesAU commented 6 years ago

We can't rely on using an invalid email address to trigger a send error; because that likely won't get past the validation part of our APIs under test. So instead, we can now configure a forced-fail instance.

Also added { envelope } response in success callback on sendMail, matching what nodemailer does.

JulesAU commented 6 years ago

Published with my PR to here:

https://www.npmjs.com/package/@motowhere/nodemailer-mock-transport

bcoe commented 6 years ago

@JulesAU thanks for the update, sorry for dropping this on the floor. Any chance you could get the tests passing and we can work on landing this?

JulesAU commented 6 years ago

@bcoe Don't think the failing build has anything to do with my PR - seems project build was failing 11 months ago and before...