yahoo/mojito-validate — pull request #3 by Yujun-Qian (2013-04-22 06:54:12+00:00)
Title: tests unstable, need to call main's cb()?

I added some tests, see pull #1, but noticed intermittent failures. Seems to be ok now and coverage is up.

One issue involved the cmd & test code relying on process.cwd() which I think picked up a second fixture I added, `someapp-ok`. I worked around this in the test code.

I think subsequent instability is because validator.validate doesn't call the callback on completion, and the test code uses a timeout instead.

Can you update the code to call the callback on completion? Down the road I want the option to call your validate command before test, jslint, and start commands, which should use that behavior...

Thanks in advance.

Isao