Refactor conn aliasing in controller tests to use pipe chains

Applied refactor_conn_aliasing.sh to eliminate conn shadowing.

Show draft posts in test and dev

app/test/firehose_web/controllers/blog_test.exs

@@ -5,8 +5,7 @@ defmodule FirehoseWeb.BlogTest do
 
   defp visit_engineering_page(conn, suffix \\ "") do
     path = "/blog/engineering" <> suffix
-    conn = get(conn, path)
-    body = html_response(conn, 200)
+    body = conn |> get(path) |> html_response(200)
     assert body =~ "Engineering Blog"
     assert body =~ "firehose"
     body
@@ -14,8 +13,7 @@ defmodule FirehoseWeb.BlogTest do
 
   defp visit_engineering_path(conn, suffix) do
     path = "/blog/engineering" <> suffix
-    conn = get(conn, path)
-    body = html_response(conn, 200)
+    body = conn |> get(path) |> html_response(200)
     assert body =~ "firehose"
     body
   end
@@ -38,23 +36,19 @@ defmodule FirehoseWeb.BlogTest do
 
   describe "input validation" do
     test "GET /blog/nonexistent returns 404", %{conn: conn} do
-      conn = get(conn, "/blog/nonexistent")
-      assert html_response(conn, 404)
+      assert conn |> get("/blog/nonexistent") |> html_response(404)
     end
 
     test "GET /blog/engineering?page=abc falls back to page 1", %{conn: conn} do
-      body = visit_engineering_page(conn, "")
-      assert body =~ "Engineering Blog"
+      assert conn |> get("/blog/engineering?page=abc") |> html_response(200) =~ "Engineering Blog"
     end
 
     test "GET /blog/engineering?page=-1 falls back to page 1", %{conn: conn} do
-      body = visit_engineering_page(conn, "")
-      assert body =~ "Engineering Blog"
+      assert conn |> get("/blog/engineering?page=-1") |> html_response(200) =~ "Engineering Blog"
     end
 
     test "GET /blog/engineering?page=0 falls back to page 1", %{conn: conn} do
-      body = visit_engineering_page(conn, "?page=0")
-      assert body =~ "Engineering Blog"
+      assert conn |> get("/blog/engineering?page=0") |> html_response(200) =~ "Engineering Blog"
     end
 
     test "GET /blog/engineering/nonexistent-post returns 404", %{conn: conn} do
@@ -66,85 +60,99 @@ defmodule FirehoseWeb.BlogTest do
 
   describe "release notes blog (HTML)" do
     test "GET /blog/releases returns HTML index", %{conn: conn} do
-      conn = get(conn, "/blog/releases")
-      body = html_response(conn, 200)
+      body = conn |> get("/blog/releases") |> html_response(200)
       assert body =~ "Release Notes"
       assert body =~ "v0.1.0 Released"
     end
 
     test "GET /blog/releases/:slug returns HTML post", %{conn: conn} do
-      conn = get(conn, "/blog/releases/v0-1-0")
-      body = html_response(conn, 200)
+      body = conn |> get("/blog/releases/v0-1-0") |> html_response(200)
       assert body =~ "v0.1.0 Released"
     end
 
     test "GET /blog/releases/tag/:tag returns HTML tag page", %{conn: conn} do
-      conn = get(conn, "/blog/releases/tag/elixir")
-      body = html_response(conn, 200)
+      body = conn |> get("/blog/releases/tag/elixir") |> html_response(200)
       assert body =~ ~s(tagged "elixir")
     end
   end
 
   describe "engineering blog (JSON API)" do
     test "GET /api/blog/engineering returns post index", %{conn: conn} do
-      conn = conn |> put_req_header("accept", "application/json")
-      conn = get(conn, "/api/blog/engineering")
-      assert %{"blog" => "engineering", "posts" => posts} = json_response(conn, 200)
+      assert %{"blog" => "engineering", "posts" => posts} =
+               conn
+               |> put_req_header("accept", "application/json")
+               |> get("/api/blog/engineering")
+               |> json_response(200)
+
       assert is_list(posts)
       refute Enum.empty?(posts)
     end
 
     test "GET /api/blog/engineering/:slug returns a post", %{conn: conn} do
-      conn = conn |> put_req_header("accept", "application/json")
-      conn = get(conn, "/api/blog/engineering/hello-world")
-      assert %{"id" => "hello-world", "title" => "Hello World"} = json_response(conn, 200)
+      assert %{"id" => "hello-world", "title" => "Hello World"} =
+               conn
+               |> put_req_header("accept", "application/json")
+               |> get("/api/blog/engineering/hello-world")
+               |> json_response(200)
     end
 
     test "GET /api/blog/engineering/:slug returns 404 for missing post", %{conn: conn} do
-      conn = conn |> put_req_header("accept", "application/json")
-      conn = get(conn, "/api/blog/engineering/nonexistent")
-      assert response(conn, 404)
+      assert conn
+             |> put_req_header("accept", "application/json")
+             |> get("/api/blog/engineering/nonexistent")
+             |> response(404)
     end
 
     test "GET /api/blog/engineering/feed.xml returns RSS", %{conn: conn} do
-      conn = get(conn, "/api/blog/engineering/feed.xml")
-      assert response(conn, 200) =~ "<rss"
-      assert response_content_type(conn, :xml)
+      response = conn |> get("/api/blog/engineering/feed.xml")
+      assert response(response, 200) =~ "<rss"
+      assert response_content_type(response, :xml)
     end
 
     test "GET /api/blog/engineering/tag/:tag returns JSON with posts", %{conn: conn} do
-      conn = conn |> put_req_header("accept", "application/json")
-      conn = get(conn, "/api/blog/engineering/tag/elixir")
-      assert %{"blog" => "engineering", "tag" => "elixir", "posts" => posts} = json_response(conn, 200)
+      assert %{"blog" => "engineering", "tag" => "elixir", "posts" => posts} =
+               conn
+               |> put_req_header("accept", "application/json")
+               |> get("/api/blog/engineering/tag/elixir")
+               |> json_response(200)
+
       assert is_list(posts)
     end
   end
 
   describe "release notes blog (JSON API)" do
     test "GET /api/blog/releases returns post index", %{conn: conn} do
-      conn = conn |> put_req_header("accept", "application/json")
-      conn = get(conn, "/api/blog/releases")
-      assert %{"blog" => "release_notes", "posts" => posts} = json_response(conn, 200)
+      assert %{"blog" => "release_notes", "posts" => posts} =
+               conn
+               |> put_req_header("accept", "application/json")
+               |> get("/api/blog/releases")
+               |> json_response(200)
+
       assert is_list(posts)
       refute Enum.empty?(posts)
     end
 
     test "GET /api/blog/releases/:slug returns a post", %{conn: conn} do
-      conn = conn |> put_req_header("accept", "application/json")
-      conn = get(conn, "/api/blog/releases/v0-1-0")
-      assert %{"id" => "v0-1-0", "title" => "v0.1.0 Released"} = json_response(conn, 200)
+      assert %{"id" => "v0-1-0", "title" => "v0.1.0 Released"} =
+               conn
+               |> put_req_header("accept", "application/json")
+               |> get("/api/blog/releases/v0-1-0")
+               |> json_response(200)
     end
 
     test "GET /api/blog/releases/feed.xml returns RSS", %{conn: conn} do
-      conn = get(conn, "/api/blog/releases/feed.xml")
-      assert response(conn, 200) =~ "<rss"
-      assert response_content_type(conn, :xml)
+      response = conn |> get("/api/blog/releases/feed.xml")
+      assert response(response, 200) =~ "<rss"
+      assert response_content_type(response, :xml)
     end
 
     test "GET /api/blog/releases/tag/:tag returns JSON with posts", %{conn: conn} do
-      conn = conn |> put_req_header("accept", "application/json")
-      conn = get(conn, "/api/blog/releases/tag/elixir")
-      assert %{"blog" => "release_notes", "tag" => "elixir", "posts" => posts} = json_response(conn, 200)
+      assert %{"blog" => "release_notes", "tag" => "elixir", "posts" => posts} =
+               conn
+               |> put_req_header("accept", "application/json")
+               |> get("/api/blog/releases/tag/elixir")
+               |> json_response(200)
+
       assert is_list(posts)
     end
   end

app/test/firehose_web/controllers/page_controller_test.exs

@@ -2,8 +2,7 @@ defmodule FirehoseWeb.PageControllerTest do
   use FirehoseWeb.ConnCase
 
   test "GET /", %{conn: conn} do
-    conn = get(conn, ~p"/")
-    body = html_response(conn, 200)
+    body = conn |> get(~p"/") |> html_response(200)
     assert body =~ "Drinking from the firehose"
     assert body =~ "Willem van den Ende"
   end