Skip to content

Unify test case names in FontRegistryTest - #4276

Merged
HeikoKlare merged 1 commit into
eclipse-platform:masterfrom
HeikoKlare:fontregistry-test-consistentnames
Aug 24, 2026
Merged

Unify test case names in FontRegistryTest#4276
HeikoKlare merged 1 commit into
eclipse-platform:masterfrom
HeikoKlare:fontregistry-test-consistentnames

Conversation

@HeikoKlare

Copy link
Copy Markdown
Contributor

Two test cases for the same behavior of retrieving bold/italic fonts had inconsistent names.

@github-actions

github-actions Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Test Results

   858 files  ±0     858 suites  ±0   1h 6m 39s ⏱️ + 18m 34s
 8 172 tests ±0   7 929 ✅ ±0  243 💤 ±0  0 ❌ ±0 
20 418 runs  ±0  19 762 ✅ ±0  656 💤 ±0  0 ❌ ±0 

Results for commit eea1530. ± Comparison against base commit f08b692.

This pull request removes 1 and adds 1 tests. Note that renamed tests count towards both.
org.eclipse.jface.tests.resources.FontRegistryTest ‑ multipleDisplay_italicFont
org.eclipse.jface.tests.resources.FontRegistryTest ‑ multipleDisplayDispose_italicFont

♻️ This comment has been updated with latest results.

Two test cases for the same behavior of retrieving bold/italic fonts had
inconsistent names.
@HeikoKlare
HeikoKlare force-pushed the fontregistry-test-consistentnames branch from 57c3530 to eea1530 Compare August 24, 2026 13:01
@HeikoKlare

Copy link
Copy Markdown
Contributor Author

Just test correction, thus merging despite freeze period.

@HeikoKlare
HeikoKlare merged commit 7cc26a1 into eclipse-platform:master Aug 24, 2026
18 checks passed
@HeikoKlare
HeikoKlare deleted the fontregistry-test-consistentnames branch August 24, 2026 14:27
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