disable tests that fail due to #62

cismet · Mar 24, 2017 · 497a6bc · 497a6bc
1 parent 86a99d9
commit 497a6bc
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 1 deletion.
diff --git a/src/main/java/de/cismet/commons/utils/MultiPagePictureReader.java b/src/main/java/de/cismet/commons/utils/MultiPagePictureReader.java
@@ -31,7 +31,13 @@
 import de.cismet.commons.security.handler.SimpleHttpAccessHandler;
 
 /**
- * DOCUMENT ME!
+ * FIXME: 
+ * This class seems to use an outdated Version of JAI.
+ * 
+ * java.lang.NoClassDefFoundError: com/sun/image/codec/jpeg/JPEGCodec
+ * java.lang.NoClassDefFoundError: com/sun/image/codec/jpeg/ImageFormatException
+ * 
+ * See #62
  *
  * @version  $Revision$, $Date$
  */
@@ -212,6 +218,7 @@ public MultiPagePictureReader(final URL imageURL,
             throw new IOException("Could not open '" + imageURL.toExternalForm() + "'.", ex);
         }
 
+        // FIXME: Don't use com.sun classes directly
         decoder = ImageCodec.createImageDecoder(codec, stream, null);
 
         pageCount = decoder.getNumPages();

diff --git a/src/test/java/de/cismet/commons/utils/MultiPagePictureReaderTest.java b/src/test/java/de/cismet/commons/utils/MultiPagePictureReaderTest.java
@@ -12,6 +12,7 @@
 import org.junit.Test;
 import static org.junit.Assert.*;
 import org.junit.BeforeClass;
+import org.junit.Ignore;
 
 /**
  *
@@ -52,7 +53,15 @@ private String getCurrentMethodName()
         return new Throwable().getStackTrace()[1].getMethodName();
     }
 
+    /**
+     * Testr fails on OpenJDK due to missing deprecated com/sun/image/codec/jpeg 
+     * classes.See #62
+     * 
+     * @throws IOException
+     * @throws URISyntaxException 
+     */
     @Test
+    @Ignore
     public void test010readMultiPageTiffSmallJpeg() throws IOException, URISyntaxException
     {
         LOGGER.info("TEST " + this.getCurrentMethodName());
@@ -124,7 +133,15 @@ public void test030readMultiPageTiffSmallZip() throws IOException, URISyntaxExce
         }
     }
 
+    /**
+     * Testr fails on OpenJDK due to missing deprecated com/sun/image/codec/jpeg 
+     * classes.See #62
+     * 
+     * @throws IOException
+     * @throws URISyntaxException 
+     */
     @Test
+    @Ignore
     public void test040readMultiPageTiffBigJpeg() throws IOException, URISyntaxException
     {
         LOGGER.info("TEST " + this.getCurrentMethodName());
@@ -145,7 +162,15 @@ public void test040readMultiPageTiffBigJpeg() throws IOException, URISyntaxExcep
         assertNotNull(multiPagePictureReader.loadPage(0));
     }
 
+    /**
+     * Testr fails on OpenJDK due to missing deprecated com/sun/image/codec/jpeg 
+     * classes.See #62
+     * 
+     * @throws IOException
+     * @throws URISyntaxException 
+     */
     @Test
+    @Ignore
     public void test050readSinglePageJpeg() throws IOException, URISyntaxException
     {
         LOGGER.info("TEST " + this.getCurrentMethodName());