-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add test coverage to collectors #2606
Comments
cc @squaremo If you can post your multi-cluster envtest stuff that would be awesome! |
This is what I've done for a similar testing requirement: https://github.com/weaveworks-experiments/fleeet/blob/main/module/controllers/remote_test.go#L137 |
See for example an integration test for collector in another PR It would be useful to consider it in this ticket |
We have a larger suite built out now that hit major parts of the collector. Closing. |
We need automated test coverage on:
pkg/collector
pkg/rolecollector
pkg/objectscollector
The text was updated successfully, but these errors were encountered: