Skip to content

Commit daf2c84

Browse files
committed
Added close html filter implementation with Jsoup
1 parent 16fb43f commit daf2c84

2 files changed

Lines changed: 27 additions & 5 deletions

File tree

src/main/java/com/hubspot/jinjava/lib/filter/CloseHtmlFilter.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@
44
import com.hubspot.jinjava.doc.annotations.JinjavaParam;
55
import com.hubspot.jinjava.doc.annotations.JinjavaSnippet;
66
import com.hubspot.jinjava.interpret.JinjavaInterpreter;
7+
import java.util.Objects;
8+
import org.jsoup.Jsoup;
79

810
@JinjavaDoc(
911
value = "Closes open HTML tags in a string",
@@ -19,10 +21,6 @@ public String getName() {
1921

2022
@Override
2123
public Object filter(Object var, JinjavaInterpreter interpreter, String... args) {
22-
/*
23-
Closes open HTML tags.
24-
*/
25-
26-
return null;
24+
return Jsoup.parse(Objects.toString(var)).toString();
2725
}
2826
}
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
package com.hubspot.jinjava.lib.filter;
2+
3+
import static org.assertj.core.api.Assertions.assertThat;
4+
5+
import com.google.common.collect.ImmutableMap;
6+
import com.hubspot.jinjava.BaseInterpretingTest;
7+
import com.hubspot.jinjava.BaseJinjavaTest;
8+
import org.junit.Before;
9+
import org.junit.Test;
10+
11+
public class CloseHtmlFilterTest extends BaseInterpretingTest {
12+
CloseHtmlFilter f;
13+
14+
@Before
15+
public void setup() {
16+
f = new CloseHtmlFilter();
17+
}
18+
19+
@Test
20+
public void itClosesTags() {
21+
String openTags = "<p>Hello, world!";
22+
assertThat(f.filter(openTags, interpreter)).isEqualTo("<p>Hello, world!</p>");
23+
}
24+
}

0 commit comments

Comments
 (0)