-
Notifications
You must be signed in to change notification settings - Fork 112
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
Add test for invalid UTF8 #508
Conversation
@vatsa287 please check once. |
451dfaf
to
11befe0
Compare
@hp77-creator Please do |
11befe0
to
649677a
Compare
Did it. Although I ran the following commands before submitting the PR:
both worked fine, so I submitted it, I guess, |
@vatsa287 can you check this? |
Fixes #307
I have added a test which will check for
InvalidUtf8
when the nodeId is not proper.I tried adding separate test for
InvalidUtf8
error code but Rust doesn't allow setting invalid utf-8 characters in a string. Hence have created a separate test.