Repository: commons-io Updated Branches: refs/heads/master 73202a1e3 -> 27de9894f
Replace custom temporary folder management with JUnit's TemporaryFolder. This will allow Maven to run tests concurrently. Project: http://git-wip-us.apache.org/repos/asf/commons-io/repo Commit: http://git-wip-us.apache.org/repos/asf/commons-io/commit/27de9894 Tree: http://git-wip-us.apache.org/repos/asf/commons-io/tree/27de9894 Diff: http://git-wip-us.apache.org/repos/asf/commons-io/diff/27de9894 Branch: refs/heads/master Commit: 27de9894f9e9102eac91fa928eb1443001a89feb Parents: 73202a1 Author: Gary Gregory <ggreg...@apache.org> Authored: Wed Sep 27 13:05:55 2017 -0600 Committer: Gary Gregory <ggreg...@apache.org> Committed: Wed Sep 27 13:05:55 2017 -0600 ---------------------------------------------------------------------- .../comparator/ComparatorAbstractTestCase.java | 24 +++++++++++++------- .../comparator/CompositeFileComparatorTest.java | 10 ++++---- .../comparator/DefaultFileComparatorTest.java | 4 ++-- .../comparator/DirectoryFileComparatorTest.java | 4 ++-- .../comparator/ExtensionFileComparatorTest.java | 6 ++--- .../LastModifiedFileComparatorTest.java | 6 ++--- .../io/comparator/NameFileComparatorTest.java | 6 ++--- .../io/comparator/PathFileComparatorTest.java | 6 ++--- .../io/comparator/SizeFileComparatorTest.java | 12 +++++----- 9 files changed, 43 insertions(+), 35 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/commons-io/blob/27de9894/src/test/java/org/apache/commons/io/comparator/ComparatorAbstractTestCase.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/io/comparator/ComparatorAbstractTestCase.java b/src/test/java/org/apache/commons/io/comparator/ComparatorAbstractTestCase.java index 3d919bd..69c913e 100644 --- a/src/test/java/org/apache/commons/io/comparator/ComparatorAbstractTestCase.java +++ b/src/test/java/org/apache/commons/io/comparator/ComparatorAbstractTestCase.java @@ -16,26 +16,34 @@ */ package org.apache.commons.io.comparator; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertNull; +import static org.junit.Assert.assertSame; +import static org.junit.Assert.assertTrue; + import java.io.File; import java.util.ArrayList; import java.util.Comparator; import java.util.List; import org.apache.commons.io.FileUtils; -import org.apache.commons.io.testtools.FileBasedTestCase; import org.junit.After; +import org.junit.Rule; import org.junit.Test; - -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertNotNull; -import static org.junit.Assert.assertNull; -import static org.junit.Assert.assertSame; -import static org.junit.Assert.assertTrue; +import org.junit.rules.TemporaryFolder; /** * Base Test case for Comparator implementations. */ -public abstract class ComparatorAbstractTestCase extends FileBasedTestCase { +public abstract class ComparatorAbstractTestCase { + + @Rule + public TemporaryFolder temporaryFolder = new TemporaryFolder(); + + protected File getTestDirectory() { + return temporaryFolder.getRoot(); + } /** comparator instance */ protected AbstractFileComparator comparator; http://git-wip-us.apache.org/repos/asf/commons-io/blob/27de9894/src/test/java/org/apache/commons/io/comparator/CompositeFileComparatorTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/io/comparator/CompositeFileComparatorTest.java b/src/test/java/org/apache/commons/io/comparator/CompositeFileComparatorTest.java index 26610c6..9d98791 100644 --- a/src/test/java/org/apache/commons/io/comparator/CompositeFileComparatorTest.java +++ b/src/test/java/org/apache/commons/io/comparator/CompositeFileComparatorTest.java @@ -16,9 +16,8 @@ */ package org.apache.commons.io.comparator; -import org.apache.commons.io.testtools.TestUtils; -import org.junit.Before; -import org.junit.Test; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertTrue; import java.io.BufferedOutputStream; import java.io.File; @@ -28,8 +27,9 @@ import java.util.ArrayList; import java.util.Comparator; import java.util.List; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertTrue; +import org.apache.commons.io.testtools.TestUtils; +import org.junit.Before; +import org.junit.Test; /** * Test case for {@link CompositeFileComparator}. http://git-wip-us.apache.org/repos/asf/commons-io/blob/27de9894/src/test/java/org/apache/commons/io/comparator/DefaultFileComparatorTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/io/comparator/DefaultFileComparatorTest.java b/src/test/java/org/apache/commons/io/comparator/DefaultFileComparatorTest.java index 89ce2b0..392f465 100644 --- a/src/test/java/org/apache/commons/io/comparator/DefaultFileComparatorTest.java +++ b/src/test/java/org/apache/commons/io/comparator/DefaultFileComparatorTest.java @@ -16,10 +16,10 @@ */ package org.apache.commons.io.comparator; -import org.junit.Before; - import java.io.File; +import org.junit.Before; + /** * Test case for {@link DefaultFileComparator}. */ http://git-wip-us.apache.org/repos/asf/commons-io/blob/27de9894/src/test/java/org/apache/commons/io/comparator/DirectoryFileComparatorTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/io/comparator/DirectoryFileComparatorTest.java b/src/test/java/org/apache/commons/io/comparator/DirectoryFileComparatorTest.java index 2128d07..e4649d3 100644 --- a/src/test/java/org/apache/commons/io/comparator/DirectoryFileComparatorTest.java +++ b/src/test/java/org/apache/commons/io/comparator/DirectoryFileComparatorTest.java @@ -16,11 +16,11 @@ */ package org.apache.commons.io.comparator; +import java.io.File; + import org.junit.Before; import org.junit.Test; -import java.io.File; - /** * Test case for {@link DirectoryFileComparator}. */ http://git-wip-us.apache.org/repos/asf/commons-io/blob/27de9894/src/test/java/org/apache/commons/io/comparator/ExtensionFileComparatorTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/io/comparator/ExtensionFileComparatorTest.java b/src/test/java/org/apache/commons/io/comparator/ExtensionFileComparatorTest.java index cd47ab9..dc5d73c 100644 --- a/src/test/java/org/apache/commons/io/comparator/ExtensionFileComparatorTest.java +++ b/src/test/java/org/apache/commons/io/comparator/ExtensionFileComparatorTest.java @@ -16,13 +16,13 @@ */ package org.apache.commons.io.comparator; -import org.junit.Before; -import org.junit.Test; +import static org.junit.Assert.assertTrue; import java.io.File; import java.util.Comparator; -import static org.junit.Assert.assertTrue; +import org.junit.Before; +import org.junit.Test; /** * Test case for {@link ExtensionFileComparator}. http://git-wip-us.apache.org/repos/asf/commons-io/blob/27de9894/src/test/java/org/apache/commons/io/comparator/LastModifiedFileComparatorTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/io/comparator/LastModifiedFileComparatorTest.java b/src/test/java/org/apache/commons/io/comparator/LastModifiedFileComparatorTest.java index 0a5ded2..0fa5750 100644 --- a/src/test/java/org/apache/commons/io/comparator/LastModifiedFileComparatorTest.java +++ b/src/test/java/org/apache/commons/io/comparator/LastModifiedFileComparatorTest.java @@ -16,14 +16,14 @@ */ package org.apache.commons.io.comparator; -import org.apache.commons.io.testtools.TestUtils; -import org.junit.Before; - import java.io.BufferedOutputStream; import java.io.File; import java.io.FileOutputStream; import java.io.IOException; +import org.apache.commons.io.testtools.TestUtils; +import org.junit.Before; + /** * Test case for {@link LastModifiedFileComparator}. */ http://git-wip-us.apache.org/repos/asf/commons-io/blob/27de9894/src/test/java/org/apache/commons/io/comparator/NameFileComparatorTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/io/comparator/NameFileComparatorTest.java b/src/test/java/org/apache/commons/io/comparator/NameFileComparatorTest.java index cabc0e2..d60bbc3 100644 --- a/src/test/java/org/apache/commons/io/comparator/NameFileComparatorTest.java +++ b/src/test/java/org/apache/commons/io/comparator/NameFileComparatorTest.java @@ -16,13 +16,13 @@ */ package org.apache.commons.io.comparator; -import org.junit.Before; -import org.junit.Test; +import static org.junit.Assert.assertTrue; import java.io.File; import java.util.Comparator; -import static org.junit.Assert.assertTrue; +import org.junit.Before; +import org.junit.Test; /** * Test case for {@link NameFileComparator}. http://git-wip-us.apache.org/repos/asf/commons-io/blob/27de9894/src/test/java/org/apache/commons/io/comparator/PathFileComparatorTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/io/comparator/PathFileComparatorTest.java b/src/test/java/org/apache/commons/io/comparator/PathFileComparatorTest.java index 0d9b0a1..5d36833 100644 --- a/src/test/java/org/apache/commons/io/comparator/PathFileComparatorTest.java +++ b/src/test/java/org/apache/commons/io/comparator/PathFileComparatorTest.java @@ -16,13 +16,13 @@ */ package org.apache.commons.io.comparator; -import org.junit.Before; -import org.junit.Test; +import static org.junit.Assert.assertTrue; import java.io.File; import java.util.Comparator; -import static org.junit.Assert.assertTrue; +import org.junit.Before; +import org.junit.Test; /** * Test case for {@link PathFileComparator}. http://git-wip-us.apache.org/repos/asf/commons-io/blob/27de9894/src/test/java/org/apache/commons/io/comparator/SizeFileComparatorTest.java ---------------------------------------------------------------------- diff --git a/src/test/java/org/apache/commons/io/comparator/SizeFileComparatorTest.java b/src/test/java/org/apache/commons/io/comparator/SizeFileComparatorTest.java index 88b60ec..f698b02 100644 --- a/src/test/java/org/apache/commons/io/comparator/SizeFileComparatorTest.java +++ b/src/test/java/org/apache/commons/io/comparator/SizeFileComparatorTest.java @@ -16,18 +16,18 @@ */ package org.apache.commons.io.comparator; -import org.apache.commons.io.testtools.TestUtils; -import org.junit.Before; -import org.junit.Test; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertTrue; import java.io.BufferedOutputStream; import java.io.File; import java.io.FileOutputStream; import java.io.IOException; -import static org.junit.Assert.assertEquals; -import static org.junit.Assert.assertFalse; -import static org.junit.Assert.assertTrue; +import org.apache.commons.io.testtools.TestUtils; +import org.junit.Before; +import org.junit.Test; /** * Test case for {@link SizeFileComparator}.