-
Notifications
You must be signed in to change notification settings - Fork 28
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
Gh-478: Addition of config values written to database + updating db initialization and migration + better tooling in ntegration tests for Node #62
Conversation
…ny days - and plus formatting
… choosen in configurator_standard
…onfiguration' command
…ndand and is gone
…ith tools for dump config in integration tests are not flexible enough
… integration tests from GH-305
…ing happened; grrr
…ockchain_service_url; some small changes for BlockchainBridgeClandastine and verify_bill_payments()
let (serving_node_2_config, serving_node_2_wallet) = | ||
build_config(&blockchain_server, &seed, derivation_path(0, 2)); | ||
let (serving_node_3_config, serving_node_3_wallet) = | ||
build_config(&blockchain_server, &seed, derivation_path(0, 3)); | ||
|
||
let amount = 10u64 | ||
* u64::try_from(node_lib::accountant::PAYMENT_CURVES.permanent_debt_allowed_gwub).unwrap(); |
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.
gwub
is actually short for $SUB in Gwei
so if you felt so inclined to change it to gwei
or qwaq
or something else that would be fine. Or you could leave it too... that's fine. It's one of those 'you had to be there to know' kind of things.
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 think we will likely change the naming of this sometime soon - we are looking to change the naming of the payment curve values anyways, since they are very confusing!
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.
gwasq
?
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'm going to take care of the nomenclature in a different card because I've started that in there and I want to do it just at one place.
node/src/lib.rs
Outdated
@@ -4,7 +4,7 @@ | |||
#[macro_use] | |||
pub mod sub_lib; | |||
|
|||
#[cfg_attr(test, macro_use)] | |||
//#[cfg_attr(test, macro_use)] |
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.
Should this be deleted?
No description provided.