dev/wild
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix account urls

Browse source
This commit is contained in:
Daniel Yrovas 2025-01-09 15:28:23 +11:00
parent 8e1bb5c333
commit e218290e08
Signed by: danielyrovas
SSH key fingerprint: SHA256:1avlGZQpGW038lBkNI5lOS7f0hIPM7ecJen2/P1MCCU
19 changed files with 102 additions and 102 deletions
Download patch file Download diff file Expand all files Collapse all files

28
test/wild_web/controllers/user_session_controller_test.exs
View file

@ -7,10 +7,10 @@ defmodule WildWeb.UserSessionControllerTest do
%{user: user_fixture()}
end
describe "POST /users/log_in" do
describe "POST /account/log_in" do
test "logs the user in", %{conn: conn, user: user} do
conn =
post(conn, ~p"/users/log_in", %{
post(conn, ~p"/account/log_in", %{
"user" => %{"email" => user.email, "password" => valid_user_password()}
})
@ -21,13 +21,13 @@ defmodule WildWeb.UserSessionControllerTest do
conn = get(conn, ~p"/")
response = html_response(conn, 200)
assert response =~ user.email
assert response =~ ~p"/users/settings"
assert response =~ ~p"/users/log_out"
assert response =~ ~p"/account/settings"
assert response =~ ~p"/account/log_out"
end
test "logs the user in with remember me", %{conn: conn, user: user} do
conn =
post(conn, ~p"/users/log_in", %{
post(conn, ~p"/account/log_in", %{
"user" => %{
"email" => user.email,
"password" => valid_user_password(),
@ -43,7 +43,7 @@ defmodule WildWeb.UserSessionControllerTest do
conn =
conn
|> init_test_session(user_return_to: "/foo/bar")
|> post(~p"/users/log_in", %{
|> post(~p"/account/log_in", %{
"user" => %{
"email" => user.email,
"password" => valid_user_password()
@ -57,7 +57,7 @@ defmodule WildWeb.UserSessionControllerTest do
test "login following registration", %{conn: conn, user: user} do
conn =
conn
|> post(~p"/users/log_in", %{
|> post(~p"/account/log_in", %{
"_action" => "registered",
"user" => %{
"email" => user.email,
@ -72,7 +72,7 @@ defmodule WildWeb.UserSessionControllerTest do
test "login following password update", %{conn: conn, user: user} do
conn =
conn
|> post(~p"/users/log_in", %{
|> post(~p"/account/log_in", %{
"_action" => "password_updated",
"user" => %{
"email" => user.email,
@ -80,31 +80,31 @@ defmodule WildWeb.UserSessionControllerTest do
}
})
assert redirected_to(conn) == ~p"/users/settings"
assert redirected_to(conn) == ~p"/account/settings"
assert Phoenix.Flash.get(conn.assigns.flash, :info) =~ "Password updated successfully"
end
test "redirects to login page with invalid credentials", %{conn: conn} do
conn =
post(conn, ~p"/users/log_in", %{
post(conn, ~p"/account/log_in", %{
"user" => %{"email" => "invalid@email.com", "password" => "invalid_password"}
})
assert Phoenix.Flash.get(conn.assigns.flash, :error) == "Invalid email or password"
assert redirected_to(conn) == ~p"/users/log_in"
assert redirected_to(conn) == ~p"/account/log_in"
end
end
describe "DELETE /users/log_out" do
describe "DELETE /account/log_out" do
test "logs the user out", %{conn: conn, user: user} do
conn = conn |> log_in_user(user) |> delete(~p"/users/log_out")
conn = conn |> log_in_user(user) |> delete(~p"/account/log_out")
assert redirected_to(conn) == ~p"/"
refute get_session(conn, :user_token)
assert Phoenix.Flash.get(conn.assigns.flash, :info) =~ "Logged out successfully"
end
test "succeeds even if the user is not logged in", %{conn: conn} do
conn = delete(conn, ~p"/users/log_out")
conn = delete(conn, ~p"/account/log_out")
assert redirected_to(conn) == ~p"/"
refute get_session(conn, :user_token)
assert Phoenix.Flash.get(conn.assigns.flash, :info) =~ "Logged out successfully"

8
test/wild_web/live/user_confirmation_instructions_live_test.exs
View file

@ -13,12 +13,12 @@ defmodule WildWeb.UserConfirmationInstructionsLiveTest do
describe "Resend confirmation" do
test "renders the resend confirmation page", %{conn: conn} do
{:ok, _lv, html} = live(conn, ~p"/users/confirm")
{:ok, _lv, html} = live(conn, ~p"/account/confirm")
assert html =~ "Resend confirmation instructions"
end
test "sends a new confirmation token", %{conn: conn, user: user} do
{:ok, lv, _html} = live(conn, ~p"/users/confirm")
{:ok, lv, _html} = live(conn, ~p"/account/confirm")
{:ok, conn} =
lv
@ -35,7 +35,7 @@ defmodule WildWeb.UserConfirmationInstructionsLiveTest do
test "does not send confirmation token if user is confirmed", %{conn: conn, user: user} do
Repo.update!(Accounts.User.confirm_changeset(user))
{:ok, lv, _html} = live(conn, ~p"/users/confirm")
{:ok, lv, _html} = live(conn, ~p"/account/confirm")
{:ok, conn} =
lv
@ -50,7 +50,7 @@ defmodule WildWeb.UserConfirmationInstructionsLiveTest do
end
test "does not send confirmation token if email is invalid", %{conn: conn} do
{:ok, lv, _html} = live(conn, ~p"/users/confirm")
{:ok, lv, _html} = live(conn, ~p"/account/confirm")
{:ok, conn} =
lv

10
test/wild_web/live/user_confirmation_live_test.exs
View file

@ -13,7 +13,7 @@ defmodule WildWeb.UserConfirmationLiveTest do
describe "Confirm user" do
test "renders confirmation page", %{conn: conn} do
{:ok, _lv, html} = live(conn, ~p"/users/confirm/some-token")
{:ok, _lv, html} = live(conn, ~p"/account/confirm/some-token")
assert html =~ "Confirm Account"
end
@ -23,7 +23,7 @@ defmodule WildWeb.UserConfirmationLiveTest do
Accounts.deliver_user_confirmation_instructions(user, url)
end)
{:ok, lv, _html} = live(conn, ~p"/users/confirm/#{token}")
{:ok, lv, _html} = live(conn, ~p"/account/confirm/#{token}")
result =
lv
@ -41,7 +41,7 @@ defmodule WildWeb.UserConfirmationLiveTest do
assert Repo.all(Accounts.UserToken) == []
# when not logged in
{:ok, lv, _html} = live(conn, ~p"/users/confirm/#{token}")
{:ok, lv, _html} = live(conn, ~p"/account/confirm/#{token}")
result =
lv
@ -59,7 +59,7 @@ defmodule WildWeb.UserConfirmationLiveTest do
build_conn()
|> log_in_user(user)
{:ok, lv, _html} = live(conn, ~p"/users/confirm/#{token}")
{:ok, lv, _html} = live(conn, ~p"/account/confirm/#{token}")
result =
lv
@ -72,7 +72,7 @@ defmodule WildWeb.UserConfirmationLiveTest do
end
test "does not confirm email with invalid token", %{conn: conn, user: user} do
{:ok, lv, _html} = live(conn, ~p"/users/confirm/invalid-token")
{:ok, lv, _html} = live(conn, ~p"/account/confirm/invalid-token")
{:ok, conn} =
lv

12
test/wild_web/live/user_forgot_password_live_test.exs
View file

@ -9,18 +9,18 @@ defmodule WildWeb.UserForgotPasswordLiveTest do
describe "Forgot password page" do
test "renders email page", %{conn: conn} do
{:ok, lv, html} = live(conn, ~p"/users/reset_password")
{:ok, lv, html} = live(conn, ~p"/account/reset_password")
assert html =~ "Forgot your password?"
assert has_element?(lv, ~s|a[href="#{~p"/users/register"}"]|, "Register")
assert has_element?(lv, ~s|a[href="#{~p"/users/log_in"}"]|, "Log in")
assert has_element?(lv, ~s|a[href="#{~p"/account/register"}"]|, "Register")
assert has_element?(lv, ~s|a[href="#{~p"/account/log_in"}"]|, "Log in")
end
test "redirects if already logged in", %{conn: conn} do
result =
conn
|> log_in_user(user_fixture())
|> live(~p"/users/reset_password")
|> live(~p"/account/reset_password")
|> follow_redirect(conn, ~p"/")
assert {:ok, _conn} = result
@ -33,7 +33,7 @@ defmodule WildWeb.UserForgotPasswordLiveTest do
end
test "sends a new reset password token", %{conn: conn, user: user} do
{:ok, lv, _html} = live(conn, ~p"/users/reset_password")
{:ok, lv, _html} = live(conn, ~p"/account/reset_password")
{:ok, conn} =
lv
@ -48,7 +48,7 @@ defmodule WildWeb.UserForgotPasswordLiveTest do
end
test "does not send reset password token if email is invalid", %{conn: conn} do
{:ok, lv, _html} = live(conn, ~p"/users/reset_password")
{:ok, lv, _html} = live(conn, ~p"/account/reset_password")
{:ok, conn} =
lv

18
test/wild_web/live/user_login_live_test.exs
View file

@ -6,7 +6,7 @@ defmodule WildWeb.UserLoginLiveTest do
describe "Log in page" do
test "renders log in page", %{conn: conn} do
{:ok, _lv, html} = live(conn, ~p"/users/log_in")
{:ok, _lv, html} = live(conn, ~p"/account/log_in")
assert html =~ "Log in"
assert html =~ "Register"
@ -17,7 +17,7 @@ defmodule WildWeb.UserLoginLiveTest do
result =
conn
|> log_in_user(user_fixture())
|> live(~p"/users/log_in")
|> live(~p"/account/log_in")
|> follow_redirect(conn, "/")
assert {:ok, _conn} = result
@ -29,7 +29,7 @@ defmodule WildWeb.UserLoginLiveTest do
password = "123456789abcd"
user = user_fixture(%{password: password})
{:ok, lv, _html} = live(conn, ~p"/users/log_in")
{:ok, lv, _html} = live(conn, ~p"/account/log_in")
form =
form(lv, "#login_form", user: %{email: user.email, password: password, remember_me: true})
@ -42,7 +42,7 @@ defmodule WildWeb.UserLoginLiveTest do
test "redirects to login page with a flash error if there are no valid credentials", %{
conn: conn
} do
{:ok, lv, _html} = live(conn, ~p"/users/log_in")
{:ok, lv, _html} = live(conn, ~p"/account/log_in")
form =
form(lv, "#login_form",
@ -53,19 +53,19 @@ defmodule WildWeb.UserLoginLiveTest do
assert Phoenix.Flash.get(conn.assigns.flash, :error) == "Invalid email or password"
assert redirected_to(conn) == "/users/log_in"
assert redirected_to(conn) == "/account/log_in"
end
end
describe "login navigation" do
test "redirects to registration page when the Register button is clicked", %{conn: conn} do
{:ok, lv, _html} = live(conn, ~p"/users/log_in")
{:ok, lv, _html} = live(conn, ~p"/account/log_in")
{:ok, _login_live, login_html} =
lv
|> element(~s|main a:fl-contains("Sign up")|)
|> render_click()
|> follow_redirect(conn, ~p"/users/register")
|> follow_redirect(conn, ~p"/account/register")
assert login_html =~ "Register"
end
@ -73,13 +73,13 @@ defmodule WildWeb.UserLoginLiveTest do
test "redirects to forgot password page when the Forgot Password button is clicked", %{
conn: conn
} do
{:ok, lv, _html} = live(conn, ~p"/users/log_in")
{:ok, lv, _html} = live(conn, ~p"/account/log_in")
{:ok, conn} =
lv
|> element(~s|main a:fl-contains("Forgot your password?")|)
|> render_click()
|> follow_redirect(conn, ~p"/users/reset_password")
|> follow_redirect(conn, ~p"/account/reset_password")
assert conn.resp_body =~ "Forgot your password?"
end

14
test/wild_web/live/user_registration_live_test.exs
View file

@ -6,7 +6,7 @@ defmodule WildWeb.UserRegistrationLiveTest do
describe "Registration page" do
test "renders registration page", %{conn: conn} do
{:ok, _lv, html} = live(conn, ~p"/users/register")
{:ok, _lv, html} = live(conn, ~p"/account/register")
assert html =~ "Register"
assert html =~ "Log in"
@ -16,14 +16,14 @@ defmodule WildWeb.UserRegistrationLiveTest do
result =
conn
|> log_in_user(user_fixture())
|> live(~p"/users/register")
|> live(~p"/account/register")
|> follow_redirect(conn, "/")
assert {:ok, _conn} = result
end
test "renders errors for invalid data", %{conn: conn} do
{:ok, lv, _html} = live(conn, ~p"/users/register")
{:ok, lv, _html} = live(conn, ~p"/account/register")
result =
lv
@ -38,7 +38,7 @@ defmodule WildWeb.UserRegistrationLiveTest do
describe "register user" do
test "creates account and logs the user in", %{conn: conn} do
{:ok, lv, _html} = live(conn, ~p"/users/register")
{:ok, lv, _html} = live(conn, ~p"/account/register")
email = unique_user_email()
form = form(lv, "#registration_form", user: valid_user_attributes(email: email))
@ -56,7 +56,7 @@ defmodule WildWeb.UserRegistrationLiveTest do
end
test "renders errors for duplicated email", %{conn: conn} do
{:ok, lv, _html} = live(conn, ~p"/users/register")
{:ok, lv, _html} = live(conn, ~p"/account/register")
user = user_fixture(%{email: "test@email.com"})
@ -73,13 +73,13 @@ defmodule WildWeb.UserRegistrationLiveTest do
describe "registration navigation" do
test "redirects to login page when the Log in button is clicked", %{conn: conn} do
{:ok, lv, _html} = live(conn, ~p"/users/register")
{:ok, lv, _html} = live(conn, ~p"/account/register")
{:ok, _login_live, login_html} =
lv
|> element(~s|main a:fl-contains("Log in")|)
|> render_click()
|> follow_redirect(conn, ~p"/users/log_in")
|> follow_redirect(conn, ~p"/account/log_in")
assert login_html =~ "Log in"
end

20
test/wild_web/live/user_reset_password_live_test.exs
View file

@ -19,13 +19,13 @@ defmodule WildWeb.UserResetPasswordLiveTest do
describe "Reset password page" do
test "renders reset password with valid token", %{conn: conn, token: token} do
{:ok, _lv, html} = live(conn, ~p"/users/reset_password/#{token}")
{:ok, _lv, html} = live(conn, ~p"/account/reset_password/#{token}")
assert html =~ "Reset Password"
end
test "does not render reset password with invalid token", %{conn: conn} do
{:error, {:redirect, to}} = live(conn, ~p"/users/reset_password/invalid")
{:error, {:redirect, to}} = live(conn, ~p"/account/reset_password/invalid")
assert to == %{
flash: %{"error" => "Reset password link is invalid or it has expired."},
@ -34,7 +34,7 @@ defmodule WildWeb.UserResetPasswordLiveTest do
end
test "renders errors for invalid data", %{conn: conn, token: token} do
{:ok, lv, _html} = live(conn, ~p"/users/reset_password/#{token}")
{:ok, lv, _html} = live(conn, ~p"/account/reset_password/#{token}")
result =
lv
@ -50,7 +50,7 @@ defmodule WildWeb.UserResetPasswordLiveTest do
describe "Reset Password" do
test "resets password once", %{conn: conn, token: token, user: user} do
{:ok, lv, _html} = live(conn, ~p"/users/reset_password/#{token}")
{:ok, lv, _html} = live(conn, ~p"/account/reset_password/#{token}")
{:ok, conn} =
lv
@ -61,7 +61,7 @@ defmodule WildWeb.UserResetPasswordLiveTest do
}
)
|> render_submit()
|> follow_redirect(conn, ~p"/users/log_in")
|> follow_redirect(conn, ~p"/account/log_in")
refute get_session(conn, :user_token)
assert Phoenix.Flash.get(conn.assigns.flash, :info) =~ "Password reset successfully"
@ -69,7 +69,7 @@ defmodule WildWeb.UserResetPasswordLiveTest do
end
test "does not reset password on invalid data", %{conn: conn, token: token} do
{:ok, lv, _html} = live(conn, ~p"/users/reset_password/#{token}")
{:ok, lv, _html} = live(conn, ~p"/account/reset_password/#{token}")
result =
lv
@ -89,13 +89,13 @@ defmodule WildWeb.UserResetPasswordLiveTest do
describe "Reset password navigation" do
test "redirects to login page when the Log in button is clicked", %{conn: conn, token: token} do
{:ok, lv, _html} = live(conn, ~p"/users/reset_password/#{token}")
{:ok, lv, _html} = live(conn, ~p"/account/reset_password/#{token}")
{:ok, conn} =
lv
|> element(~s|main a:fl-contains("Log in")|)
|> render_click()
|> follow_redirect(conn, ~p"/users/log_in")
|> follow_redirect(conn, ~p"/account/log_in")
assert conn.resp_body =~ "Log in"
end
@ -104,13 +104,13 @@ defmodule WildWeb.UserResetPasswordLiveTest do
conn: conn,
token: token
} do
{:ok, lv, _html} = live(conn, ~p"/users/reset_password/#{token}")
{:ok, lv, _html} = live(conn, ~p"/account/reset_password/#{token}")
{:ok, conn} =
lv
|> element(~s|main a:fl-contains("Register")|)
|> render_click()
|> follow_redirect(conn, ~p"/users/register")
|> follow_redirect(conn, ~p"/account/register")
assert conn.resp_body =~ "Register"
end

36
test/wild_web/live/user_settings_live_test.exs
View file

@ -10,17 +10,17 @@ defmodule WildWeb.UserSettingsLiveTest do
{:ok, _lv, html} =
conn
|> log_in_user(user_fixture())
|> live(~p"/users/settings")
|> live(~p"/account/settings")
assert html =~ "Change Email"
assert html =~ "Change Password"
end
test "redirects if user is not logged in", %{conn: conn} do
assert {:error, redirect} = live(conn, ~p"/users/settings")
assert {:error, redirect} = live(conn, ~p"/account/settings")
assert {:redirect, %{to: path, flash: flash}} = redirect
assert path == ~p"/users/log_in"
assert path == ~p"/account/log_in"
assert %{"error" => "You must log in to access this page."} = flash
end
end
@ -35,7 +35,7 @@ defmodule WildWeb.UserSettingsLiveTest do
test "updates the user email", %{conn: conn, password: password, user: user} do
new_email = unique_user_email()
{:ok, lv, _html} = live(conn, ~p"/users/settings")
{:ok, lv, _html} = live(conn, ~p"/account/settings")
result =
lv
@ -50,7 +50,7 @@ defmodule WildWeb.UserSettingsLiveTest do
end
test "renders errors with invalid data (phx-change)", %{conn: conn} do
{:ok, lv, _html} = live(conn, ~p"/users/settings")
{:ok, lv, _html} = live(conn, ~p"/account/settings")
result =
lv
@ -66,7 +66,7 @@ defmodule WildWeb.UserSettingsLiveTest do
end
test "renders errors with invalid data (phx-submit)", %{conn: conn, user: user} do
{:ok, lv, _html} = live(conn, ~p"/users/settings")
{:ok, lv, _html} = live(conn, ~p"/account/settings")
result =
lv
@ -92,7 +92,7 @@ defmodule WildWeb.UserSettingsLiveTest do
test "updates the user password", %{conn: conn, user: user, password: password} do
new_password = valid_user_password()
{:ok, lv, _html} = live(conn, ~p"/users/settings")
{:ok, lv, _html} = live(conn, ~p"/account/settings")
form =
form(lv, "#password_form", %{
@ -108,7 +108,7 @@ defmodule WildWeb.UserSettingsLiveTest do
new_password_conn = follow_trigger_action(form, conn)
assert redirected_to(new_password_conn) == ~p"/users/settings"
assert redirected_to(new_password_conn) == ~p"/account/settings"
assert get_session(new_password_conn, :user_token) != get_session(conn, :user_token)
@ -119,7 +119,7 @@ defmodule WildWeb.UserSettingsLiveTest do
end
test "renders errors with invalid data (phx-change)", %{conn: conn} do
{:ok, lv, _html} = live(conn, ~p"/users/settings")
{:ok, lv, _html} = live(conn, ~p"/account/settings")
result =
lv
@ -138,7 +138,7 @@ defmodule WildWeb.UserSettingsLiveTest do
end
test "renders errors with invalid data (phx-submit)", %{conn: conn} do
{:ok, lv, _html} = live(conn, ~p"/users/settings")
{:ok, lv, _html} = live(conn, ~p"/account/settings")
result =
lv
@ -172,27 +172,27 @@ defmodule WildWeb.UserSettingsLiveTest do
end
test "updates the user email once", %{conn: conn, user: user, token: token, email: email} do
{:error, redirect} = live(conn, ~p"/users/settings/confirm_email/#{token}")
{:error, redirect} = live(conn, ~p"/account/settings/confirm_email/#{token}")
assert {:live_redirect, %{to: path, flash: flash}} = redirect
assert path == ~p"/users/settings"
assert path == ~p"/account/settings"
assert %{"info" => message} = flash
assert message == "Email changed successfully."
refute Accounts.get_user_by_email(user.email)
assert Accounts.get_user_by_email(email)
# use confirm token again
{:error, redirect} = live(conn, ~p"/users/settings/confirm_email/#{token}")
{:error, redirect} = live(conn, ~p"/account/settings/confirm_email/#{token}")
assert {:live_redirect, %{to: path, flash: flash}} = redirect
assert path == ~p"/users/settings"
assert path == ~p"/account/settings"
assert %{"error" => message} = flash
assert message == "Email change link is invalid or it has expired."
end
test "does not update email with invalid token", %{conn: conn, user: user} do
{:error, redirect} = live(conn, ~p"/users/settings/confirm_email/oops")
{:error, redirect} = live(conn, ~p"/account/settings/confirm_email/oops")
assert {:live_redirect, %{to: path, flash: flash}} = redirect
assert path == ~p"/users/settings"
assert path == ~p"/account/settings"
assert %{"error" => message} = flash
assert message == "Email change link is invalid or it has expired."
assert Accounts.get_user_by_email(user.email)
@ -200,9 +200,9 @@ defmodule WildWeb.UserSettingsLiveTest do
test "redirects if user is not logged in", %{token: token} do
conn = build_conn()
{:error, redirect} = live(conn, ~p"/users/settings/confirm_email/#{token}")
{:error, redirect} = live(conn, ~p"/account/settings/confirm_email/#{token}")
assert {:redirect, %{to: path, flash: flash}} = redirect
assert path == ~p"/users/log_in"
assert path == ~p"/account/log_in"
assert %{"error" => message} = flash
assert message == "You must log in to access this page."
end

2
test/wild_web/user_auth_test.exs
View file

@ -231,7 +231,7 @@ defmodule WildWeb.UserAuthTest do
conn = conn |> fetch_flash() |> UserAuth.require_authenticated_user([])
assert conn.halted
assert redirected_to(conn) == ~p"/users/log_in"
assert redirected_to(conn) == ~p"/account/log_in"
assert Phoenix.Flash.get(conn.assigns.flash, :error) ==
"You must log in to access this page."