add interface to SftpFile

[jayhilden]

It would be really nice if you put an interface in front of SftpFile, this would make mocking for unit testing much, much easier.

Thank you.

[TrevorPilley]

+1, also an ISftpClient interface would also help with this

[CasperWSchmidt]

This should be implemented! Since SftpFile has no public constructor, no virtual properties/methods and no interface unit testing is actually nearly impossible!
Besides that, using interfaces is always best practice :)

[jayserdny]

+1

[CraigHead]

+1 ... 4 years later. Can this be added please? It's difficult to build unit tests for project that implement SSH.NET without this interface!

[CasperWSchmidt]

I decided to solve the issue by simply writing my own interface and implementation as a wrapper/adapter.

[drieseng]

This is available in the 2020.0.0 release.

[MartijnSchoemaker]

I still don't see an interface on SftpFile. Or do I miss something?

[drieseng]

Let me check later, it may be an oversight on my part.

[drieseng]

I was a bit too fast. I read this issue as a request to add an ISftpClient interface.
Can (one of) you submit a PR for adding an ISftpFile interface?

[wxtsxt]

This is still hard to mock ISftpClient when ISftpFile is not there.

[CasperWSchmidt]

@wxtsxt see above^^ You are more than welcome to submit a PR adding the interface :)

Unfortunately I don't have the time myself, and as noted above, we simply created a wrapper as a workaround.

[wxtsxt]

With merge request !812 SftpClient (and ISftpClient) returns interface ISftpFile instead of SftpFile.
Such change allows to create unit tests with mocks (sample using NSubstitude):

            var file = Substitute.For<ISftpFile>();
            file.FullName.Returns("my file name");

            var sftpClient = Substitute.For<ISftpClient>();
            sftpClient.ListDirectory(myPath).Returns(new List<ISftpFile> { file });

Please review the MR.