Completing test

Author: weaverryan

src/Turbo/src/Bridge/Mercure/Broadcaster.php

@@ -11,9 +11,11 @@
 
 namespace Symfony\UX\Turbo\Bridge\Mercure;
 
+use Doctrine\Common\Util\ClassUtils;
 use Symfony\Component\ExpressionLanguage\ExpressionLanguage;
 use Symfony\Component\Mercure\HubInterface;
 use Symfony\Component\Mercure\Update;
+use Symfony\Component\VarExporter\LazyObjectInterface;
 use Symfony\UX\Turbo\Broadcaster\BroadcasterInterface;
 
 /**
@@ -61,7 +63,11 @@ public function broadcast(object $entity, string $action, array $options): void
             return;
         }
 
-        $entityClass = $entity::class;
+        if ($entity instanceof LazyObjectInterface) {
+            $entityClass = get_parent_class($entity);
+        } else {
+            $entityClass = ClassUtils::getClass($entity);
+        }
 
         if (!isset($options['rendered_action'])) {
             throw new \InvalidArgumentException(sprintf('Cannot broadcast entity of class "%s" as option "rendered_action" is missing.', $entityClass));

src/Turbo/src/Broadcaster/TwigBroadcaster.php

@@ -11,6 +11,8 @@
 
 namespace Symfony\UX\Turbo\Broadcaster;
 
+use Doctrine\Common\Util\ClassUtils;
+use Symfony\Component\VarExporter\LazyObjectInterface;
 use Twig\Environment;
 
 /**
@@ -42,8 +44,15 @@ public function broadcast(object $entity, string $action, array $options): void
             $options['id'] = $id;
         }
 
+        // handle proxies (both styles)
+        if ($entity instanceof LazyObjectInterface) {
+            $class = get_parent_class($entity);
+        } else {
+            $class = ClassUtils::getClass($entity);
+        }
+
         if (null === $template = $options['template'] ?? null) {
-            $template = $entity::class;
+            $template = $class;
             foreach ($this->templatePrefixes as $namespace => $prefix) {
                 if (str_starts_with($template, $namespace)) {
                     $template = substr_replace($template, $prefix, 0, \strlen($namespace));

src/Turbo/tests/BroadcastTest.php

@@ -33,7 +33,7 @@ protected function setUp(): void
         parent::setUp();
     }
 
-    public function testBroadcast(): void
+    public function testBroadcastBasic(): void
     {
         ($client = self::createPantherClient())->request('GET', '/books');
 
@@ -94,7 +94,12 @@ public function testBroadcastWithProxy(): void
         // testing that Artist is updated, even though it's saved as Proxy
         ($client = self::createPantherClient())->request('GET', '/artistFromSong');
 
-        $crawler = $client->submitForm('Submit');
+        // submit first time to create the artist
+        $client->submitForm('Submit');
+        $this->assertSelectorWillContain('#artists', 'testing artist');
+
+        // submit again to update the artist
+        $client->submitForm('Submit');
         $this->assertSelectorWillContain('#artists', 'testing artist after change');
         // this part is from the stream template
         $this->assertSelectorWillContain('#artists', ', updated)');

src/Turbo/tests/app/Kernel.php

@@ -33,7 +33,6 @@
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 use Symfony\Component\HttpKernel\Kernel as BaseKernel;
-use Symfony\Component\Mercure\Hub;
 use Symfony\Component\Mercure\HubInterface;
 use Symfony\Component\Mercure\Update;
 use Symfony\Component\Routing\Loader\Configurator\RoutingConfigurator;
@@ -133,7 +132,7 @@ protected function configureRoutes(RoutingConfigurator $routes): void
         $routes->add('songs', '/songs')->controller('kernel::songs');
         $routes->add('artists', '/artists')->controller('kernel::artists');
         $routes->add('artist', '/artists/{id}')->controller('kernel::artist');
-        $routes->add('artist', '/artistFromSong')->controller('kernel::artistFromSong');
+        $routes->add('artist_from_song', '/artistFromSong')->controller('kernel::artistFromSong');
     }
 
     public function getProjectDir(): string
@@ -267,24 +266,29 @@ public function artist(Request $request, EntityManagerInterface $doctrine, Envir
 
     public function artistFromSong(Request $request, EntityManagerInterface $doctrine, Environment $twig): Response
     {
-        if (!$request->isMethod('POST')) {
-            $artist = new Artist();
-            $artist->name = 'testing artist';
+        $song = null;
+        if ($request->isMethod('POST')) {
+            // on first post, create the objects
+            // on second, update the artist
+            $id = $request->get('id');
+            if (!$id) {
+                $artist = new Artist();
+                $artist->name = 'testing artist';
 
-            $song = new Song();
-            $song->artist = $artist;
-            $song->title = 'testing song title';
+                $song = new Song();
+                $song->artist = $artist;
+                $song->title = 'testing song title';
 
-            $doctrine->persist($artist);
-            $doctrine->persist($song);
-            $doctrine->flush();
-        } else {
-            $id = $request->get('id');
-            $song = $doctrine->find(Song::class, $id);
-            $artist = $song->artist;
-            $artist->name = $artist->name.' after change';
+                $doctrine->persist($artist);
+                $doctrine->persist($song);
+                $doctrine->flush();
+            } else {
+                $song = $doctrine->find(Song::class, $id);
+                $artist = $song->artist;
+                $artist->name = $artist->name.' after change';
 
-            $doctrine->flush();
+                $doctrine->flush();
+            }
         }
 
         return new Response($twig->render('artist_from_song.html.twig', [

src/Turbo/tests/app/templates/artist_from_song.html.twig

@@ -5,8 +5,10 @@
 
     <div id="artists" {{ turbo_stream_listen('App\\Entity\\Artist') }}></div>
 
-    <form method="post" enctype="application/x-www-form-urlencoded">
-        <input name="id" value="{{ song.id }}">
-        <button>Submit</button>
-    </form>
+    <turbo-frame id="api">
+        <form method="post" enctype="application/x-www-form-urlencoded">
+            <input name="id" value="{{ song ? song.id }}">
+            <button>Submit</button>
+        </form>
+    </turbo-frame>
 {% endblock %}