-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Issue #11446: Update CheckerTest method testUnmappableCharacters #14492
Issue #11446: Update CheckerTest method testUnmappableCharacters #14492
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.
Items
7314b14
to
01b7ebc
Compare
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.
ok to merge if CI pass
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.
please investigate test coverage reduction
https://github.com/checkstyle/checkstyle/actions/runs/7941288117/job/21683472819?pr=14492
it is blocker to merge
6edb023
to
d021f7f
Compare
d021f7f
to
0b66794
Compare
@romani ping |
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.
Items
<mutatedClass>com.puppycrawl.tools.checkstyle.Checker</mutatedClass> | ||
<mutatedMethod>setCharset</mutatedMethod> | ||
<mutator>org.pitest.mutationtest.engine.gregor.mutators.experimental.MemberVariableMutator</mutator> | ||
<description>Removed assignment to member variable charset</description> |
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, this is not acceptable.
We can only remove mutation survival, extension is not allowed.
Please investigate why it happened.
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.
item:
checkerConfig.addProperty("charset", "IBM1098"); | ||
|
||
verify(checkerConfig, getPath("InputCheckerTestCharset.java"), expected); | ||
verifyWithInlineConfigParser(getPath("InputCheckerTestCharset.java"), expected); |
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.
please reuase verifyWithInlineXmlConfig
method, it fully supports xml configs
@chrccl , ping |
We have lost contact with author, anyone is welcome to reuse changes here to close the issue. |
Issue #11446: Update CheckerTest method testUnmappableCharacters