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 a rule to check that affirm is used for Assertion #1238
Comments
@pnatashap thanks for the report, but here some unclear moments:
How to fix:
Please fix the bug report in order it to get resolved faster. |
@pnatashap can you please assign this issue to me? |
@IlyaPechersky I do not have permissions to assign issues, you can ask Yegor or just create a PR. |
@pnatashap, I looked at different options, but I don’t see a suitable one. Can you please suggest me which AST class is best to use to describe org.llorllale.cactoos.matchers.Assertion<>()? |
@IlyaPechersky actually you need to create a new rule, so you can check the full list and find something similar https://docs.pmd-code.org/latest/pmd_rules_java.html. For example, XPath rules is easy to write, but it is not so powerfull, some of them are implemented in code. Example rule with the similar functionality https://github.com/pmd/pmd/blob/master/pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/bestpractices/CheckResultSetRule.java |
In case of using org.llorllale.cactoos.matchers.Assertion<>() affirm method should be called to perform validation.
The text was updated successfully, but these errors were encountered: