Update tests to add missing assert and = which check permission for LiveViews.

test/decentralised_book_index_web/live/live_views_permission_test.exs

@@ -47,11 +47,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
   describe "Book Edit /books/:id/new" do
     test "can't be accessed by regular user", %{conn: conn, user: user} do
       for user <- [nil, user] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        assert {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+                 conn
-        conn
+                 |> log_in_user(user)
-        |> log_in_user(user)
+                 |> live("/books/new")
-        |> live("/books/new")
       end
     end
 
@@ -78,11 +77,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
 
     test "can't be accessed by regular user", %{conn: conn, user: user, book: book} do
       for user <- [nil, user] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        assert {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+                 conn
-        conn
+                 |> log_in_user(user)
-        |> log_in_user(user)
+                 |> live("/books/#{book.id}/edit")
-        |> live("/books/#{book.id}/edit")
       end
     end
 
@@ -136,11 +134,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
   describe "Author Edit /authors/:id/new" do
     test "can't be accessed by regular user", %{conn: conn, user: user} do
       for user <- [nil, user] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        assert {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+                 conn
-        conn
+                 |> log_in_user(user)
-        |> log_in_user(user)
+                 |> live("/authors/new")
-        |> live("/authors/new")
       end
     end
 
@@ -167,11 +164,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
 
     test "can't be accessed by regular user", %{conn: conn, user: user, author: author} do
       for user <- [nil, user] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        assert {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+                 conn
-        conn
+                 |> log_in_user(user)
-        |> log_in_user(user)
+                 |> live("/authors/#{author.id}/edit")
-        |> live("/authors/#{author.id}/edit")
       end
     end
 
@@ -225,11 +221,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
 
     test "can't be accessed by regular user", %{conn: conn, user: user, publisher: publisher} do
       for user <- [nil, user] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        assert {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+                 conn
-        conn
+                 |> log_in_user(user)
-        |> log_in_user(user)
+                 |> live("/publishers/#{publisher.id}")
-        |> live("/publishers/#{publisher.id}")
       end
     end
 
@@ -253,11 +248,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
   describe "Publisher Edit /publishers/:id/new" do
     test "can't be accessed by regular user", %{conn: conn, user: user} do
       for user <- [nil, user] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+          conn
-        conn
+          |> log_in_user(user)
-        |> log_in_user(user)
+          |> live("/publishers/new")
-        |> live("/publishers/new")
       end
     end
 
@@ -284,11 +278,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
 
     test "can't be accessed by regular user", %{conn: conn, user: user, publisher: publisher} do
       for user <- [nil, user] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        assert {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+                 conn
-        conn
+                 |> log_in_user(user)
-        |> log_in_user(user)
+                 |> live("/publishers/#{publisher.id}/edit")
-        |> live("/publishers/#{publisher.id}/edit")
       end
     end
 
@@ -347,11 +340,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
       server: server
     } do
       for user <- [nil, user, moderator] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        assert {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+                 conn
-        conn
+                 |> log_in_user(user)
-        |> log_in_user(user)
+                 |> live("/servers/#{server.id}")
-        |> live("/servers/#{server.id}")
       end
     end
 
@@ -370,11 +362,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
   describe "Server Edit /servers/:id/new" do
     test "can't be accessed by non-admin user", %{conn: conn, user: user, moderator: moderator} do
       for user <- [nil, user, moderator] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        assert {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+                 conn
-        conn
+                 |> log_in_user(user)
-        |> log_in_user(user)
+                 |> live("/servers/new")
-        |> live("/servers/new")
       end
     end
 
@@ -402,11 +393,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
       server: server
     } do
       for user <- [nil, user, moderator] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        assert {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+                 conn
-        conn
+                 |> log_in_user(user)
-        |> log_in_user(user)
+                 |> live("/servers/#{server.id}/edit")
-        |> live("/servers/#{server.id}/edit")
       end
     end
 
@@ -470,11 +460,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
       moderator: moderator
     } do
       for user <- [nil, regular_user, moderator] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        assert {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+                 conn
-        conn
+                 |> log_in_user(user)
-        |> log_in_user(user)
+                 |> live("/users/#{regular_user.id}")
-        |> live("/users/#{regular_user.id}")
       end
     end
 
@@ -497,11 +486,10 @@ defmodule DecentralisedBookIndexWeb.LiveViewsPermissionsTest do
       moderator: moderator
     } do
       for user <- [nil, regular_user, moderator] do
-        {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}}
+        assert {:error, {:redirect, %{flash: %{"error" => "Unauthorized!"}, to: "/"}}} =
-
+                 conn
-        conn
+                 |> log_in_user(user)
-        |> log_in_user(user)
+                 |> live("/users/#{regular_user.id}/edit")
-        |> live("/users/#{regular_user.id}/edit")
       end
     end