From 327fd231cf92791d8e143d22d5cba5835075c81d Mon Sep 17 00:00:00 2001
From: Claudio Ortolina <cloud8421@gmail.com>
Date: Thu, 8 May 2025 09:45:13 +0100
Subject: [PATCH] Test that side panel renders tracks

---
 .../live/collection_live/show.html.heex       |  2 +-
 .../live/collection_live/show_test.exs        | 41 +++++++++++++++++++
 2 files changed, 42 insertions(+), 1 deletion(-)

diff --git a/lib/music_library_web/live/collection_live/show.html.heex b/lib/music_library_web/live/collection_live/show.html.heex
index 118a0cb2..91083a78 100644
--- a/lib/music_library_web/live/collection_live/show.html.heex
+++ b/lib/music_library_web/live/collection_live/show.html.heex
@@ -324,7 +324,7 @@
         >
           {medium_title(medium)}
         </h4>
-        <ul class="w-full table table-auto">
+        <ul id={"disc-#{medium.number}"} class="w-full table table-auto">
           <li
             :for={track <- medium.tracks}
             class="contents leading-5 text-zinc-700 dark:text-zinc-300 list-none"
diff --git a/test/music_library_web/live/collection_live/show_test.exs b/test/music_library_web/live/collection_live/show_test.exs
index 1799715b..9bb243dc 100644
--- a/test/music_library_web/live/collection_live/show_test.exs
+++ b/test/music_library_web/live/collection_live/show_test.exs
@@ -58,4 +58,45 @@ defmodule MusicLibraryWeb.CollectionLive.ShowTest do
       end
     end
   end
+
+  describe "Side panel" do
+    test "shows a record's tracks", %{conn: conn} do
+      record = record()
+
+      release_response = Fixtures.Release.release(:marbles)
+
+      Req.Test.stub(MusicBrainz.API, fn conn ->
+        Req.Test.json(conn, release_response)
+      end)
+
+      session =
+        conn
+        |> visit(~p"/collection/#{record.id}")
+        |> assert_has("button", text: "Show Tracks")
+        |> unwrap(fn view ->
+          # we can't directly click the "Show Tracks" button as
+          # its phx-click event uses a JS command. Bit of a hack, but we can simulate
+          # the click by pretending we're dealing with a JS hook, and trigger the event
+          # that is sent by the JS command.
+          view
+          |> Phoenix.LiveViewTest.render_hook(:load_release_with_tracks, %{})
+        end)
+        |> assert_has("a", text: "Connect your Last.fm account")
+
+      release =
+        MusicBrainz.Release.from_api_response(release_response)
+
+      for medium <- release.media do
+        session
+        |> within("#disc-#{medium.number}", fn inner_session ->
+          for track <- medium.tracks do
+            inner_session
+            |> assert_has("li", text: escape(track.title))
+          end
+
+          inner_session
+        end)
+      end
+    end
+  end
 end