test: replaced anonymous functions with closuers

[rkp1146]

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

[targos]

I suppose it was closed by mistake?

[targos]

Changes LGTM.

Commit message should look like: test: replace anonymous functions with arrow functions

[gireeshpunathil]

@rkp1146 - your email is not associated with your GitHub account while you pushed the change.
ref: https://github.com/settings/emails
ref: https://help.github.com/articles/setting-your-commit-email-address-on-github/

[gireeshpunathil]

fast-track? 👍

[gireeshpunathil]

CI: https://ci.nodejs.org/job/node-test-pull-request/18793/

[gireeshpunathil]

Resume build CI: https://ci.nodejs.org/job/node-test-commit/23491/

[gireeshpunathil]

failure in windows-fanned seems unrelated.

Resume CI: https://ci.nodejs.org/job/node-test-commit-windows-fanned/22620/

[gireeshpunathil]

the single failure in windows-fanned is unrelated; landing.

03:26:53 not ok 250 parallel/test-http2-client-upload
03:26:53   ---
03:26:53   duration_ms: 0.321
03:26:53   severity: fail
03:26:53   exitcode: 1
03:26:53   stack: |-
03:26:53     events.js:174
03:26:53           throw er; // Unhandled 'error' event
03:26:53           ^
03:26:53     
03:26:53     Error: read ECONNRESET
03:26:53         at TCP.onStreamRead (internal/stream_base_commons.js:139:27)
03:26:53     Emitted 'error' event at:
03:26:53         at emitErrorNT (internal/streams/destroy.js:82:8)
03:26:53         at emitErrorAndCloseNT (internal/streams/destroy.js:50:3)
03:26:53         at process.internalTickCallback (internal/process/next_tick.js:72:19)
03:26:53   ...

[gireeshpunathil]

looks like this changes are already merged through #24420 . @rkp1146 - can you confirm you worked on the assignment that we provided? In either case, looks like some mishandling. Please let me know your interest on working on another PR, thanks!