Fix to make it work in unit-test mode

lookback · Oct 27, 2016 · 7aa35bf · 7aa35bf
1 parent 3371f90
commit 7aa35bf
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/lib/mailer.js b/lib/mailer.js
@@ -187,7 +187,7 @@ const factory = (options) => {
       // `preview` prop in the data context to apply to the layout.
       if (tmpl.__helpers.has('preview')) {
         preview = tmpl.__helpers.get('preview');
-      } else if (data.preview) {
+      } else if (data && data.preview) {  // data is optional
         preview = data.preview;
       }
 

diff --git a/lib/utils.js b/lib/utils.js
@@ -154,7 +154,21 @@ Utils = {
   },
 
   readFile(relativePathFromApp) {
-    const file = Path.join(ROOT, relativePathFromApp);
+    let file
+    if (Meteor.isAppTest
+      || Meteor.isTest) {
+      // note:
+      //  * this DOES WORK with Meteor.isTest (=unit test mode)
+      //  * Meteor.isAppTest is NOT TESTED yet (=in-app test mode)
+      //
+      // background-info: we had NO luck with "Assets.absoluteFilePath(relativePathFromApp)",
+      //  so lets build the path ourselves.
+      //  see discussion https://github.com/lookback/meteor-emails/issues/76
+      file = Path.join(process.cwd(), 'assets', 'app', relativePathFromApp)
+    } else {
+      // = standard mode (development || production)
+      file = Path.join(ROOT, relativePathFromApp);
+    }
 
     try {
       return fs.readFileSync(file, {