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 ef07344f4 Remove obsolete Java 1.4 test
ef07344f4 is described below

commit ef07344f4e57c39a254fafa27c33905f3f7fc0ca
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Tue Nov 12 08:46:14 2024 -0500

    Remove obsolete Java 1.4 test
---
 .../commons/io/DirectoryWalkerTestCaseJava4.java   | 522 ---------------------
 1 file changed, 522 deletions(-)

diff --git 
a/src/test/java/org/apache/commons/io/DirectoryWalkerTestCaseJava4.java 
b/src/test/java/org/apache/commons/io/DirectoryWalkerTestCaseJava4.java
deleted file mode 100644
index c9cf22b42..000000000
--- a/src/test/java/org/apache/commons/io/DirectoryWalkerTestCaseJava4.java
+++ /dev/null
@@ -1,522 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.apache.commons.io;
-
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertFalse;
-import static org.junit.jupiter.api.Assertions.assertThrows;
-import static org.junit.jupiter.api.Assertions.assertTrue;
-import static org.junit.jupiter.api.Assertions.fail;
-
-import java.io.File;
-import java.io.FileFilter;
-import java.io.IOException;
-import java.util.ArrayList;
-import java.util.Collection;
-import java.util.List;
-
-import org.apache.commons.io.filefilter.FileFilterUtils;
-import org.apache.commons.io.filefilter.IOFileFilter;
-import org.apache.commons.io.filefilter.NameFileFilter;
-import org.apache.commons.io.filefilter.OrFileFilter;
-import org.junit.jupiter.api.Test;
-
-/**
- * This is used to test DirectoryWalker for correctness when using Java4 (no 
generics).
- *
- * @see DirectoryWalker
- */
-@SuppressWarnings({"unchecked", "rawtypes"}) // Java4
-public class DirectoryWalkerTestCaseJava4 {
-
-    /**
-     * Test DirectoryWalker implementation that finds files in a directory 
hierarchy
-     * applying a file filter.
-     */
-    static class TestCancelWalker extends DirectoryWalker {
-        private final String cancelFileName;
-        private final boolean suppressCancel;
-
-        TestCancelWalker(final String cancelFileName, final boolean 
suppressCancel) {
-            this.cancelFileName = cancelFileName;
-            this.suppressCancel = suppressCancel;
-        }
-
-        /**
-         * find files.
-         */
-        protected List find(final File startDirectory) throws IOException {
-            final List results = new ArrayList();
-            walk(startDirectory, results);
-            return results;
-        }
-
-        /**
-         * Handles Cancel.
-         */
-        @Override
-        protected void handleCancelled(final File startDirectory, final 
Collection results,
-                                       final CancelException cancel) throws 
IOException {
-            if (!suppressCancel) {
-                super.handleCancelled(startDirectory, results, cancel);
-            }
-        }
-
-        /**
-         * Handles a directory end by adding the File to the result set.
-         */
-        @Override
-        protected void handleDirectoryEnd(final File directory, final int 
depth, final Collection results) throws IOException {
-            results.add(directory);
-            if (cancelFileName.equals(directory.getName())) {
-                throw new CancelException(directory, depth);
-            }
-        }
-
-        /**
-         * Handles a file by adding the File to the result set.
-         */
-        @Override
-        protected void handleFile(final File file, final int depth, final 
Collection results) throws IOException {
-            results.add(file);
-            if (cancelFileName.equals(file.getName())) {
-                throw new CancelException(file, depth);
-            }
-        }
-    }
-    /**
-     * Test DirectoryWalker implementation that always returns false
-     * from handleDirectoryStart()
-     */
-    private static final class TestFalseFileFinder extends TestFileFinder {
-
-        protected TestFalseFileFinder(final FileFilter filter, final int 
depthLimit) {
-            super(filter, depthLimit);
-        }
-
-        /**
-         * Always returns false.
-         */
-        @Override
-        protected boolean handleDirectory(final File directory, final int 
depth, final Collection results) {
-            return false;
-        }
-    }
-    /**
-     * Test DirectoryWalker implementation that finds files in a directory 
hierarchy
-     * applying a file filter.
-     */
-    private static class TestFileFinder extends DirectoryWalker {
-
-        protected TestFileFinder(final FileFilter filter, final int 
depthLimit) {
-            super(filter, depthLimit);
-        }
-
-        protected TestFileFinder(final IOFileFilter dirFilter, final 
IOFileFilter fileFilter, final int depthLimit) {
-            super(dirFilter, fileFilter, depthLimit);
-        }
-
-        /**
-         * find files.
-         */
-        protected List<File> find(final File startDirectory) {
-            final List<File> results = new ArrayList<>();
-            try {
-                walk(startDirectory, results);
-            } catch (final IOException ex) {
-                fail(ex.toString());
-            }
-            return results;
-        }
-
-        /**
-         * Handles a directory end by adding the File to the result set.
-         */
-        @Override
-        protected void handleDirectoryEnd(final File directory, final int 
depth, final Collection results) {
-            results.add(directory);
-        }
-
-        /**
-         * Handles a file by adding the File to the result set.
-         */
-        @Override
-        protected void handleFile(final File file, final int depth, final 
Collection results) {
-            results.add(file);
-        }
-    }
-    /**
-     * Test DirectoryWalker implementation that finds files in a directory 
hierarchy
-     * applying a file filter.
-     */
-    static class TestMultiThreadCancelWalker extends DirectoryWalker {
-        private final String cancelFileName;
-        private final boolean suppressCancel;
-        private boolean cancelled;
-        public List results;
-
-        TestMultiThreadCancelWalker(final String cancelFileName, final boolean 
suppressCancel) {
-            this.cancelFileName = cancelFileName;
-            this.suppressCancel = suppressCancel;
-        }
-
-        /**
-         * find files.
-         */
-        protected List find(final File startDirectory) throws IOException {
-            results = new ArrayList();
-            walk(startDirectory, results);
-            return results;
-        }
-
-        /**
-         * Handles Cancel.
-         */
-        @Override
-        protected void handleCancelled(final File startDirectory, final 
Collection results,
-                                       final CancelException cancel) throws 
IOException {
-            if (!suppressCancel) {
-                super.handleCancelled(startDirectory, results, cancel);
-            }
-        }
-
-        /**
-         * Handles a directory end by adding the File to the result set.
-         */
-        @Override
-        protected void handleDirectoryEnd(final File directory, final int 
depth, final Collection results) throws IOException {
-            results.add(directory);
-            assertFalse(cancelled);
-            if (cancelFileName.equals(directory.getName())) {
-                cancelled = true;
-            }
-        }
-
-        /**
-         * Handles a file by adding the File to the result set.
-         */
-        @Override
-        protected void handleFile(final File file, final int depth, final 
Collection results) throws IOException {
-            results.add(file);
-            assertFalse(cancelled);
-            if (cancelFileName.equals(file.getName())) {
-                cancelled = true;
-            }
-        }
-
-        /**
-         * Handles Cancelled.
-         */
-        @Override
-        protected boolean handleIsCancelled(final File file, final int depth, 
final Collection results) throws IOException {
-            return cancelled;
-        }
-    }
-    // Directories
-    private static final File current = FileUtils.current();
-    private static final File javaDir = new File("src/main/java");
-    private static final File orgDir = new File(javaDir, "org");
-    private static final File apacheDir = new File(orgDir, "apache");
-
-    private static final File commonsDir = new File(apacheDir, "commons");
-    private static final File ioDir = new File(commonsDir, "io");
-    private static final File outputDir = new File(ioDir, "output");
-    private static final File[] dirs = {orgDir, apacheDir, commonsDir, ioDir, 
outputDir};
-    // Files
-    private static final File fileNameUtils = new File(ioDir, 
"FilenameUtils.java");
-    private static final File ioUtils = new File(ioDir, "IOUtils.java");
-
-    private static final File proxyWriter = new File(outputDir, 
"ProxyWriter.java");
-    private static final File nullStream = new File(outputDir, 
"NullOutputStream.java");
-    private static final File[] ioFiles = {fileNameUtils, ioUtils};
-    private static final File[] outputFiles = {proxyWriter, nullStream};
-    // Filters
-    private static final IOFileFilter dirsFilter = createNameFilter(dirs);
-
-    private static final IOFileFilter ioFilesFilter = 
createNameFilter(ioFiles);
-
-    private static final IOFileFilter outputFilesFilter = 
createNameFilter(outputFiles);
-
-    private static final IOFileFilter ioDirAndFilesFilter = new 
OrFileFilter(dirsFilter, ioFilesFilter);
-
-    private static final IOFileFilter dirsAndFilesFilter = new 
OrFileFilter(ioDirAndFilesFilter, outputFilesFilter);
-
-    // Filter to exclude SVN files
-    private static final IOFileFilter NOT_SVN = 
FileFilterUtils.makeSVNAware(null);
-
-    /**
-     * Create a name filter containing the names of the files
-     * in the array.
-     */
-    private static IOFileFilter createNameFilter(final File[] files) {
-        final String[] names = new String[files.length];
-        for (int i = 0; i < files.length; i++) {
-            names[i] = files[i].getName();
-        }
-        return new NameFileFilter(names);
-    }
-
-    /**
-     * Check the files in the array are in the results list.
-     */
-    private void checkContainsFiles(final String prefix, final File[] files, 
final Collection results) {
-        for (int i = 0; i < files.length; i++) {
-            assertTrue(results.contains(files[i]), prefix + "[" + i + "] " + 
files[i]);
-        }
-    }
-
-    /**
-     * Extract the directories.
-     */
-    private List directoriesOnly(final Collection<File> results) {
-        final List list = new ArrayList(results.size());
-        for (final File file : results) {
-            if (file.isDirectory()) {
-                list.add(file);
-            }
-        }
-        return list;
-    }
-
-    /**
-     * Extract the files.
-     */
-    private List filesOnly(final Collection<File> results) {
-        final List list = new ArrayList(results.size());
-        for (final File file : results) {
-            if (file.isFile()) {
-                list.add(file);
-            }
-        }
-        return list;
-    }
-
-    /**
-     * Test Cancel
-     * @throws IOException
-     */
-    @Test
-    public void testCancel() throws IOException {
-        String cancelName = null;
-
-        // Cancel on a file
-        try {
-            cancelName = "DirectoryWalker.java";
-            new TestCancelWalker(cancelName, false).find(javaDir);
-            fail("CancelException not thrown for '" + cancelName + "'");
-        } catch (final DirectoryWalker.CancelException cancel) {
-            assertEquals(cancelName, cancel.getFile().getName(), "File:  " + 
cancelName);
-            assertEquals(5, cancel.getDepth(), "Depth: " + cancelName);
-        }
-
-        // Cancel on a directory
-        try {
-            cancelName = "commons";
-            new TestCancelWalker(cancelName, false).find(javaDir);
-            fail("CancelException not thrown for '" + cancelName + "'");
-        } catch (final DirectoryWalker.CancelException cancel) {
-            assertEquals(cancelName, cancel.getFile().getName(), "File:  " + 
cancelName);
-            assertEquals(3, cancel.getDepth(), "Depth: " + cancelName);
-        }
-
-        // Suppress CancelException (use same file name as preceding test)
-        final List results = new TestCancelWalker(cancelName, 
true).find(javaDir);
-        final File lastFile = (File) results.get(results.size() - 1);
-        assertEquals(cancelName, lastFile.getName(), "Suppress:  " + 
cancelName);
-    }
-
-    /**
-     * Test Filtering
-     */
-    @Test
-    public void testFilter() {
-        final List<File> results = new TestFileFinder(dirsAndFilesFilter, 
-1).find(javaDir);
-        assertEquals(1 + dirs.length + ioFiles.length + outputFiles.length, 
results.size(), "Result Size");
-        assertTrue(results.contains(javaDir), "Start Dir");
-        checkContainsFiles("Dir", dirs, results);
-        checkContainsFiles("IO File", ioFiles, results);
-        checkContainsFiles("Output File", outputFiles, results);
-    }
-
-    /**
-     * Test Filtering and limit to depth 0
-     */
-    @Test
-    public void testFilterAndLimitA() {
-        final List<File> results = new TestFileFinder(NOT_SVN, 
0).find(javaDir);
-        assertEquals(1, results.size(), "[A] Result Size");
-        assertTrue(results.contains(javaDir), "[A] Start Dir");
-    }
-
-    /**
-     * Test Filtering and limit to depth 1
-     */
-    @Test
-    public void testFilterAndLimitB() {
-        final List<File> results = new TestFileFinder(NOT_SVN, 
1).find(javaDir);
-        assertEquals(2, results.size(), "[B] Result Size");
-        assertTrue(results.contains(javaDir), "[B] Start Dir");
-        assertTrue(results.contains(orgDir), "[B] Org Dir");
-    }
-
-    /**
-     * Test Filtering and limit to depth 3
-     */
-    @Test
-    public void testFilterAndLimitC() {
-        final List<File> results = new TestFileFinder(NOT_SVN, 
3).find(javaDir);
-        assertEquals(4, results.size(), "[C] Result Size");
-        assertTrue(results.contains(javaDir), "[C] Start Dir");
-        assertTrue(results.contains(orgDir), "[C] Org Dir");
-        assertTrue(results.contains(apacheDir), "[C] Apache Dir");
-        assertTrue(results.contains(commonsDir), "[C] Commons Dir");
-    }
-
-    /**
-     * Test Filtering and limit to depth 5
-     */
-    @Test
-    public void testFilterAndLimitD() {
-        final List<File> results = new TestFileFinder(dirsAndFilesFilter, 
5).find(javaDir);
-        assertEquals(1 + dirs.length + ioFiles.length, results.size(), "[D] 
Result Size");
-        assertTrue(results.contains(javaDir), "[D] Start Dir");
-        checkContainsFiles("[D] Dir", dirs, results);
-        checkContainsFiles("[D] File", ioFiles, results);
-    }
-
-    /**
-     * Test separate dir and file filters
-     */
-    @Test
-    public void testFilterDirAndFile1() {
-        final List<File> results = new TestFileFinder(dirsFilter, 
ioFilesFilter, -1).find(javaDir);
-        assertEquals(1 + dirs.length + ioFiles.length, results.size(), 
"[DirAndFile1] Result Size");
-        assertTrue(results.contains(javaDir), "[DirAndFile1] Start Dir");
-        checkContainsFiles("[DirAndFile1] Dir", dirs, results);
-        checkContainsFiles("[DirAndFile1] File", ioFiles, results);
-    }
-
-    /**
-     * Test separate dir and file filters
-     */
-    @Test
-    public void testFilterDirAndFile2() {
-        final List<File> results = new TestFileFinder(null, null, 
-1).find(javaDir);
-        assertTrue(results.size() > 1 + dirs.length + ioFiles.length, 
"[DirAndFile2] Result Size");
-        assertTrue(results.contains(javaDir), "[DirAndFile2] Start Dir");
-        checkContainsFiles("[DirAndFile2] Dir", dirs, results);
-        checkContainsFiles("[DirAndFile2] File", ioFiles, results);
-    }
-
-    /**
-     * Test separate dir and file filters
-     */
-    @Test
-    public void testFilterDirAndFile3() {
-        final List<File> results = new TestFileFinder(dirsFilter, null, 
-1).find(javaDir);
-        final List resultDirs = directoriesOnly(results);
-        assertEquals(1 + dirs.length, resultDirs.size(), "[DirAndFile3] Result 
Size");
-        assertTrue(results.contains(javaDir), "[DirAndFile3] Start Dir");
-        checkContainsFiles("[DirAndFile3] Dir", dirs, resultDirs);
-    }
-
-    /**
-     * Test separate dir and file filters
-     */
-    @Test
-    public void testFilterDirAndFile4() {
-        final List<File> results = new TestFileFinder(null, ioFilesFilter, 
-1).find(javaDir);
-        final List resultFiles = filesOnly(results);
-        assertEquals(ioFiles.length, resultFiles.size(), "[DirAndFile4] Result 
Size");
-        assertTrue(results.contains(javaDir), "[DirAndFile4] Start Dir");
-        checkContainsFiles("[DirAndFile4] File", ioFiles, resultFiles);
-    }
-
-    /**
-     * test an invalid start directory
-     */
-    @Test
-    public void testHandleStartDirectoryFalse() {
-
-        final List<File> results = new TestFalseFileFinder(null, 
-1).find(current);
-        assertEquals(0, results.size(), "Result Size");
-
-    }
-
-    /**
-     * Test Limiting to current directory
-     */
-    @Test
-    public void testLimitToCurrent() {
-        final List<File> results = new TestFileFinder(null, 0).find(current);
-        assertEquals(1, results.size(), "Result Size");
-        assertTrue(results.contains(FileUtils.current()), "Current Dir");
-    }
-
-    /**
-     * Test an invalid start directory
-     */
-    @Test
-    public void testMissingStartDirectory() {
-
-        // TODO is this what we want with invalid directory?
-        final File invalidDir = new File("invalid-dir");
-        final List<File> results = new TestFileFinder(null, 
-1).find(invalidDir);
-        assertEquals(1, results.size(), "Result Size");
-        assertTrue(results.contains(invalidDir), "Current Dir");
-
-        assertThrows(NullPointerException.class, () -> new 
TestFileFinder(null, -1).find(null));
-    }
-
-    /**
-     * Test Cancel
-     * @throws IOException
-     */
-    @Test
-    public void testMultiThreadCancel() throws IOException {
-        String cancelName = "DirectoryWalker.java";
-        TestMultiThreadCancelWalker walker = new 
TestMultiThreadCancelWalker(cancelName, false);
-        // Cancel on a file
-        try {
-            walker.find(javaDir);
-            fail("CancelException not thrown for '" + cancelName + "'");
-        } catch (final DirectoryWalker.CancelException cancel) {
-            final File last = (File) walker.results.get(walker.results.size() 
- 1);
-            assertEquals(cancelName, last.getName());
-            assertEquals(5, cancel.getDepth(), "Depth: " + cancelName);
-        }
-
-        // Cancel on a directory
-        try {
-            cancelName = "commons";
-            walker = new TestMultiThreadCancelWalker(cancelName, false);
-            walker.find(javaDir);
-            fail("CancelException not thrown for '" + cancelName + "'");
-        } catch (final DirectoryWalker.CancelException cancel) {
-            assertEquals(cancelName, cancel.getFile().getName(), "File:  " + 
cancelName);
-            assertEquals(3, cancel.getDepth(), "Depth: " + cancelName);
-        }
-
-        // Suppress CancelException (use same file name as preceding test)
-        walker = new TestMultiThreadCancelWalker(cancelName, true);
-        final List results = walker.find(javaDir);
-        final File lastFile = (File) results.get(results.size() - 1);
-        assertEquals(cancelName, lastFile.getName(), "Suppress:  " + 
cancelName);
-
-    }
-
-}

Reply via email to