-
Notifications
You must be signed in to change notification settings - Fork 3.6k
chain_plugin db intrinsic table RPC calls incorrectly handling --lower and --upper in certain scenarios #9805
Conversation
…r and --upper in certain scenarios
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.
As this fixes a bug, can you add a unit test case so we don't regress in the future?
…tly handling --lower and --upper in certain scenarios
Added unit tests |
Removed |
tests/get_table_tests.cpp
Outdated
@@ -116,6 +116,17 @@ BOOST_AUTO_TEST_CASE_TEMPLATE( get_scope_test, TESTER_T, backing_store_ts) { try | |||
BOOST_REQUIRE_EQUAL(name("initc"_n), result.rows[1].scope); | |||
} | |||
|
|||
param.lower_bound = "initb"; | |||
param.upper_bound = "initd"; |
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.
I missed that there was a scope with "initd", so you need to set upper_bound = "inite" (or past the last scope, so you could do param.upper_bound = "z"; which should guarantee that is past the last scope, since I doubt it has any scope values that start with "z".
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.
Updated
Change Description
This PR fixes issue in https://blockone.atlassian.net/browse/EPE-515 where table RPC calls incorrectly handling --lower and --upper in certain scenarios
Change Type
Select ONE:
Testing Changes
Select ANY that apply:
Consensus Changes
API Changes
Documentation Additions