Skip to content

test: limit test-default ITs to 4 in parallel - #25422

Draft
mcollovati wants to merge 3 commits into
mainfrom
test/ci-flaky-tests
Draft

test: limit test-default ITs to 4 in parallel#25422
mcollovati wants to merge 3 commits into
mainfrom
test/ci-flaky-tests

Conversation

@mcollovati

Copy link
Copy Markdown
Collaborator

Without a cap TestBench starts all of them at once, and the browser burst floods the dev server while it is still installing packages and pre-bundling frontend dependencies.

@github-actions github-actions Bot added the +0.0.1 label Sep 1, 2026
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Test Results

 1 433 files  ±0   1 517 suites  ±0   1h 36m 44s ⏱️ + 2m 32s
11 859 tests ±0  11 792 ✅ ±0  67 💤 ±0  0 ❌ ±0 
12 178 runs  ±0  12 110 ✅ ±0  68 💤 ±0  0 ❌ ±0 

Results for commit b0bc72a. ± Comparison against base commit c740619.

♻️ This comment has been updated with latest results.

@Artur-

Artur- commented Sep 2, 2026

Copy link
Copy Markdown
Member

Is it a parallel limit that is needed or a generic wait for the app to fully start before starting ITs?

@mcollovati

Copy link
Copy Markdown
Collaborator Author

Wait before starting tests might work. But anyway running 50 parallel browsers seems a bit too much to me

Without a cap TestBench starts all of them at once, and the browser
burst floods the dev server while it is still installing packages and
pre-bundling frontend dependencies.
Deriving groups from ${excludedGroups} resolved to the empty value the
same profile sets, leaving no filter at all, so the category list now
lives in its own property that the profile can reference.
@sonarqubecloud

sonarqubecloud Bot commented Sep 3, 2026

Copy link
Copy Markdown

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants