-
Notifications
You must be signed in to change notification settings - Fork 688
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
SONARJAVA-4260 add RestAssuredMockMvc assertions to list of test assertions #4951
Conversation
sonarqube is raising 1 issue, but for me, it is not relevant for this case and to maintain the consistency with the rest of the code it is better to keep the comment at the end of the line |
@@ -1,6 +1,6 @@ | |||
{ | |||
"ruleKey": "S1130", | |||
"hasTruePositives": true, | |||
"falseNegatives": 25, | |||
"falseNegatives": 26, |
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.
Why did this change? S1130 looks unrelated to tests.
java-checks/src/main/java/org/sonar/java/checks/helpers/UnitTestUtils.java
Outdated
Show resolved
Hide resolved
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.
There's some SonarQube issue as well.
Feel free to just reply if everything is fine and I misunderstood something.
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.
Discussed in person. LGTM.
Quality Gate passedIssues Measures |
SONARJAVA-4260
Part of