node/test
Ben Noordhuis c11612026f net: omit superfluous 'connect' event
Don't emit a 'connect' event on sockets that are handed off to
net.Server 'connection' event listeners.

1. It's superfluous because the connection has already been established
   at that point.

2. The implementation is arguably wrong because the event is emitted on
   the same tick of the event loop while the rule of thumb is to always
   emit it on the next one.

This has been tried before in commit f0a440d but was reverted again in
ede1acc because the change was incomplete (at least one test hadn't
been updated).

Fixes #1047 (again).
2013-03-01 02:09:36 +01:00
..
addons test: modify async native test.js to test for #4820 2013-02-21 13:14:07 -08:00
disabled Merge remote-tracking branch 'ry/v0.8' into master 2013-02-18 10:21:08 -08:00
fixtures test: Fix debugger repl tests 2013-02-26 16:49:17 -08:00
gc test: use the debug build of node-weak when necessary 2012-06-13 17:58:28 -07:00
internet test: Move test-net-connect-timeout to test/internet 2013-02-26 17:39:04 -08:00
message process: separate nextTick domain logic 2013-02-27 16:37:10 -08:00
pummel net: omit superfluous 'connect' event 2013-03-01 02:09:36 +01:00
simple net: omit superfluous 'connect' event 2013-03-01 02:09:36 +01:00
common.js lib, src: remove errno global 2013-02-28 23:11:47 +01:00