-
Notifications
You must be signed in to change notification settings - Fork 592
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
New Event Filtering: Add rekt test for "All" filter #7119
Comments
/area test-and-release |
Hey @Cali0707 I would like to work on this can I take this? |
@Rahul-Kumar-prog my bad for marking this ready to work, it is still not ready until after #7135 is merged. How about I ping you here when this is ready to be worked on and we can see if you want it then? |
yeah sure @Cali0707, till then can you give me any other issue. in which I have to work with test cases. |
@Rahul-Kumar-prog you can always filter issues by labels. For example, in the eventing kafka broker: https://github.com/knative-extensions/eventing-kafka-broker/issues?q=is%3Aopen+is%3Aissue+label%3Aarea%2Ftest |
/assign |
Problem
To expand the rekt tests of the new event filters, we should create a rekt test of the "Al" filter
Exit Criteria
A rekt test using an "All" filter which has:
The test should then send 6 events, one which corresponds to each filter and one which should not match any filter. The test should verify that the events which should be matched are delivered to the sink and that the remaining event is not.
Time Estimate (optional):
How many developer-days do you think this may take to resolve? `
Additional context (optional)
#5204
The text was updated successfully, but these errors were encountered: