Skip to content

Commit

Permalink
[MNG-6261] - using File apis to compare
Browse files Browse the repository at this point in the history
  • Loading branch information
fabcipriano committed Nov 19, 2018
1 parent 86d4a91 commit e50878f
Show file tree
Hide file tree
Showing 4 changed files with 102 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -852,10 +852,11 @@ private ModelData readParent( Model childModel, ModelSource childSource, ModelBu
File pomFile = parentData.getModel().getPomFile();
if ( pomFile != null )
{
FileModelSource pomSource = new FileModelSource( pomFile );
ModelSource expectedParentSource = getParentPomFile( childModel, childSource );

if ( expectedParentSource == null || ( expectedParentSource instanceof ModelSource2
&& !pomFile.toURI().equals( ( (ModelSource2) expectedParentSource ).getLocationURI() ) ) )
&& !pomSource.equals( expectedParentSource ) ) )
{
parentData = readParentExternally( childModel, request, problems );
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -80,4 +80,27 @@ public URI getLocationURI()
{
return getFile().toURI();
}

@Override
public boolean equals( Object obj )
{
if ( this == obj )
{
return true;
}

if ( ( obj == null ) || !( obj instanceof FileModelSource ) )
{
return false;
}
FileModelSource other = ( FileModelSource ) obj;
return getFile().equals( other.getFile() );
}

@Override
public int hashCode()
{
return super.hashCode();
}

}
Original file line number Diff line number Diff line change
@@ -0,0 +1,74 @@
package org.apache.maven.model.building;

/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
* distributed with this work for additional information
* regarding copyright ownership. The ASF licenses this file
* to you under the Apache License, Version 2.0 (the
* "License"); you may not use this file except in compliance
* with the License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing,
* software distributed under the License is distributed on an
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
* KIND, either express or implied. See the License for the
* specific language governing permissions and limitations
* under the License.
*/
import java.io.File;
import junit.framework.TestCase;
import static junit.framework.TestCase.assertFalse;
import static junit.framework.TestCase.assertTrue;
import org.apache.commons.lang3.SystemUtils;
import static org.junit.Assume.assumeTrue;
import org.junit.Rule;
import org.junit.Test;
import org.junit.rules.TemporaryFolder;

/**
*
* @author fabiano
*/
public class FileModelSourceTest
{

@Rule
public TemporaryFolder folder = new TemporaryFolder();

/**
* Test of equals method, of class FileModelSource.
*/
@Test
public void testEquals()
throws Exception
{
File tempFile = File.createTempFile( "pomTest-", ".xml" );
tempFile.deleteOnExit();
FileModelSource instance = new FileModelSource( tempFile );

assertFalse( instance.equals( null ) );
assertFalse( instance.equals( new Object() ) );
assertTrue( instance.equals( instance ) );
assertTrue( instance.equals( new FileModelSource( tempFile ) ) );
}

@Test
public void testWindowsPaths()
throws Exception
{
assumeTrue( SystemUtils.IS_OS_WINDOWS );

File upperCaseFolder = folder.newFolder( "TESTE" );
String absolutePath = upperCaseFolder.getAbsolutePath();
File lowerCaseFolder = new File( absolutePath.toLowerCase() );

FileModelSource upperCaseFolderSouce = new FileModelSource( upperCaseFolder );
FileModelSource lowerCaseFolderSouce = new FileModelSource( lowerCaseFolder );

assertTrue( upperCaseFolderSouce.equals( lowerCaseFolderSouce ) );
}

}
3 changes: 3 additions & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -147,6 +147,9 @@ under the License.
<contributor>
<name>Joseph Walton (MNG-5297)</name>
</contributor>
<contributor>
<name>Fabiano Cipriano de Oliveira (MNG-6261)</name>
</contributor>
</contributors>

<!--bootstrap-start-comment-->
Expand Down

0 comments on commit e50878f

Please sign in to comment.