From 87f78e7250ad58c8c7ad271380a7b3ab8769435d Mon Sep 17 00:00:00 2001
From: Kieran Eglin <kieran.eglin@gmail.com>
Date: Thu, 29 Feb 2024 12:19:12 -0800
Subject: [PATCH] Added docs; fixed up stragglers

---
 .../components/custom_components/text_components.ex        | 7 +++++--
 .../controllers/sources/source_html/index.html.heex        | 2 +-
 .../controllers/media_profile_controller_test.exs          | 4 ++--
 test/pinchflat_web/controllers/source_controller_test.exs  | 4 ++--
 4 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/lib/pinchflat_web/components/custom_components/text_components.ex b/lib/pinchflat_web/components/custom_components/text_components.ex
index 44b2cb0..e362110 100644
--- a/lib/pinchflat_web/components/custom_components/text_components.ex
+++ b/lib/pinchflat_web/components/custom_components/text_components.ex
@@ -31,13 +31,16 @@ defmodule PinchflatWeb.CustomComponents.TextComponents do
     """
   end
 
+  @doc """
+  Renders an icon as a link with the given href.
+  """
   attr :href, :string, required: true
   attr :icon, :string, required: true
   attr :class, :string, default: ""
-  # TODO: docs
+
   def icon_link(assigns) do
     ~H"""
-    <.link navigate={@href} class={["hover:text-secondary duration-200 ease-in-out", @class]}>
+    <.link href={@href} class={["hover:text-secondary duration-200 ease-in-out", @class]}>
       <CoreComponents.icon name={@icon} />
     </.link>
     """
diff --git a/lib/pinchflat_web/controllers/sources/source_html/index.html.heex b/lib/pinchflat_web/controllers/sources/source_html/index.html.heex
index e720165..3dbfec2 100644
--- a/lib/pinchflat_web/controllers/sources/source_html/index.html.heex
+++ b/lib/pinchflat_web/controllers/sources/source_html/index.html.heex
@@ -1,5 +1,5 @@
 <div class="mb-6 flex gap-3 flex-row items-center justify-between">
-  <h2 class="text-title-md2 font-bold text-black dark:text-white">All Sources</h2>
+  <h2 class="text-title-md2 font-bold text-black dark:text-white">Sources</h2>
   <nav>
     <.link navigate={~p"/sources/new"}>
       <.button color="bg-primary" rounding="rounded-full">
diff --git a/test/pinchflat_web/controllers/media_profile_controller_test.exs b/test/pinchflat_web/controllers/media_profile_controller_test.exs
index 16e5133..c5cdd19 100644
--- a/test/pinchflat_web/controllers/media_profile_controller_test.exs
+++ b/test/pinchflat_web/controllers/media_profile_controller_test.exs
@@ -17,7 +17,7 @@ defmodule PinchflatWeb.MediaProfileControllerTest do
   describe "index" do
     test "lists all media_profiles", %{conn: conn} do
       conn = get(conn, ~p"/media_profiles")
-      assert html_response(conn, 200) =~ "All Media Profiles"
+      assert html_response(conn, 200) =~ "Media Profiles"
     end
   end
 
@@ -45,7 +45,7 @@ defmodule PinchflatWeb.MediaProfileControllerTest do
       assert redirected_to(conn) == ~p"/media_profiles/#{id}"
 
       conn = get(conn, ~p"/media_profiles/#{id}")
-      assert html_response(conn, 200) =~ "Media Profile ##{id}"
+      assert html_response(conn, 200) =~ "Media Profile"
     end
 
     test "renders errors when data is invalid", %{conn: conn} do
diff --git a/test/pinchflat_web/controllers/source_controller_test.exs b/test/pinchflat_web/controllers/source_controller_test.exs
index c9be7af..4f641f5 100644
--- a/test/pinchflat_web/controllers/source_controller_test.exs
+++ b/test/pinchflat_web/controllers/source_controller_test.exs
@@ -32,7 +32,7 @@ defmodule PinchflatWeb.SourceControllerTest do
   describe "index" do
     test "lists all sources", %{conn: conn} do
       conn = get(conn, ~p"/sources")
-      assert html_response(conn, 200) =~ "All Sources"
+      assert html_response(conn, 200) =~ "Sources"
     end
   end
 
@@ -61,7 +61,7 @@ defmodule PinchflatWeb.SourceControllerTest do
       assert redirected_to(conn) == ~p"/sources/#{id}"
 
       conn = get(conn, ~p"/sources/#{id}")
-      assert html_response(conn, 200) =~ "Source ##{id}"
+      assert html_response(conn, 200) =~ "Source"
     end
 
     test "renders errors when data is invalid", %{conn: conn, invalid_attrs: invalid_attrs} do