Skip to content

test: remove TODO

Rodrigo Muino Tomonari requested to merge github/fork/Trott/cluster-http-pipe into master

All instances of the socket path in req, res, and the return value from http.createServer() are properties starting with _ indicating that they should be treated as internal.

Testing those values would tie the test to the implementation. So, do not check the socket path. If it should be checked, perhaps a feature issue is in order to expose the value.

/cc @bnoordhuis who put the comment there, in case what's going on here is that I Just Don't Get It. (Should it ideally be exposed in localAddress and it's just not?)

Merge request reports