From 7e965f4aded194cda350555745ac51cba6883dbe Mon Sep 17 00:00:00 2001
From: "P. Reis" <patrickpereirareal1@gmail.com>
Date: Fri, 5 Jul 2024 17:59:12 -0300
Subject: [PATCH] refactor(test.ts): explain why set DATABASE_URL to
 sqlite://:memory:

---
 src/test.ts | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/test.ts b/src/test.ts
index 7aecab7a..ae2af52a 100644
--- a/src/test.ts
+++ b/src/test.ts
@@ -96,7 +96,10 @@ export const createTestDB = async (databaseUrl?: string) => {
   let kysely: Kysely<DittoTables>;
 
   if (dialect === 'sqlite') {
-    Deno.env.set('DATABASE_URL', 'sqlite://:memory:'); // hack, refactor all, 021 migration
+    // migration 021_pgfts_index.ts calls 'Conf.db.dialect',
+    // and this calls the DATABASE_URL environment variable.
+    // The following line ensures to NOT use the DATABASE_URL that may exist in an .env file.
+    Deno.env.set('DATABASE_URL', 'sqlite://:memory:');
 
     kysely = new Kysely<DittoTables>({
       dialect: new DenoSqlite3Dialect({
@@ -104,7 +107,6 @@ export const createTestDB = async (databaseUrl?: string) => {
       }),
     });
   } else {
-    //kysely = await DittoDB.getInstance();
     kysely = new Kysely({
       dialect: {
         createAdapter() {