Richard Feldman

JSON decoding tests seems like a strong enough motivation for me! @mgold do I recall correctly that you'd already implemented these somewhere?

That's interesting, although I'm not sure if it saves enough to warrant its own function. The ones in OP save you a whole case-expression, whereas `decodableBy` only saves you an...

I was able to reproduce this by checking out that commit. 👍 When I ran `elm-test` on that commit, the generated JS (`elm-comidi/elm-stuff/generated-code/elm-community/elm-test/elmTestOutput.js`) included this: ```js var _elm_community$shrink$Shrink$merge = F3(...

I thought maybe the problem was the Node test runner doing funky hacks, but I tried running this in [the HTML runner](http://package.elm-lang.org/packages/elm-community/html-test-runner/1.0.6/Test-Runner-Html) and it's reproducible there too! I stepped through...

Definitely seems like an Elm bug to me!

Evan [commented on Slack](https://elmlang.slack.com/archives/C13L7S5GR/p1514943870000154): > Known bugs that could potentially do that are (1) cyclic values that get generated in a bad order and (2) TCO that involves capturing variables...

Turns out there are at least two compiler bugs here. @eeue56 [commented on Slack](https://elmlang.slack.com/archives/C13L7S5GR/p1514940171000052): > elm-test shouldn't be using the elm-community/shrink solution any more -- the problem might fix itself...

Okay, apparently the 0.19 compiler already includes package name as part of type unification, so neither bug should happen anymore. I'm gonna leave this open as a "let's circle back...

I tried this on Node 8.4.0 and the current `elm-test` and it worked fine. 🤔 @battermann can you check out https://github.com/rtfeldman/node-test-runner locally and run `npm install && npm test &&...

> 1. Is this a good idea? I think so, yes! I tried it out and it immediately revealed that I wanted `SetHue` to enforce that the minimum `model.hue` value...