test: fix tests

Signed-off-by: Thomas Citharel <tcit@tcit.fr>

lib/mobilizon/posts/posts.ex

@@ -110,10 +110,13 @@ defmodule Mobilizon.Posts do
   def update_post(%Post{} = post, attrs) do
     Cachable.clear_all_caches(post)
 
-    post
-    |> Repo.preload([:tags, :media])
-    |> Post.changeset(attrs)
-    |> Repo.update()
+    with {:ok, %Post{} = post} <-
+           post
+           |> Repo.preload(@post_preloads)
+           |> Post.changeset(attrs)
+           |> Repo.update() do
+      {:ok, Repo.preload(post, @post_preloads)}
+    end
   end
 
   @doc """

lib/service/export/icalendar.ex

@@ -8,7 +8,7 @@ defmodule Mobilizon.Service.Export.ICalendar do
   alias Mobilizon.{Config, Events}
   alias Mobilizon.Events.{Event, EventOptions}
   alias Mobilizon.Service.Export.{Cachable, Common}
-  alias Mobilizon.Service.Formatter.{HTML, Text}
+  alias Mobilizon.Service.Formatter.HTML
 
   @behaviour Cachable
 

test/federation/activity_pub/transmogrifier/posts_test.exs

@@ -2,15 +2,12 @@ defmodule Mobilizon.Federation.ActivityPub.Transmogrifier.PostsTest do
   use Mobilizon.DataCase
 
   import Mobilizon.Factory
-  import Mox
   alias Mobilizon.Actors.Actor
   alias Mobilizon.Federation.ActivityPub.{Activity, Transmogrifier}
   alias Mobilizon.Federation.ActivityStream.Convertible
   alias Mobilizon.Posts.Post
 
   describe "handle incoming posts" do
-    setup :verify_on_exit!
-
     test "it ignores an incoming post if we already have it" do
       post = insert(:post)
       post = Repo.preload(post, [:author, :attributed_to, :picture, :media])

test/service/workers/legacy_notifier_builder_test.exs

@@ -187,17 +187,18 @@ defmodule Mobilizon.Service.Workers.LegacyNotifierBuilderTest do
           "object_id" => to_string(comment_id)
         })
 
-      NotifierMock
-      |> expect(:ready?, fn -> true end)
-      |> expect(:send, fn %User{},
-                          %Activity{
-                            type: :comment,
-                            subject: :participation_event_comment,
-                            object_type: :comment
-                          },
-                          [single_activity: true] ->
-        {:ok, :sent}
-      end)
+      # # Disabled as announcement is sent straight away
+      # NotifierMock
+      # |> expect(:ready?, fn -> true end)
+      # |> expect(:send, fn %User{},
+      #                     %Activity{
+      #                       type: :comment,
+      #                       subject: :participation_event_comment,
+      #                       object_type: :comment
+      #                     },
+      #                     [single_activity: true] ->
+      #   {:ok, :sent}
+      # end)
 
       assert :ok == LegacyNotifierBuilder.perform(%Oban.Job{args: args})
     end