rejectedWith is not returning error when if error message is different from actual val

Having the same issue here. Code to replicate: <div class="highl

<a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/us

rejectedWith not checking the error message about chai-as-promised HOT 3 OPEN

mani-vandrangi commented on June 10, 2024 11

rejectedWith not checking the error message

from chai-as-promised.

Comments (3)

siddharthvp commented on June 10, 2024 6

Having the same issue here.

Code to replicate:

it('test', async () => {
	  function reject42() {
	  	return Promise.reject(42);
	  }
	  await expect(reject42()).to.be.rejectedWith(999);
});

it('test', async () => {
	  function reject42() {
	  	return Promise.reject(42);
	  }
	  await expect(reject42()).to.be.eventually.rejectedWith(999);
});

In both cases test passes rather than fail.

from chai-as-promised.

nilsreichardt commented on June 10, 2024 1

@masterpandabear But this is only true, if no rejected value is part of the error. Here are some false-positive example:

const mocha = require("mocha");
const chai = require("chai");
const chaiAsPromised = require("chai-as-promised");
chai.use(chaiAsPromised);

it('test 1', async () => {
    function reject42() {
        return Promise.reject(new Error('42'))
    }

    await chai.expect(reject42()).to.eventually.be.rejectedWith('4')
});

it('test 2', async () => {
    function reject42() {
        return Promise.reject(new Error('42'))
    }

    await chai.expect(reject42()).to.eventually.be.rejectedWith('2')
});

Both tests are passing, which is not expected (for me), because it's obvious that 42 != 4 and 42 != 2.

Reproducible source code: https://github.com/nilsreichardt/playground/blob/chai-as-promised-error-message-bug/test/index_test.js

from chai-as-promised.

masterpandabear commented on June 10, 2024

A bit late but I guess this library looks for actual thrown errors for rejected values, for example this fails as expected

    it('test', async () => {
      function reject42 () {
        return Promise.reject(new Error('42'))
      }

      await expect(reject42()).to.eventually.be.rejectedWith('999')
    })