test: Use common.expectsError in tests (!17484) · Merge requests · Rodrigo Test / Test Group-nodejs / node · GitLab

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Rodrigo Muino Tomonari requested to merge github/fork/mithunsasidharan/common_3 into master Dec 06, 2017

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