[commons-compress] branch master updated: add a few assertions

[bo...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

bodewig pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-compress.git


The following commit(s) were added to refs/heads/master by this push:
     new 560f243  add a few assertions
560f243 is described below

commit 560f243b97c2342f3569758bf0a51e5cb3f2c326
Author: Stefan Bodewig <bo...@apache.org>
AuthorDate: Sat Jan 25 14:31:39 2020 +0100

    add a few assertions
---
 .../org/apache/commons/compress/archivers/tar/SparseFilesTest.java  | 6 +++---
 1 file changed, 3 insertions(+), 3 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 43bdee7..028de5f 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
@@ -236,14 +236,14 @@ public class SparseFilesTest extends AbstractTestCase {
         Runtime runtime = Runtime.getRuntime();
         Process process = runtime.exec("tar -xf " + tarFile.getPath() + " -C " + resultDir.getPath());
         // wait until the extract finishes
-        process.waitFor();
+        assertEquals(0, process.waitFor());
 
         for (File file : resultDir.listFiles()) {
-            if(file.getName().equals(sparseFileName)) {
+            if (file.getName().equals(sparseFileName)) {
                 return new FileInputStream(file);
             }
         }
-
+        fail("didn't find " + sparseFileName + " after extracting " + tarFile);
         return null;
     }
 }