This should work:

const { assert, match } = require('@sinonjs/referee');

await assert.rejects(Promise.reject('nope'), match.string);

I'd love to do a PR, but my company requires me to get legal approval, etc., which would take a while. Regarding the matcher, I can't seem to get it to work. Do you have an example?

Regarding matchers, I think you can already pass something like match(/foo/) as the second argument.

Yes, I agree that the second argument should be optional, as stated in the docs. This is a bug.

Would you care to send a PR with a fix?

The assert.rejects function currently requires a second argument, which much be identical to what the target function rejected with. This can be difficult to determine in many cases. It would be great if the second argument was optional (for what it's worth, the docs say the second param is optional here, but maybe these aren't up to date: https://sinonjs.github.io/referee/#rejects). Here's an example (using mocha) of how I'd like this to work:

const {assert} = require('@sinonjs/referee');

async function thrower() {
    throw new Error('foo');
}

describe("assert.reject", () => {
    it("does not require a second argument", async () => {
        await assert.rejects(thrower());
    });
});

Or perhaps it could follow the same api as assert.match(). Then I could do something like:

await assert.rejects(thrower(), /foo/);

Codecov Report

Merging #81 (f559ea4) into master (70f48d0) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #81   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           18        18           
  Lines           81        81           
=========================================
  Hits            81        81           

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 70f48d0...f559ea4. Read the comment docs.

Codecov Report

Merging #215 (b9dab93) into master (da45fde) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #215   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           27        27           
  Lines          512       512           
=========================================
  Hits           512       512           

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update da45fde...b9dab93. Read the comment docs.

<!-- Summary - mandatory

give a concise (one or two short sentences) description of what what problem is being solved by this PR

Example: Fix issue #123456 by re-structuring the colour selection conditional in method paintBlue -->

<!-- (Problem in detail) - optional -->

Background

<!--

When relevant, give a more thorough description of what the problem the PR is trying to solve. Examples of good topics for this section are:

• Link to an existing GitHub issue describing the problem
• Describing the problem in greater detail than the TL;DR section above
• How you discovered the issue, if it's not already described as an issue on GitHub
• Discussion of different approaches to solving this problem and why you chose your proposed solution -->

<!-- (Solution in detail) - optional -->

Solution

<!--

When contributing code (and not just fixing typos, documentation and configuration), please describe why/how your solution works. This helps reviewers spot any mistakes in the implementation.

Example: "This solution works by adding a paintBlue() method" Then your reviewer might spot a mistake in the implementation, if paintBlue() uses the colour red. -->

<!-- mandatory -->

How to verify

1. Check out this branch
2. npm ci
3. <your-steps-here>

And run tests in Node 14, the current LTS release

Codecov Report

Merging #80 (288bcc1) into master (7aa37d2) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #80   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           18        18           
  Lines           81        81           
=========================================
  Hits            81        81           

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 7aa37d2...288bcc1. Read the comment docs.

And run tests in Node 14, the current LTS release

And format the files

Codecov Report

Merging #79 (585bd27) into master (5d6a5b1) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #79   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           18        18           
  Lines           81        81           
=========================================
  Hits            81        81           

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 5d6a5b1...585bd27. Read the comment docs.

And format the files

As done with https://github.com/sinonjs/sinon/pull/2324