On Jun 18, 2021, at 6:36 AM, Alan Bateman <al...@openjdk.java.net<mailto:al...@openjdk.java.net>> wrote:
Adding an override of transferTo may require new tests. @bplb Do you if we have good tests for all the scenarios where input stream returned by Channels.newInputStream is the source? There are not a lot of tests for the `InputStream` returned by `Channels.newInputStream`: `java/nio/channels/Channels`: `Basic.java` - reading from a wrapped `FileChannel` `ReadByte.java` - trivial `Basic2.java` - reading at random offsets from a wrapped `AsynchronousSocketChannel` `ReadOffset.java` - trivial Elsewhere: `sun/nio/ch/TempBuffer.java` `jdk/nio/zipfs/ZipFSTester.java` I don’t see that `transferTo()` on such a stream is tested anywhere.
