Fix race conditions in tests and reorder precompile workflow

Test fixes: Replace GenServer.stop with Process.exit and use try/catch
to handle exits during cleanup, preventing flaky test failures.

Workflow: Reorder steps to precompile before testing so tests validate
the actual precompiled artifacts users will download.

Author: brendon9x

.github/workflows/precompile.yml

@@ -37,21 +37,18 @@ jobs:
       - name: Install dependencies
         run: mix deps.get
 
-      - name: Compile (for native x86_64 tests)
-        run: mix compile
+      - name: Precompile NIFs (native + cross-compile)
+        run: |
+          export ELIXIR_MAKE_CACHE_DIR=$(pwd)/cache
+          mkdir -p $ELIXIR_MAKE_CACHE_DIR
+          MIX_ENV=prod mix elixir_make.precompile
 
       - name: Run tests on x86_64
         run: mix test --exclude integration
         env:
           CLICKHOUSE_HOST: localhost
           CLICKHOUSE_PORT: 9000
 
-      - name: Precompile NIFs (native + cross-compile)
-        run: |
-          export ELIXIR_MAKE_CACHE_DIR=$(pwd)/cache
-          mkdir -p $ELIXIR_MAKE_CACHE_DIR
-          MIX_ENV=prod mix elixir_make.precompile
-
       - name: Upload artifacts to release
         uses: softprops/action-gh-release@v1
         if: startsWith(github.ref, 'refs/tags/')
@@ -99,21 +96,18 @@ jobs:
           sudo apt-get install -y build-essential cmake libssl-dev
           mix deps.get
 
-      - name: Compile
-        run: mix compile
+      - name: Precompile NIFs
+        run: |
+          export ELIXIR_MAKE_CACHE_DIR=$(pwd)/cache
+          mkdir -p $ELIXIR_MAKE_CACHE_DIR
+          MIX_ENV=prod mix elixir_make.precompile
 
       - name: Run tests on ARM64
         run: mix test --exclude integration
         env:
           CLICKHOUSE_HOST: localhost
           CLICKHOUSE_PORT: 9000
 
-      - name: Precompile NIFs
-        run: |
-          export ELIXIR_MAKE_CACHE_DIR=$(pwd)/cache
-          mkdir -p $ELIXIR_MAKE_CACHE_DIR
-          MIX_ENV=prod mix elixir_make.precompile
-
       - name: Upload artifacts to release
         uses: softprops/action-gh-release@v1
         if: startsWith(github.ref, 'refs/tags/')
@@ -191,18 +185,18 @@ jobs:
           brew install cmake openssl
           mix deps.get
 
-      - name: Run tests on x86_64 macOS
-        run: mix test --exclude integration
-        env:
-          CLICKHOUSE_HOST: localhost
-          CLICKHOUSE_PORT: 9000
-
       - name: Precompile NIFs
         run: |
           export ELIXIR_MAKE_CACHE_DIR=$(pwd)/cache
           mkdir -p $ELIXIR_MAKE_CACHE_DIR
           MIX_ENV=prod mix elixir_make.precompile
 
+      - name: Run tests on x86_64 macOS
+        run: mix test --exclude integration
+        env:
+          CLICKHOUSE_HOST: localhost
+          CLICKHOUSE_PORT: 9000
+
       - name: Upload artifacts to release
         uses: softprops/action-gh-release@v1
         if: startsWith(github.ref, 'refs/tags/')
@@ -264,18 +258,18 @@ jobs:
           brew install cmake openssl
           mix deps.get
 
-      - name: Run tests on ARM64 macOS
-        run: mix test --exclude integration
-        env:
-          CLICKHOUSE_HOST: localhost
-          CLICKHOUSE_PORT: 9000
-
       - name: Precompile NIFs
         run: |
           export ELIXIR_MAKE_CACHE_DIR=$(pwd)/cache
           mkdir -p $ELIXIR_MAKE_CACHE_DIR
           MIX_ENV=prod mix elixir_make.precompile
 
+      - name: Run tests on ARM64 macOS
+        run: mix test --exclude integration
+        env:
+          CLICKHOUSE_HOST: localhost
+          CLICKHOUSE_PORT: 9000
+
       - name: Upload artifacts to release
         uses: softprops/action-gh-release@v1
         if: startsWith(github.ref, 'refs/tags/')

test/bang_test.exs

@@ -15,11 +15,12 @@ defmodule Chex.BangTest do
       if Process.alive?(conn) do
         try do
           Connection.execute(conn, "DROP TABLE IF EXISTS #{table}")
-        rescue
-          _ -> :ok
+        catch
+          :exit, _ -> :ok
         end
 
-        GenServer.stop(conn)
+        # Use Process.exit to avoid race conditions
+        Process.exit(conn, :normal)
       end
     end)
 

test/block_test.exs

@@ -15,11 +15,12 @@ defmodule Chex.BlockTest do
       if Process.alive?(conn) do
         try do
           Chex.Connection.execute(conn, "DROP TABLE IF EXISTS #{table}")
-        rescue
-          _ -> :ok
+        catch
+          :exit, _ -> :ok
         end
 
-        GenServer.stop(conn)
+        # Use Process.exit to avoid race conditions
+        Process.exit(conn, :normal)
       end
     end)
 

test/connection_test.exs

@@ -13,11 +13,12 @@ defmodule Chex.ConnectionTest do
       if Process.alive?(conn) do
         try do
           Chex.Connection.execute(conn, "DROP TABLE IF EXISTS #{table}")
-        rescue
-          _ -> :ok
+        catch
+          :exit, _ -> :ok
         end
 
-        GenServer.stop(conn)
+        # Use Process.exit to avoid race conditions
+        Process.exit(conn, :normal)
       end
     end)
 

test/nesting_integration_test.exs

@@ -15,11 +15,12 @@ defmodule Chex.NestingIntegrationTest do
       if Process.alive?(conn) do
         try do
           Connection.execute(conn, "DROP TABLE IF EXISTS #{table}")
-        rescue
-          _ -> :ok
+        catch
+          :exit, _ -> :ok
         end
 
-        GenServer.stop(conn)
+        # Use Process.exit to avoid race conditions
+        Process.exit(conn, :normal)
       end
     end)
 

test/query_test.exs

@@ -15,11 +15,12 @@ defmodule Chex.QueryTest do
       if Process.alive?(conn) do
         try do
           Connection.execute(conn, "DROP TABLE IF EXISTS #{table}")
-        rescue
-          _ -> :ok
+        catch
+          :exit, _ -> :ok
         end
 
-        GenServer.stop(conn)
+        # Use Process.exit to avoid race conditions
+        Process.exit(conn, :normal)
       end
     end)