-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 more unit tests for Label.class
related to label filtering
#9959
Add more unit tests for Label.class
related to label filtering
#9959
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left some comments. But I think you should consider how STRIMZI_LABELS_EXCLUSION_PATTERN
inthe Labels
class is used and try to apply a similar pattern for both.
user-operator/src/main/java/io/strimzi/operator/user/UserOperatorConfig.java
Outdated
Show resolved
Hide resolved
user-operator/src/main/java/io/strimzi/operator/user/model/KafkaUserModel.java
Outdated
Show resolved
Hide resolved
user-operator/src/main/java/io/strimzi/operator/user/model/KafkaUserModel.java
Outdated
Show resolved
Hide resolved
user-operator/src/test/java/io/strimzi/operator/user/UserControllerMockTest.java
Outdated
Show resolved
Hide resolved
user-operator/src/test/java/io/strimzi/operator/user/ResourceUtils.java
Outdated
Show resolved
Hide resolved
Signed-off-by: see-quick <maros.orsak159@gmail.com>
Signed-off-by: see-quick <maros.orsak159@gmail.com>
Signed-off-by: see-quick <maros.orsak159@gmail.com>
Signed-off-by: see-quick <maros.orsak159@gmail.com>
Signed-off-by: see-quick <maros.orsak159@gmail.com>
Signed-off-by: see-quick <maros.orsak159@gmail.com>
91b0303
to
c766fe2
Compare
Signed-off-by: see-quick <maros.orsak159@gmail.com>
user-operator/src/main/java/io/strimzi/operator/user/model/KafkaUserModel.java
Outdated
Show resolved
Hide resolved
Signed-off-by: see-quick <maros.orsak159@gmail.com>
Signed-off-by: see-quick <maros.orsak159@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess all we really need are the beter tests? 🤦
operator-common/src/main/java/io/strimzi/operator/common/model/Labels.java
Outdated
Show resolved
Hide resolved
Signed-off-by: see-quick <maros.orsak159@gmail.com>
Label.class
related to label filtering
Type of change
Description
This PR adds more unit tests for
Label.class
to filter labels in all resources (e.g., Kafka, KafkaUser...).Checklist