Only return posts in TwAPI conversations.
authorRoger Braun <roger@rogerbraun.net>
Fri, 30 Jun 2017 14:41:09 +0000 (16:41 +0200)
committerRoger Braun <roger@rogerbraun.net>
Fri, 30 Jun 2017 14:41:09 +0000 (16:41 +0200)
lib/pleroma/web/activity_pub/activity_pub.ex
test/web/activity_pub/activity_pub_test.exs
test/web/twitter_api/twitter_api_controller_test.exs
test/web/twitter_api/twitter_api_test.exs

index 25560f4c96f5e7bc071a5011eb1d6eeb78a1fa57..79c1d5b6951aedb79b53d6fa8448f20fd367e147 100644 (file)
@@ -77,7 +77,7 @@ defmodule Pleroma.Web.ActivityPub.ActivityPub do
 
   def fetch_activities_for_context(context) do
     query = from activity in Activity,
-      where: fragment("? @> ?", activity.data, ^%{ context: context }),
+      where: fragment("? @> ?", activity.data, ^%{ type: "Create", context: context }),
       order_by: [desc: :inserted_at]
     Repo.all(query)
   end
index 0dfaaa99cd0963c88637128afe715cf0f6304704..45ead0993e8e968b5ba445b723ad609842d2cf0b 100644 (file)
@@ -69,9 +69,10 @@ defmodule Pleroma.Web.ActivityPub.ActivityPubTest do
 
   describe "fetch activities in context" do
     test "retrieves activities that have a given context" do
-      {:ok, activity} = ActivityBuilder.insert(%{"context" => "2hu"})
-      {:ok, activity_two} = ActivityBuilder.insert(%{"context" => "2hu"})
-      {:ok, _activity_three} = ActivityBuilder.insert(%{"context" => "3hu"})
+      {:ok, activity} = ActivityBuilder.insert(%{"type" => "Create", "context" => "2hu"})
+      {:ok, activity_two} = ActivityBuilder.insert(%{"type" => "Create", "context" => "2hu"})
+      {:ok, _activity_three} = ActivityBuilder.insert(%{"type" => "Create", "context" => "3hu"})
+      {:ok, _activity_four} = ActivityBuilder.insert(%{"type" => "Announce", "context" => "2hu"})
 
       activities = ActivityPub.fetch_activities_for_context("2hu")
 
index 5ccced73941a40314e2ed3532fddc002fa3ad7f7..4450bd5e521eab723c3e9ee1aa27a9a9ecf52f68 100644 (file)
@@ -85,9 +85,9 @@ defmodule Pleroma.Web.TwitterAPI.ControllerTest do
   describe "GET /statusnet/conversation/:id.json" do
     test "returns the statuses in the conversation", %{conn: conn} do
       {:ok, _user} = UserBuilder.insert
-      {:ok, _activity} = ActivityBuilder.insert(%{"context" => "2hu"})
-      {:ok, _activity_two} = ActivityBuilder.insert(%{"context" => "2hu"})
-      {:ok, _activity_three} = ActivityBuilder.insert(%{"context" => "3hu"})
+      {:ok, _activity} = ActivityBuilder.insert(%{"type" => "Create", "context" => "2hu"})
+      {:ok, _activity_two} = ActivityBuilder.insert(%{"type" => "Create", "context" => "2hu"})
+      {:ok, _activity_three} = ActivityBuilder.insert(%{"type" => "Create", "context" => "3hu"})
 
       {:ok, object} = Object.context_mapping("2hu") |> Repo.insert
       conn = conn
index f591bef4a02ad6f1cbb772d25095208e2047e4ff..080e5f86ae4ecfada5de1731cdfaf252bfd26bee 100644 (file)
@@ -218,9 +218,9 @@ defmodule Pleroma.Web.TwitterAPI.TwitterAPITest do
 
   test "fetch statuses in a context using the conversation id" do
     {:ok, user} = UserBuilder.insert()
-    {:ok, activity} = ActivityBuilder.insert(%{"context" => "2hu"})
-    {:ok, activity_two} = ActivityBuilder.insert(%{"context" => "2hu"})
-    {:ok, _activity_three} = ActivityBuilder.insert(%{"context" => "3hu"})
+    {:ok, activity} = ActivityBuilder.insert(%{"type" => "Create", "context" => "2hu"})
+    {:ok, activity_two} = ActivityBuilder.insert(%{"type" => "Create", "context" => "2hu"})
+    {:ok, _activity_three} = ActivityBuilder.insert(%{"type" => "Create", "context" => "3hu"})
 
     {:ok, object} = Object.context_mapping("2hu") |> Repo.insert