Not the same thing. Original test was based on node's `child_process` docs, which implies that `code` should be `null` on error, or the exit code otherwise, so an exit code of 1 was incorrectly  considered successful.

Come to think of it, we should probably change this to `if code is 0 then resolve() else reject()`. I can do a PR for that if you'd like.