add test for trailing text

mainmatter · Oct 6, 2017 · 44fba8f · 44fba8f
1 parent d49229f
commit 44fba8f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/tests/integration/components/dummy-component-test.js b/tests/integration/components/dummy-component-test.js
@@ -51,16 +51,18 @@ describe('HBS Minifier plugin', function() {
   });
 
   it('Collapse leading/trailing text from ElementNode nodes', function() {
-    this.render(hbs`<div>x        {{foo}}     y   </div>`);
+    this.render(hbs`<div>x        {{foo}}     y   {{bar}}    z</div>`);
 
     let childNodes = this.$('div')[0].childNodes;
-    expect(childNodes.length).to.equal(3);
+    expect(childNodes.length).to.equal(5);
     expect(childNodes[0]).to.be.a('text');
     expect(childNodes[0]).to.have.a.property('textContent', 'x ');
     expect(childNodes[1]).to.be.a('text');
     expect(childNodes[1]).to.have.a.property('textContent', 'foo');
     expect(childNodes[2]).to.be.a('text');
     expect(childNodes[2]).to.have.a.property('textContent', ' y ');
+    expect(childNodes[3]).to.have.a.property('textContent', 'bar');
+    expect(childNodes[4]).to.have.a.property('textContent', ' z');
   });
 
   it('does not strip inside of <pre> tags', function() {

diff --git a/utils/helpers.js b/utils/helpers.js
@@ -12,7 +12,7 @@ const isWhitespaceTextNode = function(node) {
 
 const hasLeadingOrTrailingWhiteSpace = function(chars) {
   chars = encodeURIComponent(chars);
-  return matchHorizontalTABAndNewLineEnd.test(chars) || matchHorizontalTABAndNewLineEnd.test(chars);
+  return matchHorizontalTABAndNewLineBegin.test(chars) || matchHorizontalTABAndNewLineEnd.test(chars);
 };
 
 const getElementAttribute = function(node, attrName) {