Skip to content

Migrates two more performance tests from PerformanceTestCaseJunit4 to JUnit 5 as part of #3639 - #4494

Open
ravnskjaer wants to merge 1 commit into
eclipse-platform:masterfrom
ravnskjaer:junit5-perf-tests-3639
Open

ravnskjaer wants to merge 1 commit into
eclipse-platform:masterfrom
ravnskjaer:junit5-perf-tests-3639

Conversation

@ravnskjaer

Copy link
Copy Markdown
Contributor
  • OpenCloseWindowTest
  • OpenClosePerspectiveTest

Changes

  • Remove dependency on PerformanceTestCaseJunit4.
  • Migrate OpenCloseWindowTest to use PerformanceMeter directly.
  • Migrate OpenClosePerspectiveTest to use PerformanceTestCaseJunit5.
  • Use CloseTestWindowsExtension instead of CloseTestWindowsRule.
  • Use @ParameterizedTest instead of RunWith(Parameterized.class) annotation.

Part of the work for #3639

@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Test Results

   867 files  ±0     867 suites  ±0   46m 21s ⏱️ - 13m 47s
 8 465 tests ±0   8 223 ✅ ±0  242 💤 ±0  0 ❌ ±0 
21 273 runs  ±0  20 590 ✅ ±0  683 💤 ±0  0 ❌ ±0 

Results for commit 661d76b. ± Comparison against base commit 3faf11c.

♻️ This comment has been updated with latest results.

…lipse-platform#3639

- Remove dependency on PerformanceTestCaseJunit4.
- Migrate OpenCloseWindowTest to use PerformanceMeter directly.
- Migrate OpenClosePerspectiveTest to use PerformanceTestCaseJunit5.
- Use CloseTestWindowsExtension instead of CloseTestWindowsRule.
- Use @ParameterizedTest instead of @RunWith(Parameterized.class)

Part of the work for eclipse-platform#3639
@ravnskjaer
ravnskjaer force-pushed the junit5-perf-tests-3639 branch from 870a677 to 661d76b Compare October 16, 2026 20:03
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