ApplicationRequirements: Add test, more text for pool size.

This commit is contained in:
lain 2020-12-08 17:30:10 +01:00
parent 5b9b7b4888
commit 50d16a9e27
2 changed files with 27 additions and 2 deletions

View file

@ -194,9 +194,15 @@ defp check_repo_pool_size!(:ok) do
not Pleroma.Config.get([:dangerzone, :override_repo_pool_size], false) do not Pleroma.Config.get([:dangerzone, :override_repo_pool_size], false) do
Logger.error(""" Logger.error("""
!!!CONFIG WARNING!!! !!!CONFIG WARNING!!!
The database pool size has been altered from the recommended value of 10.\n
Please revert or ensure your database is tuned appropriately and then set\n The database pool size has been altered from the recommended value of 10.
Please revert or ensure your database is tuned appropriately and then set
`config :pleroma, :dangerzone, override_repo_pool_size: true`. `config :pleroma, :dangerzone, override_repo_pool_size: true`.
If you are experiencing database timeouts, please check the "Optimizing
your PostgreSQL performance" section in the documentation. If you still
encounter issues after that, please open an issue on the tracker.
""") """)
{:error, "Repo.pool_size different than recommended value."} {:error, "Repo.pool_size different than recommended value."}

View file

@ -12,6 +12,25 @@ defmodule Pleroma.ApplicationRequirementsTest do
alias Pleroma.Config alias Pleroma.Config
alias Pleroma.Repo alias Pleroma.Repo
describe "check_repo_pool_size!/1" do
test "raises if the pool size is unexpected" do
clear_config([Pleroma.Repo, :pool_size], 11)
assert_raise Pleroma.ApplicationRequirements.VerifyError,
"Repo.pool_size different than recommended value.",
fn ->
capture_log(&Pleroma.ApplicationRequirements.verify!/0)
end
end
test "doesn't raise if the pool size is unexpected but the respective flag is set" do
clear_config([Pleroma.Repo, :pool_size], 11)
clear_config([:dangerzone, :override_repo_pool_size], true)
assert Pleroma.ApplicationRequirements.verify!() == :ok
end
end
describe "check_welcome_message_config!/1" do describe "check_welcome_message_config!/1" do
setup do: clear_config([:welcome]) setup do: clear_config([:welcome])
setup do: clear_config([Pleroma.Emails.Mailer]) setup do: clear_config([Pleroma.Emails.Mailer])