test: Use common.expectsError in tests

[mithunsasidharan]

Refactored tests to replace assert.throws(fn, common.expectsError(err)); with common.expectsError(fn, err); in following test files :

• test/parallel/test-buffer-slow.js
• test/parallel/test-buffer-tostring-range.js
• test/parallel/test-child-process-send-type-error.js
• test/parallel/test-child-process-spawnsync-kill-signal.js
• test/parallel/test-child-process-stdio.js
• test/parallel/test-child-process-validate-stdio.js
• test/parallel/test-common.js
• test/parallel/test-console-instance.js
• test/parallel/test-dgram-bind.js
• test/parallel/test-dgram-create-socket-handle.js

Checklist

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

Affected core subsystem(s)

test

[apapirovski]

Hi @mithunsasidharan — thank you for contributing. Could you use the checklist in your PR to indicate which changes have happened or whether you've run the tests locally? It seems like all your PRs have the checkboxes completely unchecked. In general, it's helpful to at least run make test -j4 locally.

[mithunsasidharan]

@apapirovski : Sorry for missing out on that. I usually run all test for any changes I make make -j4 test including changes in this PR. I'll be creating few more PRs for the same change in left over files. Let me know if you need any more info from my side. Thanks a lot.

[apapirovski]

LGTM. Thanks for updating the checklist 👍

[maclover7]

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

[apapirovski]

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

CI was a little wonky the last time, just running again to be safe.

[apapirovski]

Landed in d8c896c