Add test for staged cluster administration #4
Conversation
ghost
assigned
There was a problem hiding this comment.
Maybe this should also escape quote characters in Arg before wrapping it?
re:replace(Arg, "\"", "\\\\\"", [global, {return, list}]).
|
I'd like to see the return values checked more as well. I'd also be nice to add a scenario like the one mentioned over in the riak pull request where you stage an add, down the node, and then do the add. |
|
The feedback on this test was overlooked and not addressed when staged clustering was merged in awhile back. In any case, I'm merging in side branches of I agree that this test should be improved in the future, and created a new issue to track that: #8 |
Add a test for `node_confirms` as put option
basho#4) * Add test for reaping tombs, and reflecting API changes associated with change * Extend test to cover protocols and failure And add to group * Must have tictacaae enabled But exchange tick set so that it will never run * Update nextgenrepl_reaptombs.erl Remove rogue config item * Update to test with bucket types * Remove rogue whitespace ProtocolVersionError was not matching due to rogue white space * Update nextgenrepl_reaptombs.erl * Update rebar.config
See basho/riak_core#177 and basho/riak_core#181