remove the MavenProject dependency from the GitDirLocator as it is not needed

Author: TheSnoozer

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

@@ -1510,7 +1510,7 @@ private void appendPropertiesToReactorProjects(LogInterface log, Properties prop
    * @return the File representation of the .git directory
    */
   private File lookupGitDirectory() throws GitCommitIdExecutionException {
-    return new GitDirLocator(project).lookupGitDirectory(dotGitDirectory);
+    return new GitDirLocator(project.getBasedir()).lookupGitDirectory(dotGitDirectory);
   }
 
   private void logProperties(LogInterface log, Properties propertiesToPublish) {

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

@@ -23,26 +23,25 @@
 import java.io.FileReader;
 import java.io.IOException;
 import java.nio.file.Path;
-import java.util.List;
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
-import org.apache.maven.project.MavenProject;
 import org.eclipse.jgit.lib.Constants;
 
 /**
  * This class encapsulates logic to locate a valid .git directory of the currently used project. If
  * it's not already specified, this logic will try to find it.
  */
 public class GitDirLocator {
-  final MavenProject mavenProject;
+  final File projectBasedir;
 
   /**
    * Constructor to encapsulates all references required to locate a valid .git directory
    *
-   * @param mavenProject The currently used (maven) project.
+   * @param projectBasedir The project basedir that will be used as last resort to search
+   *                       the parent project hierarchy until a .git directory is found.
    */
-  public GitDirLocator(MavenProject mavenProject) {
-    this.mavenProject = mavenProject;
+  public GitDirLocator(File projectBasedir) {
+    this.projectBasedir = projectBasedir;
   }
 
   /**
@@ -82,17 +81,13 @@ public File lookupGitDirectory(@Nonnull File manuallyConfiguredDir) {
   }
 
   /**
-   * Search up all the maven parent project hierarchy until a .git directory is found.
+   * Search up all the parent project hierarchy until a .git directory is found.
    *
    * @return File which represents the location of the .git directory or NULL if none found.
    */
   @Nullable
   private File findProjectGitDirectory() {
-    if (this.mavenProject == null) {
-      return null;
-    }
-
-    File basedir = mavenProject.getBasedir();
+    File basedir = this.projectBasedir;
     while (basedir != null) {
       File gitdir = new File(basedir, Constants.DOT_GIT);
       if (gitdir.exists()) {

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

@@ -22,21 +22,14 @@
 
 import java.io.File;
 import java.nio.file.Files;
-import java.util.Collections;
-import java.util.List;
-import org.apache.maven.project.MavenProject;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.rules.TemporaryFolder;
 import org.junit.runner.RunWith;
-import org.mockito.Mock;
 import org.mockito.junit.MockitoJUnitRunner;
 
 @RunWith(MockitoJUnitRunner.class)
 public class GitDirLocatorTest {
-
-  @Mock MavenProject project;
-
   @Rule
   public TemporaryFolder folder = new TemporaryFolder();
 
@@ -46,7 +39,7 @@ public void shouldUseTheManuallySpecifiedDirectory() throws Exception {
     File dotGitDir = folder.newFolder("temp");
     try {
       // when
-      GitDirLocator locator = new GitDirLocator(project);
+      GitDirLocator locator = new GitDirLocator(dotGitDir);
       File foundDirectory = locator.lookupGitDirectory(dotGitDir);
 
       // then
@@ -79,7 +72,7 @@ public void shouldResolveRelativeSubmodule() throws Exception {
 
     try {
       // when
-      GitDirLocator locator = new GitDirLocator(project);
+      GitDirLocator locator = new GitDirLocator(dotGitDir);
       File foundDirectory = locator.lookupGitDirectory(dotGitDir);
 
       // then