patch to fix empty node handling within expressions

Author: jescalan

lib/index.js

@@ -24,7 +24,8 @@ module.exports = function reshapeEvalCode (_locals = {}) {
 }
 
 function evalNode (node, locals, opts) {
-  const newNode = parse(gen.call({ runtime: opts.runtime }, node, opts)(locals))
-  if (node.location) { newNode.location = node.location }
+  let newNode = parse(gen.call({ runtime: opts.runtime }, node, opts)(locals))
+  if (newNode.length === 0) { newNode = [{ type: 'text', content: '' }] }
+  if (node.location) { newNode[0].location = node.location }
   return newNode
 }

package.json

@@ -13,13 +13,13 @@
     "reshape-plugin-util": "^0.2.1"
   },
   "devDependencies": {
-    "ava": "^0.20.0",
-    "codecov": "^2.2.0",
-    "nyc": "^11.0.3",
+    "ava": "^0.22.0",
+    "codecov": "^2.3.0",
+    "nyc": "^11.1.0",
     "reshape": "^0.4.1",
     "reshape-expressions": "^0.1.5",
     "snazzy": "^7.0.0",
-    "standard": "^10.0.2"
+    "standard": "^10.0.3"
   },
   "engines": {
     "node": ">= 6"

test/index.js

@@ -26,3 +26,9 @@ test('merges from reshape options locals', (t) => {
     .process('<p>{{ foo }}</p>')
     .then((res) => { t.is(res.output(), '<p>bar</p>') })
 })
+
+test('correctly handles empty nodes', (t) => {
+  return reshape({ plugins: [expressions(), evalCode()], locals: { foo: 'bar' } })
+    .process('<p class="{{ true ? \'\' : \'\' }}"></p>')
+    .then((res) => { t.is(res.output(), '<p class></p>') })
+})