You're right, it's only a test dependency. That's because testing the server with a client that share the same code is not really healthy ;-)
On Mon, Sep 12, 2011 at 19:50, Davis Ford <[email protected]> wrote: > Guillaume - one more quick question, in this test case: > > > http://svn.apache.org/repos/asf/mina/sshd/trunk/sshd-core/src/test/java/org/apache/sshd/SftpTest.java > > What exactly are you using JSch for? I pulled JSch into my project > thinking I also needed it based on the test case, but I've implemented an > SFTP server using just the SshServer and SftpSubsystem, so I'm thinking I > have no dependency on JSch and I can remove it. > > It looks like maybe you are just using it as a client in the test cases -- > is this correct? > > Regards, > Davis > > -----Original Message----- > From: Davis Ford [mailto:[email protected]] > Sent: Monday, September 12, 2011 1:16 PM > To: [email protected] > Subject: RE: sftp questions > > I think it makes sense to do so if you're basing all your other error > handling on the IOException checked exception. > > Since that interface (FileSystemView) has methods that could fail, there's > really no other recourse except throw a RuntimeException and that is > currently swallowed. If you do decide to change it on the trunk, let me > know, and I'll do a git pull. > > Thanks! > > -----Original Message----- > From: Guillaume Nodet [mailto:[email protected]] > Sent: Monday, September 12, 2011 1:12 PM > To: [email protected] > Subject: Re: sftp questions > > Well, changing the interface is possible if needed I think. > > On Mon, Sep 12, 2011 at 18:14, Davis Ford <[email protected]> wrote: > > > The only issue with throwing IOException is that it is a checked > exception, > > and the interface method FileSystemView#getFile(String file) doesn't > declare > > it, so I have to change the interface to throw it. > > -- ------------------------ Guillaume Nodet ------------------------ Blog: http://gnodet.blogspot.com/ ------------------------ Open Source SOA http://fusesource.com
