admin/Moltbot
1
0
Fork 0
Code Issues Pull Requests Actions 6 Packages Projects Releases Wiki Activity

perf(test): tighten background abort timing windows

Browse Source
This commit is contained in:
Peter Steinberger
2026-02-22 10:21:46 +00:00
parent c995f9be07
commit 9ab7b85a66
1 changed files with 10 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
src/agents/bash-tools.exec.background-abort.e2e.test.ts
View File

@@ -7,9 +7,12 @@ import {
import { createExecTool } from "./bash-tools.exec.js";
import { killProcessTree } from "./shell-utils.js";
const BACKGROUND_HOLD_CMD = 'node -e "setTimeout(() => {}, 1000)"';
const BACKGROUND_HOLD_CMD = 'node -e "setTimeout(() => {}, 500)"';
const ABORT_SETTLE_MS = process.platform === "win32" ? 200 : 60;
const ABORT_WAIT_TIMEOUT_MS = process.platform === "win32" ? 1_500 : 600;
const ABORT_WAIT_TIMEOUT_MS = process.platform === "win32" ? 1_500 : 450;
const POLL_INTERVAL_MS = 15;
const FINISHED_WAIT_TIMEOUT_MS = process.platform === "win32" ? 8_000 : 1_200;
const BACKGROUND_TIMEOUT_SEC = process.platform === "win32" ? 0.2 : 0.12;
afterEach(() => {
resetProcessRegistryForTests();
@@ -24,8 +27,8 @@ async function waitForFinishedSession(sessionId: string) {
return Boolean(finished);
},
{
timeout: process.platform === "win32" ? 10_000 : 2_000,
interval: 20,
timeout: FINISHED_WAIT_TIMEOUT_MS,
interval: POLL_INTERVAL_MS,
},
)
.toBe(true);
@@ -63,7 +66,7 @@ async function expectBackgroundSessionSurvivesAbort(params: {
const finished = getFinishedSession(sessionId);
return Date.now() - startedAt >= ABORT_SETTLE_MS && !finished && running?.exited === false;
},
{ timeout: ABORT_WAIT_TIMEOUT_MS, interval: 20 },
{ timeout: ABORT_WAIT_TIMEOUT_MS, interval: POLL_INTERVAL_MS },
)
.toBe(true);
@@ -125,7 +128,7 @@ test("background exec still times out after tool signal abort", async () => {
executeParams: {
command: BACKGROUND_HOLD_CMD,
background: true,
timeout: 0.2,
timeout: BACKGROUND_TIMEOUT_SEC,
},
abortAfterStart: true,
});
@@ -146,7 +149,7 @@ test("yielded background exec still times out", async () => {
executeParams: {
command: BACKGROUND_HOLD_CMD,
yieldMs: 5,
timeout: 0.2,
timeout: BACKGROUND_TIMEOUT_SEC,
},
});
});