Skip to content

Conversation

@vogella
Copy link
Contributor

@vogella vogella commented Jan 6, 2026

Migrate performance tests to JUnit 5 to avoid dependency on PerformanceTestCaseJunit4.

  • Update UIPerformanceTestRule to implement BeforeAllCallback and AfterAllCallback.
  • Migrate GenerateIdentifiersTest and LabelProviderTest to use PerformanceMeter directly.
  • Use CloseTestWindowsExtension instead of CloseTestWindowsRule.
  • Add necessary JUnit 5 imports to MANIFEST.MF.

This is part of the work for issue #3639.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

Test Results

 3 015 files  ±0   3 015 suites  ±0   2h 11m 32s ⏱️ + 4m 26s
 8 258 tests ±0   8 010 ✅ ±0  248 💤 ±0  0 ❌ ±0 
23 598 runs  ±0  22 807 ✅ ±0  791 💤 ±0  0 ❌ ±0 

Results for commit c3dff4d. ± Comparison against base commit e81ba1c.

♻️ This comment has been updated with latest results.

@vogella
Copy link
Contributor Author

vogella commented Jan 7, 2026

Fixed the unused flushState parameter warning in ComputeSizeTest.java

1 similar comment
@vogella
Copy link
Contributor Author

vogella commented Jan 7, 2026

Fixed the unused flushState parameter warning in ComputeSizeTest.java

@vogella vogella force-pushed the migrate-performance-tests-junit5 branch 4 times, most recently from b8a6899 to f7c3c4a Compare January 11, 2026 19:57
Migrate performance tests to JUnit 5 to avoid dependency on PerformanceTestCaseJunit4.
- Update UIPerformanceTestRule to implement BeforeAllCallback and AfterAllCallback.
- Migrate GenerateIdentifiersTest and LabelProviderTest to use PerformanceMeter directly.
- Use CloseTestWindowsExtension instead of CloseTestWindowsRule.
- Add necessary JUnit 5 imports to MANIFEST.MF.
@vogella vogella force-pushed the migrate-performance-tests-junit5 branch from f7c3c4a to c3dff4d Compare January 12, 2026 08:49
@vogella vogella merged commit 001deea into eclipse-platform:master Jan 12, 2026
18 checks passed
@vogella vogella deleted the migrate-performance-tests-junit5 branch January 12, 2026 10: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