Feature/fix child process type checking

https://github.com/joyent/node/pull/8392 introduced a test that doesn't fail when the code it is testing is deleted.

Also, behaviour of execFile and spawn have drifted apart again, unnecessarily, I think. This brings them into line again.

/to @cjihrig  
Node core doesn't usually use this "leading ," style, are you OK if I convert to `var` as part of this cleanup?