search

browserify / commonjs-assert

Node.js's require('assert') for all engines
MIT License
295 stars 58 forks source link

Unexpected result from assert.fail #34

Closed don01001000 closed 5 years ago

don01001000 commented 6 years ago

It seems to me based on the documentation that assert.fail(message) should throw an exception for which the message property is just the message passed in. However, it's being set to whatever messaged is wrapped in single quotes and followed by undefined undefined.

The output of the above program is as follows:

{ [AssertionError: 'boom' undefined undefined]
  name: 'AssertionError',
  actual: 'boom',
  expected: undefined,
  operator: undefined,
  message: '\'boom\' undefined undefined',
  generatedMessage: true }

According to the documentation:

If message is provided only it will be used as the error message, the other arguments will be stored as properties on the thrown object.

BridgeAR commented 5 years ago

This is resolved by #44. Please update to v2.0.0.