Simplify test configuration by removing redundant environment variables

Removed PARADEDB_WITH_PG:
- pg gem is now always included unconditionally in Gemfile
- Rails ParadeDB is a PostgreSQL library, pg should always be available
- Matches pattern from django-paradedb which always includes psycopg

Removed PARADEDB_INTEGRATION:
- Redundant check - just test if PARADEDB_TEST_DSN exists
- If DSN is set, use PostgreSQL; otherwise use SQLite
- Simpler logic with same behavior

This aligns better with django-paradedb's simpler approach.

Author: ankitml

.github/workflows/ci.yml

@@ -11,7 +11,6 @@ env:
   POSTGRES_USER: postgres
   POSTGRES_PASSWORD: postgres
   POSTGRES_DB: postgres
-  PARADEDB_WITH_PG: "1"
 
 jobs:
   test:
@@ -49,8 +48,6 @@ jobs:
           --health-retries 12
 
     env:
-      PARADEDB_WITH_PG: "1"
-      PARADEDB_INTEGRATION: "1"
       PARADEDB_TEST_DSN: postgresql://postgres@localhost:5432/postgres
       PGPASSWORD: postgres
 

Gemfile

@@ -2,8 +2,5 @@ source "https://rubygems.org"
 
 gem "activerecord", "~> 8.1"
 gem "activesupport", "~> 8.1"
+gem "pg", "~> 1.5"
 gem "sqlite3", "~> 2.0"
-
-if ENV["PARADEDB_WITH_PG"] == "1"
-  gem "pg", "~> 1.5"
-end

scripts/run_integration_tests.sh

@@ -4,8 +4,6 @@ set -euo pipefail
 
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 
-export PARADEDB_WITH_PG=1
-
 source "${SCRIPT_DIR}/rbenv_bootstrap.sh"
 
 # Only start Docker container if not in CI (CI uses services)
@@ -18,7 +16,6 @@ USER="${PARADEDB_USER:-postgres}"
 PASSWORD="${PARADEDB_PASSWORD:-postgres}"
 DB="${PARADEDB_DB:-postgres}"
 
-export PARADEDB_INTEGRATION=1
 export PARADEDB_TEST_DSN="postgres://${USER}:${PASSWORD}@localhost:${PORT}/${DB}"
 export PGPASSWORD="${PASSWORD}"
 

spec/spec_helper.rb

@@ -7,9 +7,11 @@
 ActiveRecord::Base.logger = nil
 
 def establish_test_connection
-  if ENV["PARADEDB_INTEGRATION"] == "1" && ENV["PARADEDB_TEST_DSN"]
+  if ENV["PARADEDB_TEST_DSN"]
+    # Integration tests: use real ParadeDB/PostgreSQL
     ActiveRecord::Base.establish_connection(ENV["PARADEDB_TEST_DSN"])
   else
+    # Unit tests: use SQLite in-memory database
     ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: ":memory:")
   end
 end