test: run postgres.js tests with deno and bun (#713)

Deno ran out of the box. Bun needed some changes due to error
stacktraces differences.

Author: v0idpwn

.github/workflows/elixir.yml

@@ -160,6 +160,12 @@ jobs:
         with:
           node-version: 'lts/*'
           cache-dependency-path: 'test/integration/js/'
+      - uses: denoland/setup-deno@v2
+        with:
+          deno-version: 'v2.x'
+      - uses: oven-sh/setup-bun@v2
+        with:
+          bun-version: latest
       - name: Set up Rust
         uses: dtolnay/rust-toolchain@v1
         with:

test/integration/external_test.exs

@@ -52,26 +52,25 @@ defmodule Supavisor.Integration.ExternalTest do
       assert_run(ctx, ~w[postgres/index.js])
     end
 
-    # These currently do not pass
-    # @tag runtime: "bun", mode: "session"
-    # test "Bun session", ctx do
-    #   assert_run ctx, ~w[postgres/index.js], suite: "js"
-    # end
-    #
-    # @tag runtime: "bun", mode: "transaction"
-    # test "Bun transaction", ctx do
-    #   assert_run ctx, ~w[postgres/index.js], suite: "js"
-    # end
-    #
-    # @tag runtime: "deno", mode: "session"
-    # test "Deno session", ctx do
-    #   assert_run ctx, ~w[run --allow-all postgres/index.js], suite: "js"
-    # end
-    #
-    # @tag runtime: "deno", mode: "transaction"
-    # test "Deno transaction", ctx do
-    #   assert_run ctx, ~w[run --allow-all postgres/index.js], suite: "js"
-    # end
+    @tag runtime: "bun", mode: "session"
+    test "Bun session", ctx do
+      assert_run(ctx, ~w[postgres/index.js], suite: "js")
+    end
+
+    @tag runtime: "bun", mode: "transaction"
+    test "Bun transaction", ctx do
+      assert_run(ctx, ~w[postgres/index.js], suite: "js")
+    end
+
+    @tag runtime: "deno", mode: "session"
+    test "Deno session", ctx do
+      assert_run(ctx, ~w[run --allow-all postgres/index.js], suite: "js")
+    end
+
+    @tag runtime: "deno", mode: "transaction"
+    test "Deno transaction", ctx do
+      assert_run(ctx, ~w[run --allow-all postgres/index.js], suite: "js")
+    end
   end
 
   defp assert_run(ctx, args, opts \\ []) do

test/integration/js/postgres/index.js

@@ -1575,29 +1575,48 @@ t('numeric is returned as string', async() => [
   typeof (await sql`select 1.2 as x`)[0].x
 ])
 
+// This test needed changes to be compatible with Bun,
+// because Bun stacktraces behave differently from Node.js/deno ones
 t('Async stack trace', async() => {
   const sql = postgres({ ...options, debug: false })
+
+  const extractLine = (stackLine) => {
+    const match = stackLine.match(/:(\d+)(?::|$)/)
+    return match ? parseInt(match[1]) : 0
+  }
+
   return [
-    parseInt(new Error().stack.split('\n')[1].match(':([0-9]+):')[1]) + 1,
-    parseInt(await sql`error`.catch(x => x.stack.split('\n').pop().match(':([0-9]+):')[1]))
+    extractLine(new Error().stack.split('\n')[1]) + 1,
+    await sql`error`.catch(x => {
+      const lines = x.stack.split('\n')
+      for (const line of lines) {
+        if (line.includes('postgres/index.js') && !line.includes('native:')) {
+          return extractLine(line)
+        }
+      }
+      return extractLine(lines[lines.length - 1])
+    })
   ]
 })
 
 t('Debug has long async stack trace', async() => {
   const sql = postgres({ ...options, debug: true })
 
-  return [
-    'watyo',
-    await yo().catch(x => x.stack.match(/wat|yo/g).join(''))
-  ]
-
   function yo() {
     return wat()
   }
 
   function wat() {
     return sql`error`
   }
+
+  return [
+    true,
+    await yo().catch(x => {
+      const stackLines = x.stack.split('\n').filter(line => line.includes('postgres/index.js'))
+      return stackLines.length >= 2
+    })
+  ]
 })
 
 t('Error contains query string', async() => [

test/integration/js/shared/test.js

@@ -19,7 +19,20 @@ t.timeout = (process.env.TIMEOUT || 5) | 0
 
 async function test(o, name, options, fn) {
   typeof options !== 'object' && (fn = options, options = {})
-  const line = new Error().stack.split('\n')[3].match(':([0-9]+):')[1]
+  
+  const stack = new Error().stack.split('\n')
+  let line
+  
+  for (let i = 2; i < stack.length; i++) {
+    const stackLine = stack[i]
+    if (!stackLine.includes('test.js') && !stackLine.includes('native:')) {
+      const match = stackLine.match(/:(\d+)(?::|$)/)
+      if (match) {
+        line = match[1]
+        break
+      }
+    }
+  }
 
   await 1