use proper workers for fetching pins instead of an ad-hoc task #788

Merged
floatingghost merged 8 commits from post-fetching-when-invalid-user into develop 2024-05-31 08:58:52 +00:00
2 changed files with 0 additions and 36 deletions
Showing only changes of commit b68ca47fd8 - Show all commits

View file

@ -399,31 +399,6 @@ test "does NOT schedule background fetching of `replies` beyond max thread depth
assert all_enqueued(worker: Pleroma.Workers.RemoteFetcherWorker) == []
end
test "it does not explode if it cannot process the user behind a post" do
# this will break the nickname ascii check
user_ap_data = "test/fixtures/users_mock/user.json"
|> File.read!()
|> String.replace("{{nickname}}", "あっこ")
|> Jason.decode!()
|> Map.delete("featured")
user_ap_id = user_ap_data["id"]
Tesla.Mock.mock_global(fn %{url: ^user_ap_id} ->
%Tesla.Env{status: 200, body: Jason.encode!(user_ap_data),
headers: HttpRequestMock.activitypub_object_headers()}
end)
data =
File.read!("test/fixtures/mastodon-post-activity.json")
|> Jason.decode!()
|> Map.put("actor", user_ap_id)
|> Map.put("to", ["https://www.w3.org/ns/activitystreams#Public"])
|> Map.put("cc", [])
assert {:error, :not_found} = Transmogrifier.handle_incoming(data)
end
end
describe "`handle_incoming/2`, Pleroma format `replies` handling" do

View file

@ -146,17 +146,6 @@ test "it accepts quote posts" do
# It fetched the quoted post
assert Object.normalize("https://misskey.io/notes/8vs6wxufd0")
end
test "it does not explode if it cannot process the user behind a post" do
# this will break the nickname ascii check
user_ap_data = "test/fixtures/users_mock/user.json"
|> File.read!()
|> String.replace("{{nickname}}", "あっこ")
Tesla.Mock.mock_global(fn %{url: "https://example.com/users/あっこ"} ->
%Tesla.Env{status: 200, body: user_ap_data}
end)
end
end
describe "prepare outgoing" do