test: improve assertion message in dgram test

[shakeelmohamed]

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• commit message follows commit guidelines

Affected core subsystem(s)

test

[joyeecheung]

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

[BridgeAR]

Please do not use multiline template strings. This adds lots of whitespace to the message. Instead a string concat should be used here (and the template string in the second line).

[BridgeAR]

You might also just remove the message all together as that would also increase the readability a lot.

[jasnell]

@shakeelmohamed ... thank you so much for the pull request and for participating in the code and learn! As @BridgeAR indicates, multiline template strings are less than ideal and are not something that we use generally. Also, the added message does not quite fit the check. As @BridgeAR suggests, I would just remove the message altogether and leave it as assert.strictEqual(async, true).

[shakeelmohamed]

Thanks for the feedback @BridgeAR @jasnell, I did the lazy person's force push (delete remote branch, squash locally, re-push branch) and had to create a new PR: #16121