Improve TestJavadocReportTest#testTestJavadoc

J8 warns and continues with missing dependency, J9 fails.
In fact test was wrong: dependency should have been on classpath

git-svn-id: https://svn.apache.org/repos/asf/maven/plugins/trunk@1816952 13f79535-47bb-0310-9956-ffa450edef68

Author: rfscholte

maven-javadoc-plugin/src/test/java/org/apache/maven/plugins/javadoc/TestJavadocReportTest.java

@@ -55,5 +55,8 @@ public void testTestJavadoc()
         File generatedFile =
             new File( getBasedir(), "target/test/unit/test-javadoc-test/target/site/apidocs/maven/AppTest.html" );
         assertTrue( FileUtils.fileExists( generatedFile.getAbsolutePath() ) );
+        
+        File options = new File( getBasedir(), "target/test/unit/test-javadoc-test/target/site/apidocs/options");
+        FileUtils.fileRead( options ).contains( "junit-3.8.1.jar" );
     }
 }

maven-javadoc-plugin/src/test/java/org/apache/maven/plugins/javadoc/stubs/TestJavadocMavenProjectStub.java

@@ -23,6 +23,7 @@
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
+import java.util.Set;
 
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.artifact.DefaultArtifact;
@@ -50,12 +51,6 @@ public TestJavadocMavenProjectStub()
         setUrl( getModel().getUrl() );
         setPackaging( getModel().getPackaging() );
 
-        Artifact junit = new DefaultArtifact( "junit", "junit", VersionRange.createFromVersion( "3.8.1" ),
-                                              Artifact.SCOPE_TEST, "jar", null, new DefaultArtifactHandler( "jar" ),
-                                              false );
-        junit.setFile( new File( getBasedir() + "/junit/junit/3.8.1/junit-3.8.1.jar" ) );
-        setTestArtifacts( Collections.singletonList( junit ) );
-
         Build build = new Build();
         build.setFinalName( getModel().getArtifactId() );
         build.setDirectory( super.getBasedir() + "/target/test/unit/test-javadoc-test/target" );
@@ -87,4 +82,14 @@ public MavenProject getExecutionProject()
     {
         return this;
     }
+    
+    @Override
+    public Set<Artifact> getArtifacts()
+    {
+        Artifact junit = new DefaultArtifact( "junit", "junit", VersionRange.createFromVersion( "3.8.1" ),
+                                              Artifact.SCOPE_TEST, "jar", null, new DefaultArtifactHandler( "jar" ),
+                                              false );
+        junit.setFile( new File( getBasedir() + "/junit/junit/3.8.1/junit-3.8.1.jar" ) );
+        return Collections.singleton( junit );
+    }
 }