PMD Results
The following document contains the results of PMD 6.55.0.
Violations By Priority
Priority 3
org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–3682 |
| UnusedLocalVariable | Avoid unused local variables such as 'actualFailIfNoTests'. | 1690 |
| CollapsibleIfStatements | These nested if statements could be combined | 2726–2728 |
| CollapsibleIfStatements | These nested if statements could be combined | 2759–2773 |
org/apache/maven/plugin/surefire/StartupReportConfiguration.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–279 |
org/apache/maven/plugin/surefire/SurefireHelper.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–301 |
org/apache/maven/plugin/surefire/booterclient/BooterSerializer.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–196 |
org/apache/maven/plugin/surefire/booterclient/ForkNumberBucket.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateMethod | Avoid unused private methods such as 'getHighestDrawnNumber()'. | 77 |
org/apache/maven/plugin/surefire/booterclient/ForkStarter.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–834 |
| CollapsibleIfStatements | These nested if statements could be combined | 618–625 |
org/apache/maven/plugin/surefire/booterclient/JarManifestForkConfiguration.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–245 |
org/apache/maven/plugin/surefire/booterclient/lazytestprovider/TestProvidingInputStream.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–154 |
org/apache/maven/plugin/surefire/booterclient/output/ForkedProcessEventNotifier.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–208 |
org/apache/maven/plugin/surefire/booterclient/output/InPluginProcessDumpSingleton.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–95 |
org/apache/maven/plugin/surefire/extensions/SurefireForkChannel.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–281 |
| EmptyControlStatement | Empty try body - you could rename the resource to 'ignored' | 178–185 |
org/apache/maven/plugin/surefire/report/DefaultReporterFactory.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–428 |
org/apache/maven/plugin/surefire/report/StatelessXmlReporter.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–713 |
org/apache/maven/plugin/surefire/report/Utf8RecodingDeferredFileOutputStream.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–224 |
org/apache/maven/surefire/stream/CommandEncoder.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–131 |
org/apache/maven/surefire/stream/EventDecoder.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–435 |
Priority 4
org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Collections.emptyList' due to existing static import 'java.util.Collections.emptyList' | 1705 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Collections.emptyList' due to existing static import 'java.util.Collections.emptyList' | 2062 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Collections.singletonList' due to existing static import 'java.util.Collections.singletonList' | 2077 |
org/apache/maven/plugin/surefire/booterclient/ChecksumCalculator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 105 |
org/apache/maven/plugin/surefire/report/DefaultReporterFactory.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.FAILURE' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.FAILURE' | 182 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.ERROR' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.ERROR' | 183 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.FLAKE' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.FLAKE' | 184 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReportEntryType.SUCCESS' due to existing static import 'org.apache.maven.plugin.surefire.report.ReportEntryType.SUCCESS' | 217 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReportEntryType.FAILURE' due to existing static import 'org.apache.maven.plugin.surefire.report.ReportEntryType.FAILURE' | 219 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReportEntryType.ERROR' due to existing static import 'org.apache.maven.plugin.surefire.report.ReportEntryType.ERROR' | 221 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.FLAKE' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.FLAKE' | 228 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.ERROR' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.ERROR' | 230 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.FAILURE' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.FAILURE' | 230 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.SUCCESS' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.SUCCESS' | 233 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReportEntryType.SUCCESS' due to existing static import 'org.apache.maven.plugin.surefire.report.ReportEntryType.SUCCESS' | 283 |
org/apache/maven/plugin/surefire/report/StatelessXmlReporter.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReportEntryType.SUCCESS' due to existing static import 'org.apache.maven.plugin.surefire.report.ReportEntryType.SUCCESS' | 172 |
org/apache/maven/plugin/surefire/report/TestSetRunListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 306 |
org/apache/maven/plugin/surefire/report/Utf8RecodingDeferredFileOutputStream.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 137 |
| UselessParentheses | Useless parentheses. | 138 |
org/apache/maven/plugin/surefire/report/WrappedReportEntry.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 154 |
org/apache/maven/plugin/surefire/util/ScannerUtil.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 35 |
| UselessParentheses | Useless parentheses. | 49 |
Files
org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–3682 |
| UnusedLocalVariable | Avoid unused local variables such as 'actualFailIfNoTests'. | 3 | 1690 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Collections.emptyList' due to existing static import 'java.util.Collections.emptyList' | 4 | 1705 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Collections.emptyList' due to existing static import 'java.util.Collections.emptyList' | 4 | 2062 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'Collections.singletonList' due to existing static import 'java.util.Collections.singletonList' | 4 | 2077 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 2726–2728 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 2759–2773 |
org/apache/maven/plugin/surefire/StartupReportConfiguration.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–279 |
org/apache/maven/plugin/surefire/SurefireHelper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–301 |
org/apache/maven/plugin/surefire/booterclient/BooterSerializer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–196 |
org/apache/maven/plugin/surefire/booterclient/ChecksumCalculator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 105 |
org/apache/maven/plugin/surefire/booterclient/ForkNumberBucket.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateMethod | Avoid unused private methods such as 'getHighestDrawnNumber()'. | 3 | 77 |
org/apache/maven/plugin/surefire/booterclient/ForkStarter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–834 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 618–625 |
org/apache/maven/plugin/surefire/booterclient/JarManifestForkConfiguration.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–245 |
org/apache/maven/plugin/surefire/booterclient/lazytestprovider/TestProvidingInputStream.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–154 |
org/apache/maven/plugin/surefire/booterclient/output/ForkedProcessEventNotifier.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–208 |
org/apache/maven/plugin/surefire/booterclient/output/InPluginProcessDumpSingleton.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–95 |
org/apache/maven/plugin/surefire/extensions/SurefireForkChannel.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–281 |
| EmptyControlStatement | Empty try body - you could rename the resource to 'ignored' | 3 | 178–185 |
org/apache/maven/plugin/surefire/report/DefaultReporterFactory.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–428 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.FAILURE' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.FAILURE' | 4 | 182 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.ERROR' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.ERROR' | 4 | 183 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.FLAKE' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.FLAKE' | 4 | 184 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReportEntryType.SUCCESS' due to existing static import 'org.apache.maven.plugin.surefire.report.ReportEntryType.SUCCESS' | 4 | 217 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReportEntryType.FAILURE' due to existing static import 'org.apache.maven.plugin.surefire.report.ReportEntryType.FAILURE' | 4 | 219 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReportEntryType.ERROR' due to existing static import 'org.apache.maven.plugin.surefire.report.ReportEntryType.ERROR' | 4 | 221 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.FLAKE' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.FLAKE' | 4 | 228 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.ERROR' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.ERROR' | 4 | 230 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.FAILURE' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.FAILURE' | 4 | 230 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'TestResultType.SUCCESS' due to existing static import 'org.apache.maven.plugin.surefire.report.DefaultReporterFactory.TestResultType.SUCCESS' | 4 | 233 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReportEntryType.SUCCESS' due to existing static import 'org.apache.maven.plugin.surefire.report.ReportEntryType.SUCCESS' | 4 | 283 |
org/apache/maven/plugin/surefire/report/StatelessXmlReporter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–713 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'ReportEntryType.SUCCESS' due to existing static import 'org.apache.maven.plugin.surefire.report.ReportEntryType.SUCCESS' | 4 | 172 |
org/apache/maven/plugin/surefire/report/TestSetRunListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 306 |
org/apache/maven/plugin/surefire/report/Utf8RecodingDeferredFileOutputStream.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–224 |
| UselessParentheses | Useless parentheses. | 4 | 137 |
| UselessParentheses | Useless parentheses. | 4 | 138 |
org/apache/maven/plugin/surefire/report/WrappedReportEntry.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 154 |
org/apache/maven/plugin/surefire/util/ScannerUtil.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 35 |
| UselessParentheses | Useless parentheses. | 4 | 49 |
org/apache/maven/surefire/stream/CommandEncoder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–131 |
org/apache/maven/surefire/stream/EventDecoder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–435 |