tests

Author: dansabin

lib/render.js

@@ -1,24 +1,29 @@
+'use strict;'
+
 const Inky = require('inky').Inky;
 const Juice = require('juice');
+const Hoek = require('hoek');
 
 module.exports = (htmlFile, cssFile, options = { inky: { enabled: true }, juice: { linkCss: false } }) => {
-    let convertedHtml;
-    if (options.inky.enabled) {
-        const inky = new Inky(options.inky);
-        convertedHtml = inky.releaseTheKraken(htmlFile);
-    }
+    return new Promise((resolve, reject) => {
+        Hoek.assert(htmlFile || htmlFile === '', new Error('HTML file is required!'));
 
-    if (options.juice.linkCss) {
-        return new Promise((resolve, reject) => {
+        let convertedHtml;
+        if (options.inky.enabled) {
+            const inky = new Inky(options.inky);
+            convertedHtml = inky.releaseTheKraken(htmlFile);
+        }
+    
+        if (options.juice.linkCss) {
             Juice.juiceResources(options.inky.enabled ? convertedHtml : htmlFile, options.juice, (err, htmlResult) => {
                 if (err) {
                     return reject(err);
                 }
 
                 return resolve(htmlResult);
             });
-        });
-    } else {
-        return Promise.resolve(Juice.inlineContent(options.inky.enabled ? convertedHtml : htmlFile, cssFile, options.juice))
-    }
+        } else {
+            return resolve(Juice.inlineContent(options.inky.enabled ? convertedHtml : htmlFile, cssFile || '', options.juice))
+        }
+    })
 }

package.json

@@ -4,20 +4,24 @@
   "description": "CLI for Merging HTML and CSS to inline for emails. Supports Inky Templates.",
   "main": "bin/driver",
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1"
+    "test": "node_modules/lab/bin/lab"
   },
   "bin": {
     "inline-email": "bin/driver"
   },
   "author": "Dan Sabin <dan@freethinking.it>",
   "license": "MIT",
   "dependencies": {
-    "command-line-args": "^5.0.0",
-    "command-line-usage": "^4.1.0",
-    "inky": "^1.3.7",
-    "juice": "^4.2.2"
+    "command-line-args": "5.0.0",
+    "command-line-usage": "4.1.0",
+    "hoek": "^5.0.2",
+    "inky": "1.3.7",
+    "juice": "4.2.2"
+  },
+  "devDependencies": {
+    "code": "3.0.2",
+    "lab": "14.3.2"
   },
-  "devDependencies": {},
   "repository": {
     "type": "git",
     "url": "git+https://github.com/freethinkingit/inline-email.git"
@@ -30,5 +34,8 @@
   "bugs": {
     "url": "https://github.com/freethinkingit/inline-email/issues"
   },
-  "homepage": "https://github.com/freethinkingit/inline-email#readme"
+  "homepage": "https://github.com/freethinkingit/inline-email#readme",
+  "env": {
+    "node": "6.12.3"
+  }
 }

test/lib/render.js

@@ -0,0 +1,38 @@
+'use strict';
+
+const Code = require('code');
+const Lab = require('lab');
+const Render = require('../../lib/render')
+
+const expect = Code.expect;
+const lab = exports.lab = Lab.script();
+
+lab.experiment('render', () => {
+    lab.test('with no args throws error requiring HTML files', () => {
+        return Render()
+            .catch((error) => {
+                expect(error.message).to.equal('HTML file is required!');
+            });
+    });
+
+    lab.test('with blank HTML files returns nothing', () => {
+        return Render('')
+        .then((htmlResult) => {
+            expect(htmlResult).to.equal('');
+        })
+    });
+
+    lab.test('with no args throws error requiring HTML files', () => {
+        return Render()
+            .catch((error) => {
+                expect(error.message).to.equal('HTML file is required!');
+            });
+    });
+
+    lab.test('convert inky tags to html', () => {
+        return Render('<row>thing</row>', null, {inky: {enabled: true}, juice: {}})
+        .then((htmlResult) => {
+            expect(htmlResult).to.equal('<table class=\"row\"><tbody><tr>thing</tr></tbody></table>');
+        })
+    });
+});