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

[umairahmed88]

my code is

    it("Only allows the owner to withdraw", async function () {
        const accounts = await ethers.getSigners();
        const attacker = accounts[1];
        const attackerConnectedContract = await fundMe.connect(attacker);
        await expect(
            attackerConnectedContract.withdraw()
        ).to.be.revertedWith("FundMe_NotOwner");
    });

I got the error

     Only allows the owner to withdraw:
 AssertionError: Expected transaction to be reverted with reason 'FundMe__NotOwner', but it reverted with a custom error

[alymurtazamemon]

@umairahmed88 Use this;

await expect(
            attackerConnectedContract.withdraw()
        ).to.be.revertedWithCustomError(fundMe, "FundMe_NotOwner");

[umairahmed88]

Wao great!
Thank you very much

[paulcoffee85]

@umairahmed88 Use this;

await expect(
            attackerConnectedContract.withdraw()
        ).to.be.revertedWithCustomError(fundMe, "FundMe_NotOwner");

Why was the "..CustomError( fundMe, "... necessary for some , but not for Patrick in the video ? Was it based on whether the FundMe.sol file was from repo or from remix in previous section?

[alymurtazamemon]

@paulcoffee85 Hardhat updates the way it handles the custom errors.

[paulcoffee85]

@alymurtazamemon good to know. Thanks
And what is the best documents to read on testing code that Patrick is using in the video ? I"m following some of it but how would one know what to do working from scratch.? Seems the hardhat documentation is limited compared to the course material. Thanks again @alymurtazamemon

[masaj84]

@paulcoffee85 not sure why, but in my case changing from this:
if (msg.sender != i_owner) revert FundMe__NotOwner();

to this:
require(msg.sender == i_owner, "FundMe__NotOwner");

worked.