Skip to content

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

Open
mk-pmb wants to merge 2 commits into
nodeftpd:masterfrom
mk-pmb:fix-test-typo-expectederr
Open

Tests: Fix typo and adapt to new jsftp API#136
mk-pmb wants to merge 2 commits into
nodeftpd:masterfrom
mk-pmb:fix-test-typo-expectederr

Conversation

@mk-pmb

@mk-pmb mk-pmb commented Nov 24, 2017

Copy link
Copy Markdown
Contributor

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

mk-pmb commented Nov 24, 2017

Copy link
Copy Markdown
Contributor Author

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant