Reproduce via random seed about node-test-runner HOT 4 CLOSED

Oh I'd just change the API to accept an Int instead of a Random.Seed

Sure, just document why this is the case and that the integer should be 32 random bits, etc. Again in the short-term you can avoid an API change thanks to Random.Pcg.toJson. Wait this is the runner that hasn't been released yet, not elm-test itself, so let's make the breaking API change now.

from node-test-runner.

Also I feel like when picking one at random, we should start with the initial time and then do like Random.int 1000 9999 to generate a relatively concise int for a seed that you could quickly holler across a room at a coworker who wants to reproduce what you're seeing.

Four digits seems like a plenty large enough possibility space given that we're not doing crypto or anything, but @mgold would be the expert on that.

from node-test-runner.

As with anything involving reusing or seeds or generating seeds from randomized output, the answer is: theoretically you are compromising the validity of the RNG and you should feel bad; in practice it's probably fine (until you get a really obscure and difficult bug).

I don't think it's worth limiting the 4 billion-ish possibilities to 9000 just to make the seeds easier to shout across the room. I've done similar things with flaky rspecs and I just wind up copying and pasting the seed at one machine.

I'm totally on board with indicating and being able to specify the initial seed, but the opaque type is working against you. With Random.Pcg you can do it by abusing toJson. If that's not merged into core (as is likely), you might have to rely on toString.

from node-test-runner.

I just wind up copying and pasting the seed at one machine.

Yeah fair point.

the opaque type is working against you

Oh I'd just change the API to accept an Int instead of a Random.Seed

from node-test-runner.