Skip to content

Commit

Permalink
make test happy
Browse files Browse the repository at this point in the history
Signed-off-by: husharp <[email protected]>
  • Loading branch information
HuSharp committed Jun 21, 2024
1 parent 41e6376 commit af36a92
Showing 1 changed file with 11 additions and 9 deletions.
20 changes: 11 additions & 9 deletions tests/server/cluster/cluster_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ func TestRegionStatistics(t *testing.T) {
re := require.New(t)
ctx, cancel := context.WithCancel(context.Background())
defer cancel()
tc, err := tests.NewTestCluster(ctx, 2)
tc, err := tests.NewTestCluster(ctx, 3)
defer tc.Destroy()
re.NoError(err)

Expand Down Expand Up @@ -223,9 +223,9 @@ func TestRegionStatistics(t *testing.T) {
time.Sleep(1000 * time.Millisecond)

leaderServer.ResignLeader()
newLeaderName := tc.WaitLeader()
re.NotEqual(newLeaderName, leaderName)
re.NotEqual(tc.WaitLeader(), leaderName)
leaderServer = tc.GetLeaderServer()
leaderName = leaderServer.GetServer().Name()
rc = leaderServer.GetRaftCluster()
r := rc.GetRegion(region.Id)
re.NotNil(r)
Expand All @@ -238,9 +238,9 @@ func TestRegionStatistics(t *testing.T) {
re.Len(regions, 1)

leaderServer.ResignLeader()
newLeaderName = tc.WaitLeader()
re.Equal(newLeaderName, leaderName)
re.NotEqual(tc.WaitLeader(), leaderName)
leaderServer = tc.GetLeaderServer()
leaderName = leaderServer.GetServer().Name()
rc = leaderServer.GetRaftCluster()
re.NotNil(r)
re.True(r.LoadedFromStorage() || r.LoadedFromSync())
Expand All @@ -255,11 +255,13 @@ func TestRegionStatistics(t *testing.T) {
re.False(r.LoadedFromStorage() && r.LoadedFromSync())

leaderServer.ResignLeader()
newLeaderName = tc.WaitLeader()
re.NotEqual(newLeaderName, leaderName)
re.NotEqual(tc.WaitLeader(), leaderName)
leaderServer = tc.GetLeaderServer()
leaderName = leaderServer.GetServer().Name()
leaderServer.ResignLeader()
newLeaderName = tc.WaitLeader()
re.Equal(newLeaderName, leaderName)
re.NotEqual(tc.WaitLeader(), leaderName)
leaderServer = tc.GetLeaderServer()
leaderName = leaderServer.GetServer().Name()

Check failure on line 264 in tests/server/cluster/cluster_test.go

View workflow job for this annotation

GitHub Actions / statics

ineffectual assignment to leaderName (ineffassign)
leaderServer = tc.GetLeaderServer()
rc = leaderServer.GetRaftCluster()
r = rc.GetRegion(region.Id)
Expand Down

0 comments on commit af36a92

Please sign in to comment.