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

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


The following commit(s) were added to refs/heads/master by this push:
     new 00df1aa0 remove unreachable code (#526)
00df1aa0 is described below

commit 00df1aa0869c0431f8a54cf8bab95f429a1b5107
Author: Elliotte Rusty Harold <elh...@users.noreply.github.com>
AuthorDate: Mon Dec 18 09:22:52 2023 -0500

    remove unreachable code (#526)
---
 .../org/apache/commons/io/filefilter/AbstractIOFileFilterTest.java | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git 
a/src/test/java/org/apache/commons/io/filefilter/AbstractIOFileFilterTest.java 
b/src/test/java/org/apache/commons/io/filefilter/AbstractIOFileFilterTest.java
index 8208785c..ac9db71f 100644
--- 
a/src/test/java/org/apache/commons/io/filefilter/AbstractIOFileFilterTest.java
+++ 
b/src/test/java/org/apache/commons/io/filefilter/AbstractIOFileFilterTest.java
@@ -111,16 +111,11 @@ public abstract class AbstractIOFileFilterTest {
         assertEquals(expected, filter.accept(file.toPath(), null),
                 "test " + testNumber + " Filter(File) " + 
filter.getClass().getName() + " not " + expected + " for " + file);
 
-        if (file != null && file.getParentFile() != null) {
+        if (file.getParentFile() != null) {
             assertEquals(expected, filter.accept(file.getParentFile(), 
file.getName()),
                     "test " + testNumber + " Filter(File, String) " + 
filter.getClass().getName() + " not " + expected + " for " + file);
             assertEquals(expected, filter.matches(file.toPath()),
                     "test " + testNumber + " Filter(File) " + 
filter.getClass().getName() + " not " + expected + " for " + file);
-        } else if (file == null) {
-            assertEquals(expected, filter.accept(file),
-                    "test " + testNumber + " Filter(File, String) " + 
filter.getClass().getName() + " not " + expected + " for null");
-            assertEquals(expected, filter.matches(null),
-                    "test " + testNumber + " Filter(File, String) " + 
filter.getClass().getName() + " not " + expected + " for null");
         }
     }
 

Reply via email to