Skip to content

Commit

Permalink
Prevent intermittent failures in RepoIndexerTest (go-gitea#19225)
Browse files Browse the repository at this point in the history
The RepoIndexerTest is failing with considerable frequency due to a race inherrent in
its design. This PR adjust this test to avoid the reliance on waiting for the populate
repo indexer to run and forcibly adds the repo to the queue. It then flushes the queue.

It may be worth separating out the tests somewhat by testing the Index function
directly away from the queue however, this forceful method should solve the current
problem.

Signed-off-by: Andrew Thornton <art27@cantab.net>
  • Loading branch information
zeripath authored and Stelios Malathouras committed Mar 28, 2022
1 parent c2f8eda commit 37e1915
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions modules/indexer/stats/indexer_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,14 @@
package stats

import (
"context"
"path/filepath"
"testing"
"time"

repo_model "code.gitea.io/gitea/models/repo"
"code.gitea.io/gitea/models/unittest"
"code.gitea.io/gitea/modules/queue"
"code.gitea.io/gitea/modules/setting"

_ "code.gitea.io/gitea/models"
Expand All @@ -32,10 +34,14 @@ func TestRepoStatsIndex(t *testing.T) {
err := Init()
assert.NoError(t, err)

time.Sleep(5 * time.Second)

repo, err := repo_model.GetRepositoryByID(1)
assert.NoError(t, err)

err = UpdateRepoIndexer(repo)
assert.NoError(t, err)

queue.GetManager().FlushAll(context.Background(), 5*time.Second)

status, err := repo_model.GetIndexerStatus(repo, repo_model.RepoIndexerTypeStats)
assert.NoError(t, err)
assert.Equal(t, "65f1bf27bc3bf70f64657658635e66094edbcb4d", status.CommitSha)
Expand Down

0 comments on commit 37e1915

Please sign in to comment.