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 563aa2c8846bf13fcd560eed7520cfb7a46618a4 Author: Gary D. Gregory <garydgreg...@gmail.com> AuthorDate: Fri Sep 26 15:24:17 2025 -0400 Use longer lines --- .../commons/io/build/AbstractOriginTest.java | 21 ++++++----------- .../io/build/AbstractStreamBuilderTest.java | 8 +++---- .../channels/ByteArraySeekableByteChannelTest.java | 27 ++++------------------ 3 files changed, 16 insertions(+), 40 deletions(-) diff --git a/src/test/java/org/apache/commons/io/build/AbstractOriginTest.java b/src/test/java/org/apache/commons/io/build/AbstractOriginTest.java index 01f7060e2..d40cbf05d 100644 --- a/src/test/java/org/apache/commons/io/build/AbstractOriginTest.java +++ b/src/test/java/org/apache/commons/io/build/AbstractOriginTest.java @@ -277,10 +277,8 @@ void testGetRandomAccessFile(final OpenOption openOption) throws IOException { @Test void testGetReadableByteChannel() throws IOException { - try (ReadableByteChannel channel = - getOriginRo().getChannel(ReadableByteChannel.class, StandardOpenOption.READ)) { - final SeekableByteChannel seekable = - channel instanceof SeekableByteChannel ? (SeekableByteChannel) channel : null; + try (ReadableByteChannel channel = getOriginRo().getChannel(ReadableByteChannel.class, StandardOpenOption.READ)) { + final SeekableByteChannel seekable = channel instanceof SeekableByteChannel ? (SeekableByteChannel) channel : null; assertNotNull(channel); assertTrue(channel.isOpen()); if (seekable != null) { @@ -313,11 +311,9 @@ void testGetReader() throws IOException { @Test void testGetWritableByteChannel() throws IOException { final boolean supportsRead; - try (WritableByteChannel channel = - getOriginRw().getChannel(WritableByteChannel.class, StandardOpenOption.WRITE)) { + try (WritableByteChannel channel = getOriginRw().getChannel(WritableByteChannel.class, StandardOpenOption.WRITE)) { supportsRead = channel instanceof ReadableByteChannel; - final SeekableByteChannel seekable = - channel instanceof SeekableByteChannel ? (SeekableByteChannel) channel : null; + final SeekableByteChannel seekable = channel instanceof SeekableByteChannel ? (SeekableByteChannel) channel : null; assertNotNull(channel); assertTrue(channel.isOpen()); if (seekable != null) { @@ -332,18 +328,15 @@ void testGetWritableByteChannel() throws IOException { } if (supportsRead) { setOriginRw(newOriginRw()); - try (ReadableByteChannel channel = - getOriginRw().getChannel(ReadableByteChannel.class, StandardOpenOption.READ)) { + try (ReadableByteChannel channel = getOriginRw().getChannel(ReadableByteChannel.class, StandardOpenOption.READ)) { assertNotNull(channel); assertTrue(channel.isOpen()); checkRead(channel); } } setOriginRw(newOriginRw()); - try (WritableByteChannel channel = - getOriginRw().getChannel(WritableByteChannel.class, StandardOpenOption.WRITE)) { - final SeekableByteChannel seekable = - channel instanceof SeekableByteChannel ? (SeekableByteChannel) channel : null; + try (WritableByteChannel channel = getOriginRw().getChannel(WritableByteChannel.class, StandardOpenOption.WRITE)) { + final SeekableByteChannel seekable = channel instanceof SeekableByteChannel ? (SeekableByteChannel) channel : null; assertNotNull(channel); assertTrue(channel.isOpen()); if (seekable != null) { diff --git a/src/test/java/org/apache/commons/io/build/AbstractStreamBuilderTest.java b/src/test/java/org/apache/commons/io/build/AbstractStreamBuilderTest.java index 68baca1cb..36c994c18 100644 --- a/src/test/java/org/apache/commons/io/build/AbstractStreamBuilderTest.java +++ b/src/test/java/org/apache/commons/io/build/AbstractStreamBuilderTest.java @@ -59,10 +59,9 @@ public char[] get() { } private static Stream<IOConsumer<Builder>> fileBasedConfigurers() throws URISyntaxException { - final URI uri = Objects.requireNonNull( - AbstractStreamBuilderTest.class.getResource(AbstractOriginTest.FILE_RES_RO)) - .toURI(); + final URI uri = Objects.requireNonNull(AbstractStreamBuilderTest.class.getResource(AbstractOriginTest.FILE_RES_RO)).toURI(); final Path path = Paths.get(AbstractOriginTest.FILE_NAME_RO); + // @formatter:off return Stream.of( b -> b.setByteArray(ArrayUtils.EMPTY_BYTE_ARRAY), b -> b.setFile(AbstractOriginTest.FILE_NAME_RO), @@ -75,6 +74,7 @@ private static Stream<IOConsumer<Builder>> fileBasedConfigurers() throws URISynt b -> b.setInputStream(new FileInputStream(AbstractOriginTest.FILE_NAME_RO)), b -> b.setChannel(Files.newByteChannel(path)), b -> b.setURI(uri)); + // @formatter:on } private void assertResult(final char[] arr, final int size) { @@ -126,7 +126,7 @@ void testBufferSizeChecker() { @MethodSource("fileBasedConfigurers") void testGetInputStream(final IOConsumer<Builder> configurer) throws Exception { final Builder builder = builder(); - configurer.accept(builder); + configurer.accept(builder); assertNotNull(builder.getInputStream()); } } diff --git a/src/test/java/org/apache/commons/io/channels/ByteArraySeekableByteChannelTest.java b/src/test/java/org/apache/commons/io/channels/ByteArraySeekableByteChannelTest.java index 6a54f84a0..5b29b3f1b 100644 --- a/src/test/java/org/apache/commons/io/channels/ByteArraySeekableByteChannelTest.java +++ b/src/test/java/org/apache/commons/io/channels/ByteArraySeekableByteChannelTest.java @@ -45,28 +45,11 @@ public class ByteArraySeekableByteChannelTest extends AbstractSeekableByteChanne static Stream<Arguments> testConstructor() { return Stream.of( - Arguments.of( - (IOSupplier<ByteArraySeekableByteChannel>) ByteArraySeekableByteChannel::new, - EMPTY_BYTE_ARRAY, - IOUtils.DEFAULT_BUFFER_SIZE), - Arguments.of( - (IOSupplier<ByteArraySeekableByteChannel>) () -> new ByteArraySeekableByteChannel(8), - EMPTY_BYTE_ARRAY, - 8), - Arguments.of( - (IOSupplier<ByteArraySeekableByteChannel>) () -> new ByteArraySeekableByteChannel(16), - EMPTY_BYTE_ARRAY, - 16), - Arguments.of( - (IOSupplier<ByteArraySeekableByteChannel>) - () -> ByteArraySeekableByteChannel.wrap(EMPTY_BYTE_ARRAY), - EMPTY_BYTE_ARRAY, - 0), - Arguments.of( - (IOSupplier<ByteArraySeekableByteChannel>) - () -> ByteArraySeekableByteChannel.wrap(testData), - testData, - testData.length)); + Arguments.of((IOSupplier<ByteArraySeekableByteChannel>) ByteArraySeekableByteChannel::new, EMPTY_BYTE_ARRAY, IOUtils.DEFAULT_BUFFER_SIZE), + Arguments.of((IOSupplier<ByteArraySeekableByteChannel>) () -> new ByteArraySeekableByteChannel(8), EMPTY_BYTE_ARRAY, 8), + Arguments.of((IOSupplier<ByteArraySeekableByteChannel>) () -> new ByteArraySeekableByteChannel(16), EMPTY_BYTE_ARRAY, 16), + Arguments.of((IOSupplier<ByteArraySeekableByteChannel>) () -> ByteArraySeekableByteChannel.wrap(EMPTY_BYTE_ARRAY), EMPTY_BYTE_ARRAY, 0), + Arguments.of((IOSupplier<ByteArraySeekableByteChannel>) () -> ByteArraySeekableByteChannel.wrap(testData), testData, testData.length)); } static Stream<Arguments> testShouldResizeWhenWritingMoreDataThanCapacity() {