Committing test case

Author: jasmith-hs

src/main/java/com/hubspot/jinjava/lib/fn/eager/EagerMacroFunction.java

@@ -71,6 +71,15 @@ public Object doEvaluate(
           interpreter
         );
         if (!result.getResult().isFullyResolved()) {
+          if (
+            result
+              .getSpeculativeBindings()
+              .keySet()
+              .stream()
+              .anyMatch(key -> localContextScope.getScope().containsKey(key))
+          ) {
+            throw new DeferredValueException("e");
+          }
           result =
             eagerEvaluateInDeferredExecutionMode(
               () -> getEvaluationResultDirectly(argMap, kwargMap, varArgs, interpreter),
@@ -294,7 +303,7 @@ private String getSetTagForAliasedVariables(String fullName) {
         .getCombinedScope()
         .entrySet()
         .stream()
-        .filter(entry -> entry.getValue() instanceof DeferredValue)
+        //        .filter(entry -> entry.getValue() instanceof DeferredValue)
         .map(Entry::getKey)
         .collect(Collectors.toMap(Function.identity(), name -> aliasName + name));
       return EagerReconstructionUtils.buildSetTag(

src/test/java/com/hubspot/jinjava/EagerTest.java

@@ -1499,4 +1499,11 @@ public void itAllowsVariableSharingAliasName() {
       "allows-variable-sharing-alias-name"
     );
   }
+
+  @Test
+  public void itAllowsModificationInAliasedMacro() {
+    expectedTemplateInterpreter.assertExpectedOutput(
+      "allows-modification-in-aliased-macro"
+    );
+  }
 }

src/test/resources/eager/allows-modification-in-aliased-macro.expected.jinja

@@ -0,0 +1,6 @@
+{% import 'eager/settings.jinja' as shared %}
+
+{% if deferred %}
+{{ shared.load_settings() }}
+{% endif %}
+{{ shared.settings }}

src/test/resources/eager/allows-modification-in-aliased-macro.jinja

@@ -0,0 +1,6 @@
+{% import 'eager/settings.jinja' as shared %}
+
+{% if deferred %}
+{{ shared.load_settings() }}
+{% endif %}
+{{ shared.settings }}

src/test/resources/tags/macrotag/eager/settings.jinja

@@ -0,0 +1,5 @@
+{% set settings = {} %}
+
+{% macro load_settings() %}
+{% do settings.put('foo', 'bar') %}
+{% endmacro %}