Coerce noEscape-statements to a string

Fixes #1838

Author: jaylinski

lib/handlebars/compiler/javascript-compiler.js

@@ -426,7 +426,7 @@ JavaScriptCompiler.prototype = {
     if (this.isInline()) {
       this.replaceStack(current => [' != null ? ', current, ' : ""']);
 
-      this.pushSource(this.appendToBuffer(this.popStack()));
+      this.pushSource(this.appendToBuffer([this.popStack(), '+', "''"]));
     } else {
       let local = this.popStack();
       this.pushSource([

spec/javascript-compiler.js

@@ -34,6 +34,7 @@ describe('javascript-compiler api', function() {
         .toCompileTo('food');
     });
   });
+
   describe('#compilerInfo', function() {
     var $superCheck, $superInfo;
     beforeEach(function() {
@@ -58,6 +59,7 @@ describe('javascript-compiler api', function() {
         .toCompileTo('food ');
     });
   });
+
   describe('buffer', function() {
     var $superAppend, $superCreate;
     beforeEach(function() {
@@ -116,4 +118,13 @@ describe('javascript-compiler api', function() {
       });
     });
   });
+
+  describe('options', function() {
+    it('should append `noEscape` statements as string', function() {
+      expectTemplate('{{a}}{{b}}')
+        .withCompileOptions({ noEscape: true })
+        .withInput({ a: 1, b: 1 })
+        .toCompileTo('11');
+    });
+  });
 });