Merge pull request #459 from EB-Forks/test/disable-legacy-include-dir…

…ective test: Assert that disabling legacy includes works correctly
mde · Sep 10, 2019 · 3dc11e9 · 3dc11e9
2 parents b364cc3 + 5f6e6ef
commit 3dc11e9
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/test/ejs.js b/test/ejs.js
@@ -1167,6 +1167,16 @@ suite('preprocessor include', function () {
     throw new Error('expected inclusion error');
   });
 
+  test('legacy includes are a syntax error when disabled', function() {
+    try {
+      ejs.render('<%- include is-syntax-error %>', null, {legacyInclude: false});
+    }
+    catch (err) {
+      assert.ok(err.message.indexOf('missing ) after argument list') > -1);
+      return;
+    }
+    throw new Error('expected SyntaxError from legacy include being disabled');
+  });
 });
 
 suite('comments', function () {