fix: return variable name in include error

harttle · Mar 23, 2020 · 93433a8 · 93433a8
1 parent 6bf242b
commit 93433a8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/builtin/tags/include.ts b/src/builtin/tags/include.ts
@@ -29,7 +29,7 @@ export default {
         ? yield renderer.renderTemplates(liquid.parse(evalQuotedToken(file)), ctx)
         : yield evalToken(file, ctx))
       : file.getText()
-    assert(filepath, () => `illegal filename "${file}":"${filepath}"`)
+    assert(filepath, () => `illegal filename "${file.getText()}":"${filepath}"`)
 
     const saved = ctx.saveRegister('blocks', 'blockMode')
     ctx.setRegister('blocks', {})

diff --git a/test/integration/builtin/tags/include.ts b/test/integration/builtin/tags/include.ts
@@ -44,7 +44,7 @@ describe('tags/include', function () {
     })
     return liquid.renderFile('/parent.html').catch(function (e) {
       expect(e.name).to.equal('RenderError')
-      expect(e.message).to.match(/illegal filename/)
+      expect(e.message).to.match(/illegal filename "not-exist"/)
     })
   })