You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by pe...@apache.org on 2021/01/09 03:20:45 UTC

[commons-compress] 12/13: COMPRESS-540: Remove commented out code in test

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 <th...@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());