Skip to content

chore: remove Mockito from test dependencies#85

Merged
deer merged 1 commit into
mainfrom
remove_mockito
Jun 10, 2026
Merged

chore: remove Mockito from test dependencies#85
deer merged 1 commit into
mainfrom
remove_mockito

Conversation

@deer

@deer deer commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Removes Mockito from all modules that depended on it (base-configuration, base-flow, base-mereology, base-query, base-retryable) and replaces every mock with a hand-rolled test double or a simple inline lambda. The maven-dependency-plugin and maven-surefire-plugin build configuration that each module carried solely to wire Mockito as a -javaagent is removed as well, along with the root mockito.version property and its dependencyManagement entry.

Replacements: a TrackingSubscriber inner class in SubscriberRegistryTests captures subscription, items, completion, and error state in place of mock(Subscriber.class) + ArgumentCaptor; a shared NO_OP_SUBSCRIPTION constant stands in for mocked Subscriptions in FilteringSubscriberTests and MappingSubscriberTests; AtomicBoolean/AtomicInteger counters replace mocked suppliers and retryables in ConfigurationTests and BlockingRetryTests.

@deer deer merged commit 8a4c3a9 into main Jun 10, 2026
3 checks passed
@deer deer deleted the remove_mockito branch June 10, 2026 08:26
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