Merge pull request jknack#457 from mathijs81/master

jknack · jknack · commit 6bb04c009ceb · 2015-12-25T18:37:03.000-03:00
Fix missing ; in equals sign escape
diff --git a/handlebars/src/test/java/com/github/jknack/handlebars/i386/Issue386.java b/handlebars/src/test/java/com/github/jknack/handlebars/i386/Issue386.java
@@ -11,7 +11,7 @@ public class Issue386 extends AbstractTest {
   @Test
   public void blockHelperShouldNotIntroduceANewContext() throws IOException {
     shouldCompileTo("{{#partial \"body\"}}{{&this}}{{/partial}}{{block \"body\"}}", $("foo", "bar"),
-        "{foo&#x3Dbar}");
+        "{foo&#x3D;bar}");
   }
 
   @Test
diff --git a/handlebars/src/test/java/com/github/jknack/handlebars/i430/Issue430.java b/handlebars/src/test/java/com/github/jknack/handlebars/i430/Issue430.java
@@ -12,6 +12,6 @@
 public class Issue430 extends v4Test {
   @Test
   public void shouldEscapeEqualsSignInHtml() throws IOException {
-    assertEquals("foo&#x3D", Handlebars.Utils.escapeExpression("foo=").toString());
+    assertEquals("foo&#x3D;", Handlebars.Utils.escapeExpression("foo=").toString());
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ public class Issue386 extends AbstractTest {`
`11`	`11`	`@Test`
`12`	`12`	`public void blockHelperShouldNotIntroduceANewContext() throws IOException {`
`13`	`13`	`shouldCompileTo("{{#partial \"body\"}}{{&this}}{{/partial}}{{block \"body\"}}", $("foo", "bar"),`
`14`		`- "{foo&#x3Dbar}");`
	`14`	`+ "{foo=bar}");`
`15`	`15`	`}`
`16`	`16`
`17`	`17`	`@Test`
Original file line number	Diff line number	Diff line change
`@@ -12,6 +12,6 @@`
`12`	`12`	`public class Issue430 extends v4Test {`
`13`	`13`	`@Test`
`14`	`14`	`public void shouldEscapeEqualsSignInHtml() throws IOException {`
`15`		`- assertEquals("foo&#x3D", Handlebars.Utils.escapeExpression("foo=").toString());`
	`15`	`+ assertEquals("foo=", Handlebars.Utils.escapeExpression("foo=").toString());`
`16`	`16`	`}`
`17`	`17`	`}`