Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add function generator #201

Merged
merged 10 commits into from
Oct 1, 2018
Merged

Add function generator #201

merged 10 commits into from
Oct 1, 2018

Conversation

dubzzz
Copy link
Owner

@dubzzz dubzzz commented Oct 1, 2018

Fixes #100
Fixes #199

dubzzz added 10 commits October 1, 2018 00:12
@dubzzz
Implement FunctionArbitrary
7cec66b 
The code needs to be cleaned before being merged.
Both pretty and hash must be extracted to their own files.
README has to be updated too.

Fixes #100 and #199
@dubzzz
Add a comparison function arbitrary
27f1dc7 
Comparison functions are widely used by JavaScript libraries.
As the consequence they have to be a built-in of fast-check.
@dubzzz
Add a comparison boolean function
a46fd7d
@dubzzz
Increase the number of hash collisions for compareFunc
1badd8c
@dubzzz
Force crc32 implementation to give positive values
fe2e620 
All values should be between 0x00000000 and 0xffffffff
@dubzzz
Add e2e test for compareFunc
8978540
@dubzzz
Move polyfills into utils/
68c2462
@dubzzz
Extract and rename pretty into utils/stringify
15c182e
@dubzzz
Extract hash into utils/hash
2178e25
@dubzzz
Update Arbitraries.md for functions
da69093
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 92.264% when pulling da69093 on feat/func/1 into 500434f on master.

@dubzzz dubzzz merged commit 68d1dfc into master Oct 1, 2018
@dubzzz dubzzz deleted the feat/func/1 branch October 1, 2018 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dubzzz @coveralls