PeterAlfredLee commented on a change in pull request #153:
URL: https://github.com/apache/commons-compress/pull/153#discussion_r530895371
##########
File path:
src/test/java/org/apache/commons/compress/archivers/tar/SparseFilesTest.java
##########
@@ -73,54 +75,57 @@ public void testPaxGNU() throws Throwable {
@Test
public void testExtractSparseTarsOnWindows() throws IOException {
- if (!isOnWindows) {
- return;
- }
+ assumeTrue(isOnWindows);
Review comment:
Maybe we can make it more detailed with something like :
```
assumeTrue("This test should be ignored if not running on
Windows",isOnWindows);
```
##########
File path:
src/test/java/org/apache/commons/compress/archivers/tar/SparseFilesTest.java
##########
@@ -129,14 +134,13 @@ public void testExtractOldGNU() throws IOException,
InterruptedException {
@Test
public void testExtractExtendedOldGNU() throws IOException,
InterruptedException {
- if (isOnWindows) {
- return;
- }
+ assumeFalse(isOnWindows);
Review comment:
Same one.
##########
File path:
src/test/java/org/apache/commons/compress/archivers/tar/SparseFilesTest.java
##########
@@ -169,31 +173,32 @@ public void testExtractExtendedOldGNU() throws
IOException, InterruptedException
@Test
public void testExtractPaxGNU() throws IOException, InterruptedException {
- if (isOnWindows) {
- return;
- }
+ assumeFalse(isOnWindows);
Review comment:
Same one.
##########
File path:
src/test/java/org/apache/commons/compress/archivers/tar/SparseFilesTest.java
##########
@@ -73,54 +75,57 @@ public void testPaxGNU() throws Throwable {
@Test
public void testExtractSparseTarsOnWindows() throws IOException {
- if (!isOnWindows) {
- return;
- }
+ assumeTrue(isOnWindows);
final File oldGNUSparseTar = getFile("oldgnu_sparse.tar");
final File paxGNUSparseTar = getFile("pax_gnu_sparse.tar");
try (TarArchiveInputStream paxGNUSparseInputStream = new
TarArchiveInputStream(new FileInputStream(paxGNUSparseTar))) {
// compare between old GNU and PAX 0.0
- paxGNUSparseInputStream.getNextTarEntry();
+ TarArchiveEntry paxGNUEntry =
paxGNUSparseInputStream.getNextTarEntry();
+ assertTrue(paxGNUSparseInputStream.canReadEntryData(paxGNUEntry));
try (TarArchiveInputStream oldGNUSparseInputStream = new
TarArchiveInputStream(new FileInputStream(oldGNUSparseTar))) {
- oldGNUSparseInputStream.getNextTarEntry();
+ final TarArchiveEntry oldGNUEntry =
oldGNUSparseInputStream.getNextTarEntry();
+
assertTrue(oldGNUSparseInputStream.canReadEntryData(oldGNUEntry));
assertArrayEquals(IOUtils.toByteArray(oldGNUSparseInputStream),
- IOUtils.toByteArray(paxGNUSparseInputStream));
+ IOUtils.toByteArray(paxGNUSparseInputStream));
}
// compare between old GNU and PAX 0.1
- paxGNUSparseInputStream.getNextTarEntry();
+ paxGNUEntry = paxGNUSparseInputStream.getNextTarEntry();
+ assertTrue(paxGNUSparseInputStream.canReadEntryData(paxGNUEntry));
try (TarArchiveInputStream oldGNUSparseInputStream = new
TarArchiveInputStream(new FileInputStream(oldGNUSparseTar))) {
- oldGNUSparseInputStream.getNextTarEntry();
+ final TarArchiveEntry oldGNUEntry =
oldGNUSparseInputStream.getNextTarEntry();
+
assertTrue(oldGNUSparseInputStream.canReadEntryData(oldGNUEntry));
assertArrayEquals(IOUtils.toByteArray(oldGNUSparseInputStream),
- IOUtils.toByteArray(paxGNUSparseInputStream));
+ IOUtils.toByteArray(paxGNUSparseInputStream));
}
// compare between old GNU and PAX 1.0
- paxGNUSparseInputStream.getNextTarEntry();
+ paxGNUEntry = paxGNUSparseInputStream.getNextTarEntry();
+ assertTrue(paxGNUSparseInputStream.canReadEntryData(paxGNUEntry));
try (TarArchiveInputStream oldGNUSparseInputStream = new
TarArchiveInputStream(new FileInputStream(oldGNUSparseTar))) {
- oldGNUSparseInputStream.getNextTarEntry();
+ final TarArchiveEntry oldGNUEntry =
oldGNUSparseInputStream.getNextTarEntry();
+
assertTrue(oldGNUSparseInputStream.canReadEntryData(oldGNUEntry));
assertArrayEquals(IOUtils.toByteArray(oldGNUSparseInputStream),
- IOUtils.toByteArray(paxGNUSparseInputStream));
+ IOUtils.toByteArray(paxGNUSparseInputStream));
}
}
}
@Test
public void testExtractOldGNU() throws IOException, InterruptedException {
- if (isOnWindows) {
- return;
- }
+ assumeFalse(isOnWindows);
Review comment:
Maybe we can make it more detailed with something like :
```
assumeFalse("This test should be ignored on Windows", isOnWindows);
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
