Merge pull request #66 from c4spar/fix-trim-with-filter

fix(trim): trim end not working with pipes
ventojs · Jun 9, 2024 · 7e7abe2 · 7e7abe2
2 parents 43e67e9 + d59fd1c
commit 7e7abe2
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/plugins/trim.ts b/plugins/trim.ts
@@ -20,7 +20,7 @@ export function trim(_: Environment, tokens: Token[]) {
       code = code.slice(1);
     }
 
-    if (type === "tag" && code.endsWith("-")) {
+    if ((type === "tag" || type === "filter") && code.endsWith("-")) {
       next[1] = next[1].trimStart();
       code = code.slice(0, -1);
     }

diff --git a/test/print.test.ts b/test/print.test.ts
@@ -96,6 +96,23 @@ Deno.test("Print trim", async () => {
   });
 });
 
+Deno.test("Print trim with filter", async () => {
+  await test({
+    template: `Hello {{- "World" |> toUpperCase }} !`,
+    expected: "HelloWORLD !",
+  });
+
+  await test({
+    template: `Hello {{ "World" |> toUpperCase -}} !`,
+    expected: "Hello WORLD!",
+  });
+
+  await test({
+    template: `Hello {{- "World" |> toUpperCase -}} !`,
+    expected: "HelloWORLD!",
+  });
+});
+
 Deno.test({
   name: "Print async filters",
   // @ts-ignore only used to detect node.js