From ddb7553b306fc6192e82a707e4e54bb4e21be8cc Mon Sep 17 00:00:00 2001 From: robobun Date: Sun, 26 Jul 2026 09:54:14 +0000 Subject: [PATCH] test(docker): add postgres-prepared-pipeline-reorder to prestart-map The test file was added in #33627 after prestart-map.mjs was last touched in #33622, so it is not in the map. That means: - scripts/runner.node.mjs does not push it toward the end of the shard - test/docker/coordinator.ts does not pre-warm postgres_plain for it When a shard's only postgres_plain test is this file (build #81338, debian 13 aarch64, shard [67/281]), it runs early and pays the full container cold-start (~18s) as its own wall time; the four tests themselves take ~80ms on a release build. Also flip the describe to concurrent: the four tests each own a max:1 SQL instance and share no state, so they can overlap their connect + warm-up round trip. --- test/docker/prestart-map.mjs | 1 + test/js/sql/postgres-prepared-pipeline-reorder.test.ts | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/test/docker/prestart-map.mjs b/test/docker/prestart-map.mjs index a33617c66ee6..f0de287c21dc 100644 --- a/test/docker/prestart-map.mjs +++ b/test/docker/prestart-map.mjs @@ -17,6 +17,7 @@ export const prestartMap = { "js/sql/sql-postgres-datetime": ["postgres_plain"], "js/sql/postgres-binary-numeric": ["postgres_plain"], "js/sql/postgres-multi-statement-fields": ["postgres_plain"], + "js/sql/postgres-prepared-pipeline-reorder": ["postgres_plain"], "js/sql/postgres-simple-query-pipeline": ["postgres_plain"], "js/sql/sql-onconnect-onclose-throw": ["postgres_plain", "mysql_plain"], "js/sql/sql-prepare-false": ["postgres_plain"], diff --git a/test/js/sql/postgres-prepared-pipeline-reorder.test.ts b/test/js/sql/postgres-prepared-pipeline-reorder.test.ts index 0f2b89f8a5d1..c2ac989a9eca 100644 --- a/test/js/sql/postgres-prepared-pipeline-reorder.test.ts +++ b/test/js/sql/postgres-prepared-pipeline-reorder.test.ts @@ -12,7 +12,7 @@ import { SQL } from "bun"; import { expect, test } from "bun:test"; import { describeWithContainer } from "harness"; -describeWithContainer("postgres", { image: "postgres_plain" }, container => { +describeWithContainer("postgres", { image: "postgres_plain", concurrent: true }, container => { const url = () => `postgres://bun_sql_test@${container.host}:${container.port}/bun_sql_test`; // Before the fix: B's Parse is never sent. C's Bind+Execute is written