Merge pull request #2 from elixirschool/user-auth-tests

add factory and tests

Author: SophieDeBenedetto

lib/pointing_party/card.ex

@@ -1,6 +1,7 @@
 defmodule PointingParty.Card do
   use Ecto.Schema
   import Ecto.Changeset
+  import Ecto.Query
   alias PointingParty.Repo
   alias PointingParty.Card
 
@@ -21,4 +22,10 @@ defmodule PointingParty.Card do
   def get!(id) do
     Repo.get!(Card, id)
   end
+
+  def first do
+    Card
+    |> first
+    |> Repo.one()
+  end
 end

lib/pointing_party_web/controllers/card_controller.ex

@@ -4,7 +4,7 @@ defmodule PointingPartyWeb.CardController do
 
   def index(conn, _params) do
     # temporary, just to get something on the page for now
-    card = Card.get!(1)
+    card = Card.first()
     render(conn, "index.html", card: card)
   end
 end

mix.exs

@@ -42,7 +42,8 @@ defmodule PointingParty.MixProject do
       {:phoenix_live_reload, "~> 1.2", only: :dev},
       {:gettext, "~> 0.11"},
       {:jason, "~> 1.0"},
-      {:plug_cowboy, "~> 2.0"}
+      {:plug_cowboy, "~> 2.0"},
+      {:ex_machina, "~> 2.3", only: :test}
     ]
   end
 

mix.lock

@@ -6,6 +6,7 @@
   "decimal": {:hex, :decimal, "1.8.0", "ca462e0d885f09a1c5a342dbd7c1dcf27ea63548c65a65e67334f4b61803822e", [:mix], [], "hexpm"},
   "ecto": {:hex, :ecto, "3.1.7", "fa21d06ef56cdc2fdaa62574e8c3ba34a2751d44ea34c30bc65f0728421043e5", [:mix], [{:decimal, "~> 1.6", [hex: :decimal, repo: "hexpm", optional: false]}, {:jason, "~> 1.0", [hex: :jason, repo: "hexpm", optional: true]}], "hexpm"},
   "ecto_sql": {:hex, :ecto_sql, "3.1.6", "1e80e30d16138a729c717f73dcb938590bcdb3a4502f3012414d0cbb261045d8", [:mix], [{:db_connection, "~> 2.0", [hex: :db_connection, repo: "hexpm", optional: false]}, {:ecto, "~> 3.1.0", [hex: :ecto, repo: "hexpm", optional: false]}, {:mariaex, "~> 0.9.1", [hex: :mariaex, repo: "hexpm", optional: true]}, {:myxql, "~> 0.2.0", [hex: :myxql, repo: "hexpm", optional: true]}, {:postgrex, "~> 0.14.0 or ~> 0.15.0", [hex: :postgrex, repo: "hexpm", optional: true]}, {:telemetry, "~> 0.4.0", [hex: :telemetry, repo: "hexpm", optional: false]}], "hexpm"},
+  "ex_machina": {:hex, :ex_machina, "2.3.0", "92a5ad0a8b10ea6314b876a99c8c9e3f25f4dde71a2a835845b136b9adaf199a", [:mix], [{:ecto, "~> 2.2 or ~> 3.0", [hex: :ecto, repo: "hexpm", optional: true]}, {:ecto_sql, "~> 3.0", [hex: :ecto_sql, repo: "hexpm", optional: true]}], "hexpm"},
   "file_system": {:hex, :file_system, "0.2.7", "e6f7f155970975789f26e77b8b8d8ab084c59844d8ecfaf58cbda31c494d14aa", [:mix], [], "hexpm"},
   "gettext": {:hex, :gettext, "0.17.0", "abe21542c831887a2b16f4c94556db9c421ab301aee417b7c4fbde7fbdbe01ec", [:mix], [], "hexpm"},
   "jason": {:hex, :jason, "1.1.2", "b03dedea67a99223a2eaf9f1264ce37154564de899fd3d8b9a21b1a6fd64afe7", [:mix], [{:decimal, "~> 1.0", [hex: :decimal, repo: "hexpm", optional: true]}], "hexpm"},

test/pointing_party_web/controllers/card_controller_test.exs

@@ -0,0 +1,27 @@
+defmodule PointingPartyWeb.CardControllerTest do
+  use PointingPartyWeb.ConnCase
+  import PointingParty.Factory
+
+  @username "test_user"
+
+  describe "authenticated user" do
+    setup  %{conn: conn} do
+      conn = conn
+      |> Plug.Test.init_test_session(username: @username)
+      {:ok, %{conn: conn}}
+    end
+
+    test "GET /cards", %{conn: conn} do
+      insert(:card)
+      conn = get(conn, "/cards")
+      assert html_response(conn, 200) =~ "Ticket Title0"
+    end
+  end
+
+  describe "unauthenticated user" do
+    test "GET / cards redirects to the log in page", %{conn: conn} do
+      conn = get(conn, "/cards")
+      assert redirected_to(conn) =~ "/login"
+    end
+  end
+end

test/pointing_party_web/controllers/page_controller_test.exs

@@ -1,8 +1,25 @@
 defmodule PointingPartyWeb.PageControllerTest do
   use PointingPartyWeb.ConnCase
 
-  test "GET /", %{conn: conn} do
-    conn = get(conn, "/")
-    assert html_response(conn, 200) =~ "Welcome to Phoenix!"
+  @username "test_user"
+
+  describe "authenticated user" do
+    setup  %{conn: conn} do
+      conn = conn
+      |> Plug.Conn.assign(:username, @username)
+      {:ok, %{conn: conn}}
+    end
+
+    test "GET / renders the page with log out link", %{conn: conn} do
+      conn = get(conn, "/")
+      assert html_response(conn, 200) =~ "Log Out"
+    end
+  end
+
+  describe "unauthenticated user" do
+    test "GET / renders the page with log in link", %{conn: conn} do
+      conn = get(conn, "/")
+      assert html_response(conn, 200) =~ "Log In"
+    end
   end
 end

test/support/factory.ex

@@ -0,0 +1,11 @@
+defmodule PointingParty.Factory do
+  use ExMachina.Ecto, repo: PointingParty.Repo
+  alias PointingParty.Card
+
+  def card_factory do
+    %Card{
+      title: sequence("Ticket Title"),
+      description: sequence("Ticket description")
+    }
+  end
+end

test/test_helper.exs

@@ -1,2 +1,3 @@
+{:ok, _} = Application.ensure_all_started(:ex_machina)
 ExUnit.start()
 Ecto.Adapters.SQL.Sandbox.mode(PointingParty.Repo, :manual)