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
Metadata testcases #27
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few bug's which need fixing before we merge, but in general these are a good start.
Let's also add a few tests which stress the amount of text which can be in a value, and the length of a metadata key name as well. I have a sinking suspicion that the default lmdb key length limit (511 bytes) will raise errors which may not propagate clearly through to user space if they are exceeded.
Nice work!
Will add this to stress test cases |
@rlizzo Verified, rebased and pushed. This can be merged if you are good. The pending TODOs can be tracked through corresponding issues. Keeping this open till those are fixed might make this stale |
Looks good. Let's merge! |
Description
Basic test cases for Metadata
Types of changes
Checklist: