diff options
author | Ilja <ilja@ilja.space> | 2022-04-05 13:21:09 +0200 |
---|---|---|
committer | Ilja <ilja@ilja.space> | 2022-04-05 13:21:09 +0200 |
commit | 5f37db330f28556fa6c4c7b2559e6b4663ab4417 (patch) | |
tree | 006e323e1d0afe90900cf3c296b1d01641c9bb97 | |
parent | d7c53da77adfcfc5cf1fd83dd0671b9de8147e15 (diff) | |
download | pleroma-5f37db330f28556fa6c4c7b2559e6b4663ab4417.tar.gz |
Fix eratic test for POST /api/pleroma/admin/reports/:id/notes
It retrieved two ReportNotes and then checked one of them. But the order isn't guaranteed, while the test tested on the content of the first ReportNote.
I made the test on the content more generic
-rw-r--r-- | test/pleroma/web/admin_api/controllers/report_controller_test.exs | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/pleroma/web/admin_api/controllers/report_controller_test.exs b/test/pleroma/web/admin_api/controllers/report_controller_test.exs index 6211956c6..30dcb87e2 100644 --- a/test/pleroma/web/admin_api/controllers/report_controller_test.exs +++ b/test/pleroma/web/admin_api/controllers/report_controller_test.exs @@ -355,7 +355,6 @@ defmodule Pleroma.Web.AdminAPI.ReportControllerTest do } = note end - @tag :erratic test "it returns reports with notes", %{conn: conn, admin: admin} do conn = get(conn, "/api/pleroma/admin/reports") @@ -364,7 +363,8 @@ defmodule Pleroma.Web.AdminAPI.ReportControllerTest do [note, _] = notes assert note["user"]["nickname"] == admin.nickname - assert note["content"] == "this is disgusting!" + # We use '=~' because the order of the notes isn't guaranteed + assert note["content"] =~ "this is disgusting" assert note["created_at"] assert response["total"] == 1 end |