Lesson 9 - Unit test: Assertion Error for 'Enter Lottery'

[Basem815]

I have the following error defined in my Lottery.sol file:
error Lottery__NotEnoughEthEntered();
with the following piece of code related to that error within the same file:

function enterLottery() public payable {
        if (msg.value < i_entranceFee) {
            revert Lottery__NotEnoughEthEntered();
        }
        ...

The problem is when I try to run the following test defined in Lottery.test.js:

describe("Enter Lottery", async function () {
				it("Reverts when you don't pay enough", async () => {
					await expect(lottery.enterLottery()).to.be.revertedWith(
						"Lottery__NotEnoughEthEntered"
					);
				});
			});

I get the following error:
AssertionError: Expected transaction to be reverted with "Lottery__NotEnoughEthEntered", but other reason was found: ""

Any suggestions why I am getting this error message? The test for the constructor passes without an issue but for some reason I can't get the defined error messages to be displayed.

[alymurtazamemon]

@Basem815 Try this;

await expect(lottery.enterLottery()).to.be.revertedWithCustomError(
    lottery,
    "Lottery__NotEnoughEthEntered"
);

If it does not work then please leave the repo so I can test it locally.

[Basem815]

@Basem815 It is working fine, just had some issues in the imports. Going to push it on your repo, merge it.

Still didn't work for me. I'll try doing a clean compile. Also, the reason I had added the additional part to the imports was because it would give me the warning:
Source "@chainlink/contracts/src/v0.8/VRFConsumerBaseV2.sol" not found: File import callback not supported

It still didn't make a difference though so I'll just try a clean compile some time later. Thanks though!

[alymurtazamemon]

@Basem815 Yup, leave those warnings, it is due to the solidity plugin.

[Basem815]

@alymurtazamemon I'm no longer having that issue mentioned above. Thanks! I guess it was changing the import paths only.
I am facing another issue though with 4 of the testcases:
Error: VM Exception while processing transaction: reverted with an unrecognized custom error (return data: 0x71e83137)
I've managed to identify that it's the await lottery.performUpkeep([]); line in each of these tests that is causing the issue. Still trying to figure it out but would appreciate it if you already know what it could be!

[alymurtazamemon]

@Basem815 Which test?

[Basem815]

@alymurtazamemon Pushed the latest version to include them but referring to these ones:

1) Doesn't allow anyone to enter while the lottery is calculating
2) Returns false if the lottery isn't open
3) Can only run if checkUpkeep is true
4) Updates the lottery state, emits an event and calls the VRF Coordinator