Skip to content

Migrate TestFailReporter to JUnit 5 TestFailExtension#3680

Merged
vogella merged 1 commit intoeclipse-platform:masterfrom
vogella:migrate-testfailreporter
Jan 22, 2026
Merged

Migrate TestFailReporter to JUnit 5 TestFailExtension#3680
vogella merged 1 commit intoeclipse-platform:masterfrom
vogella:migrate-testfailreporter

Conversation

@vogella
Copy link
Contributor

@vogella vogella commented Jan 16, 2026

Extracts the TestFailReporter inner class from FileBufferFunctions into a standalone TestFailExtension class that implements JUnit 5's TestWatcher interface.

Updates FileBufferFunctions to use @RegisterExtension with the new extension.

This contributes to the JUnit 5 migration efforts.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 16, 2026

Test Results

 3 018 files  ±0   3 018 suites  ±0   2h 18m 9s ⏱️ - 11m 45s
 8 270 tests ±0   8 022 ✅ ±0  248 💤 ±0  0 ❌ ±0 
23 634 runs  ±0  22 843 ✅ ±0  791 💤 ±0  0 ❌ ±0 

Results for commit 760e14c. ± Comparison against base commit 62f3974.

♻️ This comment has been updated with latest results.

@vogella vogella force-pushed the migrate-testfailreporter branch from b2a051c to 7212335 Compare January 19, 2026 09:22
@vogella
Copy link
Contributor Author

vogella commented Jan 21, 2026

rules-replacement.md accidently got added to this commit, needs to be removed

Extracts the TestFailReporter inner class from FileBufferFunctions
into a standalone TestFailExtension class that implements
JUnit 5's TestWatcher interface.

Updates FileBufferFunctions to use @RegisterExtension with the
new extension.
@vogella vogella force-pushed the migrate-testfailreporter branch from 7212335 to 760e14c Compare January 21, 2026 15:11
@vogella vogella merged commit 34498a4 into eclipse-platform:master Jan 22, 2026
18 checks passed
@vogella vogella deleted the migrate-testfailreporter branch January 22, 2026 07:51
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