-
Notifications
You must be signed in to change notification settings - Fork 4
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
Provide Known Answer Tests #56
Comments
at the moment, we have: Lines 46 to 71 in ca3b8b6
Is that good enough or are we talking about a list of hundreds/thousands of them? |
Yeah that's great, a few is fine. Some hashes/ciphers seem to go overboard testing 1000's of edgecases, doubt that's of any use here. Wrote a LXR implementation and trying to keep it simple wrt keeping up with any algorithm changes. Hopefully a KAT file or some notes in the readme can help anyone else doing the same. |
Hashes and ciphers usually have known answer tests associated with the reference implementation.
This provides a canonical digest for anyone writing their own implementation of LXR Hash to check against.
The text was updated successfully, but these errors were encountered: