Skip to content

test: s/assert.fail/common.fail as appropriate

Rodrigo Muino Tomonari requested to merge github/fork/cjihrig/fail into master
Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

test

Description of change

Many tests use assert.fail(null, null, msg) where it would be simpler to use common.fail(msg). This is largely because common.fail() is fairly new. This commit makes the replacement when applicable.

R= @nodejs/testing

Merge request reports