hj/pleroma
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge branch 'bugfix/repeated-follow-unfollow' into 'develop'

Browse source 
Add error messages for repeated follows and unfollows

See merge request !7
This commit is contained in:
lambadalambda 2017-04-17 05:46:59 -04:00
commit ac9817bdb0
4 changed files with 56 additions and 28 deletions
Download patch file Download diff file Expand all files Collapse all files

17
test/web/twitter_api/twitter_api_test.exs
View file

@ -114,26 +114,31 @@ defmodule Pleroma.Web.TwitterAPI.TwitterAPITest do
test "Follow another user" do
user = insert(:user)
following = insert(:user)
followed = insert(:user)
{:ok, user, following, activity } = TwitterAPI.follow(user, following.id)
{ :ok, user, followed, activity } = TwitterAPI.follow(user, followed.id)
user = Repo.get(User, user.id)
follow = Repo.get(Activity, activity.id)
assert user.following == [User.ap_followers(following)]
assert user.following == [User.ap_followers(followed)]
assert follow == activity
{ :error, msg } = TwitterAPI.follow(user, followed.id)
assert msg == "Could not follow user: #{followed.nickname} is already on your list."
end
test "Unfollow another user" do
following = insert(:user)
user = insert(:user, %{following: [User.ap_followers(following)]})
followed = insert(:user)
user = insert(:user, %{following: [User.ap_followers(followed)]})
{:ok, user, _following } = TwitterAPI.unfollow(user, following.id)
{ :ok, user, _followed } = TwitterAPI.unfollow(user, followed.id)
user = Repo.get(User, user.id)
assert user.following == []
{ :error, msg } = TwitterAPI.unfollow(user, followed.id)
assert msg == "Not subscribed!"
end
test "fetch statuses in a context using the conversation id" do