fix: type mismatch when generating score in blind tests (#20)

lib/blindtest/blindtest.ex

@@ -170,17 +170,9 @@ defmodule BlindTest do
       fn {l, i}, {:ok, {conf, acc}} ->
         # skips comments and emtpy lines
         if !String.starts_with?(l, "#") and l != "" do
-          case String.replace(l, "\"", "") |> String.split(",") do
-            ["!customize" | args] ->
-              case Enum.reduce_while(args, {:ok, %Config{}}, fn kv, {:ok, config} ->
-                     case parse_custom_kv(kv) do
-                       {:ok, {k, v}} ->
-                         {:cont, {:ok, Map.put(config, k, v)}}
-
-                       err ->
-                         {:halt, err}
-                     end
-                   end) do
+          case sanitize_playlist_line(l) do
+            ["!customize" | config_kv] ->
+              case parse_config_line(config_kv) do
                 {:ok, config} ->
                   {:cont, {:ok, {config, acc}}}
 
@@ -200,18 +192,14 @@ defmodule BlindTest do
                 length(acc) > @playlist_size_limit ->
                   {:halt, {:error, "Playlist size limit reached #{@playlist_size_limit}"}}
 
-                String.contains?(uri.host, "youtu.be") || String.contains?(uri.host, "youtube") ->
-                  {:cont,
-                   {:ok,
-                    {conf,
-                     acc ++
-                       [
-                         %GuessEntry{
-                           url: url,
-                           f1s: Enum.map(String.split(f1s, "|"), &BlindTest.sanitize_input/1),
-                           f2s: Enum.map(String.split(f2s, "|"), &BlindTest.sanitize_input/1)
-                         }
-                       ]}}}
+                youtube?(uri) ->
+                  entry = %GuessEntry{
+                    url: url,
+                    f1s: Enum.map(String.split(f1s, "|"), &BlindTest.sanitize_input/1),
+                    f2s: Enum.map(String.split(f2s, "|"), &BlindTest.sanitize_input/1)
+                  }
+
+                  {:cont, {:ok, {conf, Enum.concat(acc, [entry])}}}
 
                 true ->
                   {:halt,
@@ -228,6 +216,26 @@ defmodule BlindTest do
     )
   end
 
+  defp youtube?(uri) do
+    String.contains?(uri.host, "youtu.be") || String.contains?(uri.host, "youtube")
+  end
+
+  defp sanitize_playlist_line(line) do
+    String.replace(line, "\"", "") |> String.split(",")
+  end
+
+  defp parse_config_line(config_kv) do
+    Enum.reduce_while(config_kv, {:ok, %Config{}}, fn kv, {:ok, config} ->
+      case parse_custom_kv(kv) do
+        {:ok, {k, v}} ->
+          {:cont, {:ok, Map.put(config, k, v)}}
+
+        err ->
+          {:halt, err}
+      end
+    end)
+  end
+
   @doc """
   Titleize an input string
 

lib/blindtest/game.ex

@@ -469,7 +469,10 @@ defmodule Game do
 
     Party.add_game(%Party.GameResult{name: data.name, scores: data.scores})
 
-    Nostrum.Api.create_message(data.channel_id, generate_ranking(data.scores))
+    Nostrum.Api.create_message(
+      data.channel_id,
+      "`#{data.name}` ranking:\n#{generate_ranking(data.scores)}"
+    )
 
     :keep_state_and_data
   end
@@ -527,7 +530,7 @@ defmodule Game do
     |> Enum.reduce(
       "",
       fn {{user, score}, index}, acc ->
-        "#{acc}\n#{String.pad_leading(index + 1, 3, " ")} | #{Discord.mention(user)} - **#{score}** point(s)#{if index + 1 <= 3,
+        "#{acc}\n#{String.pad_leading(Integer.to_string(index + 1), 3, " ")} | #{Discord.mention(user)} - **#{score}** point(s)#{if index + 1 <= 3,
           do: " - #{Map.get(@medals, index + 1)}"}"
       end
     )
@@ -650,7 +653,10 @@ defmodule Game.Monitor do
   end
 
   def handle_info({:DOWN, _ref, :process, object, reason}, channel_id) when reason != :killed do
-    Logger.info("Game process monitor received a game crash message", reason: reason, data: object)
+    Logger.info("Game process monitor received a game crash message",
+      reason: reason,
+      data: object
+    )
 
     Nostrum.Api.create_message!(
       channel_id,