gwlucastrig commented on code in PR #335: URL: https://github.com/apache/commons-imaging/pull/335#discussion_r1398239328
########## src/test/java/org/apache/commons/imaging/formats/tiff/TiffJpegTest.java: ########## @@ -161,6 +166,11 @@ public void testFullPixelMatch() throws IOException { // test set 2 includes a TIFF-specific alpha channel processTestSet(testSet2); + + // test set 3 includes an add-valued width and height that does + // not evently fit a block of JPEG data. It is used to test + // for the proper handling of edge-cases. + processTestSet(testSet3); Review Comment: done ########## src/test/java/org/apache/commons/imaging/formats/tiff/TiffJpegTest.java: ########## @@ -51,6 +51,11 @@ public class TiffJpegTest extends TiffBaseTest { "TestSampleArgb.tiff", "TestJpegArgb.tiff",}; + static final String[] testSet3 = { + "TestSampleRgb127x127.tiff", + "TestJpegRgb127x127.tiff",}; Review Comment: done -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@commons.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org