limepotato/akkoma
1
0
Fork 0
Code Pull requests Activity

Fix Transmogrifier tests

Browse source 
These tests relied on the removed Fetcher.fetch_object_from_id!/2 function injecting the error tuple into a log message with the exact words "Object containment failed."

We will keep this behavior by generating a similar log message, but perhaps this should do a better job of matching on the error tuple returned by Transmogrifier.handle_incoming/1
This commit is contained in:
Mark Felder 2023-12-26 14:22:04 -05:00 committed by Floatingghost
parent c241b5b09f
commit ac4cc619ea
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/pleroma/object/fetcher.ex
View file

@ -157,6 +157,10 @@ defmodule Pleroma.Object.Fetcher do
{:scheme, false} -> {:scheme, false} ->
{:error, "URI Scheme Invalid"} {:error, "URI Scheme Invalid"}
{:containment, e} ->
Logger.info("Error while fetching #{id}: Object containment failed. #{inspect(e)}")
{:error, e}
{:transmogrifier, {:error, {:reject, e}}} -> {:transmogrifier, {:error, {:reject, e}}} ->
Logger.info("Rejected #{id} while fetching: #{inspect(e)}") Logger.info("Rejected #{id} while fetching: #{inspect(e)}")
{:reject, e} {:reject, e}