fix_flaky_transfer_task_test.exs (#237)
authorilja <akkoma.dev@ilja.space>
Tue, 1 Nov 2022 14:31:29 +0000 (14:31 +0000)
committerfloatingghost <hannah@coffee-and-dreams.uk>
Tue, 1 Nov 2022 14:31:29 +0000 (14:31 +0000)
commitbe5044f785e562c4978121b74bce03f131b827e3
tree38c8792c5777584603f7c28bb5b24c48853a7f14
parentf1dfd76b988eeba78404a267a96ae3ad31e25040
fix_flaky_transfer_task_test.exs (#237)

There were async calls happening, so they weren't always finished when assert happened.

I also fixed some bugs in the erratic tests that were introduced when removing :shout.:shout is a key where restart is needed, and was changed in the test to use :rate_limit (which also requires a restart). But there was a bug in the syntax that didn't get caught because the test was tagged as erratic and therefor didn't fail. Here I fixed it.

During compilation, we had a warning `:logger is used by the current application but the current application does not depend on :logger` which is now fixed as well (see commit message for complete stacktrace).

Co-authored-by: Ilja <ilja@ilja.space>
Reviewed-on: https://akkoma.dev/AkkomaGang/akkoma/pulls/237
Co-authored-by: ilja <akkoma.dev@ilja.space>
Co-committed-by: ilja <akkoma.dev@ilja.space>
restarter/lib/pleroma.ex
restarter/mix.exs
test/pleroma/config/transfer_task_test.exs