Author: jdcasey Date: Mon Jan 14 15:59:01 2008 New Revision: 611974 URL: http://svn.apache.org/viewvc?rev=611974&view=rev Log: two more unit tests, and a little simplification to take care of duplicated error reporting coverage.
Added: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/pom.xml (with props) maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/maven/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/maven/test/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/maven/test/error/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/maven/test/error/mojoFailure/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/maven/test/error/mojoFailure/App.java (with props) maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/maven/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/maven/test/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/maven/test/error/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/maven/test/error/mojoFailure/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java (with props) maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/pom.xml (with props) maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/maven/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/maven/test/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/maven/test/error/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/maven/test/error/mojoFailure/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/maven/test/error/mojoFailure/App.java (with props) maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/maven/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/maven/test/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/maven/test/error/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/maven/test/error/mojoFailure/ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java (with props) Modified: maven/components/trunk/maven-core/src/main/aspect/org/apache/maven/errors/LifecycleErrorReporterAspect.aj maven/components/trunk/maven-core/src/main/java/org/apache/maven/errors/CoreErrorReporter.java maven/components/trunk/maven-core/src/main/java/org/apache/maven/errors/DefaultCoreErrorReporter.java maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/error/ErrorReporterPointcutTest.java Modified: maven/components/trunk/maven-core/src/main/aspect/org/apache/maven/errors/LifecycleErrorReporterAspect.aj URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/aspect/org/apache/maven/errors/LifecycleErrorReporterAspect.aj?rev=611974&r1=611973&r2=611974&view=diff ============================================================================== --- maven/components/trunk/maven-core/src/main/aspect/org/apache/maven/errors/LifecycleErrorReporterAspect.aj (original) +++ maven/components/trunk/maven-core/src/main/aspect/org/apache/maven/errors/LifecycleErrorReporterAspect.aj Mon Jan 14 15:59:01 2008 @@ -4,9 +4,8 @@ import org.codehaus.plexus.logging.Logger; import org.apache.maven.project.path.PathTranslator; import org.apache.maven.plugin.MojoExecution; -import org.apache.maven.artifact.Artifact; -import org.apache.maven.artifact.resolver.ArtifactResolver; import org.apache.maven.artifact.resolver.ArtifactResolutionException; +import org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException; import org.apache.maven.plugin.PluginConfigurationException; import org.apache.maven.plugin.descriptor.Parameter; import org.apache.maven.plugin.loader.PluginLoaderException; @@ -217,29 +216,14 @@ after( MavenProject project, String scope ) throwing( ArtifactResolutionException cause ): pm_resolveTransitiveDependencies( project, scope ) { - getReporter().reportProjectDependenciesUnresolvable( project, scope, cause ); - } - - private pointcut within_pm_downloadDependencies( MavenProject project ): - withincode( void DefaultPluginManager.downloadDependencies( MavenProject, .. ) ) - && args( project, .. ); - - private pointcut ar_resolve( Artifact artifact ): - call( * ArtifactResolver+.resolve( Artifact, ..) ) - && args( artifact, .. ); - - after( MavenProject project, Artifact artifact ) throwing( ArtifactNotFoundException cause ): - within_pm_downloadDependencies( project ) - && ar_resolve( artifact ) - { - getReporter().reportProjectDependencyArtifactNotFound( project, artifact, cause ); - } - - after( MavenProject project, Artifact artifact ) throwing( ArtifactResolutionException cause ): - within_pm_downloadDependencies( project ) - && ar_resolve( artifact ) - { - getReporter().reportProjectDependencyArtifactUnresolvable( project, artifact, cause ); + if ( cause instanceof MultipleArtifactsNotFoundException ) + { + getReporter().reportProjectDependenciesNotFound( project, scope, (MultipleArtifactsNotFoundException) cause ); + } + else + { + getReporter().reportProjectDependenciesUnresolvable( project, scope, cause ); + } } private pointcut le_getLifecycleBindings( List tasks, MavenProject configuringProject, String targetDescription ): Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/errors/CoreErrorReporter.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/errors/CoreErrorReporter.java?rev=611974&r1=611973&r2=611974&view=diff ============================================================================== --- maven/components/trunk/maven-core/src/main/java/org/apache/maven/errors/CoreErrorReporter.java (original) +++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/errors/CoreErrorReporter.java Mon Jan 14 15:59:01 2008 @@ -7,6 +7,7 @@ import org.apache.maven.artifact.resolver.ArtifactNotFoundException; import org.apache.maven.artifact.resolver.ArtifactResolutionException; import org.apache.maven.artifact.resolver.ArtifactResolutionResult; +import org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException; import org.apache.maven.artifact.versioning.ArtifactVersion; import org.apache.maven.artifact.versioning.InvalidVersionSpecificationException; import org.apache.maven.execution.MavenExecutionRequest; @@ -112,11 +113,9 @@ void reportProjectDependenciesNotFound( MavenProject project, String scope, ArtifactNotFoundException cause ); - void reportProjectDependenciesUnresolvable( MavenProject project, String scope, ArtifactResolutionException cause ); - - void reportProjectDependencyArtifactNotFound( MavenProject project, Artifact artifact, ArtifactNotFoundException cause ); + void reportProjectDependenciesNotFound( MavenProject project, String scope, MultipleArtifactsNotFoundException cause ); - void reportProjectDependencyArtifactUnresolvable( MavenProject project, Artifact artifact, ArtifactResolutionException cause ); + void reportProjectDependenciesUnresolvable( MavenProject project, String scope, ArtifactResolutionException cause ); void reportProjectMojoFailureException( MavenSession session, MojoBinding binding, MojoFailureException cause ); Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/errors/DefaultCoreErrorReporter.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/errors/DefaultCoreErrorReporter.java?rev=611974&r1=611973&r2=611974&view=diff ============================================================================== --- maven/components/trunk/maven-core/src/main/java/org/apache/maven/errors/DefaultCoreErrorReporter.java (original) +++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/errors/DefaultCoreErrorReporter.java Mon Jan 14 15:59:01 2008 @@ -10,6 +10,7 @@ import org.apache.maven.artifact.resolver.ArtifactResolutionException; import org.apache.maven.artifact.resolver.ArtifactResolutionResult; import org.apache.maven.artifact.resolver.CyclicDependencyException; +import org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException; import org.apache.maven.artifact.versioning.ArtifactVersion; import org.apache.maven.artifact.versioning.InvalidVersionSpecificationException; import org.apache.maven.execution.MavenExecutionRequest; @@ -643,25 +644,18 @@ reportTransitiveResolutionError( project, scope, cause ); } - public void reportProjectDependenciesUnresolvable( MavenProject project, - String scope, - ArtifactResolutionException cause ) + public void reportProjectDependenciesNotFound( MavenProject project, + String scope, + MultipleArtifactsNotFoundException cause ) { reportTransitiveResolutionError( project, scope, cause ); } - public void reportProjectDependencyArtifactNotFound( MavenProject project, - Artifact artifact, - ArtifactNotFoundException cause ) - { - reportArtifactError( project, artifact, cause ); - } - - public void reportProjectDependencyArtifactUnresolvable( MavenProject project, - Artifact artifact, - ArtifactResolutionException cause ) + public void reportProjectDependenciesUnresolvable( MavenProject project, + String scope, + ArtifactResolutionException cause ) { - reportArtifactError( project, artifact, cause ); + reportTransitiveResolutionError( project, scope, cause ); } private void reportTransitiveResolutionError( MavenProject project, @@ -687,24 +681,6 @@ writeProjectCoordinate( project, writer ); addTips( CoreErrorTips.getDependencyArtifactResolutionTips( project, scope, cause ), - writer ); - - registerBuildError( cause, writer.toString(), cause.getCause() ); - } - - private void reportArtifactError( MavenProject project, - Artifact depArtifact, - AbstractArtifactResolutionException cause ) - { - StringWriter writer = new StringWriter(); - - writer.write( NEWLINE ); - writer.write( "Maven could not resolve one of your project dependencies from the repository:" ); - - writeArtifactInfo( depArtifact, cause, writer, true ); - - writeProjectCoordinate( project, writer ); - addTips( CoreErrorTips.getDependencyArtifactResolutionTips( project, depArtifact, cause ), writer ); registerBuildError( cause, writer.toString(), cause.getCause() ); Added: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/pom.xml URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/pom.xml?rev=611974&view=auto ============================================================================== --- maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/pom.xml (added) +++ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/pom.xml Mon Jan 14 15:59:01 2008 @@ -0,0 +1,22 @@ +<?xml version="1.0" encoding="UTF-8"?><project> + <modelVersion>4.0.0</modelVersion> + <groupId>org.apache.maven.errortest</groupId> + <artifactId>testReportProjectDependenciesNotFound</artifactId> + <version>1</version> + + <repositories> + <repository> + <id>central</id> + <url>file://${java.io.tmpdir}</url> + </repository> + </repositories> + + <dependencies> + <dependency> + <groupId>not</groupId> + <artifactId>found</artifactId> + <version>3.8.1</version> + </dependency> + </dependencies> + +</project> \ No newline at end of file Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/pom.xml ------------------------------------------------------------------------------ svn:eol-style = native Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/pom.xml ------------------------------------------------------------------------------ svn:keywords = "Author Date Id Revision" Added: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/maven/test/error/mojoFailure/App.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/maven/test/error/mojoFailure/App.java?rev=611974&view=auto ============================================================================== --- maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/maven/test/error/mojoFailure/App.java (added) +++ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/maven/test/error/mojoFailure/App.java Mon Jan 14 15:59:01 2008 @@ -0,0 +1,13 @@ +package org.apache.maven.test.error.mojoFailure; + +/** + * Hello world! + * + */ +public class App +{ + public static void main( String[] args ) + { + System.out.println( "Hello World!" ); + } +} Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/maven/test/error/mojoFailure/App.java ------------------------------------------------------------------------------ svn:eol-style = native Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/main/java/org/apache/maven/test/error/mojoFailure/App.java ------------------------------------------------------------------------------ svn:keywords = "Author Date Id Revision" Added: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java?rev=611974&view=auto ============================================================================== --- maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java (added) +++ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java Mon Jan 14 15:59:01 2008 @@ -0,0 +1,38 @@ +package org.apache.maven.test.error.mojoFailure; + +import junit.framework.Test; +import junit.framework.TestCase; +import junit.framework.TestSuite; + +/** + * Unit test for simple App. + */ +public class AppTest + extends TestCase +{ + /** + * Create the test case + * + * @param testName name of the test case + */ + public AppTest( String testName ) + { + super( testName ); + } + + /** + * @return the suite of tests being tested + */ + public static Test suite() + { + return new TestSuite( AppTest.class ); + } + + /** + * Rigourous Test :-) + */ + public void testApp() + { + assertTrue( true ); + } +} Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java ------------------------------------------------------------------------------ svn:eol-style = native Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesNotFound/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java ------------------------------------------------------------------------------ svn:keywords = "Author Date Id Revision" Added: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/pom.xml URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/pom.xml?rev=611974&view=auto ============================================================================== --- maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/pom.xml (added) +++ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/pom.xml Mon Jan 14 15:59:01 2008 @@ -0,0 +1,22 @@ +<?xml version="1.0" encoding="UTF-8"?><project> + <modelVersion>4.0.0</modelVersion> + <groupId>org.apache.maven.errortest</groupId> + <artifactId>testReportProjectDependenciesUnresolvable</artifactId> + <version>1</version> + + <repositories> + <repository> + <id>central</id> + <url>test://host/path</url> + </repository> + </repositories> + + <dependencies> + <dependency> + <groupId>not</groupId> + <artifactId>found</artifactId> + <version>3.8.1</version> + </dependency> + </dependencies> + +</project> \ No newline at end of file Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/pom.xml ------------------------------------------------------------------------------ svn:eol-style = native Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/pom.xml ------------------------------------------------------------------------------ svn:keywords = "Author Date Id Revision" Added: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/maven/test/error/mojoFailure/App.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/maven/test/error/mojoFailure/App.java?rev=611974&view=auto ============================================================================== --- maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/maven/test/error/mojoFailure/App.java (added) +++ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/maven/test/error/mojoFailure/App.java Mon Jan 14 15:59:01 2008 @@ -0,0 +1,13 @@ +package org.apache.maven.test.error.mojoFailure; + +/** + * Hello world! + * + */ +public class App +{ + public static void main( String[] args ) + { + System.out.println( "Hello World!" ); + } +} Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/maven/test/error/mojoFailure/App.java ------------------------------------------------------------------------------ svn:eol-style = native Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/main/java/org/apache/maven/test/error/mojoFailure/App.java ------------------------------------------------------------------------------ svn:keywords = "Author Date Id Revision" Added: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java?rev=611974&view=auto ============================================================================== --- maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java (added) +++ maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java Mon Jan 14 15:59:01 2008 @@ -0,0 +1,38 @@ +package org.apache.maven.test.error.mojoFailure; + +import junit.framework.Test; +import junit.framework.TestCase; +import junit.framework.TestSuite; + +/** + * Unit test for simple App. + */ +public class AppTest + extends TestCase +{ + /** + * Create the test case + * + * @param testName name of the test case + */ + public AppTest( String testName ) + { + super( testName ); + } + + /** + * @return the suite of tests being tested + */ + public static Test suite() + { + return new TestSuite( AppTest.class ); + } + + /** + * Rigourous Test :-) + */ + public void testApp() + { + assertTrue( true ); + } +} Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java ------------------------------------------------------------------------------ svn:eol-style = native Propchange: maven/components/trunk/maven-embedder/src/test/error-reporting-projects/testReportProjectDependenciesUnresolvable/src/test/java/org/apache/maven/test/error/mojoFailure/AppTest.java ------------------------------------------------------------------------------ svn:keywords = "Author Date Id Revision" Modified: maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/error/ErrorReporterPointcutTest.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/error/ErrorReporterPointcutTest.java?rev=611974&r1=611973&r2=611974&view=diff ============================================================================== --- maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/error/ErrorReporterPointcutTest.java (original) +++ maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/error/ErrorReporterPointcutTest.java Mon Jan 14 15:59:01 2008 @@ -5,6 +5,7 @@ import org.apache.commons.httpclient.HttpException; import org.apache.commons.httpclient.HttpStatus; import org.apache.commons.httpclient.methods.GetMethod; +import org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException; import org.apache.maven.embedder.Configuration; import org.apache.maven.embedder.DefaultConfiguration; import org.apache.maven.embedder.MavenEmbedder; @@ -216,31 +217,35 @@ fail( writer.toString() ); } - // FIXME: Figure out how to keep the project-build error report from being the primary report... public void testReportErrorResolvingExtensionDirectDependencies() throws URISyntaxException, IOException { -// File projectDir = prepareProjectDir(); -// File localRepo = new File( projectDir, "local-repo" ); -// File project = new File( projectDir, "project" ); -// -// reporter.reportErrorResolvingExtensionDirectDependencies( null, null, null, null, null ); -// reporterCtl.setMatcher( MockControl.ALWAYS_MATCHER ); -// reporterCtl.setVoidCallable(); -// -// reporterCtl.replay(); -// -// MavenExecutionRequest request = new DefaultMavenExecutionRequest().setBaseDirectory( project ) -// .setLocalRepositoryPath( localRepo ) -// .setShowErrors( true ) -// .setErrorReporter( reporter ) -// .setGoals( Arrays.asList( new String[] { -// "initialize" -// } ) ); -// -// maven.execute( request ); -// -// reporterCtl.verify(); + File projectDir = prepareProjectDir(); + File localRepo = new File( projectDir, "local-repo" ); + File project = new File( projectDir, "project" ); + + // TODO: Verify that the actual error reported is the one that identified the failing project as an extension POM. + reporter.reportBadDependencyVersion( null, null, null ); + reporterCtl.setMatcher( MockControl.ALWAYS_MATCHER ); + reporterCtl.setVoidCallable(); + + reporter.reportErrorResolvingExtensionDirectDependencies( null, null, null, null, null ); + reporterCtl.setMatcher( MockControl.ALWAYS_MATCHER ); + reporterCtl.setVoidCallable(); + + reporterCtl.replay(); + + MavenExecutionRequest request = new DefaultMavenExecutionRequest().setBaseDirectory( project ) + .setLocalRepositoryPath( localRepo ) + .setShowErrors( true ) + .setErrorReporter( reporter ) + .setGoals( Arrays.asList( new String[] { + "initialize" + } ) ); + + maven.execute( request ); + + reporterCtl.verify(); } public void testReportAggregatedMojoFailureException() @@ -613,27 +618,49 @@ } public void testReportProjectDependenciesNotFound() + throws URISyntaxException, IOException { - // TODO Auto-generated method stub + File projectDir = prepareProjectDir(); + reporter.reportProjectDependenciesNotFound( null, null, (MultipleArtifactsNotFoundException) null ); + reporterCtl.setMatcher( MockControl.ALWAYS_MATCHER ); + reporterCtl.setVoidCallable(); + + reporterCtl.replay(); + + MavenExecutionRequest request = new DefaultMavenExecutionRequest().setBaseDirectory( projectDir ) + .setShowErrors( true ) + .setErrorReporter( reporter ) + .setGoals( Arrays.asList( new String[] { + "compile" + } ) ); + + maven.execute( request ); + + reporterCtl.verify(); } public void testReportProjectDependenciesUnresolvable() + throws URISyntaxException, IOException { - // TODO Auto-generated method stub + File projectDir = prepareProjectDir(); - } + reporter.reportProjectDependenciesUnresolvable( null, null, null ); + reporterCtl.setMatcher( MockControl.ALWAYS_MATCHER ); + reporterCtl.setVoidCallable(); - public void testReportProjectDependencyArtifactNotFound() - { - // TODO Auto-generated method stub + reporterCtl.replay(); - } + MavenExecutionRequest request = new DefaultMavenExecutionRequest().setBaseDirectory( projectDir ) + .setShowErrors( true ) + .setErrorReporter( reporter ) + .setGoals( Arrays.asList( new String[] { + "compile" + } ) ); - public void testReportProjectDependencyArtifactUnresolvable() - { - // TODO Auto-generated method stub + maven.execute( request ); + reporterCtl.verify(); } public void testReportProjectMojoFailureException()