PMD Results
The following document contains the results of PMD 7.5.0.
Violations By Priority
Priority 5
de/softwareforge/testing/postgres/embedded/EmbeddedPostgres.java
Rule | Violation | Line |
---|---|---|
TooManyMethods | This class has too many methods, consider refactoring it. | 76–983 |
CognitiveComplexity | The method 'cleanOldDataDirectories(File)' has a cognitive complexity of 23, current threshold is 15 | 558 |
CyclomaticComplexity | The method 'cleanOldDataDirectories(File)' has a cyclomatic complexity of 12. | 558 |
NPathComplexity | The method 'cleanOldDataDirectories(File)' has an NPath complexity of 226, current threshold is 200 | 558 |
AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 574 |
AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 590 |
AvoidReassigningParameters | Avoid reassigning parameters such as 'processName' | 624 |
TooManyMethods | This class has too many methods, consider refactoring it. | 658–982 |
de/softwareforge/testing/postgres/embedded/EmbeddedUtil.java
Rule | Violation | Line |
---|---|---|
TooManyMethods | This class has too many methods, consider refactoring it. | 37–229 |
de/softwareforge/testing/postgres/embedded/FlywayPreparer.java
Rule | Violation | Line |
---|---|---|
UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 55 |
de/softwareforge/testing/postgres/embedded/TarXzCompressedBinaryManager.java
Rule | Violation | Line |
---|---|---|
CognitiveComplexity | The method 'extractTxz(InputStream, String)' has a cognitive complexity of 20, current threshold is 15 | 159 |
CyclomaticComplexity | The method 'extractTxz(InputStream, String)' has a cyclomatic complexity of 13. | 159 |
AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 167 |
AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 178 |
AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 187–208 |
CollapsibleIfStatements | This if statement could be combined with its parent | 216–218 |
de/softwareforge/testing/postgres/embedded/ZonkyIOPostgresLocator.java
Rule | Violation | Line |
---|---|---|
PreserveStackTrace | Thrown exception does not preserve the stack trace of exception 'e' on all code paths | 81 |
PreserveStackTrace | Thrown exception does not preserve the stack trace of exception 'e' on all code paths | 94 |
UseTryWithResources | Consider using a try-with-resources statement instead of explicitly closing the resource | 139–143 |
de/softwareforge/testing/postgres/junit5/EmbeddedPgExtension.java
Rule | Violation | Line |
---|---|---|
TestClassWithoutTestCases | The class 'TestingContext' might be a test class, but it contains no test cases. | 240 |
SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 252 |
SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 260 |
Files
de/softwareforge/testing/postgres/embedded/EmbeddedPostgres.java
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyMethods | This class has too many methods, consider refactoring it. | 5 | 76–983 |
CognitiveComplexity | The method 'cleanOldDataDirectories(File)' has a cognitive complexity of 23, current threshold is 15 | 5 | 558 |
CyclomaticComplexity | The method 'cleanOldDataDirectories(File)' has a cyclomatic complexity of 12. | 5 | 558 |
NPathComplexity | The method 'cleanOldDataDirectories(File)' has an NPath complexity of 226, current threshold is 200 | 5 | 558 |
AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 5 | 574 |
AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 5 | 590 |
AvoidReassigningParameters | Avoid reassigning parameters such as 'processName' | 5 | 624 |
TooManyMethods | This class has too many methods, consider refactoring it. | 5 | 658–982 |
de/softwareforge/testing/postgres/embedded/EmbeddedUtil.java
Rule | Violation | Priority | Line |
---|---|---|---|
TooManyMethods | This class has too many methods, consider refactoring it. | 5 | 37–229 |
de/softwareforge/testing/postgres/embedded/FlywayPreparer.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryConstructor | Avoid unnecessary constructors - the compiler will generate these for you | 5 | 55 |
de/softwareforge/testing/postgres/embedded/TarXzCompressedBinaryManager.java
Rule | Violation | Priority | Line |
---|---|---|---|
CognitiveComplexity | The method 'extractTxz(InputStream, String)' has a cognitive complexity of 20, current threshold is 15 | 5 | 159 |
CyclomaticComplexity | The method 'extractTxz(InputStream, String)' has a cyclomatic complexity of 13. | 5 | 159 |
AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 5 | 167 |
AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 5 | 178 |
AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 5 | 187–208 |
CollapsibleIfStatements | This if statement could be combined with its parent | 5 | 216–218 |
de/softwareforge/testing/postgres/embedded/ZonkyIOPostgresLocator.java
Rule | Violation | Priority | Line |
---|---|---|---|
PreserveStackTrace | Thrown exception does not preserve the stack trace of exception 'e' on all code paths | 5 | 81 |
PreserveStackTrace | Thrown exception does not preserve the stack trace of exception 'e' on all code paths | 5 | 94 |
UseTryWithResources | Consider using a try-with-resources statement instead of explicitly closing the resource | 5 | 139–143 |
de/softwareforge/testing/postgres/junit5/EmbeddedPgExtension.java
Rule | Violation | Priority | Line |
---|---|---|---|
TestClassWithoutTestCases | The class 'TestingContext' might be a test class, but it contains no test cases. | 5 | 240 |
SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 5 | 252 |
SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 5 | 260 |
Suppressed Violations
Filename | Rule message | Suppression type | Reason |
---|---|---|---|
de/softwareforge/testing/postgres/embedded/EmbeddedPostgres.java | Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. | @suppresswarnings | |
de/softwareforge/testing/postgres/embedded/EmbeddedPostgres.java | Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. | @suppresswarnings | |
de/softwareforge/testing/postgres/embedded/TarXzCompressedBinaryManager.java | Ensure that resources like this AsynchronousFileChannel object are closed after use | //nopmd |