error.stack is assumed to exist

gruntjs / grunt-contrib-nodeunit

Run Nodeunit unit tests.

http://gruntjs.com/

MIT License

70 stars 26 forks source link

error.stack is assumed to exist #36

Closed dtex closed 9 years ago

dtex commented 9 years ago

I had a stack overflow in my code which does not generate an error.stack. As a result I was seeing cannot call method split() on undefined (with no stack). Took a while to trace that to here.

This doesn’t magically show the cause of the overflow, but at least it shows the “correct” error.

dtex commented 9 years ago

Note: This was in node 0.10.34 . I'm getting a nice call stack in node 0.12.2 even without this change so closing the PR.