Tag List Report
The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.
| Tag Class | Total number of occurrences | Tag strings used by tag class |
|---|---|---|
| Deprecated Work | 4 | @deprecated |
| FIXME Work | 7 | fixme, @fixme |
| Todo Work | 17 | todo, @todo |
Each tag is detailed below:
Deprecated Work
Number of occurrences found in the code: 4
| org.apache.maven.plugin.surefire.AbstractSurefireMojo | Line |
|---|---|
| use skipTests instead | 208 |
| use {@link #systemPropertyVariables} instead | 320 |
| instead use <em>disable</em> within {@code statelessTestsetReporter} since of 3.0.0-M6 | 745 |
| org.apache.maven.plugin.surefire.report.StatelessXmlReporter | Line |
| Prefer adding a new constructor that accepts a configuration object, e.g. {@link org.apache.maven.surefire.extensions.StatelessReportMojoConfiguration}. | 127 |
FIXME Work
Number of occurrences found in the code: 7
| org.apache.maven.plugin.surefire.AbstractSurefireMojo | Line |
|---|---|
| look at duplicate parameters to remove | 3083 |
| this default version should be aligned with the junit-platform-version property from pom | 3117 |
| support only from TestNG 6.14.3 | 3162 |
| check if already present as plugin dependency or project dependency | 3163 |
| configurable? or picked from test dependencies | 3167 |
| will be different with 3 and testng | 3273 |
| org.apache.maven.plugin.surefire.SurefireDependencyResolver | Line |
| --No comment-- | 204 |
Todo Work
Number of occurrences found in the code: 17
| org.apache.maven.plugin.surefire.AbstractSurefireMojo | Line |
|---|---|
| make readonly to handle system property | 747 |
| this is in separate method and can be better tested than whole method createForkConfiguration() | 2400 |
| exclude transitive deps (hamcrest etc...) | 3152 |
| why an enclosing class on the top do we real need this shading?? | 3298 |
| org.apache.maven.plugin.surefire.AbstractSurefireMojoToolchainsTest | Line |
| Is this still required? | 86 |
| Is this still required? | 93 |
| org.apache.maven.plugin.surefire.booterclient.ForkingRunListenerTest | Line |
| Test weird characters | 391 |
| org.apache.maven.plugin.surefire.booterclient.Platform | Line |
| simplify or remove when Java 8 support is dropped | 32 |
| org.apache.maven.plugin.surefire.booterclient.output.ForkClient | Line |
| move to the same package with ForkStarter | 52 |
| org.apache.maven.plugin.surefire.log.PluginConsoleLogger | Line |
| remove this as thread safety should be handled by the underlying logging framework no need to add complexity | 26 |
| org.apache.maven.plugin.surefire.report.StatelessXmlReporter | Line |
| this is no more stateless due to existence of testClassMethodRunHistoryMap since of 2.19. | 88 |
| SUREFIRE-1986 | 695 |
| org.apache.maven.plugin.surefire.report.StatelessXmlReporterTest | Line |
| assertThat( file ).doesNotExist(); | 620 |
| org.apache.maven.plugin.surefire.report.Utf8RecodingDeferredFileOutputStream | Line |
| delete( file ); uncomment L485 assertThat( file ).doesNotExist() in StatelessXmlReporterTest | 180 |
| uncomment IOException |*/ UncheckedIOException e) { | 181 |
| uncomment file.toFile() | 182 |
| should be removed after uncommented delete( file ) | 185 |