Skip to content

test(cloudflare): Increasing timeout and exit with reason#21571

Merged
JPeer264 merged 1 commit into
developfrom
jp/fix-flaky-do-rpc-test
Jun 16, 2026
Merged

test(cloudflare): Increasing timeout and exit with reason#21571
JPeer264 merged 1 commit into
developfrom
jp/fix-flaky-do-rpc-test

Conversation

@JPeer264

Copy link
Copy Markdown
Member

Hope to have a better reasoning on #21544 and re-enabling two skipped tests.

With that PR there are two things happening

  1. the timeout increases as I couldn't think of anything else other than the worker is not fast enough to spin up.
  2. When something crashes for some reason it fails fast rather than waiting for the timeout to be over, including a better error description with the code and the signal. Maybe then the reasoning is better of what is actually happening (in case it fails before)

@JPeer264 JPeer264 self-assigned this Jun 16, 2026
@JPeer264 JPeer264 requested a review from a team as a code owner June 16, 2026 10:41
@JPeer264 JPeer264 requested review from andreiborza and mydea and removed request for a team June 16, 2026 10:41

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c903756. Configure here.

Comment thread dev-packages/cloudflare-integration-tests/runner.ts
@JPeer264 JPeer264 force-pushed the jp/fix-flaky-do-rpc-test branch from c903756 to 09a5a5d Compare June 16, 2026 10:56
@JPeer264 JPeer264 merged commit 9eebe18 into develop Jun 16, 2026
42 checks passed
@JPeer264 JPeer264 deleted the jp/fix-flaky-do-rpc-test branch June 16, 2026 12:29
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.

2 participants