test: refactor test-https-set-timeout-server

* Add common.mustCall() for error handlers. * Alphabetize modules PR-URL: #13032 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Alexey Orlenko <[email protected]>

Author: Trott

test/sequential/test-https-set-timeout-server.js

@@ -21,16 +21,16 @@
 
 'use strict';
 constcommon=require('../common');
-constassert=require('assert');
 
 if(!common.hasCrypto){
 common.skip('missing crypto');
 return;
 }
-consthttps=require('https');
 
-consttls=require('tls');
+constassert=require('assert');
 constfs=require('fs');
+consthttps=require('https');
+consttls=require('tls');
 
 consttests=[];
 
@@ -69,7 +69,7 @@ test(function serverTimeout(cb){
 https.get({
 port: this.address().port,
 rejectUnauthorized: false
-}).on('error',common.noop);
+}).on('error',common.mustCall());
 }));
 });
 
@@ -90,7 +90,7 @@ test(function serverRequestTimeout(cb){
 method: 'POST',
 rejectUnauthorized: false
 });
-req.on('error',common.noop);
+req.on('error',common.mustCall());
 req.write('Hello');
 // req is in progress
 });
@@ -111,7 +111,7 @@ test(function serverResponseTimeout(cb){
 https.get({
 port: this.address().port,
 rejectUnauthorized: false
-}).on('error',common.noop);
+}).on('error',common.mustCall());
 });
 });
 
@@ -131,7 +131,7 @@ test(function serverRequestNotTimeoutAfterEnd(cb){
 https.get({
 port: this.address().port,
 rejectUnauthorized: false
-}).on('error',common.noop);
+}).on('error',common.mustCall());
 });
 });