-
Notifications
You must be signed in to change notification settings - Fork 75
Open
Labels
starmapshttps://www.starmaps.app/https://www.starmaps.app/
Description
eta: 2023-01
It looks like we have 2 ways to run integration tests:
- https://github.com/testground/testground/tree/master/pkg/cmd/itest
- https://github.com/testground/testground/tree/master/integration_tests
Some of these are testing the same commands.
The itest package makes go test -v ./pkg/... is annoying to use because integration tests are creating a lot of noise amongts unit tests.
We should merge all these tests to a single, clearer location.
A few notes:
- Using go instead of shell make sense here, the header.sh script is already quite complex.
- We need to take into account the fact that some integration tests require setup (k8s)
- Bonus point for parallelizing tests, we might be able to use a github action's matrix.
Design
- Review approaches
Tasks
- Create a integration_tests package that relies on [golang's testing tools]
- Implement the tooling required to make these tests maintainable(https://github.com/testground/testground/tree/master/pkg/cmd/itest)
- Move these tests to a github CI action, ideally with a matrix per environment (docker, k8s, exec, etc), or if possible: per test since they are quite long.
- port
test-integ-local-exec- Port the shell scripts to go
- port
test-integ-local-docker- Port the shell scripts to go
- port
test-integ-examples- Port the shell scripts to go
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
starmapshttps://www.starmaps.app/https://www.starmaps.app/
Type
Projects
Status
In Progress