Skip to content

Commit d57126e

Browse files
mnobileclaude
andcommitted
Add test for macro in standard ternary (? :) syntax
Addresses review feedback to ensure the `?`/`:` ternary also works correctly with macro calls and variable conditions. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 92a0739 commit d57126e

1 file changed

Lines changed: 13 additions & 0 deletions

File tree

src/test/java/com/hubspot/jinjava/lib/tag/MacroTagTest.java

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -386,6 +386,19 @@ public void itCallsMacroInTernaryWithVariableCondition() {
386386
assertThat(jinjava.render(template, context).trim()).isEqualTo("Hello nobody");
387387
}
388388

389+
@Test
390+
public void itCallsMacroInStandardTernaryWithVariableCondition() {
391+
String template =
392+
"{% macro greet(name) %}Hello {{ name }}{% endmacro %}" +
393+
"{{ myVar ? greet('world') : greet('nobody') }}";
394+
395+
context.put("myVar", true);
396+
assertThat(jinjava.render(template, context).trim()).isEqualTo("Hello world");
397+
398+
context.put("myVar", false);
399+
assertThat(jinjava.render(template, context).trim()).isEqualTo("Hello nobody");
400+
}
401+
389402
private Node snippet(String jinja) {
390403
return new TreeParser(interpreter, jinja).buildTree().getChildren().getFirst();
391404
}

0 commit comments

Comments
 (0)