Merge pull request #807 from exiahuang/fix-report-outputdirectory

fix outputDirectory not work at running spotbugs:spotbugs goal
spotbugs · Jul 15, 2024 · c6b6416 · c6b6416
2 parents c86c55a + 9234e2a
commit c6b6416
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/src/it/html-report/pom.xml b/src/it/html-report/pom.xml
@@ -40,6 +40,7 @@
         <configuration>
           <xmlOutput>true</xmlOutput>
           <htmlOutput>true</htmlOutput>
+          <outputDirectory>target/test-output-directory</outputDirectory>
           <failOnError>false</failOnError>
           <debug>@spotbugsTestDebug@</debug>
           <threshold>High</threshold>

diff --git a/src/it/html-report/verify.groovy b/src/it/html-report/verify.groovy
@@ -13,5 +13,5 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-File spotbugsHtml =  new File(basedir, 'target/spotbugs.html')
+File spotbugsHtml =  new File(basedir, 'target/test-output-directory/spotbugs.html')
 assert spotbugsHtml.exists()
diff --git a/src/main/groovy/org/codehaus/mojo/spotbugs/SpotBugsMojo.groovy b/src/main/groovy/org/codehaus/mojo/spotbugs/SpotBugsMojo.groovy
@@ -1098,7 +1098,7 @@ class SpotBugsMojo extends AbstractMavenReport implements SpotBugsPluginsTrait {
 
         log.debug("****** SpotBugsMojo executeSpotbugs *******")
 
-        File htmlTempFile = new File("${project.build.directory}/spotbugs.html")
+        File htmlTempFile = new File("${outputDirectory}/spotbugs.html")
         if (htmlOutput) {
             forceFileCreation(htmlTempFile)
         }