Skip to content

test: apply correct assert.fail() arguments

Rodrigo Muino Tomonari requested to merge github/fork/Trott/assert.fail into master

The assert.fail function signature has the message as the third argument but, understandably, it is often assumed that it is the first argument (or at least the first argument if no other arguments are passed).

This corrects the assert.fail() invocations in the Node.js tests.

Before: assert.fail('message'); // result: AssertionError: 'message' undefined undefined

After: assert.fail(null, null, 'message'); // result: AssertionError: message

Merge request reports