tests for loop error conditions

Author: Jeff Escalante

lib/index.js

@@ -140,7 +140,7 @@ function walk (opts, nodes) {
         throw new Error('You must provide an array or object to loop through')
       }
 
-      if (loopParams.length < 1) {
+      if (loopParams.keys.length < 1 || loopParams.keys[0] === '') {
         throw new Error('You must provide at least one loop argument')
       }
 

test/fixtures/loop_expression_error.html

@@ -0,0 +1,5 @@
+<p>x</p>
+<each loop="item in #@$%#$">
+  <p>{{index}}: {{item}}</p>
+</each>
+<p>x</p>

test/fixtures/loop_no_args.html

@@ -0,0 +1,5 @@
+<p>x</p>
+<each loop=" in [1,2,3]">
+  <p>{{index}}: {{item}}</p>
+</each>
+<p>x</p>

test/fixtures/loop_no_attr.html

@@ -0,0 +1,5 @@
+<p>x</p>
+<each>
+  <p>{{index}}: {{item}}</p>
+</each>
+<p>x</p>

test/fixtures/loop_no_collection.html

@@ -0,0 +1,5 @@
+<p>x</p>
+<each loop="item in 5">
+  <p>{{index}}: {{item}}</p>
+</each>
+<p>x</p>

test/index.js

@@ -95,6 +95,30 @@ test('nested loops', (t) => {
   })
 })
 
+test('loop - no loop attribute', (t) => {
+  return expectError('loop_no_attr', (err) => {
+    t.truthy(err.toString() === 'Error: the "elseif" tag must have a "loop" attribute')
+  })
+})
+
+test('loop - no array or object passed', (t) => {
+  return expectError('loop_no_collection', (err) => {
+    t.truthy(err.toString() === 'Error: You must provide an array or object to loop through')
+  })
+})
+
+test('loop - no loop arguments', (t) => {
+  return expectError('loop_no_args', (err) => {
+    t.truthy(err.toString() === 'Error: You must provide at least one loop argument')
+  })
+})
+
+test('loop - expression error', (t) => {
+  return expectError('loop_expression_error', (err) => {
+    t.truthy(err.toString() === 'SyntaxError: Unexpected token ILLEGAL')
+  })
+})
+
 //
 // Utility
 //