Author: carlos Date: Tue Jun 13 06:18:42 2006 New Revision: 413892 URL: http://svn.apache.org/viewvc?rev=413892&view=rev Log: Merging revision 413090-413092 from trunk. [MNG-2357] Ignore temp files, refactor code and add missing it tests
Modified: maven/components/branches/maven-2.0.x/maven-core-it/README.txt maven/components/branches/maven-2.0.x/maven-core-it/integration-tests.txt maven/components/branches/maven-2.0.x/maven-core-it/it0103/ (props changed) maven/components/branches/maven-2.0.x/maven-core-it/it0103/level1/level2/level3/ (props changed) maven/components/branches/maven-2.0.x/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java maven/components/branches/maven-2.0.x/maven-project/src/test/java/org/apache/maven/project/AbstractMavenProjectTestCase.java Modified: maven/components/branches/maven-2.0.x/maven-core-it/README.txt URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.0.x/maven-core-it/README.txt?rev=413892&r1=413891&r2=413892&view=diff ============================================================================== --- maven/components/branches/maven-2.0.x/maven-core-it/README.txt (original) +++ maven/components/branches/maven-2.0.x/maven-core-it/README.txt Tue Jun 13 06:18:42 2006 @@ -281,6 +281,12 @@ when they contain ${project.build.directory} in the string value of a Map.Entry. +it0105: MRESOURCES-18 + +it0106: When a project has modules and its parent is not preinstalled [MNG-2318] + +it0107: Verify that default implementation of an implementation for a complex object works as + expected [MNG-2293] ------------------------------------------------------------------------------- - generated sources Modified: maven/components/branches/maven-2.0.x/maven-core-it/integration-tests.txt URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.0.x/maven-core-it/integration-tests.txt?rev=413892&r1=413891&r2=413892&view=diff ============================================================================== --- maven/components/branches/maven-2.0.x/maven-core-it/integration-tests.txt (original) +++ maven/components/branches/maven-2.0.x/maven-core-it/integration-tests.txt Tue Jun 13 06:18:42 2006 @@ -1,3 +1,5 @@ +#it0106 MNG-2318 not yet fixed +#it0105 not fixed until 2.1 #it0104 Commenting out, not fixed until post-2.0.4, due to dependency on new plexus-container-default version. it0103 it0102 Propchange: maven/components/branches/maven-2.0.x/maven-core-it/it0103/ ------------------------------------------------------------------------------ --- svn:ignore (added) +++ svn:ignore Tue Jun 13 06:18:42 2006 @@ -0,0 +1 @@ +log.txt Propchange: maven/components/branches/maven-2.0.x/maven-core-it/it0103/level1/level2/level3/ ------------------------------------------------------------------------------ --- svn:ignore (added) +++ svn:ignore Tue Jun 13 06:18:42 2006 @@ -0,0 +1 @@ +target Modified: maven/components/branches/maven-2.0.x/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.0.x/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java?rev=413892&r1=413891&r2=413892&view=diff ============================================================================== --- maven/components/branches/maven-2.0.x/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java (original) +++ maven/components/branches/maven-2.0.x/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java Tue Jun 13 06:18:42 2006 @@ -289,6 +289,11 @@ // circle around and set the artifacts and class realm PluginDescriptor addedPlugin = pluginCollector.getPluginDescriptor( plugin ); + if ( addedPlugin == null ) + { + throw new IllegalStateException( "The PluginDescriptor for the plugin " + plugin + " was not found." ); + } + addedPlugin.setClassRealm( child.getContainerRealm() ); // we're only setting the plugin's artifact itself as the artifact list, to allow it to be retrieved Modified: maven/components/branches/maven-2.0.x/maven-project/src/test/java/org/apache/maven/project/AbstractMavenProjectTestCase.java URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.0.x/maven-project/src/test/java/org/apache/maven/project/AbstractMavenProjectTestCase.java?rev=413892&r1=413891&r2=413892&view=diff ============================================================================== --- maven/components/branches/maven-2.0.x/maven-project/src/test/java/org/apache/maven/project/AbstractMavenProjectTestCase.java (original) +++ maven/components/branches/maven-2.0.x/maven-project/src/test/java/org/apache/maven/project/AbstractMavenProjectTestCase.java Tue Jun 13 06:18:42 2006 @@ -70,17 +70,12 @@ URL resourceUrl = cloader.getResource( resource ); - File resourceFile = null; - if ( resourceUrl != null ) - { - resourceFile = new File( resourceUrl.getPath() ); - } - else + if ( resourceUrl == null ) { throw new FileNotFoundException( "Unable to find: " + resource ); } - return resourceFile; + return new File( resourceUrl.getPath() ); } protected ArtifactRepository getLocalRepository()