#124 : Extra directories created when default <generateGitPropertiesFilename> is used

Author: S L

src/main/java/pl/project13/maven/git/GitCommitIdMojo.java

@@ -490,7 +490,7 @@ void loadGitDataWithJGit(@NotNull Properties properties) throws IOException, Moj
 
   void generatePropertiesFile(@NotNull Properties properties, File base, String propertiesFilename) throws IOException {
     FileWriter fileWriter = null;
-    File gitPropsFile = new File(base, propertiesFilename);
+    File gitPropsFile = craftPropertiesOutputFile(base, propertiesFilename);
     try {
       Files.createParentDirs(gitPropsFile);
 
@@ -511,6 +511,19 @@ void generatePropertiesFile(@NotNull Properties properties, File base, String pr
     }
   }
 
+  @VisibleForTesting
+  File craftPropertiesOutputFile(File base, String propertiesFilename){
+    File returnPath = new File(base, propertiesFilename);
+
+    File currentPropertiesFilepath = new File(propertiesFilename);
+    if(currentPropertiesFilepath.isAbsolute()){
+       returnPath = currentPropertiesFilepath;
+    }
+
+    return returnPath;
+  }
+
+
   boolean isPomProject(@NotNull MavenProject project) {
     return project.getPackaging().equalsIgnoreCase("pom");
   }

src/test/java/pl/project13/maven/git/GitCommitIdMojoTest.java

@@ -211,4 +211,27 @@ private void assertShortDescribe(String commitDescribe, String expectedShortDesc
     commitIdMojo.loadShortDescribe(prop);
     assertThat(prop.getProperty(GitCommitIdMojo.COMMIT_SHORT_DESCRIBE)).isEqualTo(expectedShortDescribe);
   }
+
+  @Test
+  public void testCraftPropertiesOutputFileWithRelativePath() throws IOException {
+    GitCommitIdMojo commitIdMojo = new GitCommitIdMojo();
+    File baseDir = new File(".");
+    String targetDir = baseDir.getCanonicalPath() + "/";
+    String generateGitPropertiesFilename = "target/classes/git.properties";
+    
+    File result = commitIdMojo.craftPropertiesOutputFile(baseDir, generateGitPropertiesFilename);
+    assertThat(result.getCanonicalPath()).isEqualTo(targetDir + generateGitPropertiesFilename);
+  }
+
+  @Test
+  public void testCraftPropertiesOutputFileWithFullPath() throws IOException {
+    GitCommitIdMojo commitIdMojo = new GitCommitIdMojo();
+    File baseDir = new File(".");
+    String targetDir = baseDir.getCanonicalPath() + "/";
+    String generateGitPropertiesFilename = targetDir + "target/classes/git.properties";
+
+    File result = commitIdMojo.craftPropertiesOutputFile(baseDir, generateGitPropertiesFilename);
+    assertThat(result.getCanonicalPath()).isEqualTo(generateGitPropertiesFilename);
+  }
+
 }