Skip to content

Migrate org.eclipse.jface.tests to JUnit 5#3730

Merged
vogella merged 1 commit intoeclipse-platform:masterfrom
vogella:jface-tests
Feb 12, 2026
Merged

Migrate org.eclipse.jface.tests to JUnit 5#3730
vogella merged 1 commit intoeclipse-platform:masterfrom
vogella:jface-tests

Conversation

@vogella
Copy link
Contributor

@vogella vogella commented Feb 11, 2026

  • Migrated all tests in org.eclipse.jface.tests to JUnit 5.
  • Updated MANIFEST.MF to use org.junit.jupiter.api and removed JUnit 4.
  • Replaced JUnit 4 Rules with JUnit 5 Extensions or @tempdir.
  • Fixed assertion and assumption parameter order for JUnit 5.
  • Removed unused JFaceActionRule.
  • Replaced JUnit 3 AssertionFailedError with IllegalStateException in TestLazyModelContentProvider.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 11, 2026

Test Results

 3 024 files   3 024 suites   2h 11m 54s ⏱️
 8 234 tests  7 986 ✅ 248 💤 0 ❌
23 526 runs  22 735 ✅ 791 💤 0 ❌

Results for commit 514e49c.

♻️ This comment has been updated with latest results.

- Migrated all tests in org.eclipse.jface.tests to JUnit 5.
- Updated MANIFEST.MF to use org.junit.jupiter.api and removed JUnit 4.
- Replaced JUnit 4 Rules with JUnit 5 Extensions or @tempdir.
- Fixed assertion and assumption parameter order for JUnit 5.
- Removed unused JFaceActionRule.
- Replaced JUnit 3 AssertionFailedError with IllegalStateException in TestLazyModelContentProvider.
@vogella vogella merged commit 676351a into eclipse-platform:master Feb 12, 2026
18 checks passed
@vogella vogella deleted the jface-tests branch February 12, 2026 09:46
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