Merge pull request #77 from apache/fail

elharo · web-flow · commit 89d0da961850 · 2020-07-06T14:09:23.000-04:00
exceptions fail tests
diff --git a/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo.java b/src/test/java/org/apache/maven/plugins/dependency/fromDependencies/TestCopyDependenciesMojo.java
@@ -79,18 +79,10 @@ protected void setUp()
         setVariableValueToObject( mojo, "artifactHandlerManager", manager );
     }
 
-    public void assertNoMarkerFile( Artifact artifact )
+    public void assertNoMarkerFile( Artifact artifact ) throws MojoExecutionException
     {
         DefaultFileMarkerHandler handle = new DefaultFileMarkerHandler( artifact, mojo.markersDirectory );
-        try
-        {
-            assertFalse( handle.isMarkerSet() );
-        }
-        catch ( MojoExecutionException e )
-        {
-            fail( e.getLongMessage() );
-        }
-
+        assertFalse( handle.isMarkerSet() );
     }
 
     public void testCopyFile()

Original file line number	Diff line number	Diff line change
`@@ -79,18 +79,10 @@ protected void setUp()`
`79`	`79`	`setVariableValueToObject( mojo, "artifactHandlerManager", manager );`
`80`	`80`	`}`
`81`	`81`
`82`		`- public void assertNoMarkerFile( Artifact artifact )`
	`82`	`+ public void assertNoMarkerFile( Artifact artifact ) throws MojoExecutionException`
`83`	`83`	`{`
`84`	`84`	`DefaultFileMarkerHandler handle = new DefaultFileMarkerHandler( artifact, mojo.markersDirectory );`
`85`		`- try`
`86`		`- {`
`87`		`- assertFalse( handle.isMarkerSet() );`
`88`		`- }`
`89`		`- catch ( MojoExecutionException e )`
`90`		`- {`
`91`		`- fail( e.getLongMessage() );`
`92`		`- }`
`93`		`-`
	`85`	`+ assertFalse( handle.isMarkerSet() );`
`94`	`86`	`}`
`95`	`87`
`96`	`88`	`public void testCopyFile()`