Exclude bm25 test from CI suite

Author: dimamik

.github/workflows/ci.yml

@@ -23,7 +23,7 @@ jobs:
     # Additional services can be defined here if required.
     services:
       db:
-        image: postgres:17
+        image: postgres:15
         ports:
           - 5432/tcp
         env:
@@ -40,8 +40,8 @@ jobs:
     name: Test on OTP ${{matrix.otp}} / Elixir ${{matrix.elixir}}
     strategy:
       matrix:
-        otp: ['28.3']  
-        elixir: ['1.19.5']
+        otp: ['27.2']  
+        elixir: ['1.18.0']
     steps:
     # Step: Setup Elixir + Erlang image as the base.
     - name: Set up Elixir
@@ -86,9 +86,6 @@ jobs:
     - name: Install dependencies
       run: mix deps.get --check-locked
 
-    - name: Install pg_textsearch extension
-      run: |
-        docker exec ${{ job.services.db.id }} bash -lc "apt-get update && apt-get install -y wget && wget -q -O /tmp/pg_textsearch.deb https://github.com/timescale/pg_textsearch/releases/download/v0.5.0/pg-textsearch-postgresql-17_0.5.0-1_amd64.deb && dpkg -i /tmp/pg_textsearch.deb"
     # TODO These steps can be moved to `mix.exs`
 
     # Step: Compile the project treating any warnings as errors.

pg_textsearch

@@ -0,0 +1 @@
+Subproject commit 1a23bc6bbb06dcec0c7e6efa50b2a51c08882926

test/support/migrations/20260203120000_add_pg_textsearch.exs

@@ -2,25 +2,50 @@ defmodule Torus.BM25Test do
   @moduledoc false
   use Torus.Case, async: false
 
-  # @moduletag :skip
+  @moduletag :skip
+
+  # NOTE: pg_textsearch has a known limitation where uncommitted rows from
+  # rolled-back transactions remain in the BM25 memtable. This can cause
+  # ORDER BY + LIMIT queries to return stale TIDs that fail visibility checks.
+  # This is why this test is skipped by default on CI and needs to be run manually.
 
   import Ecto.Query
 
+  alias Ecto.Adapters.SQL.Sandbox
   alias Torus.Test.Repo
   alias TorusTest.Post
 
-  # NOTE: pg_textsearch has a known limitation where uncommitted rows from
-  # rolled-back transactions remain in the BM25 memtable. This can cause
-  # ORDER BY + LIMIT queries to return stale TIDs that fail visibility checks.
-  # The standalone operator (without ORDER BY) works correctly.
+  setup_all do
+    Sandbox.unboxed_run(Repo, fn ->
+      Repo.query!("CREATE EXTENSION IF NOT EXISTS pg_textsearch")
+
+      Repo.query!("""
+      CREATE INDEX IF NOT EXISTS posts_body_bm25_idx ON posts
+      USING bm25(body) WITH (text_config='english')
+      """)
+
+      Repo.query!("""
+      CREATE INDEX IF NOT EXISTS posts_title_bm25_idx ON posts
+      USING bm25(title) WITH (text_config='english')
+      """)
+    end)
+
+    :ok
+  end
 
   defp flush_bm25!(index_name \\ "posts_body_bm25_idx") do
-    Repo.query!("SELECT bm25_spill_index($1)", [index_name])
+    Sandbox.unboxed_run(Repo, fn ->
+      Repo.query!("SELECT bm25_spill_index($1)", [index_name])
+    end)
+
     :ok
   end
 
   defp reset_bm25_state! do
-    Repo.query!("TRUNCATE TABLE posts RESTART IDENTITY CASCADE")
+    Sandbox.unboxed_run(Repo, fn ->
+      Repo.query!("TRUNCATE TABLE posts RESTART IDENTITY CASCADE")
+    end)
+
     flush_bm25!()
     flush_bm25!("posts_title_bm25_idx")
     :ok