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.

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

Add arbtest #63

Merged
merged 2 commits into from
Dec 19, 2024
Merged

Add arbtest #63

merged 2 commits into from
Dec 19, 2024

Conversation

yancyribbens
Copy link
Collaborator

@yancyribbens yancyribbens commented Sep 14, 2024

Adds proptests using arbtest to test the correctness of the solutions provided by the SRD and BnB search algos. I use the dep exhaustigen to build all possible solutions and then run SRD or BnB and compare the results to show that the correct solution is found. Also, BnB contains a few proptest where a UTXO pool is randonly generated, and either one or many of the UTXOs are selected as a target with the assertion that BnB will find those UTXOS as the solution while searching.

@yancyribbens yancyribbens force-pushed the add-arbtest branch 6 times, most recently from 99c7dec to c049721 Compare October 18, 2024 15:51
@yancyribbens yancyribbens marked this pull request as ready for review October 18, 2024 15:51
@yancyribbens yancyribbens force-pushed the add-arbtest branch 7 times, most recently from fe55a4c to 23f5186 Compare October 23, 2024 22:56
@yancyribbens yancyribbens force-pushed the add-arbtest branch 2 times, most recently from d9cc49b to ab08bb8 Compare October 25, 2024 21:22
@yancyribbens
Copy link
Collaborator Author

Set maximum pool size to 20 since I had a problem where arbitrary pool size caused cargo to time out after 60 secounds

@yancyribbens
Copy link
Collaborator Author

I have been running the proptests locally for the last 150 days without error. Merging.

@yancyribbens yancyribbens merged commit d9e76f0 into p2pderivatives:master Dec 19, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant