make throwingMatcher test n arguments (currently zero or one)

[ericprud]

🚀 Feature Proposal

Make throwingMatcher test all arguments.

Motivation

Chai's throw takes multiple arguments to test error type and message.
Jest users who wish to test both of these traits must catch the error and manually perform both tests.
It would be trivial to make throwingMatcher's processResults apply each arg in turn:

      args.forEach(arg =>
        potentialResult = matcher.apply(matcherContext, [actual].concat([arg]));

Example

describe('throwingMatcher', () => {
  test('could test all args', () => {
    expect(() => {
      throw TypeError('ABC')
    }).toThrowError(TypeError, 'XYZ')
  })
})

would fail with:

    Expected the function to throw an error matching:
      "XYZ"
    Instead, it threw:
      TypeError: ABC

Pitch

• known to be useful in chai.
• trivial to implement
• encourages more comprehensive testing

[SimenB]

You can create a custom matcher for this, or get it into jest-extended, cc @mattphillips

Not sure we need it in core?

[natealcedo]

@SimenB, this was actually proposed there already. I've started implementing some of it but I've been terribly busy at work so I haven't had the time to submit a PR there. Refer to jest-community/jest-extended#157. But if @ericprud is comfortable with working on it that would be cool as well.

[mattphillips]

Yup this should be available in jest-extended soon so not sure we need it in core when we currently have toThrow in core.

[SimenB]

Ah perfect.

This is apparently a duplicate of #3659 as well, so closing 🙂

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.