This is an automated email from the ASF dual-hosted git repository. peterlee pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-compress.git
commit 34759ba88f8121e5ac4332c4b2150e758e27cbb2 Author: theobisproject <theobisproj...@gmail.com> AuthorDate: Wed Dec 30 17:26:06 2020 +0100 COMPRESS-540: Remove commented out code in test --- .../org/apache/commons/compress/archivers/tar/SparseFilesTest.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/test/java/org/apache/commons/compress/archivers/tar/SparseFilesTest.java b/src/test/java/org/apache/commons/compress/archivers/tar/SparseFilesTest.java index a095157..5f9b14b 100644 --- a/src/test/java/org/apache/commons/compress/archivers/tar/SparseFilesTest.java +++ b/src/test/java/org/apache/commons/compress/archivers/tar/SparseFilesTest.java @@ -79,8 +79,6 @@ public class SparseFilesTest extends AbstractTestCase { assertTrue(ae.isOldGNUSparse()); assertTrue(ae.isGNUSparse()); assertFalse(ae.isPaxGNUSparse()); - // TODO: Is this something which should be supported in the random access implementation. Is this even needed in the current stream implementation as it supports sparse entries now? - //assertFalse(tin.canReadEntryData(ae)); List<TarArchiveStructSparse> sparseHeaders = ae.getSparseHeaders(); assertEquals(3, sparseHeaders.size()); @@ -122,8 +120,6 @@ public class SparseFilesTest extends AbstractTestCase { assertTrue(entry.isGNUSparse()); assertTrue(entry.isPaxGNUSparse()); assertFalse(entry.isOldGNUSparse()); - // TODO: Is this something which should be supported in the random access implementation. Is this even needed in the current stream implementation as it supports sparse entries now? - //assertFalse(tin.canReadEntryData(entry)); List<TarArchiveStructSparse> sparseHeaders = entry.getSparseHeaders(); assertEquals(3, sparseHeaders.size());