Skip to content

fix: increase timeout for backgroundBuild test#1168

Closed
dobrac wants to merge 1 commit into
mainfrom
dobrac/fix-bg-build-test
Closed

fix: increase timeout for backgroundBuild test#1168
dobrac wants to merge 1 commit into
mainfrom
dobrac/fix-bg-build-test

Conversation

@dobrac
Copy link
Copy Markdown
Contributor

@dobrac dobrac commented Feb 26, 2026

The buildInBackground test was timing out with a 10 second limit. The buildInBackground method makes multiple sequential API calls (create template, get file upload links, trigger build) before returning, which exceeds the tight 10 second budget. Removed the explicit timeout override so the test inherits the workspace default of 180 seconds, which is appropriate for integration tests.


Note

Low Risk
Low risk: only changes a Vitest integration test timeout behavior to avoid flaky failures; no production logic is modified.

Overview
Removes the explicit 10_000ms timeout override from backgroundBuild.test.ts so the build template in background test can use the suite/workspace default timeout (reducing flakiness from slow sequential API calls in Template.buildInBackground).

Written by Cursor Bugbot for commit e945a4d. This will update automatically on new commits. Configure here.

The buildInBackground test was failing with a 10 second timeout. The test makes multiple sequential API calls (create template, get file upload links, trigger build) before returning, which exceeds the 10 second budget. Removed the explicit timeout override so the test inherits the workspace default of 180 seconds.
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Feb 26, 2026

⚠️ No Changeset found

Latest commit: e945a4d

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

@dobrac dobrac added the bug Something isn't working label Feb 26, 2026
@dobrac dobrac closed this Feb 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Package Artifacts

Built from 579e728. Download artifacts from this workflow run.

JS SDK (e2b@2.13.1-dobrac-fix-bg-build-test.0):

npm install ./e2b-2.13.1-dobrac-fix-bg-build-test.0.tgz

CLI (@e2b/cli@2.7.3-dobrac-fix-bg-build-test.0):

npm install ./e2b-cli-2.7.3-dobrac-fix-bg-build-test.0.tgz

Python SDK (e2b==2.14.0+dobrac-fix-bg-build-test):

pip install ./e2b-2.14.0+dobrac.fix.bg.build.test-py3-none-any.whl

@ValentaTomas ValentaTomas deleted the dobrac/fix-bg-build-test branch March 20, 2026 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant