You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by rf...@apache.org on 2017/12/02 14:53:36 UTC
svn commit: r1816958 -
/maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugins/javadoc/JavadocReportTest.java
Author: rfscholte
Date: Sat Dec 2 14:53:36 2017
New Revision: 1816958
URL: http://svn.apache.org/viewvc?rev=1816958&view=rev
Log:
JDK-8032205 was closed as not an issue, so not solved in Java9.
Need to review the conclusion
Modified:
maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugins/javadoc/JavadocReportTest.java
Modified: maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugins/javadoc/JavadocReportTest.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugins/javadoc/JavadocReportTest.java?rev=1816958&r1=1816957&r2=1816958&view=diff
==============================================================================
--- maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugins/javadoc/JavadocReportTest.java (original)
+++ maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugins/javadoc/JavadocReportTest.java Sat Dec 2 14:53:36 2017
@@ -631,7 +631,7 @@ public class JavadocReportTest
assertTrue( content.contains( "<img src=\"doc-files/maven-feather.png\" alt=\"Maven\">" ) );
JavadocVersion javadocVersion = (JavadocVersion) getVariableValueFromObject( mojo, "javadocRuntimeVersion" );
- if( javadocVersion.compareTo( JavadocVersion.parse( "1.8" ) ) >= 0 && javadocVersion.compareTo( JavadocVersion.parse( "9" ) ) < 0)
+ if( javadocVersion.compareTo( JavadocVersion.parse( "1.8" ) ) >= 0 && javadocVersion.compareTo( JavadocVersion.parse( "10" ) ) < 0)
{
// https://bugs.openjdk.java.net/browse/JDK-8032205
assertTrue( "This bug appeared in JDK8 and was planned to be fixed in JDK9, see JDK-8032205",
@@ -641,6 +641,7 @@ public class JavadocReportTest
{
assertFalse( new File( apidocs, "resources/test/doc-files/maven-feather.png" ).exists() );
}
+ assertTrue( new File( apidocs, "resources/test2/doc-files/maven-feather.png" ).exists() );
app2 = new File( apidocs, "resources/test2/App2.html" );
assertTrue( app2.exists() );