rogerpadilla / uql / 30098545976

97%

push

github

fix(test): avoid mid-hook connection churn and batch DDL in beforeAll

querier.transaction() releases the connection back to the pool when it
finishes, which is wrong for helpers that run mid-hook: clearTables (in
beforeEach) was silently forcing an extra reserve/release cycle before the
test body even ran. Switch to the low-level begin/commit/rollback primitives,
which don't touch the connection lifecycle, and apply the same batching to
dropTables/createTables in beforeAll - previously 34 serial DDL round-trips,
now 2 transactions.

This is what was actually timing out in CI, not clearTables: bun reported
the whole BunCockroachIt suite as a single "(unnamed)" beforeAll failure with
zero of its 60 tests running, not one flaky test among many passing.

3328 of 3676 branches covered (90.53%)

Branch coverage included in aggregate %.

5732 of 5858 relevant lines covered (97.85%)

600.14 hits per line