Skip to content

fix: QBFT#417

Draft
iamquang95 wants to merge 1 commit into
mainfrom
iamquang95/qbft
Draft

fix: QBFT#417
iamquang95 wants to merge 1 commit into
mainfrom
iamquang95/qbft

Conversation

@iamquang95
Copy link
Copy Markdown
Collaborator

This PR

  • Hardens the QBFT tests. Makes the test parity with charon: Previously, some params are misconfigured.
  • Add fuzz tests for QBFT matches with charon.
  • Add some more unit tests which helps the fix QBFT stricter.
  • Add more chain split test.

@iamquang95 iamquang95 changed the title fix: qbft fix: QBFT May 15, 2026
);
let (delay_ch, _) = clock.new_timer(delay);
_ = delay_ch.recv();
let delta_ms = (f64::from(jitter_ms) * deterministic_unit(seed, &msg, 0, 3)) as i32;
msg.round(),
delay
));
let key = deterministic_msg_u64(seed, &msg, 0, 4);
}

if let Some(p) = drop_prob.get(&msg.source()) {
if deterministic_unit(seed, &msg, *target, 1) < *p {
out_tx.send(msg.clone()).expect(WRITE_CHAN_ERR);
broadcasts += 1;

if deterministic_unit(seed, &msg, *target, 2) < 0.1 {
MSG_DECIDED,
];
new_msg(
message_types[deterministic_range(seed, counter, 10, message_types.len())],
message_types[deterministic_range(seed, counter, 10, message_types.len())],
instance,
peer_idx,
deterministic_i64(seed, counter, 11, 10),
instance,
peer_idx,
deterministic_i64(seed, counter, 11, 10),
deterministic_i64(seed, counter, 12, 10),
deterministic_i64(seed, counter, 11, 10),
deterministic_i64(seed, counter, 12, 10),
0,
deterministic_i64(seed, counter, 13, 10),
deterministic_i64(seed, counter, 12, 10),
0,
deterministic_i64(seed, counter, 13, 10),
deterministic_i64(seed, counter, 14, 10),
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants