fix: directory info in lookupError message, #395

demo/nodejs/index.js

@@ -1,9 +1,14 @@
 const { Liquid } = require('liquidjs')
 
 const engine = new Liquid({
-  root: __dirname,
   extname: '.liquid',
-  globals: { title: 'LiquidJS Demo' }
+  globals: { title: 'LiquidJS Demo' },
+  // root files for `.render()` and `.parse()`
+  root: __dirname,
+  // layout files for `{% layout %}`
+  layouts: './layouts',
+  // partial files for `{% include %}` and `{% render %}`
+  partials: './partials'
 })
 
 engine.registerTag('header', {

demo/nodejs/package.json

@@ -7,6 +7,6 @@
     "start": "node index.js"
   },
   "dependencies": {
-    "liquidjs": "latest"
+    "liquidjs": "^9.27.0"
   }
 }

demo/nodejs/partials/todo.liquid

@@ -0,0 +1 @@
+<li>{{index}} - {{todo}}</li>

demo/nodejs/todolist.liquid

@@ -2,6 +2,6 @@
 {%header content: title | capitalize%}
 <ul>
   {% for todo in todos %}
-  <li>{{forloop.index}} - {{todo}}</li>
+    {% render 'todo.liquid' with todo, index: forloop.index%}
   {% endfor %}
 </ul>

src/fs/loader.ts

@@ -20,16 +20,16 @@ export class Loader {
   }
 
   public * lookup (file: string, type: LookupType, sync?: boolean) {
-    const { fs, root } = this.options
+    const { fs } = this.options
-    for (const filepath of this.candidates(file, type)) {
+    const dirs = this.options[type]
+    for (const filepath of this.candidates(file, dirs)) {
       if (sync ? fs.existsSync(filepath) : yield fs.exists(filepath)) return filepath
     }
-    throw this.lookupError(file, root)
+    throw this.lookupError(file, dirs)
   }
 
-  private * candidates (file: string, type: LookupType) {
+  private * candidates (file: string, dirs: string[]) {
     const { fs, extname } = this.options
-    const dirs = this.options[type]
     for (const dir of dirs) {
       yield fs.resolve(dir, file, extname)
     }