-
Notifications
You must be signed in to change notification settings - Fork 590
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
Increase code coverage to 70% #591
Comments
#592 fixes a few. |
should /pkg/controller/testing/* be included in the coverage? |
I'll takle pkg/controller/eventing/subscription/reconcile.go |
Just posted #597 for github.com/knative/eventing/pkg/apis/eventing/v1alpha1/subscribable_channelable_validation.go |
@vaikas-google I suspect I misused the Fixes label in #597 and prow closed this as a side effect, it should be reopened I guess? |
For what it's worth, as a non-editor on the repo, I can't check any of the boxes. |
Good to know @Harwayne. That probably kills collaborative use of checkboxes since ultimately we want very few users having write access to the repo. |
/assign @akashrv |
the project is now overall at 81% with a few low spots, we can open issues to track those individually. |
@n3wscott: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Expected Behavior
Stated goal for the V0.2 release is >70% test coverage.
Actual Behavior
Some packages do not meet this goal.
I used this to create the list:
https://testgrid.knative.dev/knative-eventing#coverage&sort-by-failures=
This is a list of those packages. We are using the checklist here to keep track of which ones are being worked on. So, if you take one, check the box and comment on who's taking it so we don't duplicate the work.
The text was updated successfully, but these errors were encountered: