Tom Sydney Kerckhove

Results 241 comments of Tom Sydney Kerckhove

> Here, the problem is that your `outerArgs -> innerArg -> Property` function could do different quantifications depending on the value of `innerArg`, for example if `innerArg` is used to...

This been on my mind for a while now and I think I've been able to explain to myself why this seems to work. The shrinking happens from the outside...

Gentle ping. In the meantime I've added this to sydtest to turn off shrinking when resources are in use: https://github.com/NorfairKing/sydtest/commit/bbc7dcb69d8189a1b832cb5c7f57b27c21d59411#diff-f4f9ca3eba6b0fecf61df5940928227897178cad50db39356c016015382e29e8R178

Ha! The one time I merge something without demanding tests, it breaks something. Serves me right. The question now is whether I should try that again or demand tests this...

In the interest of fixing the bug asap, we can merge these without tests. However, I want to do so only on the condition that we do add these tests...

@clintonmead Could you make a separate PR with the fix for identity? (and tests, ideally, they are in autodocodec-api-usage. Almost every `Spec` file could have an `Identity` test added)

Ping about these. A separate PR is not needed but tests definitely are. EDIT: I just realised this will require a PR for https://github.com/NorfairKing/validity as well.

Just so we don't get in trouble with this again: are there any other numbers than integers and not integers? CI fails as well, so that definitely needs to be...