Update routing middleware to support multiple to, cc, bcc in send action

Author: johanbrook

Diff for: lib/routing.js

@@ -12,6 +12,10 @@ const CONTENT_TYPES = {
   text: 'text/plain'
 };
 
+const arrayOrString = (str) => {
+  return Array.isArray(str) ? str : str.split(',');
+};
+
 Routing = (template, settings, render, compile) => {
   check(template.name, String);
   check(template.route.path, String);
@@ -22,7 +26,7 @@ Routing = (template, settings, render, compile) => {
   const previewAction = (type) => {
     check(type, Match.OneOf('html', 'text'));
 
-    return (req, res, params, template) => {
+    return (req, res, params, _) => {
       let data = null;
 
       try {
@@ -71,7 +75,7 @@ Routing = (template, settings, render, compile) => {
     };
   };
 
-  const sendAction = (req, res, params, template) => {
+  const sendAction = (req, res, params, _) => {
     const {to = settings.testEmail, cc, bcc} = params.query;
 
     Utils.Logger.info(`Sending ${template.name}…`);
@@ -87,18 +91,18 @@ Routing = (template, settings, render, compile) => {
       }
 
       const options = {
-        to,
+        to: arrayOrString(to),
         data,
         template: template.name,
         subject: '[TEST] ' + template.name
       };
 
       if (cc) {
-        options.cc = cc;
+        options.cc = arrayOrString(cc);
       }
 
       if (bcc) {
-        options.bcc = bcc;
+        options.bcc = arrayOrString(bcc);
       }
 
       const result = Mailer.send(options);