Fix tests by leveraging Keyword.equal?/2

This commit is contained in:
Mark Felder 2023-11-28 18:54:11 +00:00
parent 8ac4458630
commit e7d6b835ae
3 changed files with 9 additions and 7 deletions

View File

@ -9,14 +9,16 @@ defmodule Pleroma.HealthcheckTest do
test "system_info/0" do test "system_info/0" do
result = Healthcheck.system_info() |> Map.from_struct() result = Healthcheck.system_info() |> Map.from_struct()
assert Map.keys(result) == [ keys = Map.keys(result)
assert Keyword.equal?(keys, [
:active, :active,
:healthy, :healthy,
:idle, :idle,
:job_queue_stats, :job_queue_stats,
:memory_used, :memory_used,
:pool_size :pool_size
] ])
end end
describe "check_health/1" do describe "check_health/1" do

View File

@ -29,13 +29,13 @@ test "change/0 converts auto_linker opts for Pleroma.Formatter", %{migration: mi
%{value: new_opts} = ConfigDB.get_by_params(%{group: :pleroma, key: Pleroma.Formatter}) %{value: new_opts} = ConfigDB.get_by_params(%{group: :pleroma, key: Pleroma.Formatter})
assert new_opts == [ assert Keyword.equal?(new_opts,
class: false, class: false,
extra: true, extra: true,
new_window: false, new_window: false,
rel: "testing", rel: "testing",
strip_prefix: false strip_prefix: false
] )
clear_config(Pleroma.Formatter, new_opts) clear_config(Pleroma.Formatter, new_opts)
assert new_opts == Pleroma.Config.get(Pleroma.Formatter) assert new_opts == Pleroma.Config.get(Pleroma.Formatter)
@ -67,6 +67,6 @@ test "transform_opts/1 returns a list of compatible opts", %{migration: migratio
strip_prefix: false strip_prefix: false
] ]
assert migration.transform_opts(old_opts) == expected_opts assert Keyword.equal?(migration.transform_opts(old_opts), expected_opts)
end end
end end

View File

@ -26,13 +26,13 @@ test "change/0 converts a map into a list", %{migration: migration} do
%{value: new_opts} = ConfigDB.get_by_params(%{group: :pleroma, key: Pleroma.Formatter}) %{value: new_opts} = ConfigDB.get_by_params(%{group: :pleroma, key: Pleroma.Formatter})
assert new_opts == [ assert Keyword.equal?(new_opts,
class: false, class: false,
extra: true, extra: true,
new_window: false, new_window: false,
rel: "F", rel: "F",
strip_prefix: false strip_prefix: false
] )
clear_config(Pleroma.Formatter, new_opts) clear_config(Pleroma.Formatter, new_opts)
assert new_opts == Pleroma.Config.get(Pleroma.Formatter) assert new_opts == Pleroma.Config.get(Pleroma.Formatter)