Anon/pleroma
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki 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
parent ea0ec5fbcf
commit 603e9f6a92
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/pleroma/object/fetcher.ex
View File

@ -75,8 +75,9 @@ defmodule Pleroma.Object.Fetcher do
{:allowed_depth, false} -> {:allowed_depth, false} ->
{:error, "Max thread distance exceeded."} {:error, "Max thread distance exceeded."}
{:containment, _} -> {:containment, e} ->
{:error, "Object containment failed."} 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)}")