ReviewOS

also looking at this

stacks/bun-query-builder

test(pg): make the Postgres-gated tests actually run in CI

#1077
Merged glennmichael123 wants to merge test/pg-gated-tests-in-ci into main
4 files +68 -35
packages/bun-query-builder/test/client.transaction-connection.test.tsmodified+3-11
Changes to packages/bun-query-builder/test/client.transaction-connection.test.ts
@@ -30,19 +30,11 @@ import { join, resolve } from 'node:path'
3030import process from 'node:process'
3131import { SQL } from 'bun'
3232import { describe, expect, it } from 'bun:test'
33import { PG_URL, probePostgres } from './pg'
34
35const pgAvailable = await probePostgres()
3336
34const PG_URL = `postgres://${process.env.USER}@localhost:5432/postgres`
3537
36let pgAvailable = false
37try {
38 const probe = new SQL(PG_URL)
39 await probe.unsafe('SELECT 1')
40 await probe.end()
41 pgAvailable = true
42}
43catch {
44 pgAvailable = false
45}
4638
4739describe.skipIf(!pgAvailable)('builder-local connection routing', () => {
4840 it('keeps transactional writes inside the transaction, and reads inside it current', () => {