pleroma

commit: e38f5f1a817d6da30e9a128ec74a2a7c78faf174
parent 5205e846eb5cfbd0adfa5031ad75e96fccbc86d8
Author: Mark Felder <feld@feld.me>
Date:   Fri, 30 Aug 2024 09:35:04 -0400

Add recognized activity types to a constant and use it in the test

Diffstat:
M
lib/pleroma/constants.ex
18
++++++++++++++++++
M
test/pleroma/web/activity_pub/activity_pub_controller_test.exs
4
+++-

2 files changed, 21 insertions(+), 1 deletion(-)
diff --git a/lib/pleroma/constants.ex b/lib/pleroma/constants.ex
@@ -85,6 +85,24 @@ defmodule Pleroma.Constants do
     ]
   )
 
+  const(activity_types,
+    do: [
+      "Create",
+      "Update",
+      "Delete",
+      "Follow",
+      "Accept",
+      "Reject",
+      "Add",
+      "Remove",
+      "Like",
+      "Announce",
+      "Undo",
+      "Flag",
+      "EmojiReact"
+    ]
+  )
+
   const(allowed_activity_types_from_strangers,
     do: [
       "Block",
diff --git a/test/pleroma/web/activity_pub/activity_pub_controller_test.exs b/test/pleroma/web/activity_pub/activity_pub_controller_test.exs
@@ -689,7 +689,9 @@ defmodule Pleroma.Web.ActivityPub.ActivityPubControllerTest do
     # Once we begin processing it through Oban we risk fetching the actor to validate the
     # activity which just leads to inserting a new user to process a Delete not relevant to us.
     test "Activities of certain types from an unknown actor are discarded", %{conn: conn} do
-      example_bad_types = ["Announce", "Delete", "Undo"]
+      example_bad_types =
+        Pleroma.Constants.activity_types() --
+          Pleroma.Constants.allowed_activity_types_from_strangers()
 
       Enum.each(example_bad_types, fn bad_type ->
         params =