diff options
Diffstat (limited to 'test/tasks')
| -rw-r--r-- | test/tasks/count_statuses_test.exs | 16 | ||||
| -rw-r--r-- | test/tasks/database_test.exs | 16 | ||||
| -rw-r--r-- | test/tasks/relay_test.exs | 24 | ||||
| -rw-r--r-- | test/tasks/user_test.exs | 43 | 
4 files changed, 49 insertions, 50 deletions
| diff --git a/test/tasks/count_statuses_test.exs b/test/tasks/count_statuses_test.exs index 6035da3c3..bb5dc88f8 100644 --- a/test/tasks/count_statuses_test.exs +++ b/test/tasks/count_statuses_test.exs @@ -22,18 +22,18 @@ defmodule Mix.Tasks.Pleroma.CountStatusesTest do      user = refresh_record(user)      user2 = refresh_record(user2) -    assert %{info: %{note_count: 2}} = user -    assert %{info: %{note_count: 1}} = user2 +    assert %{note_count: 2} = user +    assert %{note_count: 1} = user2 -    {:ok, user} = User.update_info(user, &User.Info.set_note_count(&1, 0)) -    {:ok, user2} = User.update_info(user2, &User.Info.set_note_count(&1, 0)) +    {:ok, user} = User.update_note_count(user, 0) +    {:ok, user2} = User.update_note_count(user2, 0) -    assert %{info: %{note_count: 0}} = user -    assert %{info: %{note_count: 0}} = user2 +    assert %{note_count: 0} = user +    assert %{note_count: 0} = user2      assert capture_io(fn -> Mix.Tasks.Pleroma.CountStatuses.run([]) end) == "Done\n" -    assert %{info: %{note_count: 2}} = refresh_record(user) -    assert %{info: %{note_count: 1}} = refresh_record(user2) +    assert %{note_count: 2} = refresh_record(user) +    assert %{note_count: 1} = refresh_record(user2)    end  end diff --git a/test/tasks/database_test.exs b/test/tasks/database_test.exs index b63dcac00..0c7883f33 100644 --- a/test/tasks/database_test.exs +++ b/test/tasks/database_test.exs @@ -72,26 +72,26 @@ defmodule Mix.Tasks.Pleroma.DatabaseTest do    describe "running update_users_following_followers_counts" do      test "following and followers count are updated" do        [user, user2] = insert_pair(:user) -      {:ok, %User{following: following, info: info} = user} = User.follow(user, user2) +      {:ok, %User{} = user} = User.follow(user, user2) + +      following = User.following(user)        assert length(following) == 2 -      assert info.follower_count == 0 +      assert user.follower_count == 0        {:ok, user} =          user -        |> Ecto.Changeset.change(%{following: following ++ following}) -        |> User.change_info(&Ecto.Changeset.change(&1, %{follower_count: 3})) +        |> Ecto.Changeset.change(%{follower_count: 3})          |> Repo.update() -      assert length(user.following) == 4 -      assert user.info.follower_count == 3 +      assert user.follower_count == 3        assert :ok == Mix.Tasks.Pleroma.Database.run(["update_users_following_followers_counts"])        user = User.get_by_id(user.id) -      assert length(user.following) == 2 -      assert user.info.follower_count == 0 +      assert length(User.following(user)) == 2 +      assert user.follower_count == 0      end    end diff --git a/test/tasks/relay_test.exs b/test/tasks/relay_test.exs index c866608ab..04a1e45d7 100644 --- a/test/tasks/relay_test.exs +++ b/test/tasks/relay_test.exs @@ -51,7 +51,7 @@ defmodule Mix.Tasks.Pleroma.RelayTest do        target_user = User.get_cached_by_ap_id(target_instance)        follow_activity = Utils.fetch_latest_follow(local_user, target_user)        User.follow(local_user, target_user) -      assert "#{target_instance}/followers" in refresh_record(local_user).following +      assert "#{target_instance}/followers" in User.following(local_user)        Mix.Tasks.Pleroma.Relay.run(["unfollow", target_instance])        cancelled_activity = Activity.get_by_ap_id(follow_activity.data["id"]) @@ -68,7 +68,7 @@ defmodule Mix.Tasks.Pleroma.RelayTest do        assert undo_activity.data["type"] == "Undo"        assert undo_activity.data["actor"] == local_user.ap_id        assert undo_activity.data["object"] == cancelled_activity.data -      refute "#{target_instance}/followers" in refresh_record(local_user).following +      refute "#{target_instance}/followers" in User.following(local_user)      end    end @@ -78,20 +78,18 @@ defmodule Mix.Tasks.Pleroma.RelayTest do        refute_receive {:mix_shell, :info, _} -      Pleroma.Web.ActivityPub.Relay.get_actor() -      |> Ecto.Changeset.change( -        following: [ -          "http://test-app.com/user/test1", -          "http://test-app.com/user/test1", -          "http://test-app-42.com/user/test1" -        ] -      ) -      |> Pleroma.User.update_and_set_cache() +      relay_user = Relay.get_actor() + +      ["http://mastodon.example.org/users/admin", "https://mstdn.io/users/mayuutann"] +      |> Enum.each(fn ap_id -> +        {:ok, user} = User.get_or_fetch_by_ap_id(ap_id) +        User.follow(relay_user, user) +      end)        :ok = Mix.Tasks.Pleroma.Relay.run(["list"]) -      assert_receive {:mix_shell, :info, ["test-app.com"]} -      assert_receive {:mix_shell, :info, ["test-app-42.com"]} +      assert_receive {:mix_shell, :info, ["mstdn.io"]} +      assert_receive {:mix_shell, :info, ["mastodon.example.org"]}      end    end  end diff --git a/test/tasks/user_test.exs b/test/tasks/user_test.exs index cf12d9ed6..bfd0ccbc5 100644 --- a/test/tasks/user_test.exs +++ b/test/tasks/user_test.exs @@ -58,8 +58,8 @@ defmodule Mix.Tasks.Pleroma.UserTest do        assert user.name == unsaved.name        assert user.email == unsaved.email        assert user.bio == unsaved.bio -      assert user.info.is_moderator -      assert user.info.is_admin +      assert user.is_moderator +      assert user.is_admin      end      test "user is not created" do @@ -113,11 +113,11 @@ defmodule Mix.Tasks.Pleroma.UserTest do        assert message =~ " deactivated"        user = User.get_cached_by_nickname(user.nickname) -      assert user.info.deactivated +      assert user.deactivated      end      test "user is activated" do -      user = insert(:user, info: %{deactivated: true}) +      user = insert(:user, deactivated: true)        Mix.Tasks.Pleroma.User.run(["toggle_activated", user.nickname]) @@ -125,7 +125,7 @@ defmodule Mix.Tasks.Pleroma.UserTest do        assert message =~ " activated"        user = User.get_cached_by_nickname(user.nickname) -      refute user.info.deactivated +      refute user.deactivated      end      test "no user to toggle" do @@ -139,7 +139,8 @@ defmodule Mix.Tasks.Pleroma.UserTest do    describe "running unsubscribe" do      test "user is unsubscribed" do        followed = insert(:user) -      user = insert(:user, %{following: [User.ap_followers(followed)]}) +      user = insert(:user) +      User.follow(user, followed, "accept")        Mix.Tasks.Pleroma.User.run(["unsubscribe", user.nickname]) @@ -154,8 +155,8 @@ defmodule Mix.Tasks.Pleroma.UserTest do        assert message =~ "Successfully unsubscribed"        user = User.get_cached_by_nickname(user.nickname) -      assert Enum.empty?(user.following) -      assert user.info.deactivated +      assert Enum.empty?(User.get_friends(user)) +      assert user.deactivated      end      test "no user to unsubscribe" do @@ -182,13 +183,13 @@ defmodule Mix.Tasks.Pleroma.UserTest do        assert message =~ ~r/Admin status .* true/        user = User.get_cached_by_nickname(user.nickname) -      assert user.info.is_moderator -      assert user.info.locked -      assert user.info.is_admin +      assert user.is_moderator +      assert user.locked +      assert user.is_admin      end      test "All statuses unset" do -      user = insert(:user, info: %{is_moderator: true, locked: true, is_admin: true}) +      user = insert(:user, locked: true, is_moderator: true, is_admin: true)        Mix.Tasks.Pleroma.User.run([          "set", @@ -208,9 +209,9 @@ defmodule Mix.Tasks.Pleroma.UserTest do        assert message =~ ~r/Admin status .* false/        user = User.get_cached_by_nickname(user.nickname) -      refute user.info.is_moderator -      refute user.info.locked -      refute user.info.is_admin +      refute user.is_moderator +      refute user.locked +      refute user.is_admin      end      test "no user to set status" do @@ -358,28 +359,28 @@ defmodule Mix.Tasks.Pleroma.UserTest do    describe "running toggle_confirmed" do      test "user is confirmed" do -      %{id: id, nickname: nickname} = insert(:user, info: %{confirmation_pending: false}) +      %{id: id, nickname: nickname} = insert(:user, confirmation_pending: false)        assert :ok = Mix.Tasks.Pleroma.User.run(["toggle_confirmed", nickname])        assert_received {:mix_shell, :info, [message]}        assert message == "#{nickname} needs confirmation."        user = Repo.get(User, id) -      assert user.info.confirmation_pending -      assert user.info.confirmation_token +      assert user.confirmation_pending +      assert user.confirmation_token      end      test "user is not confirmed" do        %{id: id, nickname: nickname} = -        insert(:user, info: %{confirmation_pending: true, confirmation_token: "some token"}) +        insert(:user, confirmation_pending: true, confirmation_token: "some token")        assert :ok = Mix.Tasks.Pleroma.User.run(["toggle_confirmed", nickname])        assert_received {:mix_shell, :info, [message]}        assert message == "#{nickname} doesn't need confirmation."        user = Repo.get(User, id) -      refute user.info.confirmation_pending -      refute user.info.confirmation_token +      refute user.confirmation_pending +      refute user.confirmation_token      end      test "it prints an error message when user is not exist" do | 
