pragdave / quixir

Property-based testing for Elixir
266 stars 11 forks source link

List of bounded floats generates out of bounds #10

Closed isaacsanders closed 7 years ago

isaacsanders commented 7 years ago

On the master branch of this repo http://github.com/isaacsanders/histogrammar-elixir, I have a test case, test/histogrammar/bin_test.exs, that contains a list of bounded floats.

It seems as though the list just throws out my structure and returns 42 forever.

isaacsanders commented 7 years ago

I wasn't able to replicate the bug in a REPL with just Pollution, so part of me thinks it is related to the ptestsubstitution, but that is just a guess.

pragdave commented 7 years ago

Can I go ahead and close this?

On Thu, Apr 13, 2017 at 7:08 AM, Isaac Sanders notifications@github.com wrote:

I wasn't able to replicate the bug in a REPL with just Pollution, so part of me thinks it is related to the ptestsubstitution, but that is just a guess.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/pragdave/quixir/issues/10#issuecomment-293875133, or mute the thread https://github.com/notifications/unsubscribe-auth/AAApmOSNGGtXU6h_0ASaILbKoiLS4Murks5rvhA0gaJpZM4M8ik0 .

isaacsanders commented 7 years ago

I wouldn't mind a point release, but I understand if you would rather not.

pragdave commented 7 years ago

'tis released. 0.9.2 of quixir and 0.9.1 of pollution.

Thanks again.

On Fri, Apr 14, 2017 at 12:28 PM, Isaac Sanders notifications@github.com wrote:

I wouldn't mind a point release, but I understand if you would rather not.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/pragdave/quixir/issues/10#issuecomment-294196803, or mute the thread https://github.com/notifications/unsubscribe-auth/AAApmOmBrVVURBdKQN7FWt_NYA49MkGtks5rv6y5gaJpZM4M8ik0 .

isaacsanders commented 7 years ago

:+1: happy to help, when I can manage the time to figure out the issue.