From 7060d75527c44979939e190edb7c884d94254488 Mon Sep 17 00:00:00 2001 From: Tobias Grieger Date: Tue, 6 Sep 2022 10:27:18 +0200 Subject: [PATCH] TestLeaderOnlyCommitsLogFromCurrentTerm Leader only acks to itself on `(*raft).advance` so we have to make this test a bit more like the real thing. Signed-off-by: Tobias Grieger --- raft/raft_paper_test.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/raft/raft_paper_test.go b/raft/raft_paper_test.go index 54b38b438..84c451cdf 100644 --- a/raft/raft_paper_test.go +++ b/raft/raft_paper_test.go @@ -894,6 +894,9 @@ func TestLeaderOnlyCommitsLogFromCurrentTerm(t *testing.T) { r.Step(pb.Message{From: 1, To: 1, Type: pb.MsgProp, Entries: []pb.Entry{{}}}) r.Step(pb.Message{From: 2, To: 1, Type: pb.MsgAppResp, Term: r.Term, Index: tt.index}) + rd := newReady(r, &SoftState{}, pb.HardState{}) + storage.Append(rd.Entries) + r.advance(rd) if r.raftLog.committed != tt.wcommit { t.Errorf("#%d: commit = %d, want %d", i, r.raftLog.committed, tt.wcommit) }