★ wanayoo — archive 1999 https://github.com/nodeftpd/nodeftpd/pull/136Nouvelle recherche | Portail wanayoo
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

Tests: Fix typo and adapt to new jsftp API #136

Open
wants to merge 2 commits into
base: master
from

Conversation

@mk-pmb
Copy link
Contributor

mk-pmb commented Nov 24, 2017

Fixes my typo from cf207f6 .

Original master doesn't currently pass tests on my machine (I'll open another issue for that), so instead I diff-ed the test log from before and after, they're identical except for timing.

@mk-pmb
Copy link
Contributor Author

mk-pmb commented Nov 24, 2017

jsftp changed its .raw API. I adapted our test code and now all tests pass on my machine.

@mk-pmb mk-pmb changed the title [minor][test] fix typo in tests: suppressExpecte(+d)ErrMsgs Tests: Fix typo and adapt to new jsftp API Nov 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.