zhwq1216 commented on pull request #215: URL: https://github.com/apache/commons-vfs/pull/215#issuecomment-927231575
> You'll need some kind of unit test to show that this actually fixes something. See https://github.com/apache/commons-vfs/tree/master/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/sftp I tried to add some unit tests, but AbstractSftpProviderTestCase cannot simulate the situation where the directory does not have write permissions, so it cannot verify that `channel.put` throws an exception. I checked `org.apache.commons.vfs2.provider.ftp.FtpFileObject#doGetOutputStream`, there is a code block similar to the method of catching the exception -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
