Skip to content

Commit 0321035

Browse files
committed
test: enable sendAndWait timeout test in CI
Now that the harness properly returns after exitWithNoMatchingRequestError, this test can run in CI without crashing the proxy.
1 parent 9a31b5b commit 0321035

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

nodejs/test/e2e/session.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -384,9 +384,9 @@ describe("Send Blocking Behavior", async () => {
384384
expect(events).toContain("assistant.message");
385385
});
386386

387-
// Skip in CI - this test validates client-side timeout behavior, not LLM responses.
388-
// The test intentionally times out before receiving a response, so there's no snapshot to replay.
389-
it.skipIf(process.env.CI === "true")("sendAndWait throws on timeout", async () => {
387+
// This test validates client-side timeout behavior.
388+
// The snapshot has no assistant response since we expect timeout before completion.
389+
it("sendAndWait throws on timeout", async () => {
390390
const session = await client.createSession();
391391

392392
// Use a slow command to ensure timeout triggers before completion

0 commit comments

Comments
 (0)