Skip to content

Extending/organizing MatchersTest.kt#548

Merged
TimvdLippe merged 1 commit intomockito:mainfrom
m-koops:extend-matchers-test
Oct 28, 2025
Merged

Extending/organizing MatchersTest.kt#548
TimvdLippe merged 1 commit intomockito:mainfrom
m-koops:extend-matchers-test

Conversation

@m-koops
Copy link
Copy Markdown
Contributor

@m-koops m-koops commented Oct 26, 2025

Towards fixing #492.

I was on my way to add tests for unsigned integers, but then I found the test class MatchersTest lacking test coverage for many types already included in the Methods test interface. So I have added missing tests for any(), anyOrNull() and eq() matchers.
As the number of tests in the test class is growing, I have started to organize the tests by introducing nested test classes (in JUnit4 style).

All existing test cases still remain in the class but may have moved to the related nested test class, and test methods may have been adjusted slightly to allign with sibling tests in the same nested test class.

…in the Methods test interface. Also, organized MatchersTest.kt with nested test classes.
Copy link
Copy Markdown
Contributor

@TimvdLippe TimvdLippe left a comment

Choose a reason for hiding this comment

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

Thanks!

@TimvdLippe TimvdLippe merged commit d96c318 into mockito:main Oct 28, 2025
4 checks passed
@m-koops m-koops deleted the extend-matchers-test branch December 19, 2025 12:46
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.

2 participants