Update and extends functional tests

Author: mbuliard

src/LiveComponent/tests/Fixtures/Component/ComponentWithUrlBoundProps.php

@@ -68,6 +68,12 @@ public function modifyMaybeBoundProp(LiveProp $prop): LiveProp
     #[LiveProp]
     public ?string $customAlias = null;
 
+    #[LiveProp(writable: true, url: new UrlMapping(mapPath: true))]
+    public ?string $pathProp = null;
+
+    #[LiveProp(writable: true, url: new UrlMapping(as: 'pathAlias', mapPath: true))]
+    public ?string $pathPropWithAlias = null;
+
     public function modifyBoundPropWithCustomAlias(LiveProp $liveProp): LiveProp
     {
         if ($this->customAlias) {

src/LiveComponent/tests/Fixtures/Kernel.php

@@ -216,5 +216,7 @@ protected function configureRoutes(RoutingConfigurator $routes): void
         $routes->add('homepage', '/')->controller('kernel::index');
         $routes->add('alternate_live_route', '/alt/{_live_component}/{_live_action}')->defaults(['_live_action' => 'get']);
         $routes->add('localized_route', '/locale/{_locale}/{_live_component}/{_live_action}')->defaults(['_live_action' => 'get']);
+        $routes->add('route_with_prop', '/route_with_prop/{pathProp}');
+        $routes->add('route_with_alias_prop', '/route_with_alias_prop/{pathAlias}');
     }
 }

src/LiveComponent/tests/Fixtures/templates/components/component_with_url_bound_props.html.twig

@@ -9,4 +9,6 @@
     MaybeBoundProp: {{ maybeBoundProp }}
     BoundPropWithAlias: {{ boundPropWithAlias }}
     BoundPropWithCustomAlias: {{ boundPropWithCustomAlias }}
+    PathProp: {{ pathProp }}
+    PathPropWithAlias: {{ pathPropWithAlias }}
 </div>

src/LiveComponent/tests/Functional/EventListener/AddLiveAttributesSubscriberTest.php

@@ -139,6 +139,8 @@ public function testQueryStringMappingAttribute()
             'maybeBoundProp' => ['name' => 'maybeBoundProp'],
             'boundPropWithAlias' => ['name' => 'q'],
             'boundPropWithCustomAlias' => ['name' => 'customAlias'],
+            'pathProp' => ['name' => 'pathProp'],
+            'pathPropWithAlias' => ['name' => 'pathAlias'],
         ];
 
         $this->assertEquals($expected, $queryMapping);

src/LiveComponent/tests/Functional/EventListener/LiveUrlSubscriberTest.php

@@ -0,0 +1,104 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\UX\LiveComponent\Tests\Functional\EventListener;
+
+use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
+use Symfony\UX\LiveComponent\Tests\LiveComponentTestHelper;
+use Zenstruck\Browser\Test\HasBrowser;
+
+class LiveUrlSubscriberTest extends KernelTestCase
+{
+    use HasBrowser;
+    use LiveComponentTestHelper;
+
+    public function getTestData(): iterable
+    {
+        yield 'missing_header' => [
+            'previousLocation' => null,
+            'expectedLocation' => null,
+            'props' => [],
+        ];
+        yield 'unknown_previous_location' => [
+            'previousLocation' => 'foo/bar',
+            'expectedLocation' => null,
+            'props' => [],
+        ];
+
+        yield 'no_prop' => [
+            'previousLocation' => '/route_with_prop/foo',
+            'expectedLocation' => null,
+            'props' => [],
+        ];
+
+        yield 'no_change' => [
+            'previousLocation' => '/route_with_prop/foo',
+            'expectedLocation' => '/route_with_prop/foo',
+            'props' => [
+                'pathProp' => 'foo',
+            ],
+        ];
+
+        yield 'prop_changed' => [
+            'previousLocation' => '/route_with_prop/foo',
+            'expectedLocation' => '/route_with_prop/bar',
+            'props' => [
+                'pathProp' => 'foo',
+            ],
+            'updated' => [
+                'pathProp' => 'bar',
+            ],
+        ];
+
+        yield 'alias_prop_changed' => [
+            'previousLocation' => '/route_with_alias_prop/foo',
+            'expectedLocation' => '/route_with_alias_prop/bar',
+            'props' => [
+                'pathPropWithAlias' => 'foo',
+            ],
+            'updated' => [
+                'pathPropWithAlias' => 'bar',
+            ],
+        ];
+    }
+
+    /**
+     * @dataProvider getTestData
+     */
+    public function testNoHeader(
+        ?string $previousLocation,
+        ?string $expectedLocation,
+        array $props,
+        array $updated = [],
+    ): void {
+        $component = $this->mountComponent('component_with_url_bound_props', $props);
+        $dehydrated = $this->dehydrateComponent($component);
+
+        $this->browser()
+            ->throwExceptions()
+            ->post(
+                '/_components/component_with_url_bound_props',
+                [
+                    'body' => [
+                        'data' => json_encode([
+                            'props' => $dehydrated->getProps(),
+                            'updated' => $updated,
+                        ]),
+                    ],
+                    'headers' => [
+                        'X-Live-Url' => $previousLocation,
+                    ],
+                ]
+            )
+            ->assertSuccessful()
+            ->assertHeaderEquals('X-Live-Url', $expectedLocation);
+    }
+}