fix: linting + tests
This commit is contained in:
parent
bc4bcd0089
commit
d9b659e5ea
4 changed files with 12 additions and 5 deletions
|
|
@ -375,7 +375,9 @@ defmodule MvWeb.MemberLive.Form do
|
||||||
[error | _] ->
|
[error | _] ->
|
||||||
# Try to extract message from other error types
|
# Try to extract message from other error types
|
||||||
case error do
|
case error do
|
||||||
%{message: message} when is_binary(message) -> message
|
%{message: message} when is_binary(message) ->
|
||||||
|
message
|
||||||
|
|
||||||
error when is_struct(error) ->
|
error when is_struct(error) ->
|
||||||
# Try to use Ash.ErrorKind protocol if available
|
# Try to use Ash.ErrorKind protocol if available
|
||||||
try do
|
try do
|
||||||
|
|
@ -383,7 +385,9 @@ defmodule MvWeb.MemberLive.Form do
|
||||||
rescue
|
rescue
|
||||||
Protocol.UndefinedError -> gettext("Failed to save member. Please try again.")
|
Protocol.UndefinedError -> gettext("Failed to save member. Please try again.")
|
||||||
end
|
end
|
||||||
_ -> gettext("Failed to save member. Please try again.")
|
|
||||||
|
_ ->
|
||||||
|
gettext("Failed to save member. Please try again.")
|
||||||
end
|
end
|
||||||
|
|
||||||
_ ->
|
_ ->
|
||||||
|
|
|
||||||
|
|
@ -166,7 +166,7 @@ case Accounts.User
|
||||||
# User already exists (e.g., via OIDC) - assign admin role
|
# User already exists (e.g., via OIDC) - assign admin role
|
||||||
existing_admin_user
|
existing_admin_user
|
||||||
|> Ash.Changeset.for_update(:update, %{})
|
|> Ash.Changeset.for_update(:update, %{})
|
||||||
|> Ash.Changeset.manage_relationship(:role, admin_role, type: :replace)
|
|> Ash.Changeset.manage_relationship(:role, admin_role, type: :append_and_remove)
|
||||||
|> Ash.update!()
|
|> Ash.update!()
|
||||||
|
|
||||||
{:ok, nil} ->
|
{:ok, nil} ->
|
||||||
|
|
@ -177,7 +177,7 @@ case Accounts.User
|
||||||
|> then(fn user ->
|
|> then(fn user ->
|
||||||
user
|
user
|
||||||
|> Ash.Changeset.for_update(:update, %{})
|
|> Ash.Changeset.for_update(:update, %{})
|
||||||
|> Ash.Changeset.manage_relationship(:role, admin_role, type: :replace)
|
|> Ash.Changeset.manage_relationship(:role, admin_role, type: :append_and_remove)
|
||||||
|> Ash.update!()
|
|> Ash.update!()
|
||||||
end)
|
end)
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -39,6 +39,7 @@ defmodule MvWeb.MemberLive.FormErrorHandlingTest do
|
||||||
|
|
||||||
# Should show flash error message
|
# Should show flash error message
|
||||||
assert has_element?(view, "#flash-group")
|
assert has_element?(view, "#flash-group")
|
||||||
|
|
||||||
assert html =~ "error" or html =~ "Error" or html =~ "Fehler" or
|
assert html =~ "error" or html =~ "Error" or html =~ "Fehler" or
|
||||||
html =~ "failed" or html =~ "fehlgeschlagen" or
|
html =~ "failed" or html =~ "fehlgeschlagen" or
|
||||||
html =~ "Validation failed" or html =~ "Validierung fehlgeschlagen"
|
html =~ "Validation failed" or html =~ "Validierung fehlgeschlagen"
|
||||||
|
|
@ -64,6 +65,7 @@ defmodule MvWeb.MemberLive.FormErrorHandlingTest do
|
||||||
|
|
||||||
# Should show flash error message
|
# Should show flash error message
|
||||||
assert has_element?(view, "#flash-group")
|
assert has_element?(view, "#flash-group")
|
||||||
|
|
||||||
assert html =~ "error" or html =~ "Error" or html =~ "Fehler" or
|
assert html =~ "error" or html =~ "Error" or html =~ "Fehler" or
|
||||||
html =~ "failed" or html =~ "fehlgeschlagen" or
|
html =~ "failed" or html =~ "fehlgeschlagen" or
|
||||||
html =~ "Validation failed" or html =~ "Validierung fehlgeschlagen" or
|
html =~ "Validation failed" or html =~ "Validierung fehlgeschlagen" or
|
||||||
|
|
@ -108,6 +110,7 @@ defmodule MvWeb.MemberLive.FormErrorHandlingTest do
|
||||||
|
|
||||||
# Should show flash error message
|
# Should show flash error message
|
||||||
assert has_element?(view, "#flash-group")
|
assert has_element?(view, "#flash-group")
|
||||||
|
|
||||||
assert html =~ "error" or html =~ "Error" or html =~ "Fehler" or
|
assert html =~ "error" or html =~ "Error" or html =~ "Fehler" or
|
||||||
html =~ "failed" or html =~ "fehlgeschlagen" or
|
html =~ "failed" or html =~ "fehlgeschlagen" or
|
||||||
html =~ "Validation failed" or html =~ "Validierung fehlgeschlagen"
|
html =~ "Validation failed" or html =~ "Validierung fehlgeschlagen"
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue