Add tests for the Links module.

CHANGELOG.org

@@ -36,4 +36,8 @@
     Add the API to update the Link model.
 ** 0.4.7 <2023-08-01 Tue>
     Add tests for the API.
+** 0.4.8 <2023-08-02 Wed>
+    Refactor the Links module.
+** 0.4.9 <2023-08-02 Wed>
+    Add tests for the Links module.
 

lib/link_shortener/links/links.ex

@@ -1,4 +1,4 @@
-defmodule LinkShortener.Links.Links do
+defmodule LinkShortener.Links do
   use Ecto.Schema
   import Ecto.Changeset
   import Ecto.Query
@@ -6,11 +6,12 @@ defmodule LinkShortener.Links.Links do
   alias LinkShortener.Links.Link
   alias LinkShortener.Generators.SafeString
 
-  def new_one(), do: Link.changeset(%Link{})
+  def new_one(), do: Link.changeset(%Link{}, %{})
 
   def create_one(attrs, length \\ 10, generator \\ SafeString) do
     shorten = generator.generate(length)
-    insert_one(Map.put(attrs, :shorten, shorten))
+    Map.put(attrs, :shorten, shorten)
+    |> insert_one()
   end
 
   def insert_one(attrs) do
@@ -19,25 +20,10 @@ defmodule LinkShortener.Links.Links do
     |> Repo.insert()
   end
 
-  def edit_one(id) do
+  def get_one!(id) do
-    get_one(id)
+    Repo.get!(Link, id)
-    |> Link.changeset()
   end
 
-  def update_one(%Link{} = link, changes) do
-    link
-    |> Link.changeset(changes)
-    |> Repo.update()
-  end
-
-  def insert_one(attrs) do
-    %Link{}
-    |> Link.changeset(attrs)
-    |> Repo.insert()
-  end
-
-  def delete_one(%Link{} = link), do: Repo.delete(link)
-
   def get_one_by(attrs) do
     Repo.get_by(Link, attrs)
   end
@@ -46,12 +32,26 @@ defmodule LinkShortener.Links.Links do
     get_one_by(%{shorten: shorten})
   end
 
-  def get_one(id) do
+  def get_all() do
-    Repo.get!(Link, id)
+    from(Link)
+    |> Repo.all()
   end
 
   def get_all(opts) do
     from(Link)
     |> Repo.all()
   end
+
+  def edit_one(%Link{} = link) do
+    link
+    |> Link.changeset(%{})
+  end
+
+  def update_one(%Link{} = link, changes) do
+    link
+    |> Link.changeset(changes)
+    |> Repo.update()
+  end
+
+  def delete_one(%Link{} = link), do: Repo.delete(link)
 end

lib/link_shortener_web/controllers/api/v1/link_controller.ex

@@ -1,13 +1,13 @@
 defmodule LinkShortenerWeb.Api.V1.LinkController do
   use LinkShortenerWeb, :controller
 
-  alias LinkShortener.Links.Links
+  alias LinkShortener.Links
   alias LinkShortener.Links.Link
 
   action_fallback LinkShortenerWeb.FallbackController
 
   def index(conn, _params) do
-    links = Links.get_all({})
+    links = Links.get_all()
     render(conn, "index.json", links: links)
   end
 
@@ -21,12 +21,12 @@ defmodule LinkShortenerWeb.Api.V1.LinkController do
   end
 
   def show(conn, %{"id" => id}) do
-    link = Links.get_one(id)
+    link = Links.get_one!(id)
     render(conn, "show.json", link: link)
   end
 
   def update(conn, %{"id" => id, "link" => link_params}) do
-    link = Links.get_one(id)
+    link = Links.get_one!(id)
 
     with {:ok, %Link{} = link} <- Links.update_one(link, link_params) do
       render(conn, "show.json", link: link)
@@ -34,7 +34,7 @@ defmodule LinkShortenerWeb.Api.V1.LinkController do
   end
 
   def delete(conn, %{"id" => id}) do
-    link = Links.get_one(id)
+    link = Links.get_one!(id)
 
     with {:ok, %Link{}} <- Links.delete_one(link) do
       send_resp(conn, :no_content, "")

test/link_shortener/links_test.exs

@@ -0,0 +1,106 @@
+defmodule LinkShortener.LinksTest do
+  use LinkShortener.DataCase
+
+  alias LinkShortener.Links
+
+  @create_attrs %{
+    name: "some link name",
+    url: "https://gitlab.com/KKlochko/link_shortener",
+    shorten: "git_repo",
+  }
+  @create_generated_attrs %{
+    name: "some link name",
+    url: "https://gitlab.com/KKlochko/link_shortener",
+  }
+  @update_attrs %{
+    name: "some updated link name",
+    url: "https://gitlab.com/KKlochko/link_shortener2",
+    shorten: "new_git_repo",
+  }
+  @invalid_attrs %{
+    name: nil,
+    url: nil,
+    shorten: nil,
+  }
+
+  describe "links" do
+    alias LinkShortener.Links.Link
+
+    import LinkShortener.LinksFixtures
+
+    test "new_one/1 returns the changeset" do
+      assert %Ecto.Changeset{} = Links.new_one()
+    end
+
+    test "create_one/1 with valid data creates a link" do
+      assert {:ok, %Link{} = link} = Links.create_one(@create_generated_attrs)
+      assert link.name == "some link name"
+      assert link.url == "https://gitlab.com/KKlochko/link_shortener"
+      assert String.length(link.shorten) == 10
+    end
+
+    test "create_one/2 with valid data creates a link" do
+      assert {:ok, %Link{} = link} = Links.create_one(@create_generated_attrs, 5)
+      assert link.name == "some link name"
+      assert link.url == "https://gitlab.com/KKlochko/link_shortener"
+      assert String.length(link.shorten) == 5
+    end
+
+    test "insert_one/1 with valid data creates a link" do
+      assert {:ok, %Link{} = link} = Links.insert_one(@create_attrs)
+      assert link.name == "some link name"
+      assert link.url == "https://gitlab.com/KKlochko/link_shortener"
+      assert link.shorten == "git_repo"
+    end
+
+    test "insert_one/1 with invalid data returns error changeset" do
+      assert {:error, %Ecto.Changeset{}} = Links.insert_one(@invalid_attrs)
+    end
+
+    test "get_one!/1 returns the link with given id" do
+      link = link_fixture()
+      assert Links.get_one!(link.id) == link
+    end
+
+    test "get_one_by!/1 returns the link with given shorten" do
+      link = link_fixture()
+      assert Links.get_one_by(%{shorten: link.shorten}) == link
+    end
+
+    test "get_one_by_shorten!/1 returns the link with given shorten" do
+      link = link_fixture()
+      assert Links.get_one_by_shorten(link.shorten) == link
+    end
+
+    test "get_all/0 returns all links" do
+      link = link_fixture()
+      assert Links.get_all() == [link]
+    end
+
+    test "edit_one/1 with valid data returns the changeset" do
+      link = link_fixture()
+      assert %Ecto.Changeset{} = Links.edit_one(link)
+    end
+
+    test "update_one/2 with valid data updates the link" do
+      link = link_fixture()
+
+      assert {:ok, %Link{} = link} = Links.update_one(link, @update_attrs)
+      assert link.name == "some updated link name"
+      assert link.url == "https://gitlab.com/KKlochko/link_shortener2"
+      assert link.shorten == "new_git_repo"
+    end
+
+    test "update_link/2 with invalid data returns error changeset" do
+      link = link_fixture()
+      assert {:error, %Ecto.Changeset{}} = Links.update_one(link, @invalid_attrs)
+      assert link == Links.get_one!(link.id)
+    end
+
+    test "delete_one/1 deletes the link" do
+      link = link_fixture()
+      assert {:ok, %Link{}} = Links.delete_one(link)
+      assert_raise Ecto.NoResultsError, fn -> Links.get_one!(link.id) end
+    end
+  end
+end

test/link_shortener_web/controllers/api/v1/link_controller_test.exs

@@ -4,7 +4,7 @@ defmodule LinkShortenerWeb.Api.V1.LinkControllerTest do
   import LinkShortener.LinksFixtures
 
   alias LinkShortener.Links.Link
-  alias LinkShortener.Links.Links
+  alias LinkShortener.Links
 
   @create_attrs %{
     name: "some link name",

test/support/fixtures/links_fixtures.ex

@@ -4,7 +4,7 @@ defmodule LinkShortener.LinksFixtures do
   entities via the `LinkShortener.Links` context.
   """
 
-  alias LinkShortener.Links.Links
+  alias LinkShortener.Links
 
   @doc """
   Generate a link.