fix tests

This commit is contained in:
Maksim Pechnikov 2020-06-23 09:08:24 +03:00
parent 2715c40e1d
commit 84aa9c78dd

View File

@ -7,6 +7,8 @@ defmodule Pleroma.ApplicationRequirementsTest do
import ExUnit.CaptureLog import ExUnit.CaptureLog
import Mock import Mock
alias Pleroma.Repo
describe "check_rum!" do describe "check_rum!" do
setup_with_mocks([ setup_with_mocks([
{Pleroma.ApplicationRequirements, [:passthrough], {Pleroma.ApplicationRequirements, [:passthrough],
@ -20,23 +22,19 @@ defmodule Pleroma.ApplicationRequirementsTest do
test "raises if rum is enabled and detects unapplied rum migrations" do test "raises if rum is enabled and detects unapplied rum migrations" do
Pleroma.Config.put([:database, :rum_enabled], true) Pleroma.Config.put([:database, :rum_enabled], true)
with_mocks([{Repo, [:passthrough], [exists?: fn _, _ -> false end]}]) do
assert_raise Pleroma.ApplicationRequirements.VerifyError, assert_raise Pleroma.ApplicationRequirements.VerifyError,
"Unapplied RUM Migrations detected", "Unapplied RUM Migrations detected",
fn -> fn ->
capture_log(&Pleroma.ApplicationRequirements.verify!/0) capture_log(&Pleroma.ApplicationRequirements.verify!/0)
end end
end end
end
test "raises if rum is disabled and detects rum migrations" do test "raises if rum is disabled and detects rum migrations" do
Pleroma.Config.put([:database, :rum_enabled], false) Pleroma.Config.put([:database, :rum_enabled], false)
with_mocks([ with_mocks([{Repo, [:passthrough], [exists?: fn _, _ -> true end]}]) do
{
Pleroma.Repo,
[:passthrough],
[exists?: fn _, _ -> true end]
}
]) do
assert_raise Pleroma.ApplicationRequirements.VerifyError, assert_raise Pleroma.ApplicationRequirements.VerifyError,
"RUM Migrations detected", "RUM Migrations detected",
fn -> fn ->
@ -48,29 +46,26 @@ defmodule Pleroma.ApplicationRequirementsTest do
test "doesn't do anything if rum enabled and applied migrations" do test "doesn't do anything if rum enabled and applied migrations" do
Pleroma.Config.put([:database, :rum_enabled], true) Pleroma.Config.put([:database, :rum_enabled], true)
with_mocks([ with_mocks([{Repo, [:passthrough], [exists?: fn _, _ -> true end]}]) do
{
Pleroma.Repo,
[:passthrough],
[exists?: fn _, _ -> true end]
}
]) do
assert Pleroma.ApplicationRequirements.verify!() == :ok assert Pleroma.ApplicationRequirements.verify!() == :ok
end end
end end
test "doesn't do anything if rum disabled" do test "doesn't do anything if rum disabled" do
Pleroma.Config.put([:database, :rum_enabled], false) Pleroma.Config.put([:database, :rum_enabled], false)
with_mocks([{Repo, [:passthrough], [exists?: fn _, _ -> false end]}]) do
assert Pleroma.ApplicationRequirements.verify!() == :ok assert Pleroma.ApplicationRequirements.verify!() == :ok
end end
end end
end
describe "check_migrations_applied!" do describe "check_migrations_applied!" do
setup_with_mocks([ setup_with_mocks([
{Ecto.Migrator, [], {Ecto.Migrator, [],
[ [
with_repo: fn repo, fun -> passthrough([repo, fun]) end, with_repo: fn repo, fun -> passthrough([repo, fun]) end,
migrations: fn Pleroma.Repo -> migrations: fn Repo ->
[ [
{:up, 20_191_128_153_944, "fix_missing_following_count"}, {:up, 20_191_128_153_944, "fix_missing_following_count"},
{:up, 20_191_203_043_610, "create_report_notes"}, {:up, 20_191_203_043_610, "create_report_notes"},