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() );