prisma-risk / tsoracle / 26297617366

95%

push

github

fix(openraft-toolkit): dedup leadership_events by full value (#117)

RoleClass-based dedup stripped term and follower-leader identity from
the key, so a watch-coalesced Leader(N) -> Follower(M) -> Leader(K)
sequence was suppressed (both endpoints share RoleClass::Leader).
Downstream, tsoracle-server's failover fence only runs on Leader
events, so a missed term change meant a missed fence -- the allocator
kept serving from the prior epoch's seeded floor, producing a global
ordering regression of the same class as #72.

Switches the dedup key to LeadershipState<C>, which already derives
PartialEq. Every change in role, term, or follower-leader identity is
now emitted; identical projections still compare equal and remain
suppressed, so steady-state metric ticks don't produce event storms.

Removes the private RoleClass enum and the class() method that
existed only to support the broken dedup.

Adds two regression tests in tests/lifecycle.rs:
- emits Leader { term: K } after the channel coalesces
  Follower(M) + Leader(K) past a previously-emitted Leader(N)
- suppresses an identical projection sent through the same channel,
  pinning the steady-state-tick invariant under the new key

6 of 6 new or added lines in 1 file covered. (100.0%)

6553 of 6804 relevant lines covered (96.31%)

84999.17 hits per line