lhotari opened a new pull request #10209:
URL: https://github.com/apache/pulsar/pull/10209


   ### Motivation
   
   There are some tests that fail because of port conflicts. Fixed ports 6650, 
6651 and 8080 get used in a few tests.
   
   ### Modifications
   
   - Change the tests to use dynamic ports.
   - While making changes to LeaderElectionServiceTest, make it thread safe
     - It's not thread safe to modify Mockito mocks after they have been 
published to another thread. Change the logic to use AtomicBoolean and 
AtomicReference classes to achieve the desired result. 
   - While making changes to 
PulsarMultiListenersWith*InternalListenerNameTests, make 
PulsarMultiListenersWithoutInternalListenerNameTest class extend 
PulsarMultiListenersWithInternalListenerNameTest class to reduce test code 
duplication.


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to