test: use arrow function

In `test/parallel/test-child-process-env.js`, callbacks use anonymous closure functions. It is safe to replace them with arrow functions since these callbacks don't contain references to `this`, `super` or `arguments`. This results in shorter functions. PR-URL: #24482 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Rich Trott <rtrott@gmail.com>
nodejs · Nov 28, 2018 · 0a492c730ac60fb805c7271abdc51410215ff8ed · 0a492c7
1 parent e5c85ef
commit 0a492c730ac60fb805c7271abdc51410215ff8ed
Showing with 2 additions and 2 deletions.

+2 −2 test/parallel/test-child-process-env.js
diff --git a/test/parallel/test-child-process-env.js b/test/parallel/test-child-process-env.js
@@ -48,12 +48,12 @@ let response = '';
 
 child.stdout.setEncoding('utf8');
 
-child.stdout.on('data', function(chunk) {
+child.stdout.on('data', (chunk) => {
   console.log(`stdout: ${chunk}`);
   response += chunk;
 });
 
-process.on('exit', function() {
+process.on('exit', () => {
   assert.ok(response.includes('HELLO=WORLD'));
   assert.ok(response.includes('FOO=BAR'));
   assert.ok(!response.includes('UNDEFINED=undefined'));