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 2014/02/01 14:24:43 UTC
svn commit: r1563416 -
/maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugin/javadoc/JavadocReportTest.java
Author: rfscholte
Date: Sat Feb 1 13:24:43 2014
New Revision: 1563416
URL: http://svn.apache.org/r1563416
Log:
Workaround for JDK-8032205 in test
Modified:
maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugin/javadoc/JavadocReportTest.java
Modified: maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugin/javadoc/JavadocReportTest.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugin/javadoc/JavadocReportTest.java?rev=1563416&r1=1563415&r2=1563416&view=diff
==============================================================================
--- maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugin/javadoc/JavadocReportTest.java (original)
+++ maven/plugins/trunk/maven-javadoc-plugin/src/test/java/org/apache/maven/plugin/javadoc/JavadocReportTest.java Sat Feb 1 13:24:43 2014
@@ -599,7 +599,17 @@ public class JavadocReportTest
assertTrue( app.exists() );
content = readFile( app );
assertTrue( content.contains( "<img src=\"doc-files/maven-feather.png\" alt=\"Maven\">" ) );
- assertFalse( new File( apidocs, "resources/test/doc-files/maven-feather.png" ).exists() );
+
+ float javadocVersion = (Float) getVariableValueFromObject( mojo, "fJavadocVersion" );
+ if( Float.compare( 1.8f, javadocVersion ) == 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", new File( apidocs, "resources/test/doc-files/maven-feather.png" ).exists() );
+ }
+ else
+ {
+ assertFalse( new File( apidocs, "resources/test/doc-files/maven-feather.png" ).exists() );
+ }
app2 = new File( apidocs, "resources/test2/App2.html" );
assertTrue( app2.exists() );