chase test failures

diff --git a/lib/pleroma/filter.ex b/lib/pleroma/filter.ex
index 79efc29f0f73803463d8f94ccf79e463a3fbba4a..90457dadf8641f9c100aa067ba8678dc0ef55c60 100644 (file)
--- a/lib/pleroma/filter.ex
+++ b/lib/pleroma/filter.ex
@@ -38,7 +38,8 @@ defmodule Pleroma.Filter do
     query =
       from(
         f in Pleroma.Filter,
     query =
       from(
         f in Pleroma.Filter,

-        where: f.user_id == ^user_id
+        where: f.user_id == ^user_id,
+        order_by: [desc: :id]

       )
 
     Repo.all(query)
       )
 
     Repo.all(query)

diff --git a/test/user_test.exs b/test/user_test.exs
index 0b65e89e9bdceaffe3aa86f2477935c5ac6a209a..bb47b4958820d28dd5740da17438a201c3978cab 100644 (file)
--- a/test/user_test.exs
+++ b/test/user_test.exs
@@ -873,7 +873,6 @@ defmodule Pleroma.UserTest do
 
       assert [activity] ==
                ActivityPub.fetch_activities([user2.ap_id | user2.following], %{"user" => user2})
 
       assert [activity] ==
                ActivityPub.fetch_activities([user2.ap_id | user2.following], %{"user" => user2})

-               |> ActivityPub.contain_timeline(user2)

 
       {:ok, _user} = User.deactivate(user)
 
 
       {:ok, _user} = User.deactivate(user)
 


@@ -882,7 +881,6 @@ defmodule Pleroma.UserTest do
 
       assert [] ==
                ActivityPub.fetch_activities([user2.ap_id | user2.following], %{"user" => user2})
 
       assert [] ==
                ActivityPub.fetch_activities([user2.ap_id | user2.following], %{"user" => user2})

-               |> ActivityPub.contain_timeline(user2)

     end
   end
 
     end
   end
 

diff --git a/test/web/activity_pub/activity_pub_test.exs b/test/web/activity_pub/activity_pub_test.exs
index b41f6ab81e28a5a2b4111c090bf5d7459ad7bb68..34e23b8528b4046c98c2ab94552043cd2c49f9d6 100644 (file)
--- a/test/web/activity_pub/activity_pub_test.exs
+++ b/test/web/activity_pub/activity_pub_test.exs
@@ -960,18 +960,21 @@ defmodule Pleroma.Web.ActivityPub.ActivityPubTest do
           "in_reply_to_status_id" => private_activity_2.id
         })
 
           "in_reply_to_status_id" => private_activity_2.id
         })
 

-      activities = ActivityPub.fetch_activities([user1.ap_id | user1.following])
+      activities =
+        ActivityPub.fetch_activities([user1.ap_id | user1.following])
+        |> Enum.map(fn a -> a.id end)

 
       private_activity_1 = Activity.get_by_ap_id_with_object(private_activity_1.data["id"])
 
 
       private_activity_1 = Activity.get_by_ap_id_with_object(private_activity_1.data["id"])
 

-      assert [public_activity, private_activity_1, private_activity_3] == activities
+      assert [public_activity.id, private_activity_1.id, private_activity_3.id] == activities

 
       assert length(activities) == 3
 
       activities =
         ActivityPub.fetch_activities([user1.ap_id | user1.following], %{"user" => user1})
 
       assert length(activities) == 3
 
       activities =
         ActivityPub.fetch_activities([user1.ap_id | user1.following], %{"user" => user1})

+        |> Enum.map(fn a -> a.id end)

 
 

-      assert [public_activity, private_activity_1] == activities
+      assert [public_activity.id, private_activity_1.id] == activities

       assert length(activities) == 2
     end
   end
       assert length(activities) == 2
     end
   end