This repository was archived by the owner on Jan 12, 2020. It is now read-only.
created (failing) integration test checking consistency of 3 instances#53
Merged
Conversation
Test is very simple and crude... * we start 3 services on ports 10001,10002, and 10003 each managing a simple counter * we wait for leader to settle then loop commiting a single byte message * we expect all instances to have same value for counter
Collaborator
Author
|
Oddly enough, test is failing because of an off-by-one error in my current proto rpc segregation branch but does not loop forever. I am puzzled... |
Collaborator
Author
|
Build is failing because I used Java 7 features hence running on jdk 6 will fail. Will update the change to fall back on pure Java 6. |
Collaborator
|
Looks good to me. I think we'll need integration tests for testing cluster reconfiguration ( #54), so this is great timing! Adds new dependency on AssertJ, but that looks like a nice library, and is only referenced in test mode. |
mgodave
added a commit
that referenced
this pull request
Jan 30, 2014
created (failing) integration test checking consistency of 3 instances
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
simple counter
Test is very simple and crude and I did not investigate (yet) why this test is failing...