Skip to content

Commit da41312

Browse files
committed
test: remove full-suite timing races
1 parent e6f0597 commit da41312

2 files changed

Lines changed: 13 additions & 6 deletions

File tree

packages/cli/src/lib/tile-pmtiles.test.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,6 +234,9 @@ describe("direct MBTiles to PMTiles area extraction", () => {
234234
}
235235
});
236236

237+
// The 50,000-row fixture must exceed PMTiles' 16 KiB compressed root
238+
// directory limit. SQLite setup can cross the global 15 s timeout when
239+
// the complete suite is concurrently exercising other disk-heavy tests.
237240
it("reads tiles through PMTiles leaf directories", async () => {
238241
const root = mkdtempSync(join(tmpdir(), "openmapx-pmtiles-"));
239242
try {
@@ -258,5 +261,5 @@ describe("direct MBTiles to PMTiles area extraction", () => {
258261
} finally {
259262
rmSync(root, { recursive: true, force: true });
260263
}
261-
});
264+
}, 60_000);
262265
});

services/data-manager/src/jobs/transitous/operator-feed-relay.test.ts

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1004,6 +1004,7 @@ describe("OperatorFeedRelayStore", () => {
10041004
});
10051005

10061006
it("keeps the total deadline active after a small payload is fully buffered", async () => {
1007+
vi.useFakeTimers();
10071008
let destination: string | undefined;
10081009
const relay = new OperatorFeedRelayStore({
10091010
maxEntries: 1,
@@ -1024,12 +1025,15 @@ describe("OperatorFeedRelayStore", () => {
10241025
runId: "run-buffered-total",
10251026
});
10261027
payload.stream.on("error", () => {});
1028+
if (payload.stream.readableLength === 0) {
1029+
await new Promise<void>((resolve) => payload.stream.once("readable", resolve));
1030+
}
1031+
expect(payload.stream.readableLength).toBe(Buffer.byteLength("small archive"));
1032+
const closed = waitForClose(payload.stream);
10271033

1028-
const outcome = await Promise.race([
1029-
waitForClose(payload.stream).then(() => "closed" as const),
1030-
new Promise<"still-open">((resolve) => setTimeout(() => resolve("still-open"), 100)),
1031-
]);
1032-
expect(outcome).toBe("closed");
1034+
await vi.advanceTimersByTimeAsync(21);
1035+
await closed;
1036+
vi.useRealTimers();
10331037
await vi.waitFor(() => {
10341038
expect(destination && existsSync(destination)).toBe(false);
10351039
});

0 commit comments

Comments
 (0)