XRENDERING-689: Default macro content parser doesn't preserve restricted contexts

* Pass the restricted attribute to the nested transformation context.

Author: michitux

xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/DefaultMacroContentParser.java

@@ -129,7 +129,9 @@ private XDOM createXDOM(String content, MacroTransformationContext macroContext,
 
             // Execute the content
             if (transform && macroContext.getTransformation() != null) {
-                TransformationContext txContext = new TransformationContext(result, syntax);
+                TransformationContext wrappingContext = macroContext.getTransformationContext();
+                boolean isRestricted = wrappingContext != null && wrappingContext.isRestricted();
+                TransformationContext txContext = new TransformationContext(result, syntax, isRestricted);
                 txContext.setId(macroContext.getId());
                 performTransformation((MutableRenderingContext) this.renderingContext, macroContext.getTransformation(),
                     txContext, result);

xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/test/java/org/xwiki/rendering/internal/macro/DefaultMacroContentParserTest.java

@@ -39,13 +39,15 @@
 import org.xwiki.rendering.transformation.MacroTransformationContext;
 import org.xwiki.rendering.transformation.RenderingContext;
 import org.xwiki.rendering.transformation.Transformation;
+import org.xwiki.rendering.transformation.TransformationContext;
 import org.xwiki.test.annotation.ComponentList;
 import org.xwiki.test.junit5.mockito.ComponentTest;
 import org.xwiki.test.junit5.mockito.InjectMockComponents;
 import org.xwiki.test.junit5.mockito.MockComponent;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.argThat;
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.verify;
@@ -108,7 +110,26 @@ void parseInlineWithStandaloneMacroWithTransformations() throws Exception
 
         this.macroContentParser.parse("content", this.macroContext, true, true);
 
-        verify((MutableRenderingContext) this.renderingContext).transformInContext(any(), any(),
+        verify((MutableRenderingContext) this.renderingContext).transformInContext(any(),
+            argThat(context -> !context.isRestricted()),
             eq(new XDOM(Arrays.<Block>asList(new MacroBlock("macro", Collections.emptyMap(), null, true)))));
     }
+
+    @Test
+    void parseInlineWithStandaloneMacroWithRestrictedTransformations() throws Exception
+    {
+        when(this.mockParser.parse(any(Reader.class)))
+            .thenReturn(
+                new XDOM(Collections.singletonList(new MacroBlock("macro", Collections.emptyMap(), null, false))));
+
+        this.macroContext.setTransformation(mock(Transformation.class));
+        this.macroContext.getTransformationContext().setRestricted(true);
+
+        this.macroContentParser.parse("content", this.macroContext, true, true);
+
+        verify((MutableRenderingContext) this.renderingContext).transformInContext(
+            any(), argThat(TransformationContext::isRestricted),
+            eq(new XDOM(Collections.singletonList(new MacroBlock("macro", Collections.emptyMap(), null, true))))
+        );
+    }
 }