empayre/fleet
14
0
Fork 0
mirror of https://github.com/empayre/fleet.git synced 2024-11-06 17:05:18 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix race in tests (modifying already started server config) (#8227)

Browse Source
This commit is contained in:
Lucas Manuel Rodriguez 2022-10-14 12:00:16 -03:00 committed by GitHub
parent 06a9e886a1
commit 03096ee546
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
server/service/testing_utils.go
View File

@ -253,12 +253,13 @@ func RunServerForTestsWithDS(t *testing.T, ds fleet.Datastore, opts ...*TestServ
}
limitStore, _ := memstore.New(0)
r := MakeHandler(svc, cfg, logger, limitStore, WithLoginRateLimit(throttled.PerMin(100)))
server := httptest.NewServer(r)
server := httptest.NewUnstartedServer(r)
// Set the same ReadTimeout as the actual server
server.Config.ReadTimeout = 25 * time.Second
server.Start()
t.Cleanup(func() {
server.Close()
})
// Set the same ReadTimeout as the actual server
server.Config.ReadTimeout = 25 * time.Second
return users, server
}