AssertionError: Expected transaction to be reverted with reason 'Raffle__NotEnoughEthEntered', but it reverted with a custom error

[kihiuFrank]

I'm writing a test that requires enterRaffle() to be reverted with MyCustomError - "Raffle__NotEnoughEthEntered"

describe("enterRaffle", async function () {
              it("reverts when you don't pay enough", async function () {
                  await expect(raffle.enterRaffle()).to.be.revertedWith(
                      "Raffle__NotEnoughEthEntered"
                  )
              })
          })

The test fails with error;

1) Raffle
       enterRaffle
         reverts when you don't pay enough:
     AssertionError: Expected transaction to be reverted with reason 'Raffle__NotEnoughEthEntered', but it reverted with a custom error
      at processTicksAndRejections (node:internal/process/task_queues:96:5)
      at Context.<anonymous> (test/unit/Raffle.test.js:32:19)

Why is Raffle__NotEnoughEthEntered not a custom error?

I have read other similar issues but they don't solve my problem.

Raffle.sol error & enterRaffle

error Raffle__NotEnoughEthEntered();

////////////////////////////////////////////////////////////////////////////////////////////////////

function enterRaffle() public payable {
        // you can do this,
        //require (msg.value > i_enteranceFee, "Not Enough ETH!")

        // but we will use error codes for gas efficiency since storing strings if supper expensive
        if (msg.value < i_enteranceFee) {
            revert Raffle__NotEnoughEthEntered();
        }

        if (s_raffleState != RaffleState.OPEN) {
            revert Raffle__NotOpen();
        }
        s_players.push(payable(msg.sender));

        //Emit an event when we update a dynamic array or mapping
        //name events with fuction name reversed. in this case event RaffleEnter()
        emit RaffleEnter(msg.sender);
    }

Tutorial Timestamp

15:28

@othaime-en Sir I will appreciate it a-lot if you looked into this when you can 🙏

[alymurtazamemon]

@kihiuFrank try this revertedWithCustomError

[kihiuFrank]

Thanks @alymurtazamemon That was super helpful.
Does the link explain the difference between revertedWith and revertedWithCustomError?

[alymurtazamemon]

@kihiuFrank Yes, you will find even more reverts with simple explanations with examples.

[kihiuFrank]

@kihiuFrank Yes, I forgot now the newer versions also take a contract in the parameters.

@alymurtazamemon Would you mind updating this sir?

[krakxn]

The above should fix it--revertedWithCustomError must be used for testing for custom errors, elsewise the test gives an error when the contract's custom error is triggered, which the former accounts for.

[alymurtazamemon]

@kihiuFrank It will look like this, I am not sure, Are you saying me to update this or anything else?

await expect(raffle.enterRaffle()).to.be.revertedWithCustomError(raffle, "Raffle__NotEnoughEthEntered")

[othaime-en]

Hey @kihiuFrank try and use to.be.revertedWithCustomError() instead and see if it works.
The line should read something like this

await expect(raffle.enterRaffle()).to.be.revertedWithCustomError(raffle, "Raffle__NotEnoughEthEntered")

[kihiuFrank]

Thanks @othaime-en
Always a life saver!

[othaime-en]

Welcome :)

[rbansode]

Thanks @othaime-en . Worked for me also.