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

panic in checkFailFile if code has been changed #63

Open
jbricht opened this issue Jan 16, 2024 · 1 comment
Open

panic in checkFailFile if code has been changed #63

jbricht opened this issue Jan 16, 2024 · 1 comment
Labels
need info Further information is requested

Comments

@jbricht
Copy link

jbricht commented Jan 16, 2024

Hi, thanks for the great library :)

I just encountered a very confusing panic, the "overrun" panic from drawBits. After grepping through the library a bit, i realized it was panicking while trying to replay some kind of past test failure. Deleting all the files in testdata made the panic go away.

I had just changed my code to draw more often, so I guess it was trying to run the new code on the old fail file and running out of recorded rng output?

Not sure of the best way to handle this situation, but a more helpful error message would be fantastic, if you have the time!

Again, thank you for the wonderful library.

@flyingmutant
Copy link
Owner

Hi, thanks a lot for reporting! overrun panic is an internal way to signal not enough data, it should not lead to a test failure and the only way you can observe it should be if you are running under a debugger and breaking on all the panics.

If the overrun is user-visible or does result in a test failure, does it happen with the latest version of rapid? If so, can you provide a small reproducer for the problem? That would help a lot.

@flyingmutant flyingmutant added the need info Further information is requested label Jan 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need info Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants