[TASK] Add test for recursive partial

TYPO3 · Oct 16, 2023 · 252fc56 · 252fc56
1 parent 7fa460b
commit 252fc56
Show file tree

Hide file tree

Showing 4 changed files with 142 additions and 73 deletions.
diff --git a/tests/Functional/Cases/Rendering/RecursiveRenderingTest.php b/tests/Functional/Cases/Rendering/RecursiveRenderingTest.php
@@ -0,0 +1,133 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * This file belongs to the package "TYPO3 Fluid".
+ * See LICENSE.txt that was shipped with this package.
+ */
+
+namespace TYPO3Fluid\Fluid\Tests\Functional\Cases\Rendering;
+
+use TYPO3Fluid\Fluid\Tests\Functional\AbstractFunctionalTestCase;
+use TYPO3Fluid\Fluid\View\TemplateView;
+
+final class RecursiveRenderingTest extends AbstractFunctionalTestCase
+{
+    /**
+     * @test
+     */
+    public function recursiveSectionRenderingClonesVariableStorageAndRestoresAfterLoop(): void
+    {
+        $source = file_get_contents(__DIR__ . '/../../Fixtures/Templates/RecursiveSectionRendering.html');
+        $variables = [
+            'settings' => [
+                'test' => '<strong>Bla</strong>'
+            ],
+            'items' => [
+                [
+                    'id' => 1,
+                    'items' => [
+                        [
+                            'id' => 2,
+                            'items' => [
+                                [
+                                    'id' => 3,
+                                    'items' => []
+                                ]
+                            ]
+                        ]
+                    ]
+                ],
+                [
+                    'id' => 4
+                ]
+            ]
+        ];
+        $expectations = [
+            'Item: 1.',
+            'Item: 2.',
+            'Item: 3.',
+            'Item: 4.'
+        ];
+
+        $view = new TemplateView();
+        $view->getRenderingContext()->setCache(self::$cache);
+        $view->assignMultiple($variables);
+        $view->getRenderingContext()->getTemplatePaths()->setTemplateSource($source);
+        $output = $view->render();
+        foreach ($expectations as $expectedValue) {
+            self::assertStringContainsString($expectedValue, $output);
+        }
+
+        // A second run to verify cached template parsing
+        $view = new TemplateView();
+        $view->getRenderingContext()->setCache(self::$cache);
+        $view->assignMultiple($variables);
+        $view->getRenderingContext()->getTemplatePaths()->setTemplateSource($source);
+        $output = $view->render();
+        foreach ($expectations as $expectedValue) {
+            self::assertStringContainsString($expectedValue, $output);
+        }
+    }
+
+    /**
+     * @test
+     */
+    public function recursivePartialRenderingClonesVariableStorageAndRestoresAfterLoop(): void
+    {
+        $partialPath = __DIR__ . '/../../Fixtures/Partials/';
+        $source = file_get_contents(__DIR__ . '/../../Fixtures/Templates/RecursivePartialRendering.html');
+        $variables = [
+            'settings' => [
+                'test' => '<strong>Bla</strong>'
+            ],
+            'items' => [
+                [
+                    'id' => 1,
+                    'items' => [
+                        [
+                            'id' => 2,
+                            'items' => [
+                                [
+                                    'id' => 3,
+                                    'items' => []
+                                ]
+                            ]
+                        ]
+                    ]
+                ],
+                [
+                    'id' => 4
+                ]
+            ]
+        ];
+        $expectations = [
+            'Item: 1.',
+            'Item: 2.',
+            'Item: 3.',
+            'Item: 4.'
+        ];
+
+        $view = new TemplateView();
+        $view->getRenderingContext()->setCache(self::$cache);
+        $view->assignMultiple($variables);
+        $view->getRenderingContext()->getTemplatePaths()->setPartialRootPaths([$partialPath]);
+        $view->getRenderingContext()->getTemplatePaths()->setTemplateSource($source);
+        $output = $view->render();
+        foreach ($expectations as $expectedValue) {
+            self::assertStringContainsString($expectedValue, $output);
+        }
+
+        // A second run to verify cached template parsing
+        $view = new TemplateView();
+        $view->getRenderingContext()->setCache(self::$cache);
+        $view->assignMultiple($variables);
+        $view->getRenderingContext()->getTemplatePaths()->setPartialRootPaths([$partialPath]);
+        $view->getRenderingContext()->getTemplatePaths()->setTemplateSource($source);
+        $output = $view->render();
+        foreach ($expectations as $expectedValue) {
+            self::assertStringContainsString($expectedValue, $output);
+        }
+    }
+}
diff --git a/tests/Functional/Cases/Rendering/RecursiveSectionRenderingTest.php b/tests/Functional/Cases/Rendering/RecursiveSectionRenderingTest.php
diff --git a/tests/Functional/Fixtures/Partials/RecursivePartial.html b/tests/Functional/Fixtures/Partials/RecursivePartial.html
@@ -0,0 +1,8 @@
+<f:spaceless>
+    <f:for each="{items}" as="item">
+        Item: {item.id}.
+        <f:if condition="{item.items}">
+            <f:render partial="RecursivePartial" arguments="{items: item.items}" />
+        </f:if>
+    </f:for>
+</f:spaceless>
diff --git a/tests/Functional/Fixtures/Templates/RecursivePartialRendering.html b/tests/Functional/Fixtures/Templates/RecursivePartialRendering.html
@@ -0,0 +1 @@
+<f:render partial="RecursivePartial" arguments="{items: items}" />
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		<f:render partial="RecursivePartial" arguments="{items: items}" />