`/internal/http` tests have a race
chrisdoherty4 opened this issue · 1 comments
chrisdoherty4 commented
The /internal/http tests have a race condition as it waits for the HTTP server to be deployed. Given we know we'll need some integration tests for the Kubernetes backend, it makes sense to pull this out of unit testing (given its launching a real server) and create an integration test suite that can be run independently of unit tests.
chrisdoherty4 commented
A short term patch can be to just time.Sleep() after launching the serving routine.