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 13m 10s ⏱️ - 2m 47s
 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 c616082. ± Comparison against base commit 93acda3.

♻️ 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

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.
Migrate ComputeSizeTest, LayoutTest, and ResizeTest to JUnit 5.
- Introduce LayoutPerformanceTestSuite for shared data providers.
- Remove inheritance from PerformanceTestCaseJunit4.
- Use CloseTestWindowsExtension.
- Use PerformanceMeter directly.
- Ensure INTRO_VIEW in UIPerformanceTestRule is visible.
@vogella vogella force-pushed the migrate-performance-tests-junit5 branch from 3e6b9df to c616082 Compare January 8, 2026 15:33
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