-
Notifications
You must be signed in to change notification settings - Fork 69
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
IF: Add a new set_finalizer (with weights, threshold, and local finalizer) to libtester and add more tests using it #2208
Conversation
…o libtester; add more unittests for finality transition using the new function
unittests/api_tests.cpp
Outdated
"alice"_n, "bob"_n, "carol"_n | ||
}; | ||
|
||
base_tester::finalizer_policy_input ploicy_input = { |
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.
type in all the ploicy_input
.
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.
Will correct them.
|
||
// libtester uses 1 as weight of each of the finalizer, sets (2/3 finalizers + 1) | ||
// as threshold, and makes all finalizers vote QC | ||
transaction_trace_ptr set_finalizers(const vector<account_name>& finalier_names); |
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.
typo in finalier_names
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.
Thanks!
…onst finalizer_policy_input&); correct 2 typos
Note:start |
Note:start |
threshold_equal_to_half_weight_sum_test
,votes_equal_to_threshold_test
,votes_greater_than_threshold_test
, andvotes_less_than_threshold_test
.Resolved #2193