[SUREFIRE-2100] Improve behaviour for surefire-report goal with aggregate parameter

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0.0-M7
• Fix Version/s: 3.0.0-M8
• Component/s: Maven Surefire Report Plugin
• Assignee: Konrad Windszus
• Reporter: Konrad Windszus
• Created: 17/Jun/22 10:59
• Updated: 17/Dec/22 20:04
• Resolved: 29/Sep/22 15:15

Description

The aggregate parameter affects the default reportDirectories: https://github.com/apache/maven-surefire/blob/6f0fb404f89728fb65f37c9db7711fff6af55d5d/maven-surefire-report-plugin/src/main/java/org/apache/maven/plugins/surefire/report/AbstractSurefireReportMojo.java#L201-L207.

This default behaviour only makes sense though for a Maven project containing multiple modules. For a single Maven project this default behaviour leads to the fact that always an empty report is being generated as the root project is always skipped.

Although one could argue that aggregate should only ever be set for multimodule projects, one can easily fix the logic here to also work for a singlemodule project. Also the javadoc of this argument does not state any limitations on when to use it (https://maven.apache.org/surefire/maven-surefire-report-plugin/failsafe-report-only-mojo.html#aggregate)

Issue Links

• links to: GitHub Pull Request #548