-
Notifications
You must be signed in to change notification settings - Fork 266
Conversation
7a3ae58
to
a0a7dfc
Compare
a0a7dfc
to
5455c2f
Compare
55c54e5
to
06ca4c7
Compare
test/miner_blockchain_SUITE.erl
Outdated
BinPub4 = libp2p_crypto:pubkey_to_bin(Pub4), | ||
BinPub5 = libp2p_crypto:pubkey_to_bin(Pub5), | ||
|
||
Txn6 = vars(#{?use_multi_keys => true}, 2, Priv), |
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.
Why are these being done in discrete var updates?
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.
to make sure that that transition works safely, just in case it happens. it's done the other way, as an atomic update, in the election test.
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.
oops, that's not true, will change this one.
Txn7 = blockchain_txn_vars_v1:proof(Txn7_1, Proof7), | ||
_ = [ok = ct_rpc:call(M, blockchain_worker, submit_txn, [Txn7]) || M <- Miners], | ||
|
||
ct:pal("transitioned to multikey"), |
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 we use wait_for_chain_var_update here too?
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.
yeah, fixing
|
support for helium/blockchain-core#526