test(s032): two flakes met on run 35603385277 closed at their source

The fake provider's failures and objects no longer outlive the test that set them (llm_test
and SessionCase clear on exit), and the test helper waits for the boot receipt's Task
before the sandbox goes manual, so the boot-receipt tests no longer depend on a runner's
speed.

Signed-off-by: Ayla Croft <aylacroft@proton.me>

Author: HackTuah

test/support/session_case.ex

@@ -18,6 +18,8 @@ defmodule Trinity.SessionCase do
       setup do
         Trinity.LLM.Providers.Fake.clear()
         on_exit(fn -> Trinity.SessionCase.stop_all_sessions() end)
+        # Slice 032: scripts and objects set here do not outlive the test either.
+        on_exit(fn -> Trinity.LLM.Providers.Fake.clear() end)
         :ok
       end
     end

test/test_helper.exs

@@ -19,6 +19,27 @@ ExUnit.start(
       if(fips_leg?, do: [], else: [:fips]) ++ if(local_model?, do: [], else: [:local_model])
 )
 
+# Slice 024's boot receipt is written by a transient Task after the tree starts; once the
+# receipts pool is in manual mode that write can no longer check a connection out, and the
+# boot-receipt tests fail with an OwnershipError (postgres job, run 35603385277, at slice
+# 032). Wait for the Task to finish, either way, before the mode flips: bounded, and a Task
+# that is gone has either written or logged its reason.
+boot_task = fn ->
+  Trinity.Supervisor
+  |> Supervisor.which_children()
+  |> Enum.find_value(fn
+    {Trinity.Effects.Boot, pid, _, _} when is_pid(pid) -> pid
+    _ -> nil
+  end)
+end
+
+Enum.find(1..200, fn _ ->
+  case boot_task.() do
+    nil -> true
+    _pid -> Process.sleep(50) && false
+  end
+end)
+
 Ecto.Adapters.SQL.Sandbox.mode(Trinity.Repo, :manual)
 Ecto.Adapters.SQL.Sandbox.mode(Trinity.Repo.Receipts, :manual)
 

test/trinity/llm/llm_test.exs

@@ -10,8 +10,12 @@ defmodule Trinity.LLMTest do
 
   setup :verify_on_exit!
 
+  # Cleared on the way out too: `Fake.fail(10, …)` below leaves failures behind for the next
+  # test that does not clear first (seen at slice 032 on run 35603385277, seed 568667: the
+  # memory page's consolidation test got `{:exhausted, 3, :down}` from this file's leftovers).
   setup do
     Fake.clear()
+    on_exit(fn -> Fake.clear() end)
     :ok
   end