Skip to content

fix(webide-client-tools): increase bundleFeature test timeout to 20s#513

Merged
bd82 merged 1 commit into
mainfrom
fix/webide-test-run
May 31, 2026
Merged

fix(webide-client-tools): increase bundleFeature test timeout to 20s#513
bd82 merged 1 commit into
mainfrom
fix/webide-test-run

Conversation

@alex-gilin
Copy link
Copy Markdown
Contributor

10s was too tight for webpack bundling on CI runners, causing a timeout that left stale dist folders and cascaded into a second failure (expected 1 timestamp folder, found 11).

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 31, 2026

Build Report

badge

Please note:

  1. Files only stay for around 14 days!
  2. This comment will be updated with the data of the last successful build of this PR.
Name Link
Commit 1aa3d1f
Logs https://github.com/SAP/app-studio-toolkit/actions/runs/26711054180
VSIX Files https://github.com/SAP/app-studio-toolkit/actions/runs/26711054180/artifacts/7316269878

@alex-gilin alex-gilin requested a review from bd82 May 31, 2026 07:09
Copy link
Copy Markdown
Member

@bd82 bd82 left a comment

Choose a reason for hiding this comment

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

see review comment

Comment thread packages/webide-client-tools/test/bundling_spec.js Outdated
10s was too tight for webpack bundling on CI runners, causing a timeout
that left stale dist folders and cascaded into a second failure
(expected 1 timestamp folder, found 11).
@alex-gilin alex-gilin force-pushed the fix/webide-test-run branch from f128980 to 0ba6391 Compare May 31, 2026 11:14
@alex-gilin alex-gilin changed the title fix(webide-client-tools): increase bundleFeature test timeout to 60s fix(webide-client-tools): increase bundleFeature test timeout to 20s May 31, 2026
@bd82 bd82 merged commit 0ef1622 into main May 31, 2026
3 checks passed
@bd82 bd82 deleted the fix/webide-test-run branch May 31, 2026 11:24
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