thepaperpilot/forgejo
1
0
Fork 0
Code Issues Pull requests Releases Activity Actions

fix(tests): 30s to cancel processes to avoid false negatives

Browse source 
on slower machines it can take more than 1 second to cancel leftover
tasks
This commit is contained in:
Earl Warren 2024-04-18 18:45:40 +02:00
parent c7b8a434c3
commit 6316e21be2
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/test_utils.go
View file

@ -224,7 +224,7 @@ func PrepareTestEnv(t testing.TB, skip ...int) func() {
// kill all background processes to prevent them from interfering with the fixture loading // kill all background processes to prevent them from interfering with the fixture loading
// see https://codeberg.org/forgejo/forgejo/issues/2962 // see https://codeberg.org/forgejo/forgejo/issues/2962
cancelProcesses(t, time.Second) cancelProcesses(t, 30*time.Second)
t.Cleanup(func() { cancelProcesses(t, 0) }) // cancel remaining processes in a non-blocking way t.Cleanup(func() { cancelProcesses(t, 0) }) // cancel remaining processes in a non-blocking way
// load database fixtures // load database fixtures