Behavior on missing done

[Marsup]

Hello,

I noticed people coming from mocha often forget to declare a done for synchronous code.

I know we don't intend to copy mocha but it might be useful to at least warn the user if no parameter is declared. It would be less verbose to accept both styles but I guess you did it on purpose ?

[hueniverse]

We consider testing sync code an anomaly and do require you define the argument and call it. I'm ok with a warning message, but don't think it is that important. But then again, I don't really want people to use lab... I'll let Wyatt make the call.

[Marsup]

@hueniverse It appears to be the way to go when you're using hapi though, isn't it ?

[hueniverse]

Sadly, yes. I saw the mocha guys trying to fix this. The main point is that this is no longer my problem :-)

[geek]

I think a warning message is fine, we already warn for missing exports.lab

[Marsup]

We also exit on missing exports.lab, is it desired here ? I'd think so but you tell me.

[geek]

Ya, I think that makes sense to exit early with an error

[lock]

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.