lhotari opened a new pull request, #19337: URL: https://github.com/apache/pulsar/pull/19337
### Motivation This is a follow up on previous PRs to refactor Pulsar unit tests to reduce mocking which causes flakiness since Mockito isn't thread safe. The goal is proceed in expanding the coverage of the reduction of reliance on mocking. There are 2 types of Pulsar unit tests that use a PulsarService: - Some Pulsar unit tests use a PulsarService that isn't started - Some Pulsar unit tests start the PulsarService and use less mocking ### Modifications This PR refactors the previous TestPulsarService to PulsarTestContext which is more maintainable. It supports both modes of PulsarServices, startable and non-startable. OwnerShipForCurrentServerTestBase is used as piloting the new mode. More tests will be converted after this PR has been merged. ### Documentation <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. --> - [ ] `doc` <!-- Your PR contains doc changes. Please attach the local preview screenshots (run `sh start.sh` at `pulsar/site2/website`) to your PR description, or else your PR might not get merged. --> - [ ] `doc-required` <!-- Your PR changes impact docs and you will update later --> - [x] `doc-not-needed` <!-- Your PR changes do not impact docs --> - [ ] `doc-complete` <!-- Docs have been already added --> ### Matching PR in forked repository PR in forked repository: https://github.com/lhotari/pulsar/pull/138 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
