From: lain <lain@soykaf.club>
Date: Mon, 20 Apr 2020 10:37:17 +0000 (+0200)
Subject: Merge branch 'develop' of git.pleroma.social:pleroma/pleroma into remake-remodel-dms
X-Git-Url: https://git.squeep.com/?a=commitdiff_plain;h=139b9d1338618aa0107c188a9a5505c6615768be;p=akkoma

Merge branch 'develop' of git.pleroma.social:pleroma/pleroma into remake-remodel-dms
---

139b9d1338618aa0107c188a9a5505c6615768be
diff --cc lib/pleroma/web/mastodon_api/views/notification_view.ex
index 0b05d178b,4da1ab67f..2a9951831
--- a/lib/pleroma/web/mastodon_api/views/notification_view.ex
+++ b/lib/pleroma/web/mastodon_api/views/notification_view.ex
@@@ -141,9 -122,9 +138,12 @@@ defmodule Pleroma.Web.MastodonAPI.Notif
            |> put_status(parent_activity_fn.(), reading_user, render_opts)
            |> put_emoji(activity)
  
 +        "pleroma:chat_mention" ->
 +          put_chat_message(response, activity, reading_user, render_opts)
 +
+         type when type in ["follow", "follow_request"] ->
+           response
+ 
          _ ->
            nil
        end
diff --cc test/web/activity_pub/side_effects_test.exs
index d3ad4866c,0b6b55156..2889a577c
--- a/test/web/activity_pub/side_effects_test.exs
+++ b/test/web/activity_pub/side_effects_test.exs
@@@ -33,67 -33,10 +34,72 @@@ defmodule Pleroma.Web.ActivityPub.SideE
        assert object.data["like_count"] == 1
        assert user.ap_id in object.data["likes"]
      end
+ 
+     test "creates a notification", %{like: like, poster: poster} do
+       {:ok, like, _} = SideEffects.handle(like)
+       assert Repo.get_by(Notification, user_id: poster.id, activity_id: like.id)
+     end
    end
 +
 +  describe "creation of ChatMessages" do
 +    test "notifies the recipient" do
 +      author = insert(:user, local: false)
 +      recipient = insert(:user, local: true)
 +
 +      {:ok, chat_message_data, _meta} = Builder.chat_message(author, recipient.ap_id, "hey")
 +      {:ok, chat_message_object} = Object.create(chat_message_data)
 +
 +      {:ok, create_activity_data, _meta} =
 +        Builder.create(author, chat_message_object.data["id"], [recipient.ap_id])
 +
 +      {:ok, create_activity, _meta} = ActivityPub.persist(create_activity_data, local: false)
 +
 +      {:ok, _create_activity, _meta} = SideEffects.handle(create_activity)
 +
 +      assert Repo.get_by(Notification, user_id: recipient.id, activity_id: create_activity.id)
 +    end
 +
 +    test "it creates a Chat for the local users and bumps the unread count" do
 +      author = insert(:user, local: false)
 +      recipient = insert(:user, local: true)
 +
 +      {:ok, chat_message_data, _meta} = Builder.chat_message(author, recipient.ap_id, "hey")
 +      {:ok, chat_message_object} = Object.create(chat_message_data)
 +
 +      {:ok, create_activity_data, _meta} =
 +        Builder.create(author, chat_message_object.data["id"], [recipient.ap_id])
 +
 +      {:ok, create_activity, _meta} = ActivityPub.persist(create_activity_data, local: false)
 +
 +      {:ok, _create_activity, _meta} = SideEffects.handle(create_activity)
 +
 +      # The remote user won't get a chat
 +      chat = Chat.get(author.id, recipient.ap_id)
 +      refute chat
 +
 +      # The local user will get a chat
 +      chat = Chat.get(recipient.id, author.ap_id)
 +      assert chat
 +
 +      author = insert(:user, local: true)
 +      recipient = insert(:user, local: true)
 +
 +      {:ok, chat_message_data, _meta} = Builder.chat_message(author, recipient.ap_id, "hey")
 +      {:ok, chat_message_object} = Object.create(chat_message_data)
 +
 +      {:ok, create_activity_data, _meta} =
 +        Builder.create(author, chat_message_object.data["id"], [recipient.ap_id])
 +
 +      {:ok, create_activity, _meta} = ActivityPub.persist(create_activity_data, local: false)
 +
 +      {:ok, _create_activity, _meta} = SideEffects.handle(create_activity)
 +
 +      # Both users are local and get the chat
 +      chat = Chat.get(author.id, recipient.ap_id)
 +      assert chat
 +
 +      chat = Chat.get(recipient.id, author.ap_id)
 +      assert chat
 +    end
 +  end
  end