Skip to content

Commit 09cbfba

Browse files
committed
Test that errors aren't added when parse errors are ignored
1 parent 72d938f commit 09cbfba

1 file changed

Lines changed: 15 additions & 0 deletions

File tree

src/test/java/com/hubspot/jinjava/tree/TreeParserTest.java

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
import com.hubspot.jinjava.Jinjava;
88
import com.hubspot.jinjava.JinjavaConfig;
99
import com.hubspot.jinjava.LegacyOverrides;
10+
import com.hubspot.jinjava.interpret.Context.TemporaryValueClosable;
1011
import com.hubspot.jinjava.interpret.TemplateError.ErrorType;
1112
import java.nio.charset.StandardCharsets;
1213
import org.junit.Test;
@@ -361,6 +362,20 @@ public void itDoesNotMergeAdjacentTextNodesWhenLegacyOverrideIsApplied() {
361362
assertThat(interpreter.render(overriddenTree)).isEqualTo("A\nB");
362363
}
363364

365+
@Test
366+
public void itDoesNotAddErrorWhenParseErrorsAreIgnored() {
367+
try (
368+
TemporaryValueClosable<Boolean> c = interpreter.getContext().withIgnoreParseErrors()
369+
) {
370+
String expression = "{% if ";
371+
final Node tree = new TreeParser(interpreter, expression).buildTree();
372+
assertThat(tree.getChildren()).hasSize(1);
373+
assertThat(tree.getChildren().get(0).toTreeString())
374+
.isEqualToIgnoringWhitespace(" {~ {% if ~}");
375+
}
376+
assertThat(interpreter.getErrors()).isEmpty();
377+
}
378+
364379
Node parse(String fixture) {
365380
try {
366381
return new TreeParser(

0 commit comments

Comments
 (0)