refactor: create Supabase admin client once at module level, delete auth users concurrently

Co-authored-by: bitnimble <4076797+bitnimble@users.noreply.github.com>

Author: Copilot

src/services/jest_global_setup.ts

@@ -16,11 +16,12 @@ export default async function globalSetup() {
   const pool = new Pool();
 
   // Verify tables exist (they should from supabase start + migrations)
+  const expectedTables = ['maps', 'users', 'favorites', 'difficulties'];
   const result = await pool.query(`
     SELECT table_name FROM information_schema.tables
-    WHERE table_schema = 'public' AND table_name IN ('maps', 'users', 'favorites', 'difficulties')
-  `);
-  if (result.rows.length < 4) {
+    WHERE table_schema = 'public' AND table_name = ANY($1)
+  `, [expectedTables]);
+  if (result.rows.length < expectedTables.length) {
     throw new Error(
       'Expected database tables not found. Make sure `supabase start` has been run ' +
         'and migrations have been applied.'

src/services/jest_setup.ts

@@ -7,21 +7,24 @@ import { getServerContext } from 'services/server_context';
 const projectDir = process.cwd();
 loadEnvConfig(projectDir);
 
-async function deleteAllAuthUsers() {
-  const supabaseUrl = process.env.NEXT_PUBLIC_SUPABASE_URL!;
-  const supabaseServiceKey = process.env.SUPABASE_SECRET_KEY!;
-  const supabase = createClient(supabaseUrl, supabaseServiceKey);
+const supabaseAdmin = createClient(
+  process.env.NEXT_PUBLIC_SUPABASE_URL!,
+  process.env.SUPABASE_SECRET_KEY!
+);
 
-  const { data, error } = await supabase.auth.admin.listUsers();
+async function deleteAllAuthUsers() {
+  const { data, error } = await supabaseAdmin.auth.admin.listUsers();
   if (error) {
     throw new Error(`Failed to list auth users: ${error.message}`);
   }
-  for (const user of data.users) {
-    const { error: deleteError } = await supabase.auth.admin.deleteUser(user.id);
-    if (deleteError) {
-      throw new Error(`Failed to delete auth user ${user.id}: ${deleteError.message}`);
-    }
-  }
+  await Promise.all(
+    data.users.map(async (user) => {
+      const { error: deleteError } = await supabaseAdmin.auth.admin.deleteUser(user.id);
+      if (deleteError) {
+        throw new Error(`Failed to delete auth user ${user.id}: ${deleteError.message}`);
+      }
+    })
+  );
 }
 
 async function resetTestData() {