SharedFolder might cause confusion. It's not about some network share, but about the folder that all the (reactor-)projects share. Renaming it.
git-svn-id: https://svn.apache.org/repos/asf/maven/shared/trunk@1533761 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/main/java/org/apache/maven/shared/project/utils/ProjectUtils.java b/src/main/java/org/apache/maven/shared/project/utils/ProjectUtils.java
index 3058793..5e62142 100644
--- a/src/main/java/org/apache/maven/shared/project/utils/ProjectUtils.java
+++ b/src/main/java/org/apache/maven/shared/project/utils/ProjectUtils.java
@@ -223,7 +223,7 @@
* @param project the project
* @return the shared folder
*/
- public static File getSharedFolder( MavenProject project )
+ public static File getJoinedFolder( MavenProject project )
{
if( project == null )
{
@@ -232,7 +232,7 @@
try
{
- return getSharedFolder( project.getBasedir(), project.getModel() );
+ return getJoinedFolder( project.getBasedir(), project.getModel() );
}
catch ( IOException e )
{
@@ -248,9 +248,9 @@
return null; //@todo fix exception handling
}
- private static final File getSharedFolder( File baseDirectory, Model model ) throws IOException, XmlPullParserException
+ private static final File getJoinedFolder( File baseDirectory, Model model ) throws IOException, XmlPullParserException
{
- File sharedFolder = baseDirectory;
+ File joinedFolder = baseDirectory;
for( String module : getAllModules( model ).keySet() )
{
@@ -258,19 +258,19 @@
Model submodel = readModel( moduleFile );
- File modulesSharedFolder = getSharedFolder( moduleFile.getParentFile(), submodel );
+ File modulesJoinedFolder = getJoinedFolder( moduleFile.getParentFile(), submodel );
- sharedFolder = getSharedFolder( sharedFolder, modulesSharedFolder );
+ joinedFolder = getJoinedFolder( joinedFolder, modulesJoinedFolder );
}
- return sharedFolder;
+ return joinedFolder;
}
// Don't make this method public, it has nothing to do with a MavenProject.
// If required on more places, create a separate Utils-class
- protected static final File getSharedFolder( File lhs, File rhs )
+ protected static final File getJoinedFolder( File lhs, File rhs )
{
- File sharedFolder = null;
+ File joinedFolder = null;
Stack<File> lhsStack = new Stack<File>();
@@ -296,10 +296,10 @@
if( nextFile.isDirectory() && nextFile.equals( rhsStack.pop() ) )
{
- sharedFolder = nextFile;
+ joinedFolder = nextFile;
}
}
- return sharedFolder;
+ return joinedFolder;
}
}
diff --git a/src/test/java/org/apache/maven/shared/project/utils/ProjectUtilsTest.java b/src/test/java/org/apache/maven/shared/project/utils/ProjectUtilsTest.java
index c2c37a4..7f91567 100644
--- a/src/test/java/org/apache/maven/shared/project/utils/ProjectUtilsTest.java
+++ b/src/test/java/org/apache/maven/shared/project/utils/ProjectUtilsTest.java
@@ -37,85 +37,85 @@
}
@Test
- public void getSharedFolderNullDirectories()
+ public void getJoinedFolderNullDirectories()
throws Exception
{
- assertNull( ProjectUtils.getSharedFolder( null, null ) );
+ assertNull( ProjectUtils.getJoinedFolder( null, null ) );
}
@Test
- public void getSharedFolderNullLeftFolder()
+ public void getJoinedFolderNullLeftFolder()
throws Exception
{
- assertNull( ProjectUtils.getSharedFolder( null, ROOT_FOLDER ) );
+ assertNull( ProjectUtils.getJoinedFolder( null, ROOT_FOLDER ) );
}
@Test
- public void getSharedFolderNullRightFolder()
+ public void getJoinedFolderNullRightFolder()
throws Exception
{
- assertNull( ProjectUtils.getSharedFolder( ROOT_FOLDER, null ) );
+ assertNull( ProjectUtils.getJoinedFolder( ROOT_FOLDER, null ) );
}
@Test
- public void getSharedFolderSameDirectory()
+ public void getJoinedFolderSameDirectory()
throws Exception
{
File folder = ROOT_FOLDER;
- assertEquals( folder, ProjectUtils.getSharedFolder( folder, folder ) );
+ assertEquals( folder, ProjectUtils.getJoinedFolder( folder, folder ) );
}
@Test
- public void getSharedFolderSameFile()
+ public void getJoinedFolderSameFile()
throws Exception
{
File pomFile = new File( "pom.xml" );
- assertEquals( pomFile.getParentFile(), ProjectUtils.getSharedFolder( pomFile, pomFile ) );
+ assertEquals( pomFile.getParentFile(), ProjectUtils.getJoinedFolder( pomFile, pomFile ) );
}
@Test
- public void getSharedFolderDeeperLeftFolder()
+ public void getJoinedFolderDeeperLeftFolder()
throws Exception
{
File lhsFolder = new File( ROOT_FOLDER, "src" );
File rhsFolder = ROOT_FOLDER;
- assertEquals( rhsFolder, ProjectUtils.getSharedFolder( lhsFolder, rhsFolder ) );
+ assertEquals( rhsFolder, ProjectUtils.getJoinedFolder( lhsFolder, rhsFolder ) );
}
@Test
- public void getSharedFolderDeeperRightFolder()
+ public void getJoinedFolderDeeperRightFolder()
throws Exception
{
File lhsFolder = ROOT_FOLDER;
File rhsFolder = new File( ROOT_FOLDER, "src" );
- assertEquals( lhsFolder, ProjectUtils.getSharedFolder( lhsFolder, rhsFolder ) );
+ assertEquals( lhsFolder, ProjectUtils.getJoinedFolder( lhsFolder, rhsFolder ) );
}
@Test
- public void getSharedFolderFileAndDeeperLeftFolder()
+ public void getJoinedFolderFileAndDeeperLeftFolder()
throws Exception
{
File folder = ROOT_FOLDER;
File lhsFolder = new File( folder, "src" );
File rhsFolder = new File( folder, "pom.xml" );
- assertEquals( folder, ProjectUtils.getSharedFolder( lhsFolder, rhsFolder ) );
+ assertEquals( folder, ProjectUtils.getJoinedFolder( lhsFolder, rhsFolder ) );
}
@Test
- public void getSharedFolderFileAndDeeperRightFolder()
+ public void getJoinedFolderFileAndDeeperRightFolder()
throws Exception
{
File folder = ROOT_FOLDER;
File lhsFolder = new File( folder, "pom.xml" );
File rhsFolder = new File( folder, "src" );
- assertEquals( folder, ProjectUtils.getSharedFolder( lhsFolder, rhsFolder ) );
+ assertEquals( folder, ProjectUtils.getJoinedFolder( lhsFolder, rhsFolder ) );
}
}
