-
Notifications
You must be signed in to change notification settings - Fork 262
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
[ELY-2446] Add a test class for RegexNameRewriter #1784
Conversation
Pattern pattern = Pattern.compile("\\."); | ||
|
||
RegexNameRewriter regexNameRewriter = new RegexNameRewriter(pattern, "dot", true); | ||
MutableNameRewriter rewriter = new MutableNameRewriter(regexNameRewriter); |
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.
@mittalaishwarya You don't need to use MutableNameRewriter at all to test RegexNameRewriter. You can just call method rewriteName on the regexNameRewriter
package org.wildfly.security.auth.util; | ||
|
||
import org.junit.Test; | ||
import org.wildfly.security.auth.util.RegexNameRewriter; |
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.
@mittalaishwarya Just a minor, this class is in the same package as this test. So you don't have to import it
@mittalaishwarya Otherwise this looks good! |
Hey, if you could squash these changes into one commit, that would be awesome. Thanks! |
672110b
to
3bb1838
Compare
@mittalaishwarya Looks great, thank you! |
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.
Sorry for the delay in getting this one merged, I've been away for a bit.
Thanks very much @mittalaishwarya!
https://issues.redhat.com/browse/ELY-2446