Merge pull request #14 from Financial-Times/correct-suite-name-email

Correct suite name in email

src/email.js

@@ -5,19 +5,19 @@ const nodemailer = require('nodemailer');
 const transporter = nodemailer.createTransport(REGRESSION_TRANSPORT);
 const logger = require('@financial-times/n-logger').default.logger;
 
-module.exports = function sendEmails (err, output) {
-
+module.exports = function sendEmails (err, output, { suite = 'regression' } = { }) {
+	const capitalisedSuite = suite.charAt(0).toUpperCase() + suite.slice(1);
 	const options = {
-		from: '"Signup Regression" <[email protected]>',
+		from: `"Signup ${capitalisedSuite}" <[email protected]>`,
 		to: RECIPIENTS,
-		subject: `❗Regression tests failed ${new Date()}`,
+		subject: `❗${capitalisedSuite} tests failed ${new Date()}`,
 		text: output.replace('[0;37m ','').replace('[41m ', '').replace('[42m ', '').replace('[44m ', '').replace('[45m ', '')
 	};
 
 	transporter.sendMail(options, function (emailError, data) {
 		if (emailError) {
 			return logger.info('Error sending emails', emailError);
 		}
-		logger.info('Regression test notification sent to: ', data);
+		logger.info(`${capitalisedSuite} test notification sent to: `, data);
 	});
 };

src/main.js

@@ -129,7 +129,7 @@ module.exports = class Automation {
 
 			if (error) {
 				logger.info('Sending email...', error, stderr)
-				sendEmails(stderr, stdout);
+				sendEmails(stderr, stdout, { suite });
 			}
 		});
 	}