-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
use proptest to fuzz the resolver #5921
Merged
Merged
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
56a222c
a start on using proptest to fuzz the resolver
Eh2406 1761886
cache the example url to solve performance problem
Eh2406 ce1772c
get working with minimal-versions
Eh2406 4e619d8
better generation of version numbers
Eh2406 732aa10
small clean up
Eh2406 c26553a
small clean up
Eh2406 b7285e8
handle "bad" slightly better
Eh2406 5339e92
small clean up for the cache of the example url
Eh2406 f270dda
incorporate @AltSysrq suggestions
Eh2406 17fe190
update to the new version of proptest
Eh2406 85b1976
use the new `impl Strategy for Vec<S>` to only generate valid depende…
Eh2406 0ef43cb
use the new `result_cache`
Eh2406 0206cec
double down on `prop_flat_map` to guarantee version requirements are …
Eh2406 1f98871
stronger assert in the core
Eh2406 b0bbb6a
use args for scale
Eh2406 2628ec0
works a LOT better if the simple cases are at the beginning
Eh2406 509806e
small clean up
Eh2406 d6258e9
Merge remote-tracking branch 'origin/master' into proptest
Eh2406 1eaf543
small clean up
Eh2406 2b1c39f
more controllable bad dependencies, and use Index for ranges
Eh2406 05c6ce0
remove a `prop_flat_map` use @AltSysrq suggestion again
Eh2406 856964f
fail fast on CI, don't shrink the input
Eh2406 2b16156
If resolution was successful, then unpublishing a version of a crate …
Eh2406 c497851
If resolution was unsuccessful, then it should stay unsuccessful even…
Eh2406 1b15b15
give intermediate steps names
Eh2406 d8e19fb
pull the has bad dep before `prop_flat_map`
Eh2406 2015191
`edge list` is a much cleaner approach
Eh2406 7ad9f5e
add a test to test the tester
Eh2406 6bd2540
check each registry more thoroughly
Eh2406 40d9de4
Merge remote-tracking branch 'origin/master' into proptest
Eh2406 3e7192e
pretty print the registry made by proptest
Eh2406 6763ede
proptest 0.8.7 has a better `ci_no_shrink`
Eh2406 a4da525
In theory shrinkage should be 2, but in practice we get better trees …
Eh2406 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -94,6 +94,7 @@ features = [ | |
|
||
[dev-dependencies] | ||
bufstream = "0.1" | ||
proptest = "0.8.7" | ||
|
||
[[bin]] | ||
name = "cargo" | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
s/sweet/sweet test suite
(same below)