test: fix flaky test-inspector

Using `socket.destroy()` instead of `socket.end()` fixes more-than-intermittent ECONNRESET issues on Windows. PR-URL: #9727 Fixes: #8804 Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com> Reviewed-By: Eugene Ostroukhov <eostroukhov@google.com>
nodejs · Dec 5, 2016 · 2fb8257 · 2fb8257
1 parent fc13cc6
commit 2fb8257
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/test/inspector/inspector-helper.js b/test/inspector/inspector-helper.js
@@ -286,7 +286,7 @@ TestSession.prototype.disconnect = function(childDone) {
     this.expectClose_ = true;
     this.harness_.childInstanceDone =
         this.harness_.childInstanceDone || childDone;
-    this.socket_.end();
+    this.socket_.destroy();
     console.log('[test]', 'Connection terminated');
     callback();
   });

diff --git a/test/inspector/inspector.status b/test/inspector/inspector.status
@@ -7,4 +7,3 @@ prefix inspector
 [true] # This section applies to all platforms
 
 [$system==win32]
-test-inspector  : PASS,FLAKY