Use long labels for formats in stats page
This commit is contained in:
@@ -98,4 +98,10 @@ defmodule MusicLibrary.Records.Record do
|
|||||||
def format_short_label(:blu_ray), do: "BR"
|
def format_short_label(:blu_ray), do: "BR"
|
||||||
def format_short_label(:dvd), do: "DVD"
|
def format_short_label(:dvd), do: "DVD"
|
||||||
def format_short_label(:multi), do: "MLT"
|
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
|
end
|
||||||
|
|||||||
@@ -1,11 +1,13 @@
|
|||||||
defmodule MusicLibraryWeb.StatsHTML do
|
defmodule MusicLibraryWeb.StatsHTML do
|
||||||
use MusicLibraryWeb, :html
|
use MusicLibraryWeb, :html
|
||||||
|
|
||||||
|
alias MusicLibrary.Records.Record
|
||||||
|
|
||||||
embed_templates "stats_html/*"
|
embed_templates "stats_html/*"
|
||||||
|
|
||||||
defp format_records_count(records_count_by_format) do
|
defp format_records_count(records_count_by_format) do
|
||||||
Enum.map(records_count_by_format, fn {format, count} ->
|
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)
|
end)
|
||||||
|> Enum.join(", ")
|
|> Enum.join(", ")
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -1,6 +1,8 @@
|
|||||||
defmodule MusicLibraryWeb.StatsControllerTest do
|
defmodule MusicLibraryWeb.StatsControllerTest do
|
||||||
use MusicLibraryWeb.ConnCase
|
use MusicLibraryWeb.ConnCase
|
||||||
|
|
||||||
|
alias MusicLibrary.Records.Record
|
||||||
|
|
||||||
import MusicLibrary.RecordsFixtures
|
import MusicLibrary.RecordsFixtures
|
||||||
|
|
||||||
defp create_records(_) do
|
defp create_records(_) do
|
||||||
@@ -25,7 +27,7 @@ defmodule MusicLibraryWeb.StatsControllerTest do
|
|||||||
records
|
records
|
||||||
|> Enum.frequencies_by(& &1.format)
|
|> Enum.frequencies_by(& &1.format)
|
||||||
|> Enum.each(fn {format, count} ->
|
|> 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)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user