Skip to content

Run the decorating label provider tests only once - #4454

Merged
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:lv/jface-duplicate-label-provider-suite
Oct 8, 2026
Merged

vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:lv/jface-duplicate-label-provider-suite

Conversation

@vogella

@vogella vogella commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

DecoratingLabelProviderTests was a suite of seven label provider test classes that AllLabelProviderTests already lists directly, and it was itself selected by both AllLabelProviderTests and AllTests. Each of these classes therefore ran three times per build, as the CI log shows for IDecorationContextTest. Removing the redundant suite runs them once and drops 40 duplicate test executions from org.eclipse.jface.tests.

DecoratingLabelProviderTests selected seven classes that
AllLabelProviderTests already lists, and it was itself selected by both
AllLabelProviderTests and AllTests, so each of these classes ran three
times. The redundant suite is removed.

Assisted-by: multiple AI agents and layers of automated tooling 🤖
@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Test Results

   864 files  ±0     864 suites  ±0   46m 40s ⏱️ - 1m 59s
 8 395 tests ±0   8 153 ✅ ±0  242 💤 ±0  0 ❌ ±0 
21 054 runs  ±0  20 377 ✅ ±0  677 💤 ±0  0 ❌ ±0 

Results for commit e2b02f9. ± Comparison against base commit 21e6248.

@vogella
vogella merged commit 3d3a2e2 into eclipse-platform:master Oct 8, 2026
18 checks passed
@vogella
vogella deleted the lv/jface-duplicate-label-provider-suite branch October 8, 2026 20:48
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.

1 participant