Skip to content

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

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

test: limit test-default ITs to 4 in parallel#25422
mcollovati wants to merge 4 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 435 files  ±0   1 519 suites  ±0   1h 29m 49s ⏱️ +21s
11 894 tests ±0  11 826 ✅ ±0  68 💤 ±0  0 ❌ ±0 
12 212 runs  ±0  12 144 ✅ ±0  68 💤 ±0  0 ❌ ±0 

Results for commit 046947f. ± Comparison against base commit fb60274.

♻️ 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 7, 2026

Copy link
Copy Markdown

@mcollovati

Copy link
Copy Markdown
Collaborator Author

This test module is not executed anymore in the CI environment.
The "wait for dev server start" gateway might be useful anyway. Otherwise, we can close this PR.

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