-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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.
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
Fix fired events when initialMembersipListener is used on client #7543
Fix fired events when initialMembersipListener is used on client #7543
Conversation
When initial membership event added before client is started, it was firing individual ADDED events for current memberlist, along with InitialMembershipEvent. Only latter is preserved, other events are removed. Related tests are added. Membershiplistener related tests in ClientRegressionWithMockNetworkTest are moved to MembershipListenerTest. fixes hazelcast#7430
When initial membership event added before client is started, it was firing individual ADDED events for current memberlist, along with InitialMembershipEvent. Only latter is preserved, other events are removed. Related tests are added. Membershiplistener related tests in ClientRegressionWithMockNetworkTest are moved to MembershipListenerTest. fixes hazelcast#7430 backport of hazelcast#7543
* related to issue #1181 | ||
*/ | ||
@Test | ||
public void testAddInitialMembership_whenListenerAddedViaClientConfig() throws InterruptedException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is there no assert? What is being tested?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Before the fix, It was throwing NullPointer exception according to issue mentioned in #1181 . It is testing that it does not throw any exception.
👍 |
1 similar comment
👍 |
…master Fix fired events when initialMembersipListener is used on client
When initial membership event added before client is started,
it was firing individual ADDED events for current memberlist,
along with InitialMembershipEvent. Only latter is preserved,
other events are removed.
Related tests are added.
Membershiplistener related tests in ClientRegressionWithMockNetworkTest
are moved to MembershipListenerTest.
fixes #7430