Add limiting to activity pub fetching.

lib/pleroma/web/activity_pub/activity_pub.ex

@@ -9,8 +9,11 @@ def insert(map) when is_map(map) do
 
   def fetch_public_activities do
     query = from activity in Activity,
-    where: fragment(~s(? @> '{"to": ["https://www.w3.org/ns/activitystreams#Public"]}'), activity.data)
+      where: fragment(~s(? @> '{"to": ["https://www.w3.org/ns/activitystreams#Public"]}'), activity.data),
+      limit: 20,
+      order_by: [desc: :inserted_at]
 
     Repo.all(query)
+    |> Enum.reverse
   end
 end

test/support/builders/activity_builder.ex

@@ -1,11 +1,10 @@
 defmodule Pleroma.Builders.ActivityBuilder do
   alias Pleroma.Builders.UserBuilder
   alias Pleroma.Web.ActivityPub.ActivityPub
-  alias Pleroma.User
 
-  def public_and_non_public do
+  def build(data \\ %{}, opts \\ %{}) do
-    {:ok, user} = UserBuilder.insert
+    user = opts[:user] || UserBuilder.build
-    public = %{
+    activity = %{
       "id" => 1,
       "actor" => user.ap_id,
       "to" => ["https://www.w3.org/ns/activitystreams#Public"],
@@ -14,16 +13,26 @@ def public_and_non_public do
         "content" => "test"
       }
     }
+    Map.merge(activity, data)
+  end
 
-    non_public = %{
+  def insert(data \\ %{}, opts \\ %{}) do
-      "id" => 2,
+    activity = build(data, opts)
-      "actor" => user.ap_id,
+    ActivityPub.insert(activity)
-      "to" => [],
+  end
-      "object" => %{
+
-        "type" => "Note",
+  def insert_list(times, data \\ %{}, opts \\ %{}) do
-        "content" => "test"
+    Enum.map(1..times, fn (n) ->
-      }
+      {:ok, activity} = insert(%{"id" => n})
-    }
+      activity
+    end)
+  end
+
+  def public_and_non_public do
+    {:ok, user} = UserBuilder.insert
+
+    public = build(%{"id" => 1}, %{user: user})
+    non_public = build(%{"id" => 2, "to" => []}, %{user: user})
 
     {:ok, public} = ActivityPub.insert(public)
     {:ok, non_public} = ActivityPub.insert(non_public)

test/web/activity_pub/activity_pub_test.exs

@@ -15,13 +15,24 @@ test "inserts a given map into the activity database" do
     end
   end
 
-  describe "fetch activities" do
+  describe "public fetch activities" do
-    test "retrieves all public activities" do
+    test "retrieves public activities" do
       %{public: public} = ActivityBuilder.public_and_non_public
 
       activities = ActivityPub.fetch_public_activities
       assert length(activities) == 1
       assert Enum.at(activities, 0) == public
     end
+
+    test "retrieves a maximum of 20 activities" do
+      activities = ActivityBuilder.insert_list(30)
+      last_expected = List.last(activities)
+
+      activities = ActivityPub.fetch_public_activities
+      last = List.last(activities)
+
+      assert length(activities) == 20
+      assert last == last_expected
+    end
   end
 end