dawidkwiecien opened a new pull request, #6725:
URL: https://github.com/apache/jmeter/pull/6725

   ## Description
   Fixes Sonar Qube issue (Java) JUnit assertTrue/assertFalse should be 
simplified to the corresponding dedicated assertion
   
https://sonarcloud.io/project/issues?rules=java%3AS5785&issueStatuses=OPEN%2CCONFIRMED&id=JMeter
   
   Additionaly in PackageTest.java, I've removed self-assertions for example 
AssertEquals(a1,a1) which were useless.
   
   ## Motivation and Context
   Remove issue from SonarQube
   
   ## How Has This Been Tested?
   I've runned ./gradlew test and there is no issue
   
   ## Screenshots (if appropriate):
   
   ## Types of changes
   <!--- What types of changes does your code introduce? Delete as appropriate 
-->
   - Bug fix (non-breaking change which fixes an issue)
   
   ## Checklist:
   <!--- Go over all the following points, and put an `x` in all the boxes that 
apply. -->
   <!--- If you're unsure about any of these, don't hesitate to ask. We're here 
to help! -->
   - [ x] My code follows the [code style][style-guide] of this project.
   - [ ] I have updated the documentation accordingly.
   
   [style-guide]: https://wiki.apache.org/jmeter/CodeStyleGuidelines
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to