✅(worker) Bump e2e test timeouts from 1s to 5s by Copilot · Pull Request #6386 · dubzzz/fast-check

Copilot · 2025-12-17T11:45:03Z

E2e tests were timing out with "Property timeout: exceeded limit of 1000 milliseconds" errors in CI, particularly in nonSerializableButSameData.spec.ts.

Changes

Increased assertTimeout from 1000ms to 5000ms across all 13 e2e test files in packages/worker/test/e2e/
Updated expected error message in blockEventLoop.spec.ts to match new 5000ms timeout value

The 5-second timeout provides adequate execution time for property-based tests under constrained CI environments.

Original prompt

bump the timeouts in e2e of worker to 5s, to drop these errors

FAIL @fast-check/worker test/e2e/nonSerializableButSameData.spec.ts�[2m > @fast-check/worker�[2m > should produce the same data in worker and main thread
AssertionError: expected 'Error: Property failed after 1 tests\…' to match />>>nonSerializableButSameDataProperty…/

Expected:
/>>>nonSerializableButSameDataProperty=(.*)<<</

Received:
"Error: Property failed after 1 tests
{ seed: -1454768394, path: "0", endOnFailure: true }
Counterexample: [Symbol.for("-962")]
Shrunk 0 time(s)
Got error: Property timeout: exceeded limit of 1000 milliseconds
at Timeout. (D:/a/fast-check/fast-check/packages/fast-check/lib/check/property/TimeoutProperty.js:6:30)
at listOnTimeout (node:internal/timers:605:17)
at processTimers (node:internal/timers:541:7)

Hint: Enable verbose mode in order to have the list of all failing values encountered during the run"

�[2m❯ test/e2e/nonSerializableButSameData.spec.ts:�[2m36:27
34| const workerValueRegex = />>>nonSerializableButSameDataPrope…
35| expect(message).toMatch(mainValueRegex);
36| expect(message).toMatch(workerValueRegex);
| ^
37| const mainValue = mainValueRegex.exec(message)![1];
38| const workerValue = workerValueRegex.exec(message)![1];

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

changeset-bot · 2025-12-17T11:45:07Z

⚠️ No Changeset found

Latest commit: 39569cb

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Co-authored-by: dubzzz <[email protected]>

Copilot

Pull request overview

This PR increases the timeout values for property-based tests in the worker package e2e test suite to prevent timeout failures in CI environments. The timeout increase from 1 second to 5 seconds addresses intermittent CI failures where tests were timing out before completion, particularly in nonSerializableButSameData.spec.ts.

Key Changes:

Updated assertTimeout constant from 1000ms to 5000ms across all e2e test files
Updated expected error message in blockEventLoop.spec.ts to reflect the new timeout value

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
packages/worker/test/e2e/asyncThrow.spec.ts	Updated assertTimeout from 1000 to 5000
packages/worker/test/e2e/blockEventLoop.spec.ts	Updated assertTimeout from 1000 to 5000 and updated expected error message to match new timeout
packages/worker/test/e2e/concurrentAssert.spec.ts	Updated assertTimeout from 1000 to 5000
packages/worker/test/e2e/failing.spec.ts	Updated assertTimeout from 1000 to 5000
packages/worker/test/e2e/noWorker.spec.ts	Updated assertTimeout from 1000 to 5000
packages/worker/test/e2e/nonSerializableButSameData.spec.ts	Updated assertTimeout from 1000 to 5000
packages/worker/test/e2e/nonSerializableData.spec.ts	Updated assertTimeout from 1000 to 5000
packages/worker/test/e2e/passing.spec.ts	Updated assertTimeout from 1000 to 5000
packages/worker/test/e2e/predicateIsolation.spec.ts	Updated assertTimeout from 1000 to 5000
packages/worker/test/e2e/propertyIsolation.spec.ts	Updated assertTimeout from 1000 to 5000
packages/worker/test/e2e/supportPre.spec.ts	Updated assertTimeout from 1000 to 5000
packages/worker/test/e2e/syncThrow.spec.ts	Updated assertTimeout from 1000 to 5000
packages/worker/test/e2e/unregistered.spec.ts	Updated assertTimeout from 1000 to 5000

pkg-pr-new · 2025-12-17T11:59:19Z

@fast-check/examples

@fast-check/ava

npm i https://pkg.pr.new/@fast-check/ava@6386

@fast-check/expect-type

npm i https://pkg.pr.new/@fast-check/expect-type@6386

fast-check

npm i https://pkg.pr.new/fast-check@6386

@fast-check/jest

npm i https://pkg.pr.new/@fast-check/jest@6386

@fast-check/packaged

npm i https://pkg.pr.new/@fast-check/packaged@6386

@fast-check/poisoning

npm i https://pkg.pr.new/@fast-check/poisoning@6386

@fast-check/vitest

npm i https://pkg.pr.new/@fast-check/vitest@6386

@fast-check/worker

npm i https://pkg.pr.new/@fast-check/worker@6386

commit: 39569cb

github-actions · 2025-12-17T11:59:25Z

👋 A preview of the new documentation is available at: http://69429b1cef7d8c2f28e6ceb7--dubzzz-fast-check.netlify.app

codecov · 2025-12-17T12:04:59Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.82%. Comparing base (b613932) to head (39569cb).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #6386   +/-   ##
=======================================
  Coverage   94.82%   94.82%           
=======================================
  Files         206      206           
  Lines        5503     5503           
  Branches     1442     1443    +1     
=======================================
  Hits         5218     5218           
  Misses        272      272           
  Partials       13       13

Flag	Coverage Δ
tests	`94.82% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

E2e tests were timing out with "Property timeout: exceeded limit of 1000 milliseconds" errors in CI, particularly in `nonSerializableButSameData.spec.ts`. ## Changes - Increased `assertTimeout` from 1000ms to 5000ms across all 13 e2e test files in `packages/worker/test/e2e/` - Updated expected error message in `blockEventLoop.spec.ts` to match new 5000ms timeout value The 5-second timeout provides adequate execution time for property-based tests under constrained CI environments.   <details> <summary>Original prompt</summary> > bump the timeouts in e2e of worker to 5s, to drop these errors > > > FAIL @fast-check/worker test/e2e/nonSerializableButSameData.spec.ts�[2m > @fast-check/worker�[2m > should produce the same data in worker and main thread > AssertionError: expected 'Error: Property failed after 1 tests\…' to match />>>nonSerializableButSameDataProperty…/ > > - Expected: > />>>nonSerializableButSameDataProperty=(.*)<<</ > > + Received: > "Error: Property failed after 1 tests > { seed: -1454768394, path: \"0\", endOnFailure: true } > Counterexample: [Symbol.for(\"-962\")] > Shrunk 0 time(s) > Got error: Property timeout: exceeded limit of 1000 milliseconds > at Timeout.<anonymous> (D:/a/fast-check/fast-check/packages/fast-check/lib/check/property/TimeoutProperty.js:6:30) > at listOnTimeout (node:internal/timers:605:17) > at processTimers (node:internal/timers:541:7) > > Hint: Enable verbose mode in order to have the list of all failing values encountered during the run" > > �[2m❯ test/e2e/nonSerializableButSameData.spec.ts:�[2m36:27 > 34| const workerValueRegex = />>>nonSerializableButSameDataPrope… > 35| expect(message).toMatch(mainValueRegex); > 36| expect(message).toMatch(workerValueRegex); > | ^ > 37| const mainValue = mainValueRegex.exec(message)![1]; > 38| const workerValue = workerValueRegex.exec(message)![1]; </details>  --- 💬 We'd love your input! Share your thoughts on Copilot coding agent in our [2 minute survey](https://gh.io/copilot-coding-agent-survey). --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: dubzzz <[email protected]>

Initial plan

6cb989e

Copilot AI assigned Copilot and dubzzz Dec 17, 2025

Copilot started work on behalf of dubzzz December 17, 2025 11:45 View session

⚡️(worker) Bump e2e test timeouts from 1s to 5s

39569cb

Co-authored-by: dubzzz <[email protected]>

Copilot AI changed the title ~~[WIP] Bump timeouts in e2e tests of worker to 5 seconds~~ ⚡️(worker) Bump e2e test timeouts from 1s to 5s Dec 17, 2025

Copilot finished work on behalf of dubzzz December 17, 2025 11:52

Copilot AI requested a review from dubzzz December 17, 2025 11:52

dubzzz approved these changes Dec 17, 2025

View reviewed changes

dubzzz marked this pull request as ready for review December 17, 2025 11:55

Copilot AI review requested due to automatic review settings December 17, 2025 11:55

dubzzz enabled auto-merge (squash) December 17, 2025 11:55

Copilot started reviewing on behalf of dubzzz December 17, 2025 11:56 View session

Copilot AI reviewed Dec 17, 2025

View reviewed changes

dubzzz changed the title ~~⚡️(worker) Bump e2e test timeouts from 1s to 5s~~ ✅(worker) Bump e2e test timeouts from 1s to 5s Dec 17, 2025

dubzzz disabled auto-merge December 17, 2025 12:02

dubzzz enabled auto-merge (squash) December 17, 2025 12:03

dubzzz merged commit 3a48514 into main Dec 17, 2025
60 of 95 checks passed

dubzzz deleted the copilot/bump-timeouts-in-e2e-tests branch December 17, 2025 12:04

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

✅(worker) Bump e2e test timeouts from 1s to 5s#6386

✅(worker) Bump e2e test timeouts from 1s to 5s#6386
dubzzz merged 2 commits into
mainfrom
copilot/bump-timeouts-in-e2e-tests

Copilot AI commented Dec 17, 2025 •

edited

Loading

Uh oh!

changeset-bot Bot commented Dec 17, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

pkg-pr-new Bot commented Dec 17, 2025

Uh oh!

github-actions Bot commented Dec 17, 2025

Uh oh!

Uh oh!

codecov Bot commented Dec 17, 2025 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

Copilot AI commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Uh oh!

changeset-bot Bot commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⚠️ No Changeset found

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

pkg-pr-new Bot commented Dec 17, 2025

Uh oh!

github-actions Bot commented Dec 17, 2025

Uh oh!

Uh oh!

codecov Bot commented Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Dec 17, 2025 •

edited

Loading

changeset-bot Bot commented Dec 17, 2025 •

edited

Loading

codecov Bot commented Dec 17, 2025 •

edited

Loading