-
-
Notifications
You must be signed in to change notification settings - Fork 355
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
review: test: add tests for CtCommentIMmpl class #4053
review: test: add tests for CtCommentIMmpl class #4053
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.
The test intents are good, but they can be made significantly shorter and even more clear with some slight tweaks.
Co-authored-by: Simon Larsén <[email protected]>
Co-authored-by: Simon Larsén <[email protected]>
Co-authored-by: Simon Larsén <[email protected]>
Co-authored-by: Simon Larsén <[email protected]>
Other methods in this class don't need improvement in tests so, there's no hurry to merge this one ps: I have no more commits form my side to push |
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.
LGTM, thanks @Rohitesh-Kumar-Jain
#1818
Hi, I have added some new tests for methods of the JavaOutputProcessor class :
These tests are intended to kill this mutation:
Link to #3967