diff options
author | kaniini <ariadne@dereferenced.org> | 2019-09-27 03:51:24 +0000 |
---|---|---|
committer | kaniini <ariadne@dereferenced.org> | 2019-09-27 03:51:24 +0000 |
commit | eb9aa7aa1095de150d036839c78c402019efb4b1 (patch) | |
tree | 37c808da6ae7007d203e0de6c917d093a4d6abae /test/notification_test.exs | |
parent | c4fbb56984d8f86df948cfd9b0f7c081d688c365 (diff) | |
parent | b736312123bd11e8ba87b8b39245c0f441ebd7fb (diff) | |
download | pleroma-eb9aa7aa1095de150d036839c78c402019efb4b1.tar.gz pleroma-eb9aa7aa1095de150d036839c78c402019efb4b1.zip |
Merge branch 'refactor/subscription' into 'develop'
Refactor subscription functionality
Closes #1130
See merge request pleroma/pleroma!1664
Diffstat (limited to 'test/notification_test.exs')
-rw-r--r-- | test/notification_test.exs | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/test/notification_test.exs b/test/notification_test.exs index 54c0f9877..1dbad34c1 100644 --- a/test/notification_test.exs +++ b/test/notification_test.exs @@ -33,16 +33,16 @@ defmodule Pleroma.NotificationTest do assert other_notification.activity_id == activity.id end - test "it creates a notification for subscribed users" do + test "it does not create a notification for subscribed users" do user = insert(:user) subscriber = insert(:user) User.subscribe(subscriber, user) {:ok, status} = CommonAPI.post(user, %{"status" => "Akariiiin"}) - {:ok, [notification]} = Notification.create_notifications(status) + {:ok, notifications} = Notification.create_notifications(status) - assert notification.user_id == subscriber.id + assert notifications == [] end test "does not create a notification for subscribed users if status is a reply" do @@ -182,14 +182,16 @@ defmodule Pleroma.NotificationTest do refute Notification.create_notification(activity_dupe, followed_user) end - test "it doesn't create duplicate notifications for follow+subscribed users" do + test "it doesn't create notifications for follow+subscribed users" do user = insert(:user) subscriber = insert(:user) {:ok, _, _, _} = CommonAPI.follow(subscriber, user) User.subscribe(subscriber, user) {:ok, status} = CommonAPI.post(user, %{"status" => "Akariiiin"}) - {:ok, [_notif]} = Notification.create_notifications(status) + {:ok, notifications} = Notification.create_notifications(status) + + assert notifications == [] end test "it doesn't create subscription notifications if the recipient cannot see the status" do |