test: use arrow functions in addons tests

Convert all anonymous callback functions in `test/addons/**/*.js`
to use arrow functions, except for those in
`test/addons/make-callback/test.js` (which reference `this`)
`writing-tests.md` states to use arrow functions when appropriate.
PR-URL: #30131
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>

Author: GaryGSC

test/addons/async-hello-world/test-makecallback.js

@@ -3,7 +3,7 @@ const common = require('../../common');
 constassert=require('assert');
 const{ runMakeCallback }=require(`./build/${common.buildType}/binding`);
 
-runMakeCallback(5,common.mustCall(function(err,val){
+runMakeCallback(5,common.mustCall((err,val)=>{
 assert.strictEqual(err,null);
 assert.strictEqual(val,10);
 process.nextTick(common.mustCall());

test/addons/async-hello-world/test.js

@@ -3,7 +3,7 @@ const common = require('../../common');
 constassert=require('assert');
 const{ runCall }=require(`./build/${common.buildType}/binding`);
 
-runCall(5,common.mustCall(function(err,val){
+runCall(5,common.mustCall((err,val)=>{
 assert.strictEqual(err,null);
 assert.strictEqual(val,10);
 process.nextTick(common.mustCall());

test/addons/make-callback-domain-warning/test.js

@@ -10,23 +10,23 @@ function makeCallback(object, cb) {
 }
 
 letlatestWarning=null;
-process.on('warning',function(warning){
+process.on('warning',(warning)=>{
 latestWarning=warning;
 });
 
 constd=domain.create();
 
 // When domain is disabled, no warning will be emitted
-makeCallback({domain: d},common.mustCall(function(){
+makeCallback({domain: d},common.mustCall(()=>{
 assert.strictEqual(latestWarning,null);
 
-d.run(common.mustCall(function(){
+d.run(common.mustCall(()=>{
 // No warning will be emitted when no domain property is applied
-makeCallback({},common.mustCall(function(){
+makeCallback({},common.mustCall(()=>{
 assert.strictEqual(latestWarning,null);
 
 // Warning is emitted when domain property is used and domain is enabled
-makeCallback({domain: d},common.mustCall(function(){
+makeCallback({domain: d},common.mustCall(()=>{
 assert.strictEqual(latestWarning.name,'DeprecationWarning');
 assert.strictEqual(latestWarning.code,'DEP0097');
 }));

test/addons/make-callback-recurse/test.js

@@ -71,7 +71,7 @@ assert.throws(() => {
 if(arg===1){
 // The tests are first run on bootstrap during LoadEnvironment() in
 // src/node.cc. Now run the tests through node::MakeCallback().
-setImmediate(function(){
+setImmediate(()=>{
 makeCallback({},common.mustCall(()=>{
 verifyExecutionOrder(2);
 }));

test/addons/openssl-client-cert-engine/test.js

@@ -43,14 +43,14 @@ const server = https.createServer(serverOptions, common.mustCall((req, res) => {
 headers: {}
 };
 
-constreq=https.request(clientOptions,common.mustCall(function(response){
+constreq=https.request(clientOptions,common.mustCall((response)=>{
 letbody='';
 response.setEncoding('utf8');
-response.on('data',function(chunk){
+response.on('data',(chunk)=>{
 body+=chunk;
 });
 
-response.on('end',common.mustCall(function(){
+response.on('end',common.mustCall(()=>{
 assert.strictEqual(body,'hello world');
 server.close();
 }));

test/addons/openssl-key-engine/test.js

@@ -45,14 +45,14 @@ const server = https.createServer(serverOptions, common.mustCall((req, res) => {
 headers: {}
 };
 
-constreq=https.request(clientOptions,common.mustCall(function(response){
+constreq=https.request(clientOptions,common.mustCall((response)=>{
 letbody='';
 response.setEncoding('utf8');
-response.on('data',function(chunk){
+response.on('data',(chunk)=>{
 body+=chunk;
 });
 
-response.on('end',common.mustCall(function(){
+response.on('end',common.mustCall(()=>{
 assert.strictEqual(body,'hello world');
 server.close();
 }));

test/addons/repl-domain-abort/test.js

@@ -31,7 +31,7 @@ if (common.isWindows)
 buildPath=buildPath.replace(/\\/g,'/');
 letcb_ran=false;
 
-process.on('exit',function(){
+process.on('exit',()=>{
 assert(cb_ran);
 console.log('ok');
 });

test/addons/stringbytes-external-exceed-max/test-stringbytes-external-exceed-max-by-1-ascii.js

@@ -25,7 +25,7 @@ if (!binding.ensureAllocation(2 * kStringMaxLength))
 common.skip(skipMessage);
 
 conststringLengthHex=kStringMaxLength.toString(16);
-common.expectsError(function(){
+common.expectsError(()=>{
 buf.toString('ascii');
 },{
 message: `Cannot create a string longer than 0x${stringLengthHex} `+

test/addons/stringbytes-external-exceed-max/test-stringbytes-external-exceed-max-by-1-base64.js

@@ -25,7 +25,7 @@ if (!binding.ensureAllocation(2 * kStringMaxLength))
 common.skip(skipMessage);
 
 conststringLengthHex=kStringMaxLength.toString(16);
-common.expectsError(function(){
+common.expectsError(()=>{
 buf.toString('base64');
 },{
 message: `Cannot create a string longer than 0x${stringLengthHex} `+

test/addons/stringbytes-external-exceed-max/test-stringbytes-external-exceed-max-by-1-binary.js

@@ -27,7 +27,7 @@ if (!binding.ensureAllocation(2 * kStringMaxLength))
 common.skip(skipMessage);
 
 conststringLengthHex=kStringMaxLength.toString(16);
-common.expectsError(function(){
+common.expectsError(()=>{
 buf.toString('latin1');
 },{
 message: `Cannot create a string longer than 0x${stringLengthHex} `+