test: close debug client in test-debugger-client

Killing the debuggee without first closing the socket can result in an ECONNRESET error.
nodejs · Jan 13, 2014 · 1b74892 · 1b74892
1 parent 8753bb3
commit 1b74892
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/test/simple/test-debugger-client.js b/test/simple/test-debugger-client.js
@@ -197,9 +197,14 @@ function doTest(cb, done) {
           connectCount++;
           console.log('ready!');
           cb(c, function() {
-            console.error('>>> killing node process %d\n\n', nodeProcess.pid);
-            nodeProcess.kill();
-            done();
+            c.end();
+            c.on('end', function() {
+              console.error(
+                  '>>> killing node process %d\n\n',
+                  nodeProcess.pid);
+              nodeProcess.kill();
+              done();
+            });
           });
         });
       });