diff --git a/lib/music_library/records/record.ex b/lib/music_library/records/record.ex index ea396696..82d0d8f4 100644 --- a/lib/music_library/records/record.ex +++ b/lib/music_library/records/record.ex @@ -98,4 +98,10 @@ defmodule MusicLibrary.Records.Record do def format_short_label(:blu_ray), do: "BR" def format_short_label(:dvd), do: "DVD" def format_short_label(:multi), do: "MLT" + + def format_long_label(:cd), do: "CD" + def format_long_label(:vinyl), do: "Vinyl" + def format_long_label(:blu_ray), do: "Blu-ray" + def format_long_label(:dvd), do: "DVD" + def format_long_label(:multi), do: "Multi" end diff --git a/lib/music_library_web/controllers/stats_html.ex b/lib/music_library_web/controllers/stats_html.ex index 7bc60a5e..1fa94e1d 100644 --- a/lib/music_library_web/controllers/stats_html.ex +++ b/lib/music_library_web/controllers/stats_html.ex @@ -1,11 +1,13 @@ defmodule MusicLibraryWeb.StatsHTML do use MusicLibraryWeb, :html + alias MusicLibrary.Records.Record + embed_templates "stats_html/*" defp format_records_count(records_count_by_format) do Enum.map(records_count_by_format, fn {format, count} -> - Integer.to_string(count) <> " " <> Atom.to_string(format) + Integer.to_string(count) <> " " <> Record.format_long_label(format) end) |> Enum.join(", ") end diff --git a/test/music_library_web/controllers/stats_controller_test.exs b/test/music_library_web/controllers/stats_controller_test.exs index 00d185d6..fef79894 100644 --- a/test/music_library_web/controllers/stats_controller_test.exs +++ b/test/music_library_web/controllers/stats_controller_test.exs @@ -1,6 +1,8 @@ defmodule MusicLibraryWeb.StatsControllerTest do use MusicLibraryWeb.ConnCase + alias MusicLibrary.Records.Record + import MusicLibrary.RecordsFixtures defp create_records(_) do @@ -25,7 +27,7 @@ defmodule MusicLibraryWeb.StatsControllerTest do records |> Enum.frequencies_by(& &1.format) |> Enum.each(fn {format, count} -> - assert html_response(conn, 200) =~ "#{count} #{format}" + assert html_response(conn, 200) =~ "#{count} #{Record.format_long_label(format)}" end) end