Skip to content

test(execution): fix flaky tool-invoker test - #1938

Closed
The-AarushiSingh wants to merge 1 commit into
UsefulSoftwareCo:mainfrom
The-AarushiSingh:fix/tool-invoker-test-timeout
Closed

The-AarushiSingh wants to merge 1 commit into
UsefulSoftwareCo:mainfrom
The-AarushiSingh:fix/tool-invoker-test-timeout

Conversation

@The-AarushiSingh

Copy link
Copy Markdown
Contributor

Description

Fixes #1928 - Flaky test in src/tool-invoker.test.ts.

Problem

The test "describes a return type that accepts the sandbox invocation result" times out on CI.

Solution

Added { timeout: 10000 } to the test.

Testing

  • ✅ 39/39 tests passing
  • ✅ Test completes within timeout

Closes #1928

@The-AarushiSingh
The-AarushiSingh force-pushed the fix/tool-invoker-test-timeout branch from 1add931 to c11d14c Compare September 3, 2026 13:08
- Add timeout: 10000 to prevent CI timeouts
- Add changeset for @executor-js/execution

Closes UsefulSoftwareCo#1928
@The-AarushiSingh
The-AarushiSingh force-pushed the fix/tool-invoker-test-timeout branch from c11d14c to 5af4904 Compare September 3, 2026 13:19
@RhysSullivan

Copy link
Copy Markdown
Collaborator

We're clearing the backlog ahead of the v2 launch, so we're closing this. If it still applies to v2, please open a new issue or PR against v2.

Sent from my Claude

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] flaky test in src/tool-invoker.test.ts

2 participants