berrypod/test/berrypod_web/controllers/email_settings_controller_test.exs
jamey db130a7155
All checks were successful
deploy / deploy (push) Successful in 1m19s
rework email settings for true progressive enhancement
Render all adapter field sections in the form with CSS :has(:checked)
controlling visibility. Selecting a provider instantly shows its config
fields — no JS, no page reload, no server round-trip needed.

- Render all 6 adapter configs with data-adapter attribute
- CSS :has(:checked) show/hide rules per adapter in admin stylesheet
- Namespace field names per adapter (email[brevo][api_key] etc)
- Drop 4 transactional-only providers (Resend, Postmark, Mailgun, MailPace)
- Remove noscript "Switch provider" button and controller redirect workaround
- Remove configured_adapter hidden input tracking
- Hide JS-only test email button for no-JS users via noscript style
- LiveView progressively enhances with async save and test email

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-04 23:10:37 +00:00

67 lines
2.0 KiB
Elixir

defmodule BerrypodWeb.EmailSettingsControllerTest do
use BerrypodWeb.ConnCase, async: false
import Berrypod.AccountsFixtures
alias Berrypod.Settings
setup do
original = Application.get_env(:berrypod, Berrypod.Mailer)
Application.put_env(:berrypod, Berrypod.Mailer, adapter: Swoosh.Adapters.Test)
on_exit(fn ->
Application.put_env(:berrypod, Berrypod.Mailer, original)
end)
user = user_fixture()
conn = build_conn() |> log_in_user(user)
%{conn: conn, user: user}
end
describe "POST /admin/settings/email" do
test "saves adapter config and redirects", %{conn: conn} do
conn =
post(conn, ~p"/admin/settings/email", %{
email: %{
adapter: "brevo",
brevo: %{api_key: "xkeysib-abc123def456"}
}
})
assert redirected_to(conn) == ~p"/admin/settings/email"
assert Phoenix.Flash.get(conn.assigns.flash, :info) =~ "Settings saved"
assert Settings.get_setting("email_adapter") == "brevo"
end
test "redirects with error on validation failure", %{conn: conn} do
conn =
post(conn, ~p"/admin/settings/email", %{
email: %{adapter: "brevo", brevo: %{api_key: ""}}
})
assert redirected_to(conn) =~ ~p"/admin/settings/email"
assert Phoenix.Flash.get(conn.assigns.flash, :error) =~ "required"
end
end
describe "POST /admin/settings/email/test" do
test "sends test email and redirects", %{conn: conn} do
Settings.put_setting("email_adapter", "brevo")
Settings.put_secret("email_brevo_api_key", "xkeysib-test-abc")
conn = post(conn, ~p"/admin/settings/email/test")
assert redirected_to(conn) == ~p"/admin/settings/email"
assert Phoenix.Flash.get(conn.assigns.flash, :info) =~ "Test email sent"
end
end
describe "unauthenticated" do
test "redirects to login", %{conn: _conn} do
conn = build_conn()
conn = post(conn, ~p"/admin/settings/email", %{email: %{adapter: "brevo"}})
assert redirected_to(conn) =~ ~p"/users/log-in"
end
end
end