fix tests

lib/lokal/accounts.ex

@@ -5,7 +5,7 @@ defmodule Lokal.Accounts do
 
   import Ecto.Query, warn: false
   alias Lokal.Repo
-  alias Lokal.Accounts.{User, UserToken, UserNotifier}
+  alias Lokal.Accounts.{User, UserNotifier, UserToken}
 
   ## Database getters
 

lib/lokal/accounts/user.ex

@@ -1,4 +1,8 @@
 defmodule Lokal.Accounts.User do
+  @moduledoc """
+  Schema for a registered user
+  """
+
   use Ecto.Schema
   import Ecto.Changeset
 

lib/lokal/accounts/user_notifier.ex

@@ -1,4 +1,8 @@
 defmodule Lokal.Accounts.UserNotifier do
+  @moduledoc """
+  Contains templates and messages for user messages
+  """
+
   # For simplicity, this module simply logs messages to the terminal.
   # You should replace it by a proper email or notification tool, such as:
   #

lib/lokal/accounts/user_token.ex

@@ -1,4 +1,8 @@
 defmodule Lokal.Accounts.UserToken do
+  @moduledoc """
+  Schema for a user's session token
+  """
+
   use Ecto.Schema
   import Ecto.Query
 

lib/lokal/mailer.ex

@@ -1,3 +1,7 @@
 defmodule Lokal.Mailer do
+  @moduledoc """
+  Mailer, currently uses Swoosh
+  """
+
   use Swoosh.Mailer, otp_app: :lokal
 end

lib/lokal/release.ex

@@ -1,4 +1,8 @@
 defmodule Lokal.Release do
+  @moduledoc """
+  Contains mix tasks that can used in generated releases
+  """
+
   @app :lokal
 
   def rollback(repo, version) do

lib/lokal/repo/migrator.ex

@@ -1,4 +1,8 @@
 defmodule Lokal.Repo.Migrator do
+  @moduledoc """
+  Genserver to automatically perform all migration on app start
+  """
+
   use GenServer
   require Logger
 

lib/lokal_web/component/topbar.ex

@@ -5,7 +5,7 @@ defmodule LokalWeb.Component.Topbar do
   """
 
   use LokalWeb, :component
-  alias LokalWeb.{PageLive}
+  alias LokalWeb.PageLive
 
   def topbar(assigns) do
     assigns =

lib/lokal_web/controllers/user_auth.ex

@@ -1,4 +1,8 @@
 defmodule LokalWeb.UserAuth do
+  @moduledoc """
+  Module for any user authentication functions
+  """
+
   import Plug.Conn
   import Phoenix.Controller
 

lib/lokal_web/live/live_helpers.ex

@@ -1,7 +1,11 @@
 defmodule LokalWeb.LiveHelpers do
+  @moduledoc """
+  Contains resuable methods for all liveviews
+  """
+
   import Phoenix.LiveView.Helpers
   import Phoenix.LiveView, only: [assign_new: 3]
-  alias Lokal.{Accounts}
+  alias Lokal.Accounts
 
   @doc """
   Renders a component inside the `LokalWeb.ModalComponent` component.

lib/lokal_web/live/modal_component.ex

@@ -1,4 +1,8 @@
 defmodule LokalWeb.ModalComponent do
+  @moduledoc """
+  Component that provides a floating modal
+  """
+
   use LokalWeb, :live_component
 
   @impl true

lib/lokal_web/live/page_live.ex

@@ -1,4 +1,8 @@
 defmodule LokalWeb.PageLive do
+  @moduledoc """
+  Liveview for the main home page
+  """
+
   use LokalWeb, :live_view
 
   @impl true

lib/lokal_web/telemetry.ex

@@ -1,4 +1,8 @@
 defmodule LokalWeb.Telemetry do
+  @moduledoc """
+  Telemetry genserver
+  """
+
   use Supervisor
   import Telemetry.Metrics
 

lib/lokal_web/views/layout_view.ex

@@ -1,6 +1,6 @@
 defmodule LokalWeb.LayoutView do
   use LokalWeb, :view
-  alias LokalWeb.{PageLive}
+  alias LokalWeb.PageLive
 
   # Phoenix LiveDashboard is available only in development by default,
   # so we instruct Elixir to not warn if the dashboard route is missing.

test/support/channel_case.ex

@@ -16,6 +16,7 @@ defmodule LokalWeb.ChannelCase do
   """
 
   use ExUnit.CaseTemplate
+  alias Ecto.Adapters.SQL.Sandbox
 
   using do
     quote do
@@ -29,8 +30,8 @@ defmodule LokalWeb.ChannelCase do
   end
 
   setup tags do
-    pid = Ecto.Adapters.SQL.Sandbox.start_owner!(Lokal.Repo, shared: not tags[:async])
+    pid = Sandbox.start_owner!(Lokal.Repo, shared: not tags[:async])
-    on_exit(fn -> Ecto.Adapters.SQL.Sandbox.stop_owner(pid) end)
+    on_exit(fn -> Sandbox.stop_owner(pid) end)
     :ok
   end
 end

test/support/conn_case.ex

@@ -16,6 +16,7 @@ defmodule LokalWeb.ConnCase do
   """
 
   use ExUnit.CaseTemplate
+  alias Ecto.Adapters.SQL.Sandbox
 
   using do
     quote do
@@ -32,8 +33,8 @@ defmodule LokalWeb.ConnCase do
   end
 
   setup tags do
-    pid = Ecto.Adapters.SQL.Sandbox.start_owner!(Lokal.Repo, shared: not tags[:async])
+    pid = Sandbox.start_owner!(Lokal.Repo, shared: not tags[:async])
-    on_exit(fn -> Ecto.Adapters.SQL.Sandbox.stop_owner(pid) end)
+    on_exit(fn -> Sandbox.stop_owner(pid) end)
     {:ok, conn: Phoenix.ConnTest.build_conn()}
   end
 

test/support/data_case.ex

@@ -15,6 +15,7 @@ defmodule Lokal.DataCase do
   """
 
   use ExUnit.CaseTemplate
+  alias Ecto.Adapters.SQL.Sandbox
 
   using do
     quote do
@@ -28,8 +29,8 @@ defmodule Lokal.DataCase do
   end
 
   setup tags do
-    pid = Ecto.Adapters.SQL.Sandbox.start_owner!(Lokal.Repo, shared: not tags[:async])
+    pid = Sandbox.start_owner!(Lokal.Repo, shared: not tags[:async])
-    on_exit(fn -> Ecto.Adapters.SQL.Sandbox.stop_owner(pid) end)
+    on_exit(fn -> Sandbox.stop_owner(pid) end)
     :ok
   end