Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: add e2e test coverage report #1533

Open
OverOrion opened this issue Oct 13, 2023 · 2 comments · May be fixed by #1726
Open

CI: add e2e test coverage report #1533

OverOrion opened this issue Oct 13, 2023 · 2 comments · May be fixed by #1726
Labels

Comments

@OverOrion
Copy link
Contributor

Is your feature request related to a problem? Please describe.
It would be great to have the e2e tests coverage reports as they cover much a bigger part of the code base compared to unit tests.
Some resources:

Describe the solution you'd like
Build the binary with -cover flag, run the binary with GOCOVERDIR=some_coverage_dir and at the end of the e2e test run merge those files to have a nice (hopefully) high overall coverage report.

Describe alternatives you've considered
I do not think there is any.

Additional context

The tricky part is getting those reports, because currently the CI runs the e2e tests using kind. As mentioned in the docs graceful shutdown is needed for the program to write it's report, so a simple kubectl cp command might not be enough (not tested, just a theory).

@OverOrion
Copy link
Contributor Author

@OverOrion
Copy link
Contributor Author

@nak0f nak0f linked a pull request Apr 20, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant