test: fix flaky test-gc-net-timeout

ronag · codebytere · commit d62538404e51 · 2020-03-30T11:00:18.000-07:00
If the timeout is called in the time between 'end' and 'close' that would cause a EPIPE error. Essentially making the test flaky. PR-URL: #31918 Reviewed-By: Anna Henningsen <anna@addaleax.net>
diff --git a/test/parallel/test-gc-net-timeout.js b/test/parallel/test-gc-net-timeout.js
@@ -12,6 +12,9 @@ function serverHandler(sock){
 sock.on('close', function(){
 clearTimeout(timer);
 });
+ sock.on('end', function(){
+ clearTimeout(timer);
+ });
 sock.on('error', function(err){
 assert.strictEqual(err.code, 'ECONNRESET');
 });

-Original file line number
+Diff line change
 sock.on('close',function(){
 clearTimeout(timer);
 });
 +sock.on('end',function(){
 +clearTimeout(timer);
 +});
 sock.on('error',function(err){
 assert.strictEqual(err.code,'ECONNRESET');
 });