Skip to content

Commit 2a29a43

Browse files
authored
Merge pull request #5893 from stacks-network/fix/reload-config-flaky-test
fix: flaky reload_miner_config test
2 parents c89936c + fb30553 commit 2a29a43

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

testnet/stacks-node/src/tests/nakamoto_integrations.rs

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11115,6 +11115,8 @@ fn reload_miner_config() {
1111511115
// setup sender + recipient for some test stx transfers
1111611116
// these are necessary for the interim blocks to get mined at all
1111711117
let sender_addr = tests::to_addr(&sender_sk);
11118+
let old_burn_fee_cap: u64 = 100000;
11119+
conf.burnchain.burn_fee_cap = old_burn_fee_cap;
1111811120
conf.add_initial_balance(PrincipalData::from(sender_addr).to_string(), 1000000);
1111911121
conf.add_initial_balance(PrincipalData::from(signer_addr).to_string(), 100000);
1112011122

@@ -11149,8 +11151,6 @@ fn reload_miner_config() {
1114911151
file.write_all(new_config.as_bytes()).unwrap();
1115011152
};
1115111153

11152-
update_config(100000, 50);
11153-
1115411154
let mut run_loop = boot_nakamoto::BootRunLoop::new(conf.clone()).unwrap();
1115511155
let run_loop_stopper = run_loop.get_termination_switch();
1115611156
let counters = run_loop.counters();
@@ -11180,6 +11180,8 @@ fn reload_miner_config() {
1118011180

1118111181
wait_for_first_naka_block_commit(60, &commits_submitted);
1118211182

11183+
next_block_and_mine_commit(&mut btc_regtest_controller, 60, &conf, &counters).unwrap();
11184+
next_block_and_mine_commit(&mut btc_regtest_controller, 60, &conf, &counters).unwrap();
1118311185
next_block_and_mine_commit(&mut btc_regtest_controller, 60, &conf, &counters).unwrap();
1118411186

1118511187
let burn_blocks = test_observer::get_burn_blocks();
@@ -11195,7 +11197,9 @@ fn reload_miner_config() {
1119511197
.map(|r| r.get("amt").unwrap().as_u64().unwrap())
1119611198
.sum::<u64>();
1119711199

11198-
assert_eq!(reward_amount, 200000);
11200+
let burn_amount = burn_block.get("burn_amount").unwrap().as_u64().unwrap();
11201+
11202+
assert_eq!(reward_amount + burn_amount, old_burn_fee_cap);
1119911203

1120011204
next_block_and_mine_commit(&mut btc_regtest_controller, 60, &conf, &counters).unwrap();
1120111205

@@ -11221,7 +11225,9 @@ fn reload_miner_config() {
1122111225
.map(|r| r.get("amt").unwrap().as_u64().unwrap())
1122211226
.sum::<u64>();
1122311227

11224-
assert_eq!(reward_amount, new_amount);
11228+
let burn_amount = burn_block.get("burn_amount").unwrap().as_u64().unwrap();
11229+
11230+
assert_eq!(reward_amount + burn_amount, new_amount);
1122511231

1122611232
coord_channel
1122711233
.lock()

0 commit comments

Comments
 (0)