Sort members

Author: garydgregory

src/main/java/org/apache/commons/io/channels/CloseShieldChannelHandler.java

@@ -56,10 +56,6 @@ final class CloseShieldChannelHandler implements InvocationHandler {
         SUPPORTED_INTERFACES = Collections.unmodifiableSet(interfaces);
     }
 
-    static boolean isSupported(final Class<?> interfaceClass) {
-        return SUPPORTED_INTERFACES.contains(interfaceClass);
-    }
-
     /**
      * Tests whether the given method is allowed to be called after the shield is closed.
      *
@@ -73,6 +69,10 @@ private static boolean isAllowedAfterClose(final Class<?> declaringClass, final
         return parameterCount == 0 && name.equals("supportedOptions") && NetworkChannel.class.equals(declaringClass);
     }
 
+    static boolean isSupported(final Class<?> interfaceClass) {
+        return SUPPORTED_INTERFACES.contains(interfaceClass);
+    }
+
     /**
      * Tests whether the given method returns 'this' (the channel) as per JDK spec.
      *

src/test/java/org/apache/commons/io/channels/CloseShieldChannelTest.java

@@ -126,17 +126,6 @@ void testCloseIsShielded(final Class<? extends Channel> channelInterface) throws
         verify(channel, times(2)).isOpen();
     }
 
-    @Test
-    void testWrapFileChannel(final @TempDir Path tempDir) throws IOException {
-        final Path testFile = tempDir.resolve("test.txt");
-        FileUtils.touch(testFile.toFile());
-        try (FileChannel channel = FileChannel.open(testFile); Channel shield = CloseShieldChannel.wrap(channel)) {
-            fileChannelInterfaces().forEach(iface -> assertInstanceOf(iface, shield));
-            // FileChannel is not an interface, so can not be implemented.
-            assertFalse(shield instanceof FileChannel, "not FileChannel");
-        }
-    }
-
     @Test
     void testDoesNotDoubleWrap() {
         final ByteChannel channel = mock(ByteChannel.class);
@@ -290,6 +279,17 @@ void testToString(final Class<? extends Channel> channelClass) throws Exception
         assertTrue(shieldString.contains("MyChannel"));
     }
 
+    @Test
+    void testWrapFileChannel(final @TempDir Path tempDir) throws IOException {
+        final Path testFile = tempDir.resolve("test.txt");
+        FileUtils.touch(testFile.toFile());
+        try (FileChannel channel = FileChannel.open(testFile); Channel shield = CloseShieldChannel.wrap(channel)) {
+            fileChannelInterfaces().forEach(iface -> assertInstanceOf(iface, shield));
+            // FileChannel is not an interface, so can not be implemented.
+            assertFalse(shield instanceof FileChannel, "not FileChannel");
+        }
+    }
+
     @Test
     void testWritableByteChannelMethods() throws Exception {
         final WritableByteChannel channel = mock(WritableByteChannel.class);