-
-
Notifications
You must be signed in to change notification settings - Fork 8
test: add property tests #17
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
test: add property tests #17
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.
This seems like a very different test type. If so, I’d prefer having the current file as test/main.js
, the additions as test/property-based.js
, and a test/index.js
tying them together.
I think bigint shouldn’t be supported in unist, because it can’t be roundtripped with JSON (https://github.com/syntax-tree/unist#node)
The last check looks complex: first a long line (.chain
) where it makes a certain format, then a long line (is
) where it undoes that work?
An arbitrary JSON object. 7ae6376 and 62b6329 try to clarify this
7ae6376 clarifies this some by splitting up the massive line. |
Co-authored-by: Titus <tituswormer@gmail.com>
Co-authored-by: Titus <tituswormer@gmail.com>
Co-authored-by: Titus <tituswormer@gmail.com>
good to go once CI succeeds! |
this introduces fast-check (https://dubzzz.github.io/fast-check.github.com/) as a way to automatically generate test cases exploring possible edge cases not covered in example based tests