Fork me on GitHub
#test-check
<
2017-06-16
>
carocad14:06:56

hey guys, where could I submit a bug report of the test.check library?

carocad14:06:58

maybe you could help me?. I think I might be doing a stupic mistake but I dont see it 😕

carocad14:06:43

the locals declared inside prop/for-all are not being recognized

carocad14:06:56

I cannot even make the example in the README work

carocad14:06:39

bug report submitted

gfredericks15:06:52

@carocad that's pretty strange

carocad15:06:48

yeap, I am still trying to make sense of it. I mean if it would be normal if would have popep up before the lib release

gfredericks15:06:08

@carocad looking at the travis log, the problem seems to be using previous bindings in the for-all, which doesn't support that

gfredericks15:06:19

The bindings are independent

gfredericks15:06:47

That doesn't explain your claim that the readme example doesn't work

carocad15:06:13

I can put it on my file and show you a travis fail if that helps

carocad15:06:37

btw, removing the previous binding didnt solve it in my case

gfredericks15:06:59

You'd have to refactor it to one binding with gen/let and destructing, probably

gfredericks15:06:15

The travis logs are helpful, yeah

gfredericks15:06:34

Especially if the source is online too

carocad15:06:48

it is. Currently a pr but it should be enough

carocad15:06:15

hmmm seems that I was wrong. The build is still failing but it doesnt show the same problem as before

gfredericks15:06:34

@carocad that looks like just an eastwood failure?

carocad15:06:16

yeah it seems like it. I just changed my code and the example from the repo worked locally. I am testing in travis

carocad15:06:18

yeah it worked

carocad15:06:44

so the problem was the previous bindings 😄 sorry

carocad15:06:18

I'm closing the issue

carocad15:06:10

oh it seems I cannot close the issue 😞 I put a comment explaining it was my failure though

gfredericks15:06:02

I'll close it

carocad15:06:38

thanks :thumbsup: