Remove mustache refs; avoid throwing errors, log instead

Author: mars

index.js

@@ -1,6 +1,6 @@
 // Capture environment as module variable to allow testing.
 var originalEnv = process.env;
-// This Mustache template tag should be rendered/replaced with the environment in production.
+// This template tag should be rendered/replaced with the environment in production.
 var templatedEnv = '{{REACT_APP_VARS_AS_JSON}}';
 
 
@@ -13,11 +13,9 @@ function runtimeEnv() {
     try {
       env = JSON.parse(templatedEnv);
     } catch(error) {
-      throw new Error(
-        'Runtime env vars could not be parsed. '+
-        'Content of `REACT_APP_VARS_AS_JSON` is `'+templatedEnv+'` '+
-        'Ensure `node_modules/@mars/heroku-js-runtime-env/index.js` '+
-        'is rendered as a Mustache template.'
+      console.error(
+        'Runtime env vars cannot be parsed. '+
+        'Content is `'+templatedEnv+'`'
       );
     }
 

test/production-test.js

@@ -9,11 +9,11 @@ describe('in production', function() {
     runtimeEnv.__set__('originalEnv', { NODE_ENV: 'production'})
   })
 
-  it('throws error when Mustache tag is not rendered', function() {
-    runtimeEnv.should.throw(/Runtime env vars could not be parsed/)
+  it('continues when env JSON is not rendered (writes to error log)', function() {
+    runtimeEnv.should.not.throw()
   })
 
-  describe('with Mustache template replacements', function() {
+  describe('with env JSON', function() {
     beforeEach(function() {
       runtimeEnv.__set__('templatedEnv', '{ "REACT_APP_USER": "🦄"}')
     })