databendlabs / openraft / 24136120416

86%

Build Type

push

github

Committed by drmingdrmer

Commit Message

fix: prevent redundant election when single voter is already electing

When a single-voter node calls `elect()` during `initialize()`, it
becomes a Candidate with an uncommitted vote. Before the vote flush
completes, an election tick can fire and trigger a second `elect()`,
bumping the term unnecessarily (e.g., T1 → T2). This caused flaky
test failures in `single-term-leader` mode where `new_cluster()`
expected `vote == <T1-N0:Q>` but observed `<T2-N0:Q>`.

The fix checks `candidate_ref().is_some()` to detect an in-progress
election. This is precise because the `Candidate` struct only exists
between an `elect()` call and becoming Leader — it is never present
after a node restart, unlike vote-based checks (`is_leading()`) which
would incorrectly match persisted votes from a previous run.

Coverage Stats

2 of 4 new or added lines in 1 file covered. (50.0%)

6 existing lines in 3 files now uncovered.

15959 of 18563 relevant lines covered (85.97%)

120574.6 hits per line