From 1ee08f58b4878281af0cc947763e1b0a6890a9c9 Mon Sep 17 00:00:00 2001 From: Ibrahim Kettaneh Date: Fri, 4 Oct 2024 09:29:55 -0400 Subject: [PATCH] raft: clean up TestMsgAppFlowControl A recent commit removed the nested message checks in raft_flow_control_test. However One part was missed, and this commit fixes that. Epic: None Release note: None --- pkg/raft/raft_flow_control_test.go | 5 ----- 1 file changed, 5 deletions(-) diff --git a/pkg/raft/raft_flow_control_test.go b/pkg/raft/raft_flow_control_test.go index 3064bd14e7e0..d0bf705c144f 100644 --- a/pkg/raft/raft_flow_control_test.go +++ b/pkg/raft/raft_flow_control_test.go @@ -145,11 +145,6 @@ func TestMsgAppFlowControl(t *testing.T) { r.tick() } ms := r.readMessages() - if len(ms) != 3 || ms[0].Type != pb.MsgHeartbeat || ms[1].Type != pb.MsgFortifyLeader || - ms[2].Type != pb.MsgApp || len(ms[2].Entries) != 0 { - t.Fatalf("#%d.%d: len(ms) == %d, want 3 messages including one empty MsgApp", - tt, i, len(ms)) - } require.Len(t, ms, 3) require.Equal(t, ms[0].Type, pb.MsgHeartbeat) require.Equal(t, ms[1].Type, pb.MsgFortifyLeader)