test: copyedit test-tls-psk-alpn-callback-exception-handling

Signed-off-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
PR-URL: #63485
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

Author: aduh95

test/parallel/test-tls-psk-alpn-callback-exception-handling.js

@@ -95,10 +95,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -113,7 +110,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -307,9 +304,7 @@ describe('TLS callback exception handling', () => {
 }
 }));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection should not fire'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -324,7 +319,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -354,10 +349,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -368,7 +360,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1','h2'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -397,10 +389,8 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
 awaitnewPromise((res)=>server.listen(0,res));
 
 constclient=tls.connect({
@@ -410,7 +400,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -429,9 +419,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -473,9 +461,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -526,10 +512,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -540,7 +523,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -573,10 +556,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -587,7 +567,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });

test/parallel/test-tls-psk-alpn-callback-exception-handling.js

@@ -95,10 +95,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -113,7 +110,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -307,9 +304,7 @@ describe('TLS callback exception handling', () => {
 }
 }));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection should not fire'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -324,7 +319,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -354,10 +349,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -368,7 +360,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1','h2'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -397,10 +389,8 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
 awaitnewPromise((res)=>server.listen(0,res));
 
 constclient=tls.connect({
@@ -410,7 +400,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -429,9 +419,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -473,9 +461,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -526,10 +512,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -540,7 +523,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -573,10 +556,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -587,7 +567,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });

test/parallel/test-tls-psk-alpn-callback-exception-handling.js

@@ -95,10 +95,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -113,7 +110,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -307,9 +304,7 @@ describe('TLS callback exception handling', () => {
 }
 }));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection should not fire'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -324,7 +319,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -354,10 +349,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -368,7 +360,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1','h2'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -397,10 +389,8 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
 awaitnewPromise((res)=>server.listen(0,res));
 
 constclient=tls.connect({
@@ -410,7 +400,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -429,9 +419,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -473,9 +461,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -526,10 +512,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -540,7 +523,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -573,10 +556,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -587,7 +567,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });

test/parallel/test-tls-psk-alpn-callback-exception-handling.js

@@ -95,10 +95,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -113,7 +110,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -307,9 +304,7 @@ describe('TLS callback exception handling', () => {
 }
 }));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection should not fire'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -324,7 +319,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -354,10 +349,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -368,7 +360,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1','h2'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -397,10 +389,8 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
 awaitnewPromise((res)=>server.listen(0,res));
 
 constclient=tls.connect({
@@ -410,7 +400,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -429,9 +419,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -473,9 +461,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -526,10 +512,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -540,7 +523,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -573,10 +556,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -587,7 +567,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });

test/parallel/test-tls-psk-alpn-callback-exception-handling.js

@@ -95,10 +95,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -113,7 +110,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -307,9 +304,7 @@ describe('TLS callback exception handling', () => {
 }
 }));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection should not fire'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -324,7 +319,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -354,10 +349,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -368,7 +360,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1','h2'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -397,10 +389,8 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
 awaitnewPromise((res)=>server.listen(0,res));
 
 constclient=tls.connect({
@@ -410,7 +400,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -429,9 +419,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -473,9 +461,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -526,10 +512,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -540,7 +523,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -573,10 +556,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -587,7 +567,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });

test/parallel/test-tls-psk-alpn-callback-exception-handling.js

@@ -95,10 +95,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -113,7 +110,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -307,9 +304,7 @@ describe('TLS callback exception handling', () => {
 }
 }));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection should not fire'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -324,7 +319,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -354,10 +349,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -368,7 +360,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1','h2'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -397,10 +389,8 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
 awaitnewPromise((res)=>server.listen(0,res));
 
 constclient=tls.connect({
@@ -410,7 +400,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -429,9 +419,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -473,9 +461,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -526,10 +512,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -540,7 +523,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -573,10 +556,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -587,7 +567,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });

test/parallel/test-tls-psk-alpn-callback-exception-handling.js

@@ -95,10 +95,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -113,7 +110,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -307,9 +304,7 @@ describe('TLS callback exception handling', () => {
 }
 }));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection should not fire'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -324,7 +319,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -354,10 +349,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -368,7 +360,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1','h2'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -397,10 +389,8 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
 awaitnewPromise((res)=>server.listen(0,res));
 
 constclient=tls.connect({
@@ -410,7 +400,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -429,9 +419,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -473,9 +461,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -526,10 +512,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -540,7 +523,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -573,10 +556,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -587,7 +567,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });

test/parallel/test-tls-psk-alpn-callback-exception-handling.js

@@ -95,10 +95,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -113,7 +110,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -307,9 +304,7 @@ describe('TLS callback exception handling', () => {
 }
 }));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection should not fire'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -324,7 +319,7 @@ describe('TLS callback exception handling', () => {
 }),
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -354,10 +349,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -368,7 +360,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1','h2'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -397,10 +389,8 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
 awaitnewPromise((res)=>server.listen(0,res));
 
 constclient=tls.connect({
@@ -410,7 +400,7 @@ describe('TLS callback exception handling', () => {
 ALPNProtocols: ['http/1.1'],
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -429,9 +419,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -473,9 +461,7 @@ describe('TLS callback exception handling', () => {
 
 const{ promise, resolve, reject }=createTestPromise();
 
-server.on('secureConnection',common.mustNotCall(()=>{
-reject(newError('secureConnection should not fire'));
-}));
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -526,10 +512,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -540,7 +523,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });
@@ -573,10 +556,7 @@ describe('TLS callback exception handling', () => {
 reject(e);
 }
 }));
-
-server.on('secureConnection',()=>{
-reject(newError('secureConnection should not fire'));
-});
+server.on('secureConnection',common.mustNotCall('secureConnection listener'));
 
 awaitnewPromise((res)=>server.listen(0,res));
 
@@ -587,7 +567,7 @@ describe('TLS callback exception handling', () => {
 rejectUnauthorized: false,
 });
 
-client.on('error',()=>{});
+client.on('error',common.mustCall());
 
 awaitpromise;
 });