[commons-io] 02/02: The declared exceptions are not actually thrown by test methods

[ggregory]

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

commit c9dfe728534bed57bfaf5988aa76a114a1c86a1a
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Tue Oct 17 10:15:28 2023 -0400

    The declared exceptions are not actually thrown by test methods
---
 src/test/java/org/apache/commons/io/EndianUtilsTest.java            | 2 +-
 .../java/org/apache/commons/io/FileUtilsDirectoryContainsTest.java  | 6 +++---
 src/test/java/org/apache/commons/io/IOUtilsTest.java                | 2 +-
 .../java/org/apache/commons/io/function/IOLongSupplierTest.java     | 2 +-
 src/test/java/org/apache/commons/io/function/IOStreamTest.java      | 2 +-
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/test/java/org/apache/commons/io/EndianUtilsTest.java 
b/src/test/java/org/apache/commons/io/EndianUtilsTest.java
index 79df0c66..4247644d 100644
--- a/src/test/java/org/apache/commons/io/EndianUtilsTest.java
+++ b/src/test/java/org/apache/commons/io/EndianUtilsTest.java
@@ -38,7 +38,7 @@ public class EndianUtilsTest  {
     }
 
     @Test
-    public void testEOFException() throws IOException {
+    public void testEOFException() {
         final ByteArrayInputStream input = new ByteArrayInputStream(new byte[] 
{});
         assertThrows(EOFException.class, () -> 
EndianUtils.readSwappedDouble(input));
     }
diff --git 
a/src/test/java/org/apache/commons/io/FileUtilsDirectoryContainsTest.java 
b/src/test/java/org/apache/commons/io/FileUtilsDirectoryContainsTest.java
index bba9f098..3e92fef6 100644
--- a/src/test/java/org/apache/commons/io/FileUtilsDirectoryContainsTest.java
+++ b/src/test/java/org/apache/commons/io/FileUtilsDirectoryContainsTest.java
@@ -112,7 +112,7 @@ public class FileUtilsDirectoryContainsTest {
     }
 
     @Test
-    public void testDirectoryDoesNotExist() throws IOException {
+    public void testDirectoryDoesNotExist() {
         final File dir = new File("DOESNOTEXIST");
         assertFalse(dir.exists());
         assertThrows(IllegalArgumentException.class, () -> 
FileUtils.directoryContains(dir, file1));
@@ -158,12 +158,12 @@ public class FileUtilsDirectoryContainsTest {
     }
 
     @Test
-    public void testSameFile() throws IOException {
+    public void testSameFile() {
         assertThrows(IllegalArgumentException.class, () -> 
FileUtils.directoryContains(file1, file1));
     }
 
     @Test
-    public void testUnrealizedContainment() throws IOException {
+    public void testUnrealizedContainment() {
         final File dir = new File("DOESNOTEXIST");
         final File file = new File(dir, "DOESNOTEXIST2");
         assertFalse(dir.exists());
diff --git a/src/test/java/org/apache/commons/io/IOUtilsTest.java 
b/src/test/java/org/apache/commons/io/IOUtilsTest.java
index 69a8b29e..c921abc1 100644
--- a/src/test/java/org/apache/commons/io/IOUtilsTest.java
+++ b/src/test/java/org/apache/commons/io/IOUtilsTest.java
@@ -1257,7 +1257,7 @@ public class IOUtilsTest {
         assertThrows(NullPointerException.class, () -> 
IOUtils.resourceToURL(null, ClassLoader.getSystemClassLoader()));
     }
 
-    public void testSingleEOL(final String s1, final String s2, final boolean 
ifEquals) throws IOException {
+    public void testSingleEOL(final String s1, final String s2, final boolean 
ifEquals) {
         assertEquals(ifEquals, IOUtils.contentEqualsIgnoreEOL(
                 new CharArrayReader(s1.toCharArray()),
                 new CharArrayReader(s2.toCharArray())
diff --git 
a/src/test/java/org/apache/commons/io/function/IOLongSupplierTest.java 
b/src/test/java/org/apache/commons/io/function/IOLongSupplierTest.java
index 07748ded..dce0a426 100644
--- a/src/test/java/org/apache/commons/io/function/IOLongSupplierTest.java
+++ b/src/test/java/org/apache/commons/io/function/IOLongSupplierTest.java
@@ -49,7 +49,7 @@ public class IOLongSupplierTest {
     }
 
     @Test
-    public void testAsSupplier() throws IOException {
+    public void testAsSupplier() {
         assertThrows(UncheckedIOException.class, () -> 
TestConstants.THROWING_IO_LONG_SUPPLIER.asSupplier().getAsLong());
         assertEquals(1L, getThrowsNone(() -> 
TestUtils.compareAndSetThrowsIO(atomicLong, 1L)));
         assertEquals(1L, atomicLong.get());
diff --git a/src/test/java/org/apache/commons/io/function/IOStreamTest.java 
b/src/test/java/org/apache/commons/io/function/IOStreamTest.java
index 198a361c..e1964254 100644
--- a/src/test/java/org/apache/commons/io/function/IOStreamTest.java
+++ b/src/test/java/org/apache/commons/io/function/IOStreamTest.java
@@ -402,7 +402,7 @@ public class IOStreamTest {
 
     @SuppressWarnings("resource") // custom stream not recognized by compiler 
warning machinery
     @Test
-    public void testOnCloseMultipleHandlers() throws IOException {
+    public void testOnCloseMultipleHandlers() {
         //
         final AtomicReference<String> ref = new AtomicReference<>();
         // Sanity check