Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added test for ChannelStream & fix tests for windows #95

Closed

Conversation

AspectUnk
Copy link
Contributor

ChannelStream works well for me. Unfortunately I don't have my macbook right now to run tests, but on windows I still noticed errors that I had to fix.

@Eugeny
Copy link
Member

Eugeny commented Dec 10, 2022

What a coincidence, I've just added the same test :D

Can you see if maybe you could add your changes to it? Currently it only checks ChannelStream in a client session

@Eugeny
Copy link
Member

Eugeny commented Dec 10, 2022

Also you can actually replace whoami with just "user".to_string() since the tests just test against russh itself and not a real SSH server

@AspectUnk
Copy link
Contributor Author

:D
Okay, I'll make changes soon.

@AspectUnk AspectUnk closed this Dec 10, 2022
@AspectUnk AspectUnk deleted the channelstream-test branch December 10, 2022 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants