Fawksie/akkoma
1
0
Fork 0
forked from AkkomaGang/akkoma
Code Issues Pull requests Projects Releases Packages Wiki Activity

Hook up block/unblock to TwitterAPI

Browse source
This commit is contained in:
Francis Dinh 2018-05-22 05:41:17 -04:00
parent 55f9aefd08
commit 72b93d13f8
3 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
lib/pleroma/web/twitter_api/twitter_api.ex
View file

@ -45,7 +45,8 @@ def unfollow(%User{} = follower, params) do
def block(%User{} = blocker, params) do def block(%User{} = blocker, params) do
with {:ok, %User{} = blocked} <- get_user(params), with {:ok, %User{} = blocked} <- get_user(params),
{:ok, blocker} <- User.block(blocker, blocked) do {:ok, blocker} <- User.block(blocker, blocked),
{:ok, _activity} <- ActivityPub.block(blocker, blocked) do
{:ok, blocker, blocked} {:ok, blocker, blocked}
else else
err -> err err -> err
@ -54,7 +55,8 @@ def block(%User{} = blocker, params) do
def unblock(%User{} = blocker, params) do def unblock(%User{} = blocker, params) do
with {:ok, %User{} = blocked} <- get_user(params), with {:ok, %User{} = blocked} <- get_user(params),
{:ok, blocker} <- User.unblock(blocker, blocked) do {:ok, blocker} <- User.unblock(blocker, blocked),
{:ok, _activity} <- ActivityPub.unblock(blocker, blocked) do
{:ok, blocker, blocked} {:ok, blocker, blocked}
else else
err -> err err -> err

2
test/web/twitter_api/twitter_api_controller_test.exs
View file

@ -443,7 +443,7 @@ test "without valid credentials", %{conn: conn} do
test "with credentials", %{conn: conn, user: current_user} do test "with credentials", %{conn: conn, user: current_user} do
blocked = insert(:user) blocked = insert(:user)
{:ok, current_user} = User.block(current_user, blocked) {:ok, current_user, blocked} = TwitterAPI.block(current_user, %{"user_id" => blocked.id})
assert User.blocks?(current_user, blocked) assert User.blocks?(current_user, blocked)
conn = conn =

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

@ -166,7 +166,7 @@ test "Block another user using screen_name" do
test "Unblock another user using user_id" do test "Unblock another user using user_id" do
unblocked = insert(:user) unblocked = insert(:user)
user = insert(:user) user = insert(:user)
User.block(user, unblocked) {:ok, user, _unblocked} = TwitterAPI.block(user, %{"user_id" => unblocked.id})
{:ok, user, _unblocked} = TwitterAPI.unblock(user, %{"user_id" => unblocked.id}) {:ok, user, _unblocked} = TwitterAPI.unblock(user, %{"user_id" => unblocked.id})
assert user.info["blocks"] == [] assert user.info["blocks"] == []
@ -175,7 +175,7 @@ test "Unblock another user using user_id" do
test "Unblock another user using screen_name" do test "Unblock another user using screen_name" do
unblocked = insert(:user) unblocked = insert(:user)
user = insert(:user) user = insert(:user)
User.block(user, unblocked) {:ok, user, _unblocked} = TwitterAPI.block(user, %{"screen_name" => unblocked.nickname})
{:ok, user, _unblocked} = TwitterAPI.unblock(user, %{"screen_name" => unblocked.nickname}) {:ok, user, _unblocked} = TwitterAPI.unblock(user, %{"screen_name" => unblocked.nickname})
assert user.info["blocks"] == [] assert user.info["blocks"] == []