Skip to content

Comprehensively test ExecutionListenerAdapter #5202

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
mpkorstanje wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Comprehensively test ExecutionListenerAdapter #5202

mpkorstanje wants to merge 1 commit into from
+84 −36

Conversation

@mpkorstanje
Copy link
Contributor

@mpkorstanje mpkorstanje commented Dec 5, 2025 •
edited
Loading

With mockito/mockito#146 resolved it is now possible to spy on the TestExecutionListener which enables testing the ExecutionListenerAdapter without undue verbosity.

I hereby agree to the terms of the JUnit Contributor License Agreement.

Definition of Done

@mpkorstanje mpkorstanje mentioned this pull request Dec 5, 2025
Closed
@testlens-app
Copy link

testlens-app bot commented Dec 5, 2025 •
edited
Loading

✅ All tests passed ✅

🏷️ Commit: 421912a
▶️ Tests: 22066 executed
⚪️ Checks: 16/16 completed

@mpkorstanje mpkorstanje marked this pull request as ready for review December 5, 2025 12:06
@mpkorstanje
Comprehensively test ExecutionListenerAdapter
421912a 
With mockito/mockito#146 resolved it is now
possible to spy on the TestExecutionListener which enables the
ExecutionListenerAdapter without undue verbosity.
@mpkorstanje mpkorstanje force-pushed the rien/execution-listner-adaptor-tests branch from 09ada30 to 421912a Compare December 5, 2025 12:13
@marcphilipp marcphilipp self-requested a review December 5, 2025 13:08
marcphilipp
marcphilipp approved these changes Dec 5, 2025
View reviewed changes
Copy link
Member

@marcphilipp marcphilipp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marcphilipp marcphilipp marcphilipp approved these changes

Assignees

No one assigned

Labels

component: Platform theme: discovery type: task

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mpkorstanje @marcphilipp