100% coveralls

lib/app_web/live/page_live.ex

@@ -94,8 +94,8 @@ defmodule AppWeb.PageLive do
         do: {:ok, {mimetype, width, height, variant}},
         else: {:error, "MIME types do not correspond"}
     else
-      {:image_info, nil} -> {:error, "bad file"} |> dbg()
-      {:gen_magic, {:error, reason}} -> {:error, reason} |> dbg()
+      {:image_info, nil} -> {:error, "bad file"}
+      {:gen_magic, {:error, reason}} -> {:error, reason}
     end
   end
 
@@ -175,7 +175,7 @@ defmodule AppWeb.PageLive do
   end
 
   # intermediate chunk consumption
-  def handle_progress(:image_list, _, socket), do: {:noreply, socket}
+  def handle_progress(_, _, socket), do: {:noreply, socket}
 
   @doc """
   Every time an `async task` is created, this function is called.

test/app_web/live/page_live_test.exs

@@ -91,6 +91,17 @@ defmodule AppWeb.PageLiveTest do
     assert render(lv) =~ "Image too large. Upload a smaller image up to 5MB."
   end
 
+  test "check MIME type", %{conn: _conn} do
+    path = [:code.priv_dir(:app), "static", "images", "test.png"] |> Path.join()
+    file = build_upload(path, "image/xyz")
+
+    assert AppWeb.PageLive.check_file(file, path) == {:error, "bad file"}
+
+    accepted_mime = ~w(image/jpeg image/jpg image/png image/webp)
+    assert App.Image.gen_magic_eval(path, accepted_mime) == {:error, "not acceptable"}
+    assert App.Image.gen_magic_eval("", accepted_mime) == {:error, "invalid command"}
+  end
+
   test "uploading an invalid image", %{conn: conn} do
     {:ok, lv, html} = live(conn, ~p"/")
     assert html =~ "Caption your image!"
@@ -149,6 +160,11 @@ defmodule AppWeb.PageLiveTest do
     end
   end
 
+  test "handle_intermediate_progress", %{conn: _conn} do
+    ret = AppWeb.PageLive.handle_progress(:image_list, %{}, %{})
+    assert ret == {:noreply, %{}}
+  end
+
   test "noop event handler", %{conn: _conn} do
     ret = AppWeb.PageLive.handle_event("noop", %{}, %{})