Add method AbstractMojo#mkDirForParentFile(File) central fix try can use automatic resource management

Author: Vincent Potucek

compat/maven-plugin-api/src/main/java/org/apache/maven/plugin/AbstractMojo.java

@@ -18,6 +18,7 @@
  */
 package org.apache.maven.plugin;
 
+import java.io.File;
 import java.util.Map;
 
 import org.apache.maven.plugin.logging.Log;
@@ -185,4 +186,12 @@ public Map getPluginContext() {
     public void setPluginContext(Map pluginContext) {
         this.pluginContext = pluginContext;
     }
+
+    protected File mkDirForParentFile(File file) {
+        getLog().info("Creating output file " + file);
+        if (file.getParentFile().mkdirs()) {
+            getLog().info("Created output file " + file);
+        }
+        return file;
+    }
 }

its/core-it-suite/src/test/resources/mng-0870/plugin/src/main/java/org/apache/maven/plugin/coreit/LoadMojo.java

@@ -85,7 +85,7 @@ public void execute() throws MojoExecutionException, MojoFailureException {
         getLog().info("[MAVEN-CORE-IT-LOG] Creating output file: " + file);
 
         try {
-            file.getParentFile().mkdirs();
+            mkDirForParentFile(file);
             file.createNewFile();
         } catch (IOException e) {
             throw new MojoExecutionException("Output file could not be created: " + file, e);

its/core-it-suite/src/test/resources/mng-1088/plugin/src/main/java/org/apache/maven/plugin/coreit/TouchMojo.java

@@ -69,7 +69,7 @@ public void execute() throws MojoExecutionException {
         getLog().info("[MAVEN-CORE-IT-LOG] Creating output file: " + file);
 
         try {
-            file.getParentFile().mkdirs();
+            mkDirForParentFile(file);
             file.createNewFile();
         } catch (IOException e) {
             throw new MojoExecutionException("Output file could not be created: " + file, e);

its/core-it-suite/src/test/resources/mng-2135/plugin/src/main/java/coreit/ItMojo.java

@@ -53,8 +53,7 @@ public class ItMojo extends AbstractMojo {
 
     public void execute() throws MojoExecutionException {
         try {
-            outputFile.getParentFile().mkdirs();
-            outputFile.createNewFile();
+            mkDirForParentFile(outputFile).createNewFile();
         } catch (IOException e) {
             throw new MojoExecutionException("Failed to create touch file: " + e.getMessage(), e);
         }

its/core-it-suite/src/test/resources/mng-3372/direct-using-prefix/plugin/src/main/java/org/plugin/TestMojo.java

@@ -45,7 +45,7 @@ public void execute() throws MojoExecutionException, MojoFailureException {
         File outFile = new File(buildDir, "out.txt");
         FileWriter writer = null;
         try {
-            outFile.getParentFile().mkdirs();
+            mkDirForParentFile(outFile);
 
             writer = new FileWriter(outFile);
             writer.write("Test");

its/core-it-suite/src/test/resources/mng-3652/test-plugin/src/main/java/org/apache/maven/its/mng3652/MyMojo.java

@@ -162,7 +162,7 @@ public void execute() throws MojoExecutionException {
 
         FileWriter w = null;
         try {
-            touchFile.getParentFile().mkdirs();
+            mkDirForParentFile(file);
             w = new FileWriter(touchFile);
 
             w.write(runtimeInformation.getApplicationVersion().toString());

its/core-it-suite/src/test/resources/mng-3693/maven-mng3693-plugin/src/main/java/plugin/MyMojo.java

@@ -57,7 +57,7 @@ public void execute() throws MojoExecutionException {
         File pomFile = project.getFile();
 
         File movedPomFile = new File(project.getBuild().getDirectory(), "pom.xml");
-        movedPomFile.getParentFile().mkdirs();
+        mkDirForParentFile(file);
 
         try {
             FileUtils.copyFile(pomFile, movedPomFile);

its/core-it-suite/src/test/resources/mng-5753-custom-mojo-execution-configurator/plugin/src/main/java/org/apache/maven/its/mng5753/plugin/TestMojo.java

@@ -60,7 +60,7 @@ public class TestMojo extends AbstractMojo {
     public void execute() throws MojoExecutionException {
         try {
             File file = new File(project.getBasedir(), "configuration.txt");
-            file.getParentFile().mkdirs();
+            mkDirForParentFile(file);
             Writer w = new OutputStreamWriter(new FileOutputStream(file, true), "UTF-8");
             try {
                 w.write(name);

its/core-it-suite/src/test/resources/mng-5783-plugin-dependency-filtering/plugin/src/main/java/org/apache/maven/its/mng5783/plugin/TestMojo.java

@@ -61,7 +61,7 @@ public class TestMojo extends AbstractMojo {
     public void execute() throws MojoExecutionException {
         try {
             File file = new File(target, "dependencies.txt");
-            file.getParentFile().mkdirs();
+            mkDirForParentFile(file);
             BufferedWriter w = new BufferedWriter(new OutputStreamWriter(new FileOutputStream(file), "UTF-8"));
             try {
                 for (Artifact artifact : artifacts) {

its/core-it-suite/src/test/resources/mng-6127-plugin-execution-configuration-interference/plugin/src/main/java/org/apache/maven/its/mng6127/plugin/TestMojo.java

@@ -76,7 +76,7 @@ public class TestMojo extends AbstractMojo {
 
     public void execute() throws MojoExecutionException {
         File file = new File(project.getBasedir(), "configuration.txt");
-        file.getParentFile().mkdirs();
+        mkDirForParentFile(file);
 
         Writer w = null;