fedi/forgejo
1
0
Fork 0
mirror of https://codeberg.org/forgejo/forgejo.git synced 2024-11-22 05:36:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

bug: require.Eventually must not test with assert

Browse source 
Otherwise it fails the test instead of retrying if the condition fails
at least once.
This commit is contained in:
Earl Warren 2024-11-09 00:07:18 +01:00
parent 30f795d925
commit 2541a943ce
No known key found for this signature in database
GPG key ID: 0579CB2928A78A00
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
tests/integration/actions_trigger_test.go
View file

@ -321,7 +321,7 @@ jobs:
require.Eventually(t, func() bool { require.Eventually(t, func() bool {
actionRuns = make([]*actions_model.ActionRun, 0) actionRuns = make([]*actions_model.ActionRun, 0)
require.NoError(t, db.GetEngine(db.DefaultContext).Where("repo_id=?", baseRepo.ID).Find(&actionRuns)) require.NoError(t, db.GetEngine(db.DefaultContext).Where("repo_id=?", baseRepo.ID).Find(&actionRuns))
return assert.Len(t, actionRuns, count) return len(actionRuns) == count
}, 30*time.Second, 1*time.Second) }, 30*time.Second, 1*time.Second)
// verify the expected ActionRuns were created // verify the expected ActionRuns were created