Skip to content
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 #11720: Kill surviving mutation in RequireThisCheck related to anonymous inner class #11791

Conversation

Vyom-Yadav
Copy link
Member

#11720
Hardcoded mutation tested at #11779

Reports with hardcoded mutation :

This mutation falls in the category:

  • Find a case that kills the mutation via diff report, add new test in existing PR

@pbludov
Copy link
Member

pbludov commented Jun 28, 2022

Please fix violation

[ERROR] [checkstyle] [ERROR] src\test\resources\com\puppycrawl\tools\checkstyle\checks\coding\requirethis\InputRequireThisAnonymousInnerClass.java:0: All check input files in the 'src/(it|test)/resources*/' folder must                    have trailing comments "// ok" and/or "// violation". [StyleValidationCommentInInputs]

@Vyom-Yadav Vyom-Yadav force-pushed the killSurvivingMutationInRequireThisCheck-2 branch 3 times, most recently from e3aea5f to 6390e7e Compare June 29, 2022 06:53
@Vyom-Yadav Vyom-Yadav force-pushed the killSurvivingMutationInRequireThisCheck-2 branch from 6390e7e to 9d5f66c Compare July 1, 2022 07:09
@nrmancuso
Copy link
Member

@Vyom-Yadav please make CI happy.

@Vyom-Yadav Vyom-Yadav force-pushed the killSurvivingMutationInRequireThisCheck-2 branch from 9d5f66c to 37d83e5 Compare July 1, 2022 18:26
@romani romani assigned nrmancuso and unassigned romani Jul 1, 2022
@nrmancuso nrmancuso merged commit 2a197a7 into checkstyle:master Jul 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants